[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PULL 50/55] acpi: extend aml_field() to support LockRule
From: |
Michael S. Tsirkin |
Subject: |
[Qemu-devel] [PULL 50/55] acpi: extend aml_field() to support LockRule |
Date: |
Tue, 22 Dec 2015 18:55:00 +0200 |
From: Igor Mammedov <address@hidden>
Signed-off-by: Igor Mammedov <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>
---
include/hw/acpi/aml-build.h | 8 +++++++-
hw/acpi/aml-build.c | 5 ++++-
hw/i386/acpi-build.c | 10 +++++-----
3 files changed, 16 insertions(+), 7 deletions(-)
diff --git a/include/hw/acpi/aml-build.h b/include/hw/acpi/aml-build.h
index e901371..289b4eb 100644
--- a/include/hw/acpi/aml-build.h
+++ b/include/hw/acpi/aml-build.h
@@ -49,6 +49,11 @@ typedef enum {
} AmlAccessType;
typedef enum {
+ AML_NOLOCK = 0,
+ AML_LOCK = 1,
+} AmlLockRule;
+
+typedef enum {
AML_PRESERVE = 0,
AML_WRITE_AS_ONES = 1,
AML_WRITE_AS_ZEROS = 2,
@@ -310,7 +315,8 @@ Aml *aml_while(Aml *predicate);
Aml *aml_package(uint8_t num_elements);
Aml *aml_buffer(int buffer_size, uint8_t *byte_list);
Aml *aml_resource_template(void);
-Aml *aml_field(const char *name, AmlAccessType type, AmlUpdateRule rule);
+Aml *aml_field(const char *name, AmlAccessType type, AmlLockRule lock,
+ AmlUpdateRule rule);
Aml *aml_mutex(const char *name, uint8_t sync_level);
Aml *aml_acquire(Aml *mutex, uint16_t timeout);
Aml *aml_release(Aml *mutex);
diff --git a/hw/acpi/aml-build.c b/hw/acpi/aml-build.c
index fd512a5..5dcaf32 100644
--- a/hw/acpi/aml-build.c
+++ b/hw/acpi/aml-build.c
@@ -952,11 +952,14 @@ Aml *aml_reserved_field(unsigned length)
}
/* ACPI 1.0b: 16.2.5.2 Named Objects Encoding: DefField */
-Aml *aml_field(const char *name, AmlAccessType type, AmlUpdateRule rule)
+Aml *aml_field(const char *name, AmlAccessType type, AmlLockRule lock,
+ AmlUpdateRule rule)
{
Aml *var = aml_bundle(0x81 /* FieldOp */, AML_EXT_PACKAGE);
uint8_t flags = rule << 5 | type;
+ flags |= lock << 4; /* LockRule at 4 bit offset */
+
build_append_namestring(var->buf, "%s", name);
build_append_byte(var->buf, flags);
return var;
diff --git a/hw/i386/acpi-build.c b/hw/i386/acpi-build.c
index 35ea92d..4e9855a 100644
--- a/hw/i386/acpi-build.c
+++ b/hw/i386/acpi-build.c
@@ -1130,7 +1130,7 @@ build_ssdt(GArray *table_data, GArray *linker,
aml_append(dev, aml_operation_region("PEOR", AML_SYSTEM_IO,
misc->pvpanic_port, 1));
- field = aml_field("PEOR", AML_BYTE_ACC, AML_PRESERVE);
+ field = aml_field("PEOR", AML_BYTE_ACC, AML_NOLOCK, AML_PRESERVE);
aml_append(field, aml_named_field("PEPT", 8));
aml_append(dev, field);
@@ -1170,7 +1170,7 @@ build_ssdt(GArray *table_data, GArray *linker,
/* declare CPU hotplug MMIO region and PRS field to access it */
aml_append(sb_scope, aml_operation_region(
"PRST", AML_SYSTEM_IO, pm->cpu_hp_io_base, pm->cpu_hp_io_len));
- field = aml_field("PRST", AML_BYTE_ACC, AML_PRESERVE);
+ field = aml_field("PRST", AML_BYTE_ACC, AML_NOLOCK, AML_PRESERVE);
aml_append(field, aml_named_field("PRS", 256));
aml_append(sb_scope, field);
@@ -1245,7 +1245,7 @@ build_ssdt(GArray *table_data, GArray *linker,
);
field = aml_field(stringify(MEMORY_HOTPLUG_IO_REGION), AML_DWORD_ACC,
- AML_PRESERVE);
+ AML_NOLOCK, AML_PRESERVE);
aml_append(field, /* read only */
aml_named_field(stringify(MEMORY_SLOT_ADDR_LOW), 32));
aml_append(field, /* read only */
@@ -1259,7 +1259,7 @@ build_ssdt(GArray *table_data, GArray *linker,
aml_append(scope, field);
field = aml_field(stringify(MEMORY_HOTPLUG_IO_REGION), AML_BYTE_ACC,
- AML_WRITE_AS_ZEROS);
+ AML_NOLOCK, AML_WRITE_AS_ZEROS);
aml_append(field, aml_reserved_field(160 /* bits, Offset(20) */));
aml_append(field, /* 1 if enabled, read only */
aml_named_field(stringify(MEMORY_SLOT_ENABLED), 1));
@@ -1275,7 +1275,7 @@ build_ssdt(GArray *table_data, GArray *linker,
aml_append(scope, field);
field = aml_field(stringify(MEMORY_HOTPLUG_IO_REGION), AML_DWORD_ACC,
- AML_PRESERVE);
+ AML_NOLOCK, AML_PRESERVE);
aml_append(field, /* DIMM selector, write only */
aml_named_field(stringify(MEMORY_SLOT_SLECTOR), 32));
aml_append(field, /* _OST event code, write only */
--
MST
- [Qemu-devel] [PULL 39/55] acpi: add aml_create_qword_field(), (continued)
- [Qemu-devel] [PULL 39/55] acpi: add aml_create_qword_field(), Michael S. Tsirkin, 2015/12/22
- [Qemu-devel] [PULL 40/55] acpi: aml: add helper for Opcode Arg2 Arg2 [Dst] AML pattern, Michael S. Tsirkin, 2015/12/22
- [Qemu-devel] [PULL 41/55] acpi: extend aml_add() to accept target argument, Michael S. Tsirkin, 2015/12/22
- [Qemu-devel] [PULL 42/55] acpi: add aml_decrement() and aml_subtract(), Michael S. Tsirkin, 2015/12/22
- [Qemu-devel] [PULL 43/55] acpi: add aml_call0() helper, Michael S. Tsirkin, 2015/12/22
- [Qemu-devel] [PULL 44/55] acpi: add aml_to_integer(), Michael S. Tsirkin, 2015/12/22
- [Qemu-devel] [PULL 46/55] acpi: add aml_alias(), Michael S. Tsirkin, 2015/12/22
- [Qemu-devel] [PULL 45/55] acpi: extend aml_shiftright() to accept target argument, Michael S. Tsirkin, 2015/12/22
- [Qemu-devel] [PULL 47/55] acpi: add aml_sleep(), Michael S. Tsirkin, 2015/12/22
- [Qemu-devel] [PULL 48/55] acpi: add aml_lor(), Michael S. Tsirkin, 2015/12/22
- [Qemu-devel] [PULL 50/55] acpi: extend aml_field() to support LockRule,
Michael S. Tsirkin <=
- [Qemu-devel] [PULL 49/55] acpi: add aml_lgreater(), Michael S. Tsirkin, 2015/12/22
- [Qemu-devel] [PULL 51/55] acpi: add aml_to_hexstring(), Michael S. Tsirkin, 2015/12/22
- [Qemu-devel] [PULL 52/55] acpi: add aml_to_buffer(), Michael S. Tsirkin, 2015/12/22
- [Qemu-devel] [PULL 53/55] acpi add aml_dma(), Michael S. Tsirkin, 2015/12/22
- [Qemu-devel] [PULL 54/55] acpi: extend aml_or() to accept target argument, Michael S. Tsirkin, 2015/12/22
- [Qemu-devel] [PULL 55/55] acpi: extend aml_and() to accept target argument, Michael S. Tsirkin, 2015/12/22
- Re: [Qemu-devel] [PULL 00/55] acpi, pc features, Peter Maydell, 2015/12/23