From: Andy Green <andy@warmcat.com>
To: Johannes Berg <johannes@sipsolutions.net>
Cc: linux-wireless@vger.kernel.org
Subject: Re: [PATCH] mac80211: No echo TX pkt to Monitor interfaces for injection
Date: Fri, 03 Aug 2007 11:15:17 +0100 [thread overview]
Message-ID: <46B30035.4080004@warmcat.com> (raw)
In-Reply-To: <1186135029.4647.27.camel@johannes.berg>
Somebody in the thread at some point said:
> Ah. Now I see why this is happening. It's because any packet you send
> into the monitor interface from userspace is already redirected back to
> userspace by upper levels... Hmm. It *seems* this patch is correct, but
> it changes the semantics slightly: previously, you would see transmitted
> packets with their transmit indication (whether it was acked etc) and
> now you do not get that for injected packets. Could we instead somehow
> get the upper layers to not redirect that and fix the radiotap header?
With the patch, injected packets have the same Monitor mode footprint
with hard or soft monitoring (you see the injected radiotap stuff)
because the synthesized "tx report" packet doesn't exist in the real
world shown by hard monitoring.
Acutually I think where it might all be heading is Michael Buesch's
method of selecting between hard and soft Monitor mode based on
IFF_PROMISC. Maybe stuff like the user MLE plan wants to hear about TX
reports, it's only interested in a context it is trying to associate to,
maybe it would be happy with the filtered soft Monitor (this would deal
with the objection about increased power consumption for unfiltered user
MLE hardware promisc too)... apps that truly want hard Monitor don't
want the TX reports...
If that is where we are headed then it deprecates this patch, you
actually want the TX report in soft monitor (only) which is the current
situation... we just need to get to the place that tcpdump (which sets
IFF_PROMISC for the duration) never ever sees soft Monitor...
-Andy
next prev parent reply other threads:[~2007-08-03 10:15 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-08-02 22:38 [PATCH] mac80211: No echo TX pkt to Monitor interfaces for injection warmcat
2007-08-03 9:57 ` Johannes Berg
2007-08-03 10:15 ` Andy Green [this message]
2007-08-03 10:23 ` Johannes Berg
2007-08-03 10:34 ` Andy Green
2007-08-03 10:43 ` Johannes Berg
2007-08-06 19:33 ` Johannes Berg
2007-08-07 9:47 ` Andy Green
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=46B30035.4080004@warmcat.com \
--to=andy@warmcat.com \
--cc=johannes@sipsolutions.net \
--cc=linux-wireless@vger.kernel.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).