From: Dmitry Torokhov <dmitry.torokhov@gmail.com>
To: Joey Lee <jlee@novell.com>
Cc: carlos@strangeworlds.co.uk, corentin.chary@gmail.com,
Takashi Iwai <tiwai@novell.com>,
Thomas Renninger <trenn@novell.com>,
mjg59@srcf.ucam.org, jbenc@suse.cz, linux-input@vger.kernel.org,
platform-driver-x86@vger.kernel.org
Subject: Re: [PATCH 2/3] Support enable Acer Launch Manager mode
Date: Sun, 17 Oct 2010 21:07:28 -0700 [thread overview]
Message-ID: <20101018040728.GA7782@core.coreip.homeip.net> (raw)
In-Reply-To: <4CBC4C660200002300022008@novprvlin0050.provo.novell.com>
On Sun, Oct 17, 2010 at 08:32:22PM -0600, Joey Lee wrote:
> Hi Carios,
>
> 於 五,2010-10-15 於 18:11 +0100,Carlos Corbacho 提到:
> > On Friday 15 October 2010 09:02:01 Joey Lee wrote:
> > > > Why would user chose one setup over another? I.e do we really need a
> > > > module option or maybe we should pick up behavior and stick to it?
> > >
> > > When Acer notebook ship with Windows, they will preload a userland
> > > application, the name is Launch Manager, it provide a GUI for end user
> > > to change WLAN/BT/3G on/off state.
> >
> > [...]
> >
> > > Because the Launch Manager is userland application, so wmi driver only
> > > can provide the function for it to call and don't have any way can
> > > detect it from kernel module.
> >
> > Dmitry's point is still valid - why do we want to provide the userspace
> > behaviour? Why not just handle it all in kernel using rfkill?
> >
Apparently networkign people is not quite happy with rfkill-input
behavior and would like to move policy from the kernel into userspace.
To tell the truth rfkill-input was never supposed to be the main
mechanism but rather something that is usable at bootup and then the
control would be handed off to userspace (if more complex policy is
needed).
>
> Did you mean put the wifi hotkey behavior to acer-wmi driver or any x86
> laptop driver and implement by using rfkill?
>
> The wifi hotkey behavior is highly customization and different OEM have
> different behavior. I am not sure put the wifi hotkey rule in kernel is
> a good idea.
OEMs have nothing to do with this, user is the entity that should be in
control.
>
> In the future, there need have a userland rfkill policy daemon to
> replace the rfkill-input module in kernel:
>
> Documentation/rfkill.txt
> The rfkill-input will be:
> * the deprecated rfkill-input module (an input layer handler, being
> replaced by userspace policy code) and
>
> And here have a statement in feature-removal-schedule.txt
> http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commitdiff;h=c64fb01627e24725d1f9d535e4426475a4415753#patch1
>
> +What: CONFIG_RFKILL_INPUT
> +When: 2.6.33
> +Why: Should be implemented in userspace, policy daemon.
> +Who: Johannes Berg <johannes@sipsolutions.net>
>
> So, we choice remove rfkill-input then put the logic in x86/platform
> driver?
> A simple question:
> Userland policy daemon or kernel module, which one we want to put the
> wifi hotkey behavior implementation?
>
> > We don't have Launch Manager for Linux, and quite frankly, I hope we never see
> > it - relying on random, vendor specific applications to drive this kind of
> > functionality is just asking for trouble.
> >
>
> Acer BIOS team provide the function to OS for disable the EC hehavior,
> it's available on window, why we hide it on Linux?
> Either userland daemon or kernel module who want to implement the wifi
> hotkey behavior, it need enable the launch-manager mode to disable the
> default EC behavior on wifi hotkey.
>
> If don't want provide the launch-manager mode parameter to userland, can
> we just direct enable it?
>
I believe that we simply need to tell EC to keep its hands off RF
switch. As long as key is properly reported through input layer rfkill
should be able to control the RF equipment, either via rfkill-input
in-kernel shortcut or userspace daemon (when it is ready).
Thanks.
--
Dmitry
next prev parent reply other threads:[~2010-10-18 4:07 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-10-18 2:32 [PATCH 2/3] Support enable Acer Launch Manager mode Joey Lee
2010-10-18 4:07 ` Dmitry Torokhov [this message]
2010-10-18 7:19 ` Carlos Corbacho
2010-10-18 7:55 ` Corentin Chary
2010-10-18 8:14 ` Dmitry Torokhov
2010-10-18 8:40 ` Corentin Chary
-- strict thread matches above, loose matches on Subject: below --
2010-10-30 16:28 Joey Lee
2010-10-21 3:35 Joey Lee
2010-10-19 13:25 Joey Lee
2010-10-19 13:28 ` Corentin Chary
2010-10-19 15:49 ` Dmitry Torokhov
2010-10-19 16:03 ` Corentin Chary
2010-10-19 16:49 ` Dmitry Torokhov
2010-10-19 11:15 Joey Lee
2010-10-19 13:07 ` Corentin Chary
2010-10-19 9:55 Joey Lee
2010-10-19 10:51 ` Corentin Chary
2010-10-19 8:09 Joey Lee
2010-10-19 2:48 Joey Lee
2010-10-18 4:53 Joey Lee
2010-10-18 8:20 ` Dmitry Torokhov
2010-10-15 8:02 Joey Lee
2010-10-15 17:11 ` Carlos Corbacho
2010-10-13 3:47 [PATCH 1/3] Add acer wmi hotkey events support Lee, Chun-Yi
2010-10-13 3:47 ` [PATCH 2/3] Support enable Acer Launch Manager mode Lee, Chun-Yi
2010-10-13 18:26 ` Dmitry Torokhov
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20101018040728.GA7782@core.coreip.homeip.net \
--to=dmitry.torokhov@gmail.com \
--cc=carlos@strangeworlds.co.uk \
--cc=corentin.chary@gmail.com \
--cc=jbenc@suse.cz \
--cc=jlee@novell.com \
--cc=linux-input@vger.kernel.org \
--cc=mjg59@srcf.ucam.org \
--cc=platform-driver-x86@vger.kernel.org \
--cc=tiwai@novell.com \
--cc=trenn@novell.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).