Search Linux Wireless

Re: [PATCH v11] rfkill: rewrite

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

 



On Wed, 27 May 2009, Johannes Berg wrote:
> On Tue, 2009-05-26 at 23:55 -0300, Henrique de Moraes Holschuh wrote:
> > As long as the missing userspace control functionality (the lack of which is
> > a _serious_ regression for platform drivers) is added before 2.6.31 ships, I
> > have no complains.
> > 
> > Userspace must be able to change the (soft) state using sysfs.  This is
> > not an optional part of rfkill :-)
> 
> I'm sorry, but you're wrong. Before I removed it, almost all drivers
> disabled that functionality. We do want to add it back, but certainly
> not with the mostly braindead interface it used to have.

Well, thinkpad-acpi has users of that functionality, and not just that, it
is in the middle of a two year effort to deprecate the thinkpad-acpi
specific driver attributes and switch to just rfkill.   This effort will be
destroyed if a kernel releases without support to writes to the "state"
attribute.

I need that userspace interface (writes to the state attribute) in place and
working.  It _is_ a big regression for thinkpad-acpi to have it return
-EOPNOTSUPP.

Also, regardless of the "claim" attribute (and the related
user_claim_unsupported nonsense), the old core would allow the user to
change the soft state using sysfs, so I really don't understand what you
mean when you say "almost all drivers disabled that functionality", you
can't disable writes to the "state" attribute in the old core from what I
can see.  Please correct me if I am wrong on this.

-- 
  "One disk to rule them all, One disk to find them. One disk to bring
  them all and in the darkness grind them. In the Land of Redmond
  where the shadows lie." -- The Silicon Valley Tarot
  Henrique Holschuh
--
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