* [Qemu-devel] [PATCH 0/3] hw/i386/acpi: Improve build modularity (isapc/q35/...)
@ 2019-04-27 14:40 ` Philippe Mathieu-Daudé
0 siblings, 0 replies; 12+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-04-27 14:40 UTC (permalink / raw)
To: qemu-devel
Cc: Paolo Bonzini, Marcel Apfelbaum, Igor Mammedov, Samuel Ortiz,
Michael S. Tsirkin, Yang Zhong, Thomas Huth, Eduardo Habkost,
Richard Henderson, Philippe Mathieu-Daudé
This series allow to build the ISAPC/Q35 machines independently.
Regards,
Phil.
Philippe Mathieu-Daudé (3):
hw/acpi/piix4: Move TYPE_PIIX4_PM to a public header
hw/i386/acpi: Add object_resolve_type_unambiguous to improve
modularity
hw/i386/acpi: Assert a pointer is not null BEFORE using it
hw/acpi/piix4.c | 13 -------------
hw/i386/acpi-build.c | 22 +++++++++++++++++-----
hw/isa/lpc_ich9.c | 11 -----------
include/hw/acpi/piix4.h | 2 +-
include/hw/i386/ich9.h | 2 --
5 files changed, 18 insertions(+), 32 deletions(-)
--
2.20.1
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Qemu-devel] [PATCH 0/3] hw/i386/acpi: Improve build modularity (isapc/q35/...)
@ 2019-04-27 14:40 ` Philippe Mathieu-Daudé
0 siblings, 0 replies; 12+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-04-27 14:40 UTC (permalink / raw)
To: qemu-devel
Cc: Yang Zhong, Thomas Huth, Samuel Ortiz, Michael S. Tsirkin,
Igor Mammedov, Paolo Bonzini, Richard Henderson,
Philippe Mathieu-Daudé,
Eduardo Habkost
This series allow to build the ISAPC/Q35 machines independently.
Regards,
Phil.
Philippe Mathieu-Daudé (3):
hw/acpi/piix4: Move TYPE_PIIX4_PM to a public header
hw/i386/acpi: Add object_resolve_type_unambiguous to improve
modularity
hw/i386/acpi: Assert a pointer is not null BEFORE using it
hw/acpi/piix4.c | 13 -------------
hw/i386/acpi-build.c | 22 +++++++++++++++++-----
hw/isa/lpc_ich9.c | 11 -----------
include/hw/acpi/piix4.h | 2 +-
include/hw/i386/ich9.h | 2 --
5 files changed, 18 insertions(+), 32 deletions(-)
--
2.20.1
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Qemu-devel] [PATCH 1/3] hw/acpi/piix4: Move TYPE_PIIX4_PM to a public header
@ 2019-04-27 14:40 ` Philippe Mathieu-Daudé
0 siblings, 0 replies; 12+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-04-27 14:40 UTC (permalink / raw)
To: qemu-devel
Cc: Paolo Bonzini, Marcel Apfelbaum, Igor Mammedov, Samuel Ortiz,
Michael S. Tsirkin, Yang Zhong, Thomas Huth, Eduardo Habkost,
Richard Henderson, Philippe Mathieu-Daudé
Move the TYPE_PIIX4_PM definition to the corresponding header,
so other files can use it.
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
---
hw/acpi/piix4.c | 2 --
include/hw/acpi/piix4.h | 2 ++
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/hw/acpi/piix4.c b/hw/acpi/piix4.c
index 9c079d68341..160e7308c51 100644
--- a/hw/acpi/piix4.c
+++ b/hw/acpi/piix4.c
@@ -92,8 +92,6 @@ typedef struct PIIX4PMState {
MemHotplugState acpi_memory_hotplug;
} PIIX4PMState;
-#define TYPE_PIIX4_PM "PIIX4_PM"
-
#define PIIX4_PM(obj) \
OBJECT_CHECK(PIIX4PMState, (obj), TYPE_PIIX4_PM)
diff --git a/include/hw/acpi/piix4.h b/include/hw/acpi/piix4.h
index 26c2370e308..57d7e1cda20 100644
--- a/include/hw/acpi/piix4.h
+++ b/include/hw/acpi/piix4.h
@@ -1,6 +1,8 @@
#ifndef HW_ACPI_PIIX4_H
#define HW_ACPI_PIIX4_H
+#define TYPE_PIIX4_PM "PIIX4_PM"
+
Object *piix4_pm_find(void);
#endif
--
2.20.1
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [Qemu-devel] [PATCH 1/3] hw/acpi/piix4: Move TYPE_PIIX4_PM to a public header
@ 2019-04-27 14:40 ` Philippe Mathieu-Daudé
0 siblings, 0 replies; 12+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-04-27 14:40 UTC (permalink / raw)
To: qemu-devel
Cc: Yang Zhong, Thomas Huth, Samuel Ortiz, Michael S. Tsirkin,
Igor Mammedov, Paolo Bonzini, Richard Henderson,
Philippe Mathieu-Daudé,
Eduardo Habkost
Move the TYPE_PIIX4_PM definition to the corresponding header,
so other files can use it.
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
---
hw/acpi/piix4.c | 2 --
include/hw/acpi/piix4.h | 2 ++
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/hw/acpi/piix4.c b/hw/acpi/piix4.c
index 9c079d68341..160e7308c51 100644
--- a/hw/acpi/piix4.c
+++ b/hw/acpi/piix4.c
@@ -92,8 +92,6 @@ typedef struct PIIX4PMState {
MemHotplugState acpi_memory_hotplug;
} PIIX4PMState;
-#define TYPE_PIIX4_PM "PIIX4_PM"
-
#define PIIX4_PM(obj) \
OBJECT_CHECK(PIIX4PMState, (obj), TYPE_PIIX4_PM)
diff --git a/include/hw/acpi/piix4.h b/include/hw/acpi/piix4.h
index 26c2370e308..57d7e1cda20 100644
--- a/include/hw/acpi/piix4.h
+++ b/include/hw/acpi/piix4.h
@@ -1,6 +1,8 @@
#ifndef HW_ACPI_PIIX4_H
#define HW_ACPI_PIIX4_H
+#define TYPE_PIIX4_PM "PIIX4_PM"
+
Object *piix4_pm_find(void);
#endif
--
2.20.1
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [Qemu-devel] [PATCH 2/3] hw/i386/acpi: Add object_resolve_type_unambiguous to improve modularity
@ 2019-04-27 14:40 ` Philippe Mathieu-Daudé
0 siblings, 0 replies; 12+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-04-27 14:40 UTC (permalink / raw)
To: qemu-devel
Cc: Paolo Bonzini, Marcel Apfelbaum, Igor Mammedov, Samuel Ortiz,
Michael S. Tsirkin, Yang Zhong, Thomas Huth, Eduardo Habkost,
Richard Henderson, Philippe Mathieu-Daudé
When building with CONFIG_Q35=n, we get:
LINK x86_64-softmmu/qemu-system-x86_64
/usr/bin/ld: hw/i386/acpi-build.o: in function `acpi_get_misc_info':
/source/qemu/hw/i386/acpi-build.c:243: undefined reference to `ich9_lpc_find'
collect2: error: ld returned 1 exit status
make[1]: *** [Makefile:204: qemu-system-x86_64] Error 1
This is due to a dependency in acpi-build.c on the ICH9_LPC
(via ich9_lpc_find) and PIIX4_PM (via piix4_pm_find) devices.
To allow better modularity (compile acpi-build.c with only
Q35/ICH9 or ISAPC/PIIX4), refactor the similar helper as
object_resolve_type_unambiguous(). This way we relax the
linker dependencies and can build the x86 targets with a
selection of machines (instead of all of them).
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
---
hw/acpi/piix4.c | 11 -----------
hw/i386/acpi-build.c | 20 ++++++++++++++++----
hw/isa/lpc_ich9.c | 11 -----------
include/hw/acpi/piix4.h | 2 --
include/hw/i386/ich9.h | 2 --
5 files changed, 16 insertions(+), 30 deletions(-)
diff --git a/hw/acpi/piix4.c b/hw/acpi/piix4.c
index 160e7308c51..c903e651695 100644
--- a/hw/acpi/piix4.c
+++ b/hw/acpi/piix4.c
@@ -552,17 +552,6 @@ static void piix4_pm_realize(PCIDevice *dev, Error **errp)
piix4_pm_add_propeties(s);
}
-Object *piix4_pm_find(void)
-{
- bool ambig;
- Object *o = object_resolve_path_type("", TYPE_PIIX4_PM, &ambig);
-
- if (ambig || !o) {
- return NULL;
- }
- return o;
-}
-
I2CBus *piix4_pm_init(PCIBus *bus, int devfn, uint32_t smb_io_base,
qemu_irq sci_irq, qemu_irq smi_irq,
int smm_enabled, DeviceState **piix4_pm)
diff --git a/hw/i386/acpi-build.c b/hw/i386/acpi-build.c
index 416da318ae0..123ff2b8169 100644
--- a/hw/i386/acpi-build.c
+++ b/hw/i386/acpi-build.c
@@ -35,6 +35,7 @@
#include "hw/acpi/acpi-defs.h"
#include "hw/acpi/acpi.h"
#include "hw/acpi/cpu.h"
+#include "hw/acpi/piix4.h"
#include "hw/nvram/fw_cfg.h"
#include "hw/acpi/bios-linker-loader.h"
#include "hw/loader.h"
@@ -164,10 +165,21 @@ static void init_common_fadt_data(Object *o, AcpiFadtData *data)
*data = fadt;
}
+static Object *object_resolve_type_unambiguous(const char *typename)
+{
+ bool ambig;
+ Object *o = object_resolve_path_type("", typename, &ambig);
+
+ if (ambig || !o) {
+ return NULL;
+ }
+ return o;
+}
+
static void acpi_get_pm_info(AcpiPmInfo *pm)
{
- Object *piix = piix4_pm_find();
- Object *lpc = ich9_lpc_find();
+ Object *piix = object_resolve_type_unambiguous(TYPE_PIIX4_PM);
+ Object *lpc = object_resolve_type_unambiguous(TYPE_ICH9_LPC_DEVICE);
Object *obj = piix ? piix : lpc;
QObject *o;
pm->cpu_hp_io_base = 0;
@@ -228,8 +240,8 @@ static void acpi_get_pm_info(AcpiPmInfo *pm)
static void acpi_get_misc_info(AcpiMiscInfo *info)
{
- Object *piix = piix4_pm_find();
- Object *lpc = ich9_lpc_find();
+ Object *piix = object_resolve_type_unambiguous(TYPE_PIIX4_PM);
+ Object *lpc = object_resolve_type_unambiguous(TYPE_ICH9_LPC_DEVICE);
assert(!!piix != !!lpc);
if (piix) {
diff --git a/hw/isa/lpc_ich9.c b/hw/isa/lpc_ich9.c
index ac44aa53bee..031ee9cd933 100644
--- a/hw/isa/lpc_ich9.c
+++ b/hw/isa/lpc_ich9.c
@@ -624,17 +624,6 @@ static const MemoryRegionOps ich9_rst_cnt_ops = {
.endianness = DEVICE_LITTLE_ENDIAN
};
-Object *ich9_lpc_find(void)
-{
- bool ambig;
- Object *o = object_resolve_path_type("", TYPE_ICH9_LPC_DEVICE, &ambig);
-
- if (ambig) {
- return NULL;
- }
- return o;
-}
-
static void ich9_lpc_get_sci_int(Object *obj, Visitor *v, const char *name,
void *opaque, Error **errp)
{
diff --git a/include/hw/acpi/piix4.h b/include/hw/acpi/piix4.h
index 57d7e1cda20..028bb53e3df 100644
--- a/include/hw/acpi/piix4.h
+++ b/include/hw/acpi/piix4.h
@@ -3,6 +3,4 @@
#define TYPE_PIIX4_PM "PIIX4_PM"
-Object *piix4_pm_find(void);
-
#endif
diff --git a/include/hw/i386/ich9.h b/include/hw/i386/ich9.h
index 673d13d28f2..046bcf33bed 100644
--- a/include/hw/i386/ich9.h
+++ b/include/hw/i386/ich9.h
@@ -81,8 +81,6 @@ typedef struct ICH9LPCState {
qemu_irq gsi[GSI_NUM_PINS];
} ICH9LPCState;
-Object *ich9_lpc_find(void);
-
#define Q35_MASK(bit, ms_bit, ls_bit) \
((uint##bit##_t)(((1ULL << ((ms_bit) + 1)) - 1) & ~((1ULL << ls_bit) - 1)))
--
2.20.1
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [Qemu-devel] [PATCH 2/3] hw/i386/acpi: Add object_resolve_type_unambiguous to improve modularity
@ 2019-04-27 14:40 ` Philippe Mathieu-Daudé
0 siblings, 0 replies; 12+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-04-27 14:40 UTC (permalink / raw)
To: qemu-devel
Cc: Yang Zhong, Thomas Huth, Samuel Ortiz, Michael S. Tsirkin,
Igor Mammedov, Paolo Bonzini, Richard Henderson,
Philippe Mathieu-Daudé,
Eduardo Habkost
When building with CONFIG_Q35=n, we get:
LINK x86_64-softmmu/qemu-system-x86_64
/usr/bin/ld: hw/i386/acpi-build.o: in function `acpi_get_misc_info':
/source/qemu/hw/i386/acpi-build.c:243: undefined reference to `ich9_lpc_find'
collect2: error: ld returned 1 exit status
make[1]: *** [Makefile:204: qemu-system-x86_64] Error 1
This is due to a dependency in acpi-build.c on the ICH9_LPC
(via ich9_lpc_find) and PIIX4_PM (via piix4_pm_find) devices.
To allow better modularity (compile acpi-build.c with only
Q35/ICH9 or ISAPC/PIIX4), refactor the similar helper as
object_resolve_type_unambiguous(). This way we relax the
linker dependencies and can build the x86 targets with a
selection of machines (instead of all of them).
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
---
hw/acpi/piix4.c | 11 -----------
hw/i386/acpi-build.c | 20 ++++++++++++++++----
hw/isa/lpc_ich9.c | 11 -----------
include/hw/acpi/piix4.h | 2 --
include/hw/i386/ich9.h | 2 --
5 files changed, 16 insertions(+), 30 deletions(-)
diff --git a/hw/acpi/piix4.c b/hw/acpi/piix4.c
index 160e7308c51..c903e651695 100644
--- a/hw/acpi/piix4.c
+++ b/hw/acpi/piix4.c
@@ -552,17 +552,6 @@ static void piix4_pm_realize(PCIDevice *dev, Error **errp)
piix4_pm_add_propeties(s);
}
-Object *piix4_pm_find(void)
-{
- bool ambig;
- Object *o = object_resolve_path_type("", TYPE_PIIX4_PM, &ambig);
-
- if (ambig || !o) {
- return NULL;
- }
- return o;
-}
-
I2CBus *piix4_pm_init(PCIBus *bus, int devfn, uint32_t smb_io_base,
qemu_irq sci_irq, qemu_irq smi_irq,
int smm_enabled, DeviceState **piix4_pm)
diff --git a/hw/i386/acpi-build.c b/hw/i386/acpi-build.c
index 416da318ae0..123ff2b8169 100644
--- a/hw/i386/acpi-build.c
+++ b/hw/i386/acpi-build.c
@@ -35,6 +35,7 @@
#include "hw/acpi/acpi-defs.h"
#include "hw/acpi/acpi.h"
#include "hw/acpi/cpu.h"
+#include "hw/acpi/piix4.h"
#include "hw/nvram/fw_cfg.h"
#include "hw/acpi/bios-linker-loader.h"
#include "hw/loader.h"
@@ -164,10 +165,21 @@ static void init_common_fadt_data(Object *o, AcpiFadtData *data)
*data = fadt;
}
+static Object *object_resolve_type_unambiguous(const char *typename)
+{
+ bool ambig;
+ Object *o = object_resolve_path_type("", typename, &ambig);
+
+ if (ambig || !o) {
+ return NULL;
+ }
+ return o;
+}
+
static void acpi_get_pm_info(AcpiPmInfo *pm)
{
- Object *piix = piix4_pm_find();
- Object *lpc = ich9_lpc_find();
+ Object *piix = object_resolve_type_unambiguous(TYPE_PIIX4_PM);
+ Object *lpc = object_resolve_type_unambiguous(TYPE_ICH9_LPC_DEVICE);
Object *obj = piix ? piix : lpc;
QObject *o;
pm->cpu_hp_io_base = 0;
@@ -228,8 +240,8 @@ static void acpi_get_pm_info(AcpiPmInfo *pm)
static void acpi_get_misc_info(AcpiMiscInfo *info)
{
- Object *piix = piix4_pm_find();
- Object *lpc = ich9_lpc_find();
+ Object *piix = object_resolve_type_unambiguous(TYPE_PIIX4_PM);
+ Object *lpc = object_resolve_type_unambiguous(TYPE_ICH9_LPC_DEVICE);
assert(!!piix != !!lpc);
if (piix) {
diff --git a/hw/isa/lpc_ich9.c b/hw/isa/lpc_ich9.c
index ac44aa53bee..031ee9cd933 100644
--- a/hw/isa/lpc_ich9.c
+++ b/hw/isa/lpc_ich9.c
@@ -624,17 +624,6 @@ static const MemoryRegionOps ich9_rst_cnt_ops = {
.endianness = DEVICE_LITTLE_ENDIAN
};
-Object *ich9_lpc_find(void)
-{
- bool ambig;
- Object *o = object_resolve_path_type("", TYPE_ICH9_LPC_DEVICE, &ambig);
-
- if (ambig) {
- return NULL;
- }
- return o;
-}
-
static void ich9_lpc_get_sci_int(Object *obj, Visitor *v, const char *name,
void *opaque, Error **errp)
{
diff --git a/include/hw/acpi/piix4.h b/include/hw/acpi/piix4.h
index 57d7e1cda20..028bb53e3df 100644
--- a/include/hw/acpi/piix4.h
+++ b/include/hw/acpi/piix4.h
@@ -3,6 +3,4 @@
#define TYPE_PIIX4_PM "PIIX4_PM"
-Object *piix4_pm_find(void);
-
#endif
diff --git a/include/hw/i386/ich9.h b/include/hw/i386/ich9.h
index 673d13d28f2..046bcf33bed 100644
--- a/include/hw/i386/ich9.h
+++ b/include/hw/i386/ich9.h
@@ -81,8 +81,6 @@ typedef struct ICH9LPCState {
qemu_irq gsi[GSI_NUM_PINS];
} ICH9LPCState;
-Object *ich9_lpc_find(void);
-
#define Q35_MASK(bit, ms_bit, ls_bit) \
((uint##bit##_t)(((1ULL << ((ms_bit) + 1)) - 1) & ~((1ULL << ls_bit) - 1)))
--
2.20.1
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [Qemu-devel] [PATCH 3/3] hw/i386/acpi: Assert a pointer is not null BEFORE using it
@ 2019-04-27 14:40 ` Philippe Mathieu-Daudé
0 siblings, 0 replies; 12+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-04-27 14:40 UTC (permalink / raw)
To: qemu-devel
Cc: Paolo Bonzini, Marcel Apfelbaum, Igor Mammedov, Samuel Ortiz,
Michael S. Tsirkin, Yang Zhong, Thomas Huth, Eduardo Habkost,
Richard Henderson, Philippe Mathieu-Daudé
Commit 72c194f7e75c added a non-null check on the 'obj' pointer.
Later, commit 500b11ea5095 added code which uses the 'obj'
pointer _before_ the assertion check. Move the assertion
_before_ the pointer use.
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
---
hw/i386/acpi-build.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/i386/acpi-build.c b/hw/i386/acpi-build.c
index 123ff2b8169..b4ec14e349f 100644
--- a/hw/i386/acpi-build.c
+++ b/hw/i386/acpi-build.c
@@ -186,6 +186,7 @@ static void acpi_get_pm_info(AcpiPmInfo *pm)
pm->pcihp_io_base = 0;
pm->pcihp_io_len = 0;
+ assert(obj);
init_common_fadt_data(obj, &pm->fadt);
if (piix) {
/* w2k requires FADT(rev1) or it won't boot, keep PC compatible */
@@ -204,7 +205,6 @@ static void acpi_get_pm_info(AcpiPmInfo *pm)
pm->fadt.flags |= 1 << ACPI_FADT_F_RESET_REG_SUP;
pm->cpu_hp_io_base = ICH9_CPU_HOTPLUG_IO_BASE;
}
- assert(obj);
/* The above need not be conditional on machine type because the reset port
* happens to be the same on PIIX (pc) and ICH9 (q35). */
--
2.20.1
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [Qemu-devel] [PATCH 3/3] hw/i386/acpi: Assert a pointer is not null BEFORE using it
@ 2019-04-27 14:40 ` Philippe Mathieu-Daudé
0 siblings, 0 replies; 12+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-04-27 14:40 UTC (permalink / raw)
To: qemu-devel
Cc: Yang Zhong, Thomas Huth, Samuel Ortiz, Michael S. Tsirkin,
Igor Mammedov, Paolo Bonzini, Richard Henderson,
Philippe Mathieu-Daudé,
Eduardo Habkost
Commit 72c194f7e75c added a non-null check on the 'obj' pointer.
Later, commit 500b11ea5095 added code which uses the 'obj'
pointer _before_ the assertion check. Move the assertion
_before_ the pointer use.
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
---
hw/i386/acpi-build.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/i386/acpi-build.c b/hw/i386/acpi-build.c
index 123ff2b8169..b4ec14e349f 100644
--- a/hw/i386/acpi-build.c
+++ b/hw/i386/acpi-build.c
@@ -186,6 +186,7 @@ static void acpi_get_pm_info(AcpiPmInfo *pm)
pm->pcihp_io_base = 0;
pm->pcihp_io_len = 0;
+ assert(obj);
init_common_fadt_data(obj, &pm->fadt);
if (piix) {
/* w2k requires FADT(rev1) or it won't boot, keep PC compatible */
@@ -204,7 +205,6 @@ static void acpi_get_pm_info(AcpiPmInfo *pm)
pm->fadt.flags |= 1 << ACPI_FADT_F_RESET_REG_SUP;
pm->cpu_hp_io_base = ICH9_CPU_HOTPLUG_IO_BASE;
}
- assert(obj);
/* The above need not be conditional on machine type because the reset port
* happens to be the same on PIIX (pc) and ICH9 (q35). */
--
2.20.1
^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [Qemu-devel] [PATCH 0/3] hw/i386/acpi: Improve build modularity (isapc/q35/...)
@ 2019-04-27 14:43 ` Philippe Mathieu-Daudé
0 siblings, 0 replies; 12+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-04-27 14:43 UTC (permalink / raw)
To: qemu-devel
Cc: Paolo Bonzini, Marcel Apfelbaum, Igor Mammedov, Samuel Ortiz,
Michael S. Tsirkin, Yang Zhong, Thomas Huth, Eduardo Habkost,
Richard Henderson
On 4/27/19 4:40 PM, Philippe Mathieu-Daudé wrote:
> This series allow to build the ISAPC/Q35 machines independently.
Oops I guess I forgot:
Based-on: 20190427141905.20393-1-philmd@redhat.com
(i440fx is not a machine)
https://lists.gnu.org/archive/html/qemu-devel/2019-04/msg04673.html
>
> Regards,
>
> Phil.
>
> Philippe Mathieu-Daudé (3):
> hw/acpi/piix4: Move TYPE_PIIX4_PM to a public header
> hw/i386/acpi: Add object_resolve_type_unambiguous to improve
> modularity
> hw/i386/acpi: Assert a pointer is not null BEFORE using it
>
> hw/acpi/piix4.c | 13 -------------
> hw/i386/acpi-build.c | 22 +++++++++++++++++-----
> hw/isa/lpc_ich9.c | 11 -----------
> include/hw/acpi/piix4.h | 2 +-
> include/hw/i386/ich9.h | 2 --
> 5 files changed, 18 insertions(+), 32 deletions(-)
>
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [Qemu-devel] [PATCH 0/3] hw/i386/acpi: Improve build modularity (isapc/q35/...)
@ 2019-04-27 14:43 ` Philippe Mathieu-Daudé
0 siblings, 0 replies; 12+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-04-27 14:43 UTC (permalink / raw)
To: qemu-devel
Cc: Yang Zhong, Thomas Huth, Samuel Ortiz, Michael S. Tsirkin,
Igor Mammedov, Paolo Bonzini, Richard Henderson, Eduardo Habkost
On 4/27/19 4:40 PM, Philippe Mathieu-Daudé wrote:
> This series allow to build the ISAPC/Q35 machines independently.
Oops I guess I forgot:
Based-on: 20190427141905.20393-1-philmd@redhat.com
(i440fx is not a machine)
https://lists.gnu.org/archive/html/qemu-devel/2019-04/msg04673.html
>
> Regards,
>
> Phil.
>
> Philippe Mathieu-Daudé (3):
> hw/acpi/piix4: Move TYPE_PIIX4_PM to a public header
> hw/i386/acpi: Add object_resolve_type_unambiguous to improve
> modularity
> hw/i386/acpi: Assert a pointer is not null BEFORE using it
>
> hw/acpi/piix4.c | 13 -------------
> hw/i386/acpi-build.c | 22 +++++++++++++++++-----
> hw/isa/lpc_ich9.c | 11 -----------
> include/hw/acpi/piix4.h | 2 +-
> include/hw/i386/ich9.h | 2 --
> 5 files changed, 18 insertions(+), 32 deletions(-)
>
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [Qemu-devel] [PATCH 0/3] hw/i386/acpi: Improve build modularity (isapc/q35/...)
@ 2019-04-30 19:28 ` Paolo Bonzini
0 siblings, 0 replies; 12+ messages in thread
From: Paolo Bonzini @ 2019-04-30 19:28 UTC (permalink / raw)
To: Philippe Mathieu-Daudé, qemu-devel
Cc: Marcel Apfelbaum, Igor Mammedov, Samuel Ortiz,
Michael S. Tsirkin, Yang Zhong, Thomas Huth, Eduardo Habkost,
Richard Henderson
On 27/04/19 16:43, Philippe Mathieu-Daudé wrote:
> On 4/27/19 4:40 PM, Philippe Mathieu-Daudé wrote:
>> This series allow to build the ISAPC/Q35 machines independently.
>
> Oops I guess I forgot:
> Based-on: 20190427141905.20393-1-philmd@redhat.com
> (i440fx is not a machine)
> https://lists.gnu.org/archive/html/qemu-devel/2019-04/msg04673.html
Queued (*without* the based-on-but-not-really series), thanks.
Paolo
>
>>
>> Regards,
>>
>> Phil.
>>
>> Philippe Mathieu-Daudé (3):
>> hw/acpi/piix4: Move TYPE_PIIX4_PM to a public header
>> hw/i386/acpi: Add object_resolve_type_unambiguous to improve
>> modularity
>> hw/i386/acpi: Assert a pointer is not null BEFORE using it
>>
>> hw/acpi/piix4.c | 13 -------------
>> hw/i386/acpi-build.c | 22 +++++++++++++++++-----
>> hw/isa/lpc_ich9.c | 11 -----------
>> include/hw/acpi/piix4.h | 2 +-
>> include/hw/i386/ich9.h | 2 --
>> 5 files changed, 18 insertions(+), 32 deletions(-)
>>
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [Qemu-devel] [PATCH 0/3] hw/i386/acpi: Improve build modularity (isapc/q35/...)
@ 2019-04-30 19:28 ` Paolo Bonzini
0 siblings, 0 replies; 12+ messages in thread
From: Paolo Bonzini @ 2019-04-30 19:28 UTC (permalink / raw)
To: Philippe Mathieu-Daudé, qemu-devel
Cc: Yang Zhong, Thomas Huth, Samuel Ortiz, Michael S. Tsirkin,
Igor Mammedov, Richard Henderson, Eduardo Habkost
On 27/04/19 16:43, Philippe Mathieu-Daudé wrote:
> On 4/27/19 4:40 PM, Philippe Mathieu-Daudé wrote:
>> This series allow to build the ISAPC/Q35 machines independently.
>
> Oops I guess I forgot:
> Based-on: 20190427141905.20393-1-philmd@redhat.com
> (i440fx is not a machine)
> https://lists.gnu.org/archive/html/qemu-devel/2019-04/msg04673.html
Queued (*without* the based-on-but-not-really series), thanks.
Paolo
>
>>
>> Regards,
>>
>> Phil.
>>
>> Philippe Mathieu-Daudé (3):
>> hw/acpi/piix4: Move TYPE_PIIX4_PM to a public header
>> hw/i386/acpi: Add object_resolve_type_unambiguous to improve
>> modularity
>> hw/i386/acpi: Assert a pointer is not null BEFORE using it
>>
>> hw/acpi/piix4.c | 13 -------------
>> hw/i386/acpi-build.c | 22 +++++++++++++++++-----
>> hw/isa/lpc_ich9.c | 11 -----------
>> include/hw/acpi/piix4.h | 2 +-
>> include/hw/i386/ich9.h | 2 --
>> 5 files changed, 18 insertions(+), 32 deletions(-)
>>
^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2019-04-30 19:30 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-04-27 14:40 [Qemu-devel] [PATCH 0/3] hw/i386/acpi: Improve build modularity (isapc/q35/...) Philippe Mathieu-Daudé
2019-04-27 14:40 ` Philippe Mathieu-Daudé
2019-04-27 14:40 ` [Qemu-devel] [PATCH 1/3] hw/acpi/piix4: Move TYPE_PIIX4_PM to a public header Philippe Mathieu-Daudé
2019-04-27 14:40 ` Philippe Mathieu-Daudé
2019-04-27 14:40 ` [Qemu-devel] [PATCH 2/3] hw/i386/acpi: Add object_resolve_type_unambiguous to improve modularity Philippe Mathieu-Daudé
2019-04-27 14:40 ` Philippe Mathieu-Daudé
2019-04-27 14:40 ` [Qemu-devel] [PATCH 3/3] hw/i386/acpi: Assert a pointer is not null BEFORE using it Philippe Mathieu-Daudé
2019-04-27 14:40 ` Philippe Mathieu-Daudé
2019-04-27 14:43 ` [Qemu-devel] [PATCH 0/3] hw/i386/acpi: Improve build modularity (isapc/q35/...) Philippe Mathieu-Daudé
2019-04-27 14:43 ` Philippe Mathieu-Daudé
2019-04-30 19:28 ` Paolo Bonzini
2019-04-30 19:28 ` Paolo Bonzini
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.