Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 22 Feb 2013 14:45:51 -0800
From:      Adrian Chadd <adrian@freebsd.org>
To:        PseudoCylon <moonlightakkiy@yahoo.ca>
Cc:        freebsd-bugs@freebsd.org, freebsd-wireless@freebsd.org
Subject:   Re: kern/176201: [net80211] [patch] 11n station includes unrelated ht params into ASSOC_REQ packet
Message-ID:  <CAJ-VmonQ3Vi9tGosceoepZ=42zGnvFQ8qy6LLGAvzdY6qxDi4g@mail.gmail.com>
In-Reply-To: <CAFZ_MY%2BpcGqB8U-%2B3RLiNtKEn19D8-a01akv4Vvwx4bPMNFMjQ@mail.gmail.com>
References:  <201302180246.r1I2kocv064092@freefall.freebsd.org> <CAJ-VmonXhBfxKMAD4EqhksqOKwVffQeE=JJ%2BR0qdifwAowcBkA@mail.gmail.com> <CAJ-VmonjWJoJ4CSRvtN7z3LYg=nEngcDnL49TZK96PngTkwRhg@mail.gmail.com> <201302222005.36490.bschmidt@techwires.net> <CAFZ_MY%2BpcGqB8U-%2B3RLiNtKEn19D8-a01akv4Vvwx4bPMNFMjQ@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
On 22 February 2013 13:26, PseudoCylon <moonlightakkiy@yahoo.ca> wrote:

>>> The OP problem is that we're not advertising the right capabilities
>>> when we associate, right?
>>
>> Correct.
>
> I didn't patch it right, but all of us agree on sta isn't sending
> correct param at association. With current code, sta sends back
> whatever it has received in probe resp packet.

Right, but Bernhard points out the original code did return its own
configuration, and this causes some APs to reject stations.
Hence why it's doing this.

>>> Why aren't we just advertising the VAP ampdumax and ampdudensity no
>>> matter what the operating mode? Why are we capping those parameters to
>>> the learnt-from-AP parameters?
>>
>> Because the AP would otherwise deny the association request.
>
> Should ap only allow node which caps match ap's to associate? (By the
> way, can anyone point me to the code does denial? I couldn't find it.)

I don't think so. The whole point here is to _support_ nodes whose
density/ampdusize is less than what we have configured. The only
reason we'd want to reject them is if we have some concept of "minimum
acceptable density/ampdurx" (like "minimum basic rates"), rather than
"what is it we support."



Adrian



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAJ-VmonQ3Vi9tGosceoepZ=42zGnvFQ8qy6LLGAvzdY6qxDi4g>