Search Linux Wireless

Re: [RFC 2/5] cfg80211: add support to immediately apply reg settings

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

 



On Thu, Nov 24, 2011 at 11:50 AM, Luis R. Rodriguez
<mcgrof@xxxxxxxxxxxxx> wrote:
> On Thu, Nov 24, 2011 at 11:44 AM, Johannes Berg
> <johannes@xxxxxxxxxxxxxxxx> wrote:
>> On Wed, 2011-11-23 at 10:40 -0500, Luis R. Rodriguez wrote:
>>> It is useful to apply reguatory settings immediately. We already
>>> handle applying regulatory hints immediatey for most hints but
>>> for user hints and country IE hints we want to kick hardware
>>> immediately to make these changes take effect. This adds supports
>>> to let us kick hardware to make the changes take effect as soon
>>> as possible.
>>
>> Can't say I like this -- wouldn't we rather call the regulatory update
>> callback or so?
>
> I'm not a fan either, at first I considered doing that on the
> reg_notifier() but then that also means every driver must implement a
> reg_notifier() callback, or a new reg_apply() callback. If we do it
> within cfg80211 it means all drivers get this for free and that was
> the tradeoff I wanted to review through these RFCs. Unfortunately
> setting the tx power was the only thing I saw cfg80211 could do right
> now.
>
> I'm still "meh" about this too.

I'm willing to give up on the complexity of this given that we *will*
eventually apply the settings after a scan, etc. The gains seem to be
small for what we would need to do. If we're happy to live without the
optimization I can just document it to ensure everyone knows that the
power stuff won't apply *immediately* but it will happen eventually
and typically this is soon enough.

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