headers
Jiri Benc | 1 May 12:47
Picon

Re: d80211: How does TX flow control work?

On Tue, 01 May 2007 12:41:34 +0200, Jan Kiszka wrote:
> Now I came across this issue once again. It is still present, I just
> observed it over the latest rt2x00 tree after updating shorewall and
> forgetting to fix its config.
> 
> I redirected tc to some logging filter, and this is what shorewall does
> when "CLEAR_TC=Yes" is set in shorewall.conf:
> 
> ...
> tc qdisc del dev wmaster0 root
> tc qdisc del dev wmaster0 ingress
> tc qdisc del dev wlan0 root
> tc qdisc del dev wlan0 ingress

Could you retest with the latest wireless-dev? It should be fixed there.

Thanks,

 Jiri

--

-- 
Jiri Benc
SUSE Labs
Permalink | Reply |
headers
Jan Kiszka | 1 May 12:50
Picon

Re: d80211: How does TX flow control work?

Jiri Benc wrote:
> On Tue, 01 May 2007 12:41:34 +0200, Jan Kiszka wrote:
>> Now I came across this issue once again. It is still present, I just
>> observed it over the latest rt2x00 tree after updating shorewall and
>> forgetting to fix its config.
>>
>> I redirected tc to some logging filter, and this is what shorewall does
>> when "CLEAR_TC=Yes" is set in shorewall.conf:
>>
>> ...
>> tc qdisc del dev wmaster0 root
>> tc qdisc del dev wmaster0 ingress
>> tc qdisc del dev wlan0 root
>> tc qdisc del dev wlan0 ingress
> 
> Could you retest with the latest wireless-dev? It should be fixed there.

Could you point me to the patch(es) in question? I only have 2MBit/s
downlink, so that downloading only the patch would speed things up here.

Jan
Permalink | Reply |
headers
Johannes Berg | 1 May 13:04
Favicon

Re: d80211: How does TX flow control work?

On Tue, 2007-05-01 at 12:50 +0200, Jan Kiszka wrote:

> Could you point me to the patch(es) in question? I only have 2MBit/s
> downlink, so that downloading only the patch would speed things up here.

http://git.kernel.org/?p=linux/kernel/git/linville/wireless-dev.git;a=commit;h=00a908826e778b39a802013729f7826c2d575360

johannes
Permalink | Reply |
headers
Jan Kiszka | 1 May 15:07
Picon

Re: d80211: How does TX flow control work?

Johannes Berg wrote:
> On Tue, 2007-05-01 at 12:50 +0200, Jan Kiszka wrote:
> 
>> Could you point me to the patch(es) in question? I only have 2MBit/s
>> downlink, so that downloading only the patch would speed things up here.
> 
> http://git.kernel.org/?p=linux/kernel/git/linville/wireless-dev.git;a=commit;h=00a908826e778b39a802013729f7826c2d575360
> 

Nope, doesn't help. And I wonder how this piece should do so: It just
performs a sanity check on ifconfig up, right? But the "tc qdisc del"
runs here _after_ wlan0 got configured.

Jan
Permalink | Reply |
headers
Johannes Berg | 1 May 15:26
Favicon

Re: d80211: How does TX flow control work?

On Tue, 2007-05-01 at 15:07 +0200, Jan Kiszka wrote:
> Johannes Berg wrote:
> > On Tue, 2007-05-01 at 12:50 +0200, Jan Kiszka wrote:
> > 
> >> Could you point me to the patch(es) in question? I only have 2MBit/s
> >> downlink, so that downloading only the patch would speed things up here.
> > 
> > http://git.kernel.org/?p=linux/kernel/git/linville/wireless-dev.git;a=commit;h=00a908826e778b39a802013729f7826c2d575360
> > 
> 
> Nope, doesn't help. And I wonder how this piece should do so: It just
> performs a sanity check on ifconfig up, right? But the "tc qdisc del"
> runs here _after_ wlan0 got configured.

I never managed to reproduce that part anyway. I hadn't tried removing
the qdisc when the device is down, but when it's up I never managed
to... odd

johannes
Permalink | Reply |
headers
Jan Kiszka | 1 May 16:18
Picon

Re: d80211: How does TX flow control work?

Johannes Berg wrote:
> On Tue, 2007-05-01 at 15:07 +0200, Jan Kiszka wrote:
>> Johannes Berg wrote:
>>> On Tue, 2007-05-01 at 12:50 +0200, Jan Kiszka wrote:
>>>
>>>> Could you point me to the patch(es) in question? I only have 2MBit/s
>>>> downlink, so that downloading only the patch would speed things up here.
>>> http://git.kernel.org/?p=linux/kernel/git/linville/wireless-dev.git;a=commit;h=00a908826e778b39a802013729f7826c2d575360
>>>
>> Nope, doesn't help. And I wonder how this piece should do so: It just
>> performs a sanity check on ifconfig up, right? But the "tc qdisc del"
>> runs here _after_ wlan0 got configured.
> 
> I never managed to reproduce that part anyway. I hadn't tried removing
> the qdisc when the device is down, but when it's up I never managed
> to... odd
> 

Already tried this lethal command? Works "fine" here...

#tc qdisc del dev wmaster0 root

Jan
Permalink | Reply |
headers
Jiri Benc | 1 May 17:25
Picon

Re: d80211: How does TX flow control work?

On Tue, 01 May 2007 16:18:52 +0200, Jan Kiszka wrote:
> Already tried this lethal command? Works "fine" here...
> 
> #tc qdisc del dev wmaster0 root

I cannot reproduce it too.

Could you please (_without_ applying the attached patch):

- reproduce it with the latest wireless-dev (sorry, there were some changes
  in the networking core, we need to be sure this is not fixed already)
- send the dmesg output and/or any relevant logs?

If the problem still persists, could you please try again with the
following patch applied?

(Btw, you don't need to clone wireless-dev, just make another branch in
your local git tree and pull wireless-dev into it. That way you should
download a few hundred kilobytes only.)

Thanks,

 Jiri

Subject: [PATCH] mac80211: disallow transmitting when 802.11 qdisc is removed

Signed-off-by: Jiri Benc <jbenc@...>

---
 net/mac80211/ieee80211.c |    9 +++++++++
(Continue reading)

Permalink | Reply |
headers
Michael Wu | 1 May 19:01

Re: d80211: How does TX flow control work?

On Tuesday 01 May 2007 11:25, Jiri Benc wrote:
> On Tue, 01 May 2007 16:18:52 +0200, Jan Kiszka wrote:
> > Already tried this lethal command? Works "fine" here...
> >
> > #tc qdisc del dev wmaster0 root
>
> I cannot reproduce it too.
>
I can reproduce it. I thought qdiscs couldn't be changed without taking the 
interface down first.. which is not true. It merely waits for things to quiet 
down before swapping out the qdisc.

Some code in ieee80211_master_start_xmit probably will be needed to correctly 
handle things if the ieee80211 qdisc isn't installed. I don't see an easy way 
to prevent things from working all the time if the qdisc is removed.

-Michael Wu
Permalink | Reply |
headers
Jan Kiszka | 1 May 20:57
Picon

Re: d80211: How does TX flow control work?

Michael Wu wrote:
> On Tuesday 01 May 2007 11:25, Jiri Benc wrote:
>> On Tue, 01 May 2007 16:18:52 +0200, Jan Kiszka wrote:
>>> Already tried this lethal command? Works "fine" here...
>>>
>>> #tc qdisc del dev wmaster0 root
>> I cannot reproduce it too.
>>
> I can reproduce it.

Fine, that save /me some time.

> I thought qdiscs couldn't be changed without taking the 
> interface down first.. which is not true. It merely waits for things to quiet 
> down before swapping out the qdisc.
> 
> Some code in ieee80211_master_start_xmit probably will be needed to correctly 
> handle things if the ieee80211 qdisc isn't installed. I don't see an easy way 
> to prevent things from working all the time if the qdisc is removed.

I didn't followed the discussion in details, but wasn't there once a
suggestion to hide netdevices from the user so that there would be no
chance to stumble over this issue?

Jan
Permalink | Reply |

Gmane