Message-ID: <422D737F.2020807@suse.de>
Date: Tue, 08 Mar 2005 10:42:23 +0100
From: Stefan Seyfried <seife@suse.de>
User-Agent: Mozilla Thunderbird 1.0 (X11/20041207)
MIME-Version: 1.0
To: Pavel Machek <pavel@suse.cz>
Cc: "Li, Shaohua" <shaohua.li@intel.com>, Andrew Morton <akpm@zip.com.au>,
       Bruno Ducrot <ducrot@poupinou.org>,
       kernel list <linux-kernel@vger.kernel.org>,
       ACPI mailing list <acpi-devel@lists.sourceforge.net>,
       "Brown, Len" <len.brown@intel.com>
Subject: Re: [ACPI] s4bios: does anyone use it?
References: <16A54BF5D6E14E4D916CE26C9AD305750155EBB0@pdsmsx402.ccr.corp.intel.com> <20050308091856.GB16436@elf.ucw.cz>
In-Reply-To: <20050308091856.GB16436@elf.ucw.cz>
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 8bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1850
Lines: 50

Pavel Machek wrote:
> Hi!
> 
>> >Okay, so we had 2 users in past but have 0 users now? :-).
>> I wonder how could anyone use S4BIOS in 2.6.11. S4 and S4b all came into
>> 'enter_state'. and in acpi_sleep_init:
>> 
>> 		if (i == ACPI_STATE_S4) {
>> 			if (acpi_gbl_FACS->S4bios_f) {
>> 				sleep_states[i] = 1;
>> 				printk(" S4bios");
>> 				acpi_pm_ops.pm_disk_mode =
>> PM_DISK_FIRMWARE;
>> 			}
>> 			if (sleep_states[i])
>> 				acpi_pm_ops.pm_disk_mode =
>> PM_DISK_PLATFORM;
>> 		}
>> That means we actually can't set PM_DISK_FIRMWARE (always set
>> PM_DISK_PLATFORM). Is this intended? If no, .pm_disk_mode should be a
>> mask.
> 
> pm_disk_mode is settable using /sys/power/disk, no?

No, it isn't. That was my original point: you can write "firmware" into
it, but it has no effect. This probably was a side-effect of the "make
firmware mode not default" patch from a year ago.
But the real question is: what is firmware mode good for today? Is there
a single machine where firmware mode once worked, but swsusp does not
work today?

> Anyway, what about this, then?
> 
> --- clean/Documentation/feature-removal-schedule.txt	2005-01-22 21:24:50.000000000 +0100
> +++ linux/Documentation/feature-removal-schedule.txt	2005-03-08 10:18:05.000000000 +0100

Fine with me. I think it cannot work since ~one year (when we changed
the default from "firmware if available" to "shutdown always", the code
piece cited above) and nobody complained until now, so it won't be
missed IMO.
-- 
Stefan Seyfried, QA / R&D Team Mobile Devices, SUSE LINUX N?rnberg.

"Any ideas, John?"
"Well, surrounding them's out."
-
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/