Search Linux Wireless

Re: [PATCH 2/3] iwlwifi: remove STATUS_ALIVE checking from rf_kill

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

 



On Thu, 2009-03-26 at 10:50 -0700, Helmut Schaa wrote:
> Hi,
> 
> Am Donnerstag, 26. März 2009 schrieb Reinette Chatre:
> > From: Wey-Yi Guy <wey-yi.w.guy@xxxxxxxxx>
> > 
> > Remove STATUS_ALIVE checking when HW RF KILL disabled, the bit get
> > clear in __iwl_down() function; the additional checking will fail and
> > cause RF can not be turn back on.
> 
> Are you sure this is needed? I'd argue we should only restart the adapter
> if it was alive when it got rf_killed. In case the adapter was rf_killed
> while the interface was down I don't think we want to restart the adapter
> immediately but first when the interface is taken up again.

We also need to consider if a suspend/resume happens in the middle.
Without the patch, if you enable rfkill, suspend, resume, disable
rfkill, then your interface cannot be brought up.

Reinette

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