All of lore.kernel.org
 help / color / mirror / Atom feed
From: Michael Buesch <mb@bu3sch.de>
To: Johannes Berg <johannes@sipsolutions.net>
Cc: "Stefanik Gábor" <netrolller.3d@gmail.com>,
	JMF <tolas_feup@hotmail.com>,
	"Alejandro Grijalba Martínez" <sud@latinsud.com>,
	linux-wireless <linux-wireless@vger.kernel.org>,
	"John Linville" <linville@tuxdriver.com>
Subject: Re: [PATCH] mac80211: do not alter injected seq numbers
Date: Thu, 22 May 2008 21:08:05 +0200	[thread overview]
Message-ID: <200805222108.06322.mb@bu3sch.de> (raw)
In-Reply-To: <1211482981.3698.61.camel@johannes.berg>

On Thursday 22 May 2008 21:03:01 Johannes Berg wrote:
> On Thu, 2008-05-22 at 20:52 +0200, Stefanik G=E1bor wrote:
> > On Thu, May 22, 2008 at 8:34 PM, Johannes Berg
> > <johannes@sipsolutions.net> wrote:
> > >> >> This does not work, since we use injection for other types of
> > >> >>  frames. For example management frames from hostapd. We don't=
 want
> > >> >>  (and can't) make hostapd keep track of sequence numbers.
> > >> >> You'll have to contact radiotap people and add a flag for thi=
s.
> > >> >>  This would also solve the hardware counter problem then.
> > >
> > >> The flag is IEEE80211_TX_INJECTED. Here is a new patch, for the
> > >> current wireless-testing:
> > >
> > > Reorder the two blocks and you have your answer. NO.
> > >
> > > johannes
> > >
> >=20
> > Hmm, shouldn't this version fix the hostap and other non-monitor-mo=
de cases?
>=20
> No, because hostapd uses monitor interfaces. Give up, it doesn't work
> without an extra radiotap flag.

Well, don't give up, but please contact the radiotap people for adding =
such
a flag to the header. That's really the only way to go.
There are more flags to be added, like we-are-not-interested-in-acks.

--=20
Greetings Michael.
--
To unsubscribe from this list: send the line "unsubscribe linux-wireles=
s" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

  reply	other threads:[~2008-05-22 19:08 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-04-08  0:11 [PATCH] mac80211: do not alter injected seq numbers Alejandro Grijalba
2008-04-08  8:25 ` Michael Buesch
2008-04-09  8:00   ` Johannes Berg
2008-05-22 17:48   ` JMF
     [not found]     ` <69e28c910805221115g6290e5e9nef4444efca07aea@mail.gmail.com>
2008-05-22 18:21       ` Stefanik Gábor
2008-05-22 18:34         ` Johannes Berg
2008-05-22 18:52           ` Stefanik Gábor
2008-05-22 19:03             ` Johannes Berg
2008-05-22 19:08               ` Michael Buesch [this message]
2008-05-22 19:03             ` Michael Buesch
2008-05-22 19:09           ` JMF
2008-05-22 19:15             ` Johannes Berg
2008-05-22 19:19             ` Tomas Winkler

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=200805222108.06322.mb@bu3sch.de \
    --to=mb@bu3sch.de \
    --cc=johannes@sipsolutions.net \
    --cc=linux-wireless@vger.kernel.org \
    --cc=linville@tuxdriver.com \
    --cc=netrolller.3d@gmail.com \
    --cc=sud@latinsud.com \
    --cc=tolas_feup@hotmail.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.