linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Pavel Roskin <proski@gnu.org>
To: Steve deRosier <steve@cozybit.com>
Cc: linux-wireless@vger.kernel.org
Subject: Re: radiotap rate no longer supported in mac80211?
Date: Tue, 15 Jun 2010 16:18:27 -0400	[thread overview]
Message-ID: <1276633107.14133.11.camel@mj> (raw)
In-Reply-To: <AANLkTileP2BMsZUsA-NigDcmVcAmXn6KNw0R1vMzSgYw@mail.gmail.com>

On Mon, 2010-06-14 at 15:18 -0700, Steve deRosier wrote:
> I'm trying to support per-packet setting of rate on a packet injection
> via the radiotap header.  In an earlier version of mac80211 (around
> 2.6.26), there was code in __ieee80211_parse_tx_radiotap (in
> net/mac80211/tx.c) to support the use of the the rate element from the
> radiotap header.  In current versions of wireless-testing, most of the
> code here has been removed and only the flags are parsed.
> 
> I want to return the IEEE80211_RADIOTAP_RATE portion of this function
> in order to support this.  So the questions:
> 1. Why were all fields other than IEEE80211_RADIOTAP_FLAGS removed?
> 2. Would it be OK for me to prepare and submit a patch to restore the
> rate functionality?

I posted a patch that would add rate and retry flags:
http://thread.gmane.org/gmane.linux.kernel.wireless.general/47441

I didn't see any interest in the patch.  Perhaps injecting packets at a
specific rate in not particularly needed.

Also, the patch is somewhat inelegant because of the requirement to
specify the rate when the retry count is specified.

mac80211 has an array of rates with corresponding retry counts.  The
radiotap standard has one rate and one retry count.  This doesn't map
well to the mac80211 approach.  Supporting the retry count without the
rate would require some tricky logic, and I don't know if anyone needs
that.

I don't feel good about pushing a patch that makes the code more complex
without knowing the use case.

-- 
Regards,
Pavel Roskin

  reply	other threads:[~2010-06-15 20:18 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-06-14 22:18 radiotap rate no longer supported in mac80211? Steve deRosier
2010-06-15 20:18 ` Pavel Roskin [this message]
2011-03-30 15:42   ` Roberto Riggio
2010-08-05 14:14 Capturing packets with bad FCS in monitor mode Daniel Haid
2010-08-05 22:58 ` Qasim Javed
2010-08-06 10:09   ` radiotap rate no longer supported in mac80211? Daniel Haid

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=1276633107.14133.11.camel@mj \
    --to=proski@gnu.org \
    --cc=linux-wireless@vger.kernel.org \
    --cc=steve@cozybit.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 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).