lkml.org 
[lkml]   [2007]   [Feb]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    SubjectRe: 2.6.21-rc1: known regressions (v2) (part 1)
    Date
    27 Şub 2007 Sal tarihinde, Fabio Comolli şunları yazmıştı: 
    > Confirmed, although the problem I see is probably different from
    > Ismael's one: in my case /proc/acpi/adapter/AC is present but
    > kpowersave does not work (it works in 2.6.20).
    >
    > The only file I seem to be missing is /proc/acpi/info, but I don't
    > know it is important or not.

    Then these problems are not same, İsmail's problem was an ACPI one and im sure
    that solved but yours seems a userspace problem (hal checks /proc/acpi/info
    and kpowersave uses hal) introduced by "/proc/acpi/info deprecated
    by /sys/firmware/acpi/info" [1]. You can try untested attached patch against
    hal-git tree, so i'm adding hal list into CC also.

    [1] http://www.mail-archive.com/linux-acpi@vger.kernel.org/msg04285.html
    --
    S.Çağlar Onur <caglar@pardus.org.tr>
    http://cekirdek.pardus.org.tr/~caglar/

    Linux is like living in a teepee. No Windows, no Gates and an Apache in house!
    diff --git a/hald/linux/acpi.c b/hald/linux/acpi.c
    index 30e5fdf..1714627 100644
    --- a/hald/linux/acpi.c
    +++ b/hald/linux/acpi.c
    @@ -1055,7 +1055,7 @@ acpi_synthesize_hotplug_events (void)
    HalDevice *computer;
    gchar path[HAL_PATH_MAX];

    - if (!g_file_test ("/proc/acpi/info", G_FILE_TEST_EXISTS))
    + if (!g_file_test ("/proc/acpi/", G_FILE_TEST_IS_DIR))
    return FALSE;

    if ((computer = hal_device_store_find (hald_get_gdl (), "/org/freedesktop/Hal/devices/computer")) == NULL &&
    @@ -1066,8 +1066,14 @@ acpi_synthesize_hotplug_events (void)

    /* Set appropriate properties on the computer object */
    hal_device_property_set_string (computer, "power_management.type", "acpi");
    - hal_util_set_string_elem_from_file (computer, "power_management.acpi.linux.version",
    + if (g_file_test ("/proc/acpi/info", G_FILE_TEST_EXISTS)) {
    + hal_util_set_string_elem_from_file (computer, "power_management.acpi.linux.version",
    "/proc/acpi", "info", "version", 0, FALSE);
    + }
    + else {
    + hal_util_set_string_elem_from_file (computer, "power_management.acpi.linux.version",
    + "%s/firmware/acpi", sysfs_path, "info", "version", 0, FALSE);
    + }

    /* collect batteries */
    snprintf (path, sizeof (path), "%s/acpi/battery", get_hal_proc_path ());[unhandled content-type:application/pgp-signature]
    \
     
     \ /
      Last update: 2007-02-27 19:47    [W:0.022 / U:60.680 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site