From: Dan Williams <dcbw@redhat.com>
To: Holger Schurig <hs4233@mail.mn-solutions.de>
Cc: linux-wireless@vger.kernel.org, libertas-dev@lists.infradead.org
Subject: Re: [PATCH] libertas: automatically re-associate
Date: Tue, 09 Oct 2007 11:37:36 -0400 [thread overview]
Message-ID: <1191944257.20014.3.camel@localhost.localdomain> (raw)
In-Reply-To: <200710091713.35595.hs4233@mail.mn-solutions.de>
On Tue, 2007-10-09 at 17:13 +0200, Holger Schurig wrote:
> > Seriously though, even if other drivers don't do it, that
> > doesn't mean libertas shouldn't do it. If the association
> > fails, doesn't this mean the card just keeps working and keeps
> > trying to associate forever?
>
> Yes, with a HZ/4 rate.
I don't care how _fast_ it tries to reassociate. I care that it doesn't
give up after some amount of time.
> Maybe in the meantime the user is configuring the right WEP key
> into his AP?
>
>
>
> > It's not that much more complicated to add an item to the
> > association structure for 'count' and to check that from the
> > disassociation event code, and just not reschedule association
> > if that's > 4.
>
> And after this 4 tries the device is again stuck? That's not
> nice.
>
> Therefore you need some heuristic (I don't dare to name this
> logic) to reset this counter again. But when do you do it? If
> you say "5 Minutes", then I simply change my HZ/4 into HZ*60*5
> and have the same effect.
>
> So I see that my HZ/4 is as arbitrary as your 4, but my version
> is fail-safe, because when you move away from a mis-configured
> AP to a right on, the device will automatically re-associate.
> You see, I aim for a roaming client, not for a device that stays
> put at some office desktop.
That's fine. The drivers should _not_ be trying to handle all roaming
cornercases themselves. And I believe that includes forever trying
reassociating. Userspace must come into the equation at some point,
especially with WPA!! This reassociation trick _only_ works for WEP.
And that immediately makes it less useful.
My point is that the _driver_ should only go so far in trying to guess
the intentions of the user and userspace itself. It's not the drivers
job to cover every conceivable use-case, especially where the heuristics
are quite fuzzy. That sort of thing just doesn't belong in drivers.
Dan
>
> We also get disassociated from an AP when we're away from the
> channel for too long, busy scanning for a better AP. The case
> where someone clicks on a "Deassociate Client") is neglibibly.
>
> But even when I want to have it work automatically, and without
> any manual user interaction at the WLAN client.
next prev parent reply other threads:[~2007-10-09 15:37 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-10-09 12:04 [PATCH] libertas: automatically re-associate Holger Schurig
2007-10-09 14:19 ` Dan Williams
2007-10-09 15:13 ` Holger Schurig
2007-10-09 15:37 ` Dan Williams [this message]
2007-10-10 7:43 ` Holger Schurig
2007-10-10 14:58 ` Dan Williams
2007-10-10 16:59 ` John W. Linville
2007-10-11 4:00 ` Dan Williams
2007-10-10 19:00 ` Holger Schurig
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=1191944257.20014.3.camel@localhost.localdomain \
--to=dcbw@redhat.com \
--cc=hs4233@mail.mn-solutions.de \
--cc=libertas-dev@lists.infradead.org \
--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