linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* RFC: radiotap discrepancy in Linux vs OpenBSD
@ 2007-03-26  3:24 Pavel Roskin
  2007-03-26  3:37 ` David Young
  2007-03-26  3:38 ` David Young
  0 siblings, 2 replies; 8+ messages in thread
From: Pavel Roskin @ 2007-03-26  3:24 UTC (permalink / raw)
  To: linux-wireless; +Cc: David Young, Scott Raynel

Hello!

I have noticed two different incompatible changes to enum
ieee80211_radiotap_type in ieee80211_radiotap.h.

One is found in the current wireless-2.6.git:

        IEEE80211_RADIOTAP_RX_FLAGS = 14,
        IEEE80211_RADIOTAP_TX_FLAGS = 15,
        IEEE80211_RADIOTAP_RTS_RETRIES = 16,
        IEEE80211_RADIOTAP_DATA_RETRIES = 17,

It was added together with Marvell Libertas USB driver.

Another set of the flags can be found in CVS OpenBSD:

        IEEE80211_RADIOTAP_FCS = 14,
        IEEE80211_RADIOTAP_HWQUEUE = 15,
        IEEE80211_RADIOTAP_RSSI = 16,

Actually, there is a reference to IEEE80211_RADIOTAP_FCS in Linux, but only in a
comment.

IEEE80211_RADIOTAP_FCS is not used in any driver (apparently, it's better to
keep FCS in the end of the frame, not in the header), but the other two flags
are used in more than one driver.

I think Marvell developers could act gracefully and push the flags it introduces
to higher numbers.  Doing something like that on the OpenBSD side would be
harder.  I would also like to see joining Rx and Tx flags into one 32-bit
value.

But we need some coordination when new fields are added to the protocol.
Radiotap is better than other wireless capture headers because it's better
documented and more extensible.  But extending it in an uncontrolled way would
ruin the standard.

We could consider introducing driver specific area in the header.  For instance,
Atheros chips may allow the userspace to specify a set of rates at which the
transmission will be attempted.  Uncalibrated RSSI may also be a candidate for
the driver-specific data if OpenBSD can be persuaded to abandon its present
number.

It's important that presence of driver specific fields doesn't break parsing of
the standard fields, even if new fields are made standard.  I think driver
specific flags don't belong to the it_present bitmap, but should go to the
beginning of the driver specific area.

--
Regards,
Pavel Roskin

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2007-03-28 20:33 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-03-26  3:24 RFC: radiotap discrepancy in Linux vs OpenBSD Pavel Roskin
2007-03-26  3:37 ` David Young
2007-03-26 22:45   ` Pavel Roskin
2007-03-28 18:04     ` [Radiotap] " Marcelo Tosatti
2007-03-28 20:33       ` Pavel Roskin
2007-03-26  3:38 ` David Young
2007-03-26 15:41   ` Luis R. Rodriguez
2007-03-26 16:59     ` David Young

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).