From: Furong Xu <0x1207@gmail.com>
To: Vladimir Oltean <olteanv@gmail.com>,
Alexander Lobakin <aleksander.lobakin@intel.com>,
Serge Semin <fancer.lancer@gmail.com>,
"David S. Miller" <davem@davemloft.net>,
Alexandre Torgue <alexandre.torgue@foss.st.com>,
Jose Abreu <joabreu@synopsys.com>,
Eric Dumazet <edumazet@google.com>,
Jakub Kicinski <kuba@kernel.org>, Paolo Abeni <pabeni@redhat.com>,
Maxime Coquelin <mcoquelin.stm32@gmail.com>,
Joao Pinto <jpinto@synopsys.com>
Cc: netdev@vger.kernel.org, linux-stm32@st-md-mailman.stormreply.com,
linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org, rmk+kernel@armlinux.org.uk,
linux@armlinux.org.uk, xfr@outlook.com,
Furong Xu <0x1207@gmail.com>
Subject: [PATCH net-next v8 0/7] net: stmmac: FPE via ethtool + tc
Date: Thu, 5 Sep 2024 15:02:21 +0800 [thread overview]
Message-ID: <cover.1725518135.git.0x1207@gmail.com> (raw)
Move the Frame Preemption(FPE) over to the new standard API which uses
ethtool-mm/tc-mqprio/tc-taprio.
Changes in v8:
1. use timer_delete_sync() instead of deprecated del_timer_sync()
2. check netif_running() to guarantee synchronization rules between
mod_timer() and timer_delete_sync()
3. split up stmmac_tc_ops of dwmac4, dwmac4+ and dwxgmac to give user
more descriptive error message
4. fix wrong indentation about switch-case
5. delete more unbalanced logs
Changes in v7:
1. code style fixes and clean up warnings reported by
patchwork netdev checks, no functional change intended
Changes in v6:
1. new FPE verification process based on Vladimir Oltean's proposal
2. embed ethtool_mm_state into stmmac_fpe_cfg
3. convert some bit ops to u32_replace_bits
4. register name and function name update to be more descriptive
5. split up stmmac_tc_ops of dwmac4+ and dwxgmac, they have different
implementations about mqprio
6. some code style fixes
Changes in v5:
1. fix typo in commit message
2. drop FPE capability check in tc-mqprio/tc-taprio
Changes in v4:
1. reorder FPE-related declarations and definitions into clean groups
2. move mm_lock to stmmac_fpe_cfg.lock
3. protect user configurations across NIC up/down
4. block stmmac_set_mm() when fpe_task is in progress to finish
5. convert to ethtool_dev_mm_supported() to check FPE capability in
tc-mqprio/tc-taprio
6. silence FPE workqueue start/stop logs
Changes in v3:
1. avoid races among ISR, workqueue, link update and
register configuration.
2. update FPE verification retry logic, so it retries
and fails as expected.
Changes in v2:
1. refactor FPE verification process
2. suspend/resume and kselftest-ethtool_mm, all test cases passed
3. handle TC:TXQ remapping for DWMAC CORE4+
Furong Xu (7):
net: stmmac: move stmmac_fpe_cfg to stmmac_priv data
net: stmmac: drop stmmac_fpe_handshake
net: stmmac: refactor FPE verification process
net: stmmac: configure FPE via ethtool-mm
net: stmmac: support fp parameter of tc-mqprio
net: stmmac: support fp parameter of tc-taprio
net: stmmac: silence FPE kernel logs
.../net/ethernet/stmicro/stmmac/dwmac4_core.c | 10 +-
drivers/net/ethernet/stmicro/stmmac/dwmac5.c | 96 +++++-
drivers/net/ethernet/stmicro/stmmac/dwmac5.h | 12 +-
.../ethernet/stmicro/stmmac/dwxgmac2_core.c | 9 +-
drivers/net/ethernet/stmicro/stmmac/hwif.c | 6 +-
drivers/net/ethernet/stmicro/stmmac/hwif.h | 22 +-
drivers/net/ethernet/stmicro/stmmac/stmmac.h | 30 +-
.../ethernet/stmicro/stmmac/stmmac_ethtool.c | 91 ++++++
.../net/ethernet/stmicro/stmmac/stmmac_main.c | 276 ++++++++----------
.../net/ethernet/stmicro/stmmac/stmmac_tc.c | 153 +++++++---
include/linux/stmmac.h | 28 --
11 files changed, 491 insertions(+), 242 deletions(-)
--
2.34.1
next reply other threads:[~2024-09-05 7:04 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-09-05 7:02 Furong Xu [this message]
2024-09-05 7:02 ` [PATCH net-next v8 1/7] net: stmmac: move stmmac_fpe_cfg to stmmac_priv data Furong Xu
2024-09-05 7:02 ` [PATCH net-next v8 2/7] net: stmmac: drop stmmac_fpe_handshake Furong Xu
2024-09-05 7:02 ` [PATCH net-next v8 3/7] net: stmmac: refactor FPE verification process Furong Xu
2024-09-05 13:54 ` Vladimir Oltean
2024-09-05 14:58 ` Vladimir Oltean
2024-09-05 7:02 ` [PATCH net-next v8 4/7] net: stmmac: configure FPE via ethtool-mm Furong Xu
2024-09-05 7:02 ` [PATCH net-next v8 5/7] net: stmmac: support fp parameter of tc-mqprio Furong Xu
2024-09-05 14:02 ` Vladimir Oltean
2024-09-05 7:02 ` [PATCH net-next v8 6/7] net: stmmac: support fp parameter of tc-taprio Furong Xu
2024-09-05 14:52 ` Vladimir Oltean
2024-09-05 7:02 ` [PATCH net-next v8 7/7] net: stmmac: silence FPE kernel logs Furong Xu
2024-09-05 14:53 ` Vladimir Oltean
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=cover.1725518135.git.0x1207@gmail.com \
--to=0x1207@gmail.com \
--cc=aleksander.lobakin@intel.com \
--cc=alexandre.torgue@foss.st.com \
--cc=davem@davemloft.net \
--cc=edumazet@google.com \
--cc=fancer.lancer@gmail.com \
--cc=joabreu@synopsys.com \
--cc=jpinto@synopsys.com \
--cc=kuba@kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-stm32@st-md-mailman.stormreply.com \
--cc=linux@armlinux.org.uk \
--cc=mcoquelin.stm32@gmail.com \
--cc=netdev@vger.kernel.org \
--cc=olteanv@gmail.com \
--cc=pabeni@redhat.com \
--cc=rmk+kernel@armlinux.org.uk \
--cc=xfr@outlook.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.