Re: Proposed Tx flag: don't reorder frames

Mathy Vanhoef <[email protected]> Sun, 30 Aug 2020 04:05:36 +0400
Newsgroups org.netbsd.radiotap
Message-ID <CAFXAJYzsg+NB69NpeUHuiySMsC_h2yQTko3_4-23Msfj9YOBhA@mail.gmail.com>
To follow up on this: the only downside is that OpenBSD is using the
TX flags identifier to report the hardware queue on which a frame was
received.

Since Linux and NetBSD are using TX flags, and Wireshark decided to
already accept the patch to support TX flags as well, it seems to make
sense to let this identifier represent TX flags. Then only one
implementation (OpenBSD) would have to be updated to match this. I'm
assuming nobody would be against this, since there have been more than
three weeks to discuss this, and nobody brought this up.

What's the next step? Adopt the proposal in one week if there are no
further objections?

Best regards,
Mathy

PS: Is the archive on https://lore.kernel.org/radiotap/ being updated?


On Fri, Jul 24, 2020 at 3:13 PM Johannes Berg <[email protected]> wrote:
>
> On Fri, 2020-07-24 at 09:58 +0400, Mathy Vanhoef wrote:
>
> > I propose to add a DONT_REORDER bit to the existing TX flags field.
> > When this flag is set, injected frames shouldn't be reordered relative
> > to other frames that also have this flag set (even when these frames
> > have different QoS TID values).
> >
> > This for example allows userspace to inject frames that have different
> > QoS TID fields while assuring that they are transmitted in the order
> > as they were injected. In practice this is useful to perform certain
> > experiments where the order of transmitted frames is important.
>
> Interesting.
>
> > Note that on the radiotap website the TX flags field is still listed
> > under the "suggested fields" page, but I'm assuming that by now this
> > field is approved (since it's used in Linux)?
>
> No, it never was. I made an attempt at some point, but abandoned it for
> some reason.
>
> I see no reason why it couldn't be done though, and I guess this new
> flag could be included from the start.
>
> johannes
>