linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Andy Green <andy@warmcat.com>
To: Ivo van Doorn <ivdoorn@gmail.com>
Cc: linux-wireless <linux-wireless@vger.kernel.org>
Subject: Re: [PATCH] rt73usb-add-bluenext-148f-2573
Date: Tue, 31 Jul 2007 15:34:49 +0100	[thread overview]
Message-ID: <46AF4889.4070902@warmcat.com> (raw)
In-Reply-To: <200707311611.26683.IvDoorn@gmail.com>

Somebody in the thread at some point said:

> I think I have an idea why mac80211 always reports 1 MBs,
> rt2x00 checks if PREAMBLE was enabled or not. If that is the case it sets
> the rate value to rate->val2
> Mac80211 in turn only compares the value against rate->val which
> means that it won't find the rate and assume the lowest possible rate.
> I'll fix this by always setting the rate->val value instead (It isn't
really
> important for mac80211 to know if the preamble bit was set anyway).

Somehow the rates seem to be correct now after your change to the DROP
bits...

> #define TXRX_CSR0_DROP_CRC				FIELD32(0x00020000)
> #define TXRX_CSR0_DROP_PHYSICAL			FIELD32(0x00040000)
> #define TXRX_CSR0_DROP_CONTROL			FIELD32(0x00080000)
> #define TXRX_CSR0_DROP_NOT_TO_ME		FIELD32(0x00100000)
> #define TXRX_CSR0_DROP_TO_DS				FIELD32(0x00200000)
> #define TXRX_CSR0_DROP_VERSION_ERROR	FIELD32(0x00400000)
> #define TXRX_CSR0_DROP_MULTICAST			FIELD32(0x00800000)
> #define TXRX_CSR0_DROP_BORADCAST		FIELD32(0x01000000)
> #define TXRX_CSR0_DROP_ACK_CTS			FIELD32(0x02000000)
>
> Which means it will only pass multicast, broadcast and ACK_CTS frames. :S
> Always dropping CRC errors is correct, since enabling that will
greatly upset
> any monitoring tool listening to the interface.

Well one day it will be nice if there is a stack-level way to turn it on
and off.  We can also pass in the radiotap part a flag to show if the
packet's CRC was valid or not.  The tools can then either turn on the
filtering before the packets come or filter on good packets only.

> To see if it helps try:
>
> echo 16 > csr_offset
> echo 0x0002b162 > csr_value
>
> This will disable all DROP bits (except for the CRC error) and thus should
> trigger more frames to be received by your interface. In the mean time
I'll search
> why those bits weren't cleared when enabling the interface in monitor
mode.

Yes, that seems to have done it, thanks

15:20:34.157203 54.0 Mb/s 2437 MHz (0x0480) 214dB signal Data IV:1b30
Pad 20 KeyID 0
15:20:34.157206 24.0 Mb/s 2437 MHz (0x0480) 232dB signal Acknowledgment
RA:00:11:50:ad:ce:38 (oui Unknown)
15:20:34.157942 54.0 Mb/s 2437 MHz (0x0480) 232dB signal Data IV:16aa
Pad 20 KeyID 0
15:20:34.157945 36.0 Mb/s 2437 MHz (0x0480) 214dB signal Acknowledgment
RA:00:19:d2:59:8f:d5 (oui Unknown)
15:20:34.161265 54.0 Mb/s 2437 MHz (0x0480) 214dB signal Data IV:1b31
Pad 20 KeyID 0
15:20:34.161268 24.0 Mb/s 2437 MHz (0x0480) 230dB signal Acknowledgment
RA:00:11:50:ad:ce:38 (oui Unknown)
15:20:34.165809 1.0 Mb/s 2437 MHz (0x0480) 218dB signal Beacon (froh)
[1.0* 2.0* 5.5* 11.0* Mbit] ESS CH: 6, PRIVACY
15:20:34.165812 54.0 Mb/s 2437 MHz (0x0480) 214dB signal Data IV:1b32
Pad 20 KeyID 0
15:20:34.165815 24.0 Mb/s 2437 MHz (0x0480) 232dB signal Acknowledgment
RA:00:11:50:ad:ce:38 (oui Unknown)
15:20:34.166498 54.0 Mb/s 2437 MHz (0x0480) 232dB signal Data IV:16ab
Pad 20 KeyID 0
15:20:34.166502 36.0 Mb/s 2437 MHz (0x0480) 214dB signal Acknowledgment
RA:00:19:d2:59:8f:d5 (oui Unknown)


> Which version are you using?
> wireless-dev <---
> rt2x00.git
> rt2x00 cvs

-Andy

  reply	other threads:[~2007-07-31 14:35 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-07-31 11:54 [PATCH] rt73usb-add-bluenext-148f-2573 warmcat
2007-07-31 12:12 ` Ivo van Doorn
2007-07-31 12:19   ` Andy Green
2007-07-31 12:27     ` Ivo van Doorn
2007-07-31 12:38       ` Andy Green
2007-07-31 12:58         ` Ivo van Doorn
2007-07-31 13:35           ` Andy Green
2007-07-31 14:11             ` Ivo van Doorn
2007-07-31 14:34               ` Andy Green [this message]
2007-07-31 14:50                 ` Ivo van Doorn
2007-07-31 15:06                   ` Ivo van Doorn
2007-07-31 15:14                     ` Andy Green
2007-07-31 22:37                       ` Andy Green
2007-08-01 16:50                         ` Ivo van Doorn
2007-07-31 13:49           ` 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=46AF4889.4070902@warmcat.com \
    --to=andy@warmcat.com \
    --cc=ivdoorn@gmail.com \
    --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).