public inbox for linux-wireless@vger.kernel.org
 help / color / mirror / Atom feed
From: Johannes Berg <johannes@sipsolutions.net>
To: Henrique de Moraes Holschuh <hmh@hmh.eng.br>
Cc: linux-wireless <linux-wireless@vger.kernel.org>
Subject: Re: rfkill-input madness
Date: Fri, 27 Mar 2009 22:10:37 +0100	[thread overview]
Message-ID: <1238188237.4452.17.camel@johannes.local> (raw)
In-Reply-To: <20090327143007.GA24288@khazad-dum.debian.net>

[-- Attachment #1: Type: text/plain, Size: 2840 bytes --]

On Fri, 2009-03-27 at 11:30 -0300, Henrique de Moraes Holschuh wrote:

> I think rephrasing that warning to: "set user_claim to 1 on any switches
> that you're going to mess with in response to rfkill input events" would be
> a LOT better.  I should have written it that way.

Indeed, but that's useless since almost all drivers disable userspace
claiming... I'll re-implement it later for only the software state.

> The truth is that userspace doestn't have to care about rfkill-input, unless
> it is trying to do what rfkill-input is supposed to (i.e. listening to input
> events and then trying to update switches), and if it is going to do that,
> it needs stuff that I never sent in for review.

Care to explain?

> Anyway, I got sidetracked because I was Not Happy with the userspace ABI but
> couldn't come up with anything better.  

It's not like we can change it now...

> Better at least get those patches
> into the open.  I just pushed a rebased version of the patches to somewhere
> public.
> 
> It is at:
> git://repo.or.cz/linux-2.6/linux-acpi-2.6/ibm-acpi-2.6.git rfkill
> 
> Please take a look and tell me what you think, and whether there is anything
> worth bothering with in there (FWIW, the patchset worked just fine last time
> I checked).

Ok, when I get online again :)

> As for your second question, I have been looking at the issues re. input
> devices and X.org evdev in the last two days, and the fact is that: it will
> have to be handled in userspace on most non-embedded scenarios, if things
> don't change in userspace soon.
> 
> The problem is that X.org will exclusive-grab any input devices you let it
> touch, and rfkill_input will never see any events, anyway.   So, right now,
> for the vast majority of the users, either an input device is not in use, or
> it is in exclusive-grab mode feeding X.org evdev.
> 
> At this point, I don't have a clue on what would be the best way to go about
> addressing this issue, that is causing problems not only to rfkill_input,
> but also to anyone who would like to have hotkey handling outside of X.org
> in a system daemon, but still have x.org see the keys (without ugly 'event
> repeater' hacks).
> 
> I have no idea if we should talk to the X.org people to see if we can drop
> that exclusive grab?  Allow kernel-side input handlers to ignore exclusive
> grabs?  Have rfkill-input be a you-want-it-in-the-kernel solution and simply
> don't care at all about userspace interfering with it?
> 
> I was about to try to locate someone that deals with evdev to ask his ideas
> on the subject.

Indeed, but that means *X* needs to implement all this, or something
running in X, which is undesirable... Other things are sure to run afoul
of this too, for example some hal button handlers.

johannes

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

  reply	other threads:[~2009-03-27 21:11 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-03-27 13:06 rfkill-input madness Johannes Berg
2009-03-27 14:30 ` Henrique de Moraes Holschuh
2009-03-27 21:10   ` Johannes Berg [this message]
2009-03-28  0:52     ` Henrique de Moraes Holschuh
2009-03-28 17:50       ` Johannes Berg
2009-03-30 13:23         ` Henrique de Moraes Holschuh
2009-03-30 14:11           ` Johannes Berg
2009-03-30 17:21             ` Henrique de Moraes Holschuh
2009-03-30 17:29               ` Johannes Berg

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=1238188237.4452.17.camel@johannes.local \
    --to=johannes@sipsolutions.net \
    --cc=hmh@hmh.eng.br \
    --cc=linux-wireless@vger.kernel.org \
    /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