[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 26/74] pc: acpi: memhp: move MHPD._STA method
From: |
Igor Mammedov |
Subject: |
Re: [Qemu-devel] [PATCH v2 26/74] pc: acpi: memhp: move MHPD._STA method into SSDT |
Date: |
Mon, 21 Dec 2015 18:31:40 +0100 |
On Sun, 20 Dec 2015 15:41:22 +0200
"Michael S. Tsirkin" <address@hidden> wrote:
> On Wed, Dec 16, 2015 at 03:47:35PM +0100, Igor Mammedov wrote:
[...]
> > + method = aml_method("_STA", 0, AML_NOTSERIALIZED);
> > + ifctx = aml_if(aml_equal(a_slots_nr, a_zero));
> > + {
> > + aml_append(ifctx, aml_return(a_zero));
> > + }
> > + aml_append(method, ifctx);
> > + /* present, functioning, decoding, not shown in UI */
> > + aml_append(method, aml_return(aml_int(0xB)));
> > + aml_append(ctrl_dev, method);
>
>
> Simple unserialized methods that return value depending on input
> being zero seem very common. How about a helper for this case?
>
> E.g.
>
> aml_method_if_then_else(slots_nr, aml_int(0xB), aml_int(0));
I've looked through all _STA methods that exist in DSDT
and though this method for mem hotplug doesn't belong to common pattern,
I could use you suggestion for 4 ISA devices.
Or perhaps I can save 2-3 LOC by making common
build_if_else(cond, truectx, false_ctx);
I'll try both but it looks like build_if_else() will be more useful.
And after this series is applied there is a plenty to optimize,
for example most of these _STA methods will be gone or replaced
by simple _STA = FOO expression.
>
> > }
> > aml_append(pci_scope, ctrl_dev);
> > aml_append(ctx, pci_scope);
> > diff --git a/hw/i386/acpi-dsdt-mem-hotplug.dsl
> > b/hw/i386/acpi-dsdt-mem-hotplug.dsl
> > index c2bb6a1..b4eacc9 100644
> > --- a/hw/i386/acpi-dsdt-mem-hotplug.dsl
> > +++ b/hw/i386/acpi-dsdt-mem-hotplug.dsl
> > @@ -35,14 +35,6 @@
> > External(MEMORY_SLOT_OST_EVENT, FieldUnitObj) // _OST event
> > code, write only
> > External(MEMORY_SLOT_OST_STATUS, FieldUnitObj) // _OST status
> > code, write only
> >
> > - Method(_STA, 0) {
> > - If (LEqual(MEMORY_SLOTS_NUMBER, Zero)) {
> > - Return(0x0)
> > - }
> > - /* present, functioning, decoding, not shown in UI */
> > - Return(0xB)
> > - }
> > -
> > Mutex (MEMORY_SLOT_LOCK, 0)
> >
> > Method(MEMORY_SLOT_SCAN_METHOD, 0) {
> > --
> > 1.8.3.1
- Re: [Qemu-devel] [PATCH 25/74] pc: acpi: memhp: prepare context in SSDT for moving memhp DSDT code, (continued)
[Qemu-devel] [PATCH 26/74] pc: acpi: memhp: move MHPD._STA method into SSDT, Igor Mammedov, 2015/12/09
- Re: [Qemu-devel] [PATCH 26/74] pc: acpi: memhp: move MHPD._STA method into SSDT, Marcel Apfelbaum, 2015/12/16
- [Qemu-devel] [PATCH v2 26/74] pc: acpi: memhp: move MHPD._STA method into SSDT, Igor Mammedov, 2015/12/16
- Re: [Qemu-devel] [PATCH v2 26/74] pc: acpi: memhp: move MHPD._STA method into SSDT, Marcel Apfelbaum, 2015/12/20
- Re: [Qemu-devel] [PATCH v2 26/74] pc: acpi: memhp: move MHPD._STA method into SSDT, Michael S. Tsirkin, 2015/12/20
- Re: [Qemu-devel] [PATCH v2 26/74] pc: acpi: memhp: move MHPD._STA method into SSDT, Igor Mammedov, 2015/12/21
- Re: [Qemu-devel] [PATCH v2 26/74] pc: acpi: memhp: move MHPD._STA method into SSDT,
Igor Mammedov <=
- Re: [Qemu-devel] [PATCH v2 26/74] pc: acpi: memhp: move MHPD._STA method into SSDT, Michael S. Tsirkin, 2015/12/22
- Re: [Qemu-devel] [PATCH v2 26/74] pc: acpi: memhp: move MHPD._STA method into SSDT, Igor Mammedov, 2015/12/22
[Qemu-devel] [PATCH 28/74] pc: acpi: memhp: move MHPD.MSCN method into SSDT, Igor Mammedov, 2015/12/09
[Qemu-devel] [PATCH 30/74] pc: acpi: memhp: move MHPD.MPXM method into SSDT, Igor Mammedov, 2015/12/09
[Qemu-devel] [PATCH 16/74] acpi: add aml_lor(), Igor Mammedov, 2015/12/09
[Qemu-devel] [PATCH 10/74] acpi: add aml_call0() helper, Igor Mammedov, 2015/12/09
[Qemu-devel] [PATCH 14/74] acpi: add aml_sleep(), Igor Mammedov, 2015/12/09