Search Linux Wireless

Re: [PATCH V2 1/2] cfg80211: Define a wrapper function for reporting roaming

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

 



On Wed, Dec 14, 2011 at 01:54:49PM -0500, John W. Linville wrote:
> On Wed, Dec 07, 2011 at 08:35:45PM +0530, Vasanthakumar Thiagarajan wrote:
> > On Wed, Dec 07, 2011 at 07:28:24PM +0530, Vasanthakumar Thiagarajan wrote:
> > > > 
> > > > Hmm, now that I think about it more, I think the calling convention is
> > > > really strange. Why pass the channel and bssid when the BSS struct is
> > > > already known?
> > > > 
> > > > Why not make cfg80211_roamed() do the BSS lookup (which would also
> > > > reduce the race for drivers that don't use _bss()) based on the info,
> > > > and then call a cfg80211_roamed_bss() that doesn't get channel/bssid.
> > > 
> > > Sure, i'll change it. This looks better.
> > 
> > Also, i'll create a new patch instead of the next version.
> 
> Is that patch still coming?

Please drop this, a different patch (cfg80211: Fix race in bss
timeout) which fixes the same issue has been sent.

Vasanth
--
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