lkml.org 
[lkml]   [2001]   [Apr]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: Let init know user wants to shutdown
    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/

    \
     
     \ /
      Last update: 2005-03-22 13:25    [W:2.697 / U:0.016 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site