[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [PATCH v2 1/3] Add per-guest S3/S4 state configuration



On Thu, Aug 2, 2012 at 3:36 PM, Eric Blake <eblake redhat com> wrote:
> On 08/02/2012 06:05 AM, Martin Kletzander wrote:
>> There is a new <pm/> element implemented that can control what ACPI
>> sleeping states will be advertised by BIOS and allowed to be switched
>> to by libvirt. The default keeps defaults on hypervisor, otherwise
>> forces chosen setting.
>
> You are proposing /domain/pm; but we also have /domain/os/bios, would
> this be better as a subelement /domain/os/bios/pm, since it is related
> to bios options?

I would say that /domain/os/bios/pm isn't the correct place because
/domain/os relates only to OS booting while Power Management is
outside of the scope of booting. In fact /domain/os/bios simply
controls whether the BIOS output will be over VGA or over serial.
Features specific to the system are typically kept as top level items,
which matches /domain/pm. I would however say that using the terms
ACPI, s3 and s4 isn't good design for the XML. You can leave that to
documentation but /domain/pm/suspend-to-mem and
/domain/pm/suspend-to-disk are better so that the same XML structure
can be used for PPC. They have concepts similar to s3 and s4 (I only
briefly Googled this) but they're named completely differently. The
only under place I could see this would be /domain/features/acpi. If
its there then s3 and s4 potentially make sense, but then we're left
without a place for PPC to tie in.

-- 
Doug Goldstein


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]