From: Miquel Raynal <miquel.raynal@bootlin.com>
To: Alexander Aring <alex.aring@gmail.com>,
Stefan Schmidt <stefan@datenfreihafen.org>,
linux-wpan@vger.kernel.org
Cc: David Girault <david.girault@qorvo.com>,
Romuald Despres <romuald.despres@qorvo.com>,
Frederic Blain <frederic.blain@qorvo.com>,
Nicolas Schodet <nico@ni.fr.eu.org>,
Thomas Petazzoni <thomas.petazzoni@bootlin.com>,
"David S. Miller" <davem@davemloft.net>,
Jakub Kicinski <kuba@kernel.org>, Paolo Abeni <pabeni@redhat.com>,
netdev@vger.kernel.org, Miquel Raynal <miquel.raynal@bootlin.com>
Subject: [PATCH wpan-next v4 00/11] ieee802154: Synchronous Tx support
Date: Thu, 19 May 2022 17:05:05 +0200 [thread overview]
Message-ID: <20220519150516.443078-1-miquel.raynal@bootlin.com> (raw)
Hello,
This series brings support for that famous synchronous Tx API for MLME
commands.
MLME commands will be used during scan operations. In this situation,
we need to be sure that all transfers finished and that no transfer
will be queued for a short moment.
Cheers,
Miquèl
Changes in v4:
* Made visible the mlme_tx{_pre,,_post} helpers, used them later in the
scanning code where relevant.
* Used the atomic_fetch_inc() alternative to only stop the queue when
necessary.
* Used the netif_running() helper in place of the manual check against
the IFF_UP netdev flag.
* Changed the error codes to ENETDOWN if the device was closed.
* Reworked the MLME transmissions error path so that they would not keep
the rtnl taken.
* Updated the logic to avoid erroring out on the mlme_op_pre() call
which just returns the code of the previous transmission (which we
likely do not care about here).
* Dropped the queue_stopped variable, used the existing "flags"
variable, turning it into an unsigned long so that it would accept
atomic operations. Created a WPAN_PHY_FLAG_STATE_QUEUE_STOPPED
definition for this purpose.
Changes in v3:
* Tested with lockdep enabled, a more aggressive preemption level and
the sleeping while atomic warnings enabled.
* Changed the hold/release queue mutex into a spinlock.
* Split the mlme_tx function into three, one to hold the queue, then
another part that does takes the rtnl and has the real content, and a
last helper to release the queue.
* Fixed the warning condition in the slow path.
* Used an unsigned long and test/set_bit helpers to follow the queue
state instead of an atomic_t.
Changes in v2:
* Updated the main tx function error path.
* Added a missing atomic_dec_at_test() call on the hold counter.
* Always called (upon a certain condition) the queue wakeup helper from
the release queue helper (and similarly in the hold helper) and
squashed two existing patches in it to simplify the series.
* Introduced a mutex to serialize accesses to the increment/decrement of
the hold counter and the wake up call.
* Added a warning in case an MLME Tx gets triggered while the device was
stopped.
* Used the rtnl to ensure the device cannot be stopped while an MLME
transmission is ongoing.
Changes in v1 since this series got extracted from a bigger change:
* Introduced a new atomic variable to know when the queue is actually
stopped. So far we only had an atomic to know when the queue was held
(indicates a transitioning state towards a stopped queue only) and
another atomic indicating if a transfer was still ongoing at this
point (used by the wait logic as a condition to wake up).
Miquel Raynal (11):
net: mac802154: Rename the synchronous xmit worker
net: mac802154: Rename the main tx_work struct
net: mac802154: Enhance the error path in the main tx helper
net: mac802154: Follow the count of ongoing transmissions
net: mac802154: Bring the ability to hold the transmit queue
net: mac802154: Create a hot tx path
net: mac802154: Introduce a helper to disable the queue
net: mac802154: Introduce a tx queue flushing mechanism
net: mac802154: Introduce a synchronous API for MLME commands
net: mac802154: Add a warning in the hot path
net: mac802154: Add a warning in the slow path
include/net/cfg802154.h | 13 +++-
include/net/mac802154.h | 27 -------
net/ieee802154/core.c | 3 +
net/mac802154/cfg.c | 4 +-
net/mac802154/ieee802154_i.h | 40 +++++++++-
net/mac802154/main.c | 2 +-
net/mac802154/tx.c | 147 +++++++++++++++++++++++++++++++----
net/mac802154/util.c | 71 +++++++++++++++--
8 files changed, 252 insertions(+), 55 deletions(-)
--
2.34.1
next reply other threads:[~2022-05-19 15:05 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-05-19 15:05 Miquel Raynal [this message]
2022-05-19 15:05 ` [PATCH wpan-next v4 01/11] net: mac802154: Rename the synchronous xmit worker Miquel Raynal
2022-05-19 15:05 ` [PATCH wpan-next v4 02/11] net: mac802154: Rename the main tx_work struct Miquel Raynal
2022-05-19 15:05 ` [PATCH wpan-next v4 03/11] net: mac802154: Enhance the error path in the main tx helper Miquel Raynal
2022-05-19 15:05 ` [PATCH wpan-next v4 04/11] net: mac802154: Follow the count of ongoing transmissions Miquel Raynal
2022-05-19 15:05 ` [PATCH wpan-next v4 05/11] net: mac802154: Bring the ability to hold the transmit queue Miquel Raynal
2022-05-19 15:05 ` [PATCH wpan-next v4 06/11] net: mac802154: Create a hot tx path Miquel Raynal
2022-05-19 15:05 ` [PATCH wpan-next v4 07/11] net: mac802154: Introduce a helper to disable the queue Miquel Raynal
2022-05-19 15:05 ` [PATCH wpan-next v4 08/11] net: mac802154: Introduce a tx queue flushing mechanism Miquel Raynal
2022-05-19 15:05 ` [PATCH wpan-next v4 09/11] net: mac802154: Introduce a synchronous API for MLME commands Miquel Raynal
2022-05-19 15:05 ` [PATCH wpan-next v4 10/11] net: mac802154: Add a warning in the hot path Miquel Raynal
2022-05-19 15:05 ` [PATCH wpan-next v4 11/11] net: mac802154: Add a warning in the slow path Miquel Raynal
2022-06-01 3:30 ` [PATCH wpan-next v4 00/11] ieee802154: Synchronous Tx support Alexander Aring
2022-06-01 6:12 ` Miquel Raynal
2022-06-01 21:01 ` Stefan Schmidt
2022-06-03 17:55 ` Miquel Raynal
2022-06-04 1:50 ` Alexander Aring
2022-06-06 17:03 ` Miquel Raynal
2022-06-17 14:20 ` Miquel Raynal
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=20220519150516.443078-1-miquel.raynal@bootlin.com \
--to=miquel.raynal@bootlin.com \
--cc=alex.aring@gmail.com \
--cc=davem@davemloft.net \
--cc=david.girault@qorvo.com \
--cc=frederic.blain@qorvo.com \
--cc=kuba@kernel.org \
--cc=linux-wpan@vger.kernel.org \
--cc=netdev@vger.kernel.org \
--cc=nico@ni.fr.eu.org \
--cc=pabeni@redhat.com \
--cc=romuald.despres@qorvo.com \
--cc=stefan@datenfreihafen.org \
--cc=thomas.petazzoni@bootlin.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).