From: "Mattia Dongili" <malattia@linux.it>
To: Dmitry Torokhov <dtor_core@ameritech.net>
Cc: "Brown, Len" <len.brown@intel.com>, linux-acpi@vger.kernel.org
Subject: Re: [PATCH 0/2] Deliver ACPI events upon subscription and implement multiple readers for /proc/acpi/event
Date: Wed, 25 Jan 2006 10:54:51 +0100 (CET) [thread overview]
Message-ID: <10537.83.103.117.254.1138182891.squirrel@picard.linux.it> (raw)
In-Reply-To: <200601250023.01359.dtor_core@ameritech.net>
On Wed, January 25, 2006 6:23 am, Dmitry Torokhov said:
> On Wednesday 25 January 2006 00:09, Brown, Len wrote:
[...]
> If you remember original code (without registering multiple in-kernel
> listeners) I think it was more compact that what is in the kernel
> at the moment. Also IIRC current in kernel code may start filling memory
> if acpid stops reading events for some reason.
>
> For the record I think multiple listeners are not needed since the only
> possible user is input layer and I firmly believe that ACPI should report
> keys/buttons using input layer natively so userspace can get uniform
> notification of "Sleep" button being pressed no matter whther that button
> is controlled by ACPI or it is just another key on USB keyboard.
I have no problem changing the implementation to avoid useless stuff if
the patch is going to be considered for acceptance.
>> The other argument against is why enhance an interface when
>> perhaps we should instead consider replacing it altogether...
The same thing has been said at Dmitry's submission (08/2004) and
/proc/acpi/event is still there. :)
Also, removing /proc/acpi/event will take a very long time.
Anyway what would you suggest (if you have already some plan)? I'd like to
try implementing it.
--
mattia
:wq!
next prev parent reply other threads:[~2006-01-25 9:54 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-01-25 5:09 [PATCH 0/2] Deliver ACPI events upon subscription and implement multiple readers for /proc/acpi/event Brown, Len
2006-01-25 5:23 ` Dmitry Torokhov
2006-01-25 9:54 ` Mattia Dongili [this message]
2006-01-26 17:38 ` Thomas Renninger
-- strict thread matches above, loose matches on Subject: below --
2006-01-25 1:17 Brown, Len
2006-01-25 1:46 ` Dmitry Torokhov
2006-01-25 8:16 ` Mattia Dongili
2006-01-24 23:14 Mattia Dongili
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=10537.83.103.117.254.1138182891.squirrel@picard.linux.it \
--to=malattia@linux.it \
--cc=dtor_core@ameritech.net \
--cc=len.brown@intel.com \
--cc=linux-acpi@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