All of lore.kernel.org
 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 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.