From: David Young <dyoung-e+AXbWqSrlAAvxtiuMwx3w@public.gmane.org>
To: "radiotap-sUITvd46vNxg9hUCZPvPmw@public.gmane.org"
<radiotap-sUITvd46vNxg9hUCZPvPmw@public.gmane.org>
Subject: Re: TSFT
Date: Sat, 17 Nov 2012 11:46:00 -0600 [thread overview]
Message-ID: <20121117174600.GQ21779@pixotech.com> (raw)
In-Reply-To: <1353141950.9543.3.camel-8Nb76shvtaUJvtFkdXX2HixXY32XiHfO@public.gmane.org>
On Sat, Nov 17, 2012 at 09:45:50AM +0100, Johannes Berg wrote:
> On Sat, 2012-11-17 at 00:48 -0600, David Young wrote:
> > On Fri, Nov 16, 2012 at 10:26:31PM -0800, Simon Barber wrote:
> > > In my wireshark patch I've added options to interpret TSFT as the
> > > start of end of the frame.
> >
> > I think we need 1) a methodology for an author to identify their
> > device's reference point so that they can calibrate their driver to the
> > standard and/or 2) a radiotap datum that identifies the reference point
> > in use or 3) something entirely different.
>
> FWIW, for the Atheros drivers we *just* added (1) to the Linux kernel,
> but for radiotap output we calculate the difference to the start of the
> frame, see here:
>
> http://git.kernel.org/?p=linux/kernel/git/jberg/mac80211-next.git;a=commit;h=f4bda337bbb6e245e2a07f344990adeb6a70ff35
>
> (the patches for the drivers are separate, but simply change the _START
> flag to _END)
Adjusting the NIC's TSFT in this way, so that the radiotap reference
point is according to spec, sounds right to me.
> I have no objections to (also) allowing this information to be in
> radiotap though, but I'm not sure we want to?
Given that the adjustment is easy, I'm pretty sure we should not waste
radiotap header space to describe the TSFT.
Wireshark doesn't already have more than two "interpretation modes" for
radiotap headers, does it? That would be too bad. I realize that there
may be already be a mode for a particular OS....
Dave
--
David Young
dyoung-e+AXbWqSrlAAvxtiuMwx3w@public.gmane.org Urbana, IL (217) 721-9981
next prev parent reply other threads:[~2012-11-17 17:46 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-11-17 6:26 TSFT Simon Barber
[not found] ` <50A72E17.6070207-vp0mx6+5gkqFX2APIN6yfw@public.gmane.org>
2012-11-17 6:48 ` TSFT David Young
[not found] ` <20121117064848.GO21779-Gq8g4XVd89tWk0Htik3J/w@public.gmane.org>
2012-11-17 8:45 ` TSFT Johannes Berg
[not found] ` <1353141950.9543.3.camel-8Nb76shvtaUJvtFkdXX2HixXY32XiHfO@public.gmane.org>
2012-11-17 17:46 ` David Young [this message]
[not found] ` <20121117174600.GQ21779-Gq8g4XVd89tWk0Htik3J/w@public.gmane.org>
2012-11-17 18:43 ` TSFT Johannes Berg
2012-11-17 18:56 ` TSFT Guy Harris
2012-11-17 23:26 ` TSFT Thomas Pedersen
2012-11-18 7:20 ` TSFT Simon Barber
[not found] ` <50A88C3D.5030809-vp0mx6+5gkqFX2APIN6yfw@public.gmane.org>
2012-11-18 11:07 ` TSFT Johannes Berg
[not found] ` <1353236845.9649.6.camel-8Nb76shvtaUJvtFkdXX2HixXY32XiHfO@public.gmane.org>
2012-11-20 22:43 ` TSFT Simon Barber
[not found] ` <50AC0791.3040005-vp0mx6+5gkqFX2APIN6yfw@public.gmane.org>
2012-11-21 10:42 ` TSFT Johannes Berg
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=20121117174600.GQ21779@pixotech.com \
--to=dyoung-e+axbwqsrlaavxtiumwx3w@public.gmane.org \
--cc=radiotap-sUITvd46vNxg9hUCZPvPmw@public.gmane.org \
/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).