Hi!
> This is not correct, because we want the power button to be configurable.
> The user should be able to redefine the power button's action, perhaps to
> only sleep the system. We currently surface button events to acpid, which
> then can do the right thing, including a shutdown -h now (which I assume
> notifies init).
There's no problem with configurability -- you can configure init as
well. I saw it pretty much analogic to situation with Ctrl-Alt-Del: it
also sends signal to init. Init then decides what to do. [I believe
requiring acpid for such easy stuff is not neccessary...]
Pavel
> Regards -- Andy
>
> > From: Pavel Machek [mailto:pavel@suse.cz]
> > Init should get to know that user pressed power button (so it can do
> > shutdown and poweroff). Plus, it is nice to let user know that we can
> > read such event. [I hunted bug for few hours, thinking that kernel
> > does not get the event at all].
> >
> > Here's patch to do that. Please apply,
> > Pavel
> >
> > diff -urb -x .dep* -x .hdep* -x *.[oas] -x *~ -x #* -x *CVS*
> > -x *.orig -x *.rej -x *.old -x .menu* -x asm -x local.h -x
> > System.map -x autoconf.h -x compile.h -x version.h -x
> > .version -x defkeymap.c -x uni_hash.tbl -x zImage -x vmlinu?*
> > -x TAGS -x bootsect -x *RCS* -x conmakehash -x map -x build
> > -x build -x configure -x *target* -x *.flags -x *.bak
> > clean/drivers/acpi/events/evevent.c
> > linux/drivers/acpi/events/evevent.c
> > --- clean/drivers/acpi/events/evevent.c Sun Apr 1 00:22:57 2001
> > +++ linux/drivers/acpi/events/evevent.c Wed Apr 4 01:08:11 2001
> > @@ -30,6 +30,8 @@
> > #include "acnamesp.h"
> > #include "accommon.h"
> >
> > +#include <linux/signal.h>
> > +
> > #define _COMPONENT EVENT_HANDLING
> > MODULE_NAME ("evevent")
> >
> > @@ -197,14 +172,18 @@
> > if ((status_register & ACPI_STATUS_POWER_BUTTON) &&
> > (enable_register & ACPI_ENABLE_POWER_BUTTON))
> > {
> > + printk ("acpi: Power button pressed!\n");
> > + kill_proc (1, SIGTERM, 1);
> > int_status |= acpi_ev_fixed_event_dispatch
> > (ACPI_EVENT_POWER_BUTTON);
> > }
> >
> > +
> > /* sleep button event */
> >
> > if ((status_register & ACPI_STATUS_SLEEP_BUTTON) &&
> > (enable_register & ACPI_ENABLE_SLEEP_BUTTON))
> > {
> > + printk("acpi: Sleep button pressed!\n");
> > int_status |= acpi_ev_fixed_event_dispatch
> > (ACPI_EVENT_SLEEP_BUTTON);
> > }
> >
> >
> > --
> > I'm pavel@ucw.cz. "In my country we have almost anarchy and I
> > don't care."
> > Panos Katsaloulis describing me w.r.t. patents at
> > discuss@linmodems.org
> >
-- The best software in life is free (not shareware)! Pavel GCM d? s-: !g p?:+ au- a--@ w+ v- C++@ UL+++ L++ N++ E++ W--- M- Y- R+ - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
This archive was generated by hypermail 2b29 : Sun Apr 15 2001 - 21:00:15 EST