Search Linux Wireless

Re: [PATCH 03/11] iwlwifi: fix hidden ssid discovery in passive channels

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Tomas,

> >> >> This patch gives the HW the possibility to send direct probes in passive
> >> >> channels (as long as traffic was detected in that channel) if such ssid
> >> >> was requested, so hidden ssid can be found now in those channels as well.
> >> >
> >> > this also doesn't sound like a regression to me. Nice to have for sure,
> >> > but not acceptable according to the merge window rules.
> >>
> >> If you cannot scan your AP you also not able to connect to it, drawing
> >> your card useless.
> >>
> >> > Question here is was the driver supporting this before and we broke it
> >> > in the 2.6.27 phase or not.
> >>
> >> Irrelevant, this is fixing show stopper bug.
> >
> > it is relevant. So I read this commit message as, that now we can also
> > find hidden SSIDs in passive channels. Question that you should ask then
> > is if that was supported before. Yes or no. If yes, then it is a
> > regression, no then it is a feature. And regression fixes go in features
> > wait for the next merge window.
> >
> > The term show stopper bug is irrelevant when it comes to merge window
> > rules.
> >
> I don't know if it worked before or not I didn't check. This is new code.

is this new code for new hardware or did we also use it in the 4965
case? And yes, we need to check 2.6.26 here. And then state this clearly
in the commit message.

Please also see this from the point of people that actually have to or
wanna backport patches. The better the commit message, the easier their
job to make good decisions on which patches to take.

Regards

Marcel


--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]
  Powered by Linux