On Mon, 20 Mar 2023 16:59:02 +0530
Ani Sinha wrote:
> i440fx machine versions 2.3 and newer and q35 machines supports dynamic ram
> resizing. Please see commit a1666142db6233 ("acpi-build: make ROMs RAM blocks
> resizeable") .
> Hence the warning when the ACPI table size exceeds a pre-defined va
rt/SRAT.acpihmatvirt | Bin 240 -> 240 bytes
> tests/data/acpi/virt/SSDT.memhp| Bin 1817 -> 1817 bytes
binary blob updates should be in a separate patch
with this fixed:
Reviewed-by: Igor Mammedov
> 3 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/
On Mon, 13 Mar 2023 14:58:42 -0700
Dave Jiang wrote:
> I'm attempting to implement the support of ACPI "generic port" detailed
> in the ACPI r6.5 spec in QEMU. The spec section 5.2.16.7 details the
> Generi Port Affinity Structure where it ties a Device Handle to a
> Proximity Domain. And with
(Philippe)
>
> Gavin Shan (3):
> numa: Validate cluster and NUMA node boundary if required
> hw/arm: Validate cluster and NUMA node boundary
> hw/riscv: Validate cluster and NUMA node boundary
>
> hw/arm/sbsa-ref.c | 2 ++
> hw/arm/vi
On Tue, 7 Mar 2023 13:46:36 +0100
David Hildenbrand wrote:
> On 07.03.23 11:51, Igor Mammedov wrote:
> > On Thu, 16 Feb 2023 12:47:51 +0100
> > David Hildenbrand wrote:
> >
> >> Having multiple devices, some filtering memslots and some not filtering
> >>
On Tue, 21 Feb 2023 23:05:00 +0700
Bui Quang Minh wrote:
> Simple test to check Linux boot up in x2APIC with userspace local APIC and
> TCG accelerator.
just an idea, while booting linux would give some coverage,
we probably would get much better coverage by using apic test case
from kvm unit te
On Mon, 06 Mar 2023 16:50:29 +
David Woodhouse wrote:
> On Mon, 2023-03-06 at 23:39 +0700, Bui Quang Minh wrote:
> > On 3/6/23 22:51, David Woodhouse wrote:
> > > On Mon, 2023-03-06 at 11:43 +0100, Igor Mammedov wrote:
> > > > > However, there are still p
On Mon, 06 Mar 2023 15:51:45 +
David Woodhouse wrote:
> On Mon, 2023-03-06 at 11:43 +0100, Igor Mammedov wrote:
> > > However, there are still problems while trying to extending support to
> > > APIC ID larger than 255 because there are many places assume APIC ID
On Tue, 7 Mar 2023 11:25:48 +0100
Igor Mammedov wrote:
> On Thu, 16 Feb 2023 12:47:52 +0100
> David Hildenbrand wrote:
>
> > Checking whether the memory regions are equal is sufficient: if they are
> > equal, then most certainly the contained fd is equal.
>
On Fri, 17 Feb 2023 09:20:27 -0500
"Michael S. Tsirkin" wrote:
> On Thu, Feb 16, 2023 at 12:47:50PM +0100, David Hildenbrand wrote:
> > Following up on my previous work to make virtio-mem consume multiple
> > memslots dynamically [1] that requires precise accounting between used vs.
> > reserved
On Thu, 16 Feb 2023 12:47:51 +0100
David Hildenbrand wrote:
> Having multiple devices, some filtering memslots and some not filtering
> memslots, messes up the "used_memslot" accounting. If we'd have a device
> the filters out less memory sections after a device that filters out more,
> we'd be i
place.
To me it looks like we should drop conversion to vhost_dev::mem
and replace its usage with vhost_dev::mem_sections directly
to get rid of data duplication and back and forth addr<->mr conversion.
> For now, let's just drop vhost_backend_can_merge().
>
> Signed-off-by: Davi
On Tue, 7 Feb 2023 10:50:56 +0800
"Wang, Lei" wrote:
> On 2/2/2023 7:05 PM, Igor Mammedov wrote:
> > On Fri, 6 Jan 2023 00:38:20 -0800
> > Lei Wang wrote:
> >
> >> This series aims to add a new CPU model SapphireRapids, and tries to
>
On Thu, 2 Mar 2023 18:21:28 -0500
"Michael S. Tsirkin" wrote:
> On Thu, Mar 02, 2023 at 01:07:16PM +0100, Igor Mammedov wrote:
> > On Thu, 2 Mar 2023 05:59:16 -0500
> > "Michael S. Tsirkin" wrote:
> >
> > > On Fri, Feb 24, 2023 at 04:37
On Sat, 4 Mar 2023 21:10:54 +0700
Bui Quang Minh wrote:
> On 2/28/23 23:39, Igor Mammedov wrote:
> > On Tue, 28 Feb 2023 21:34:33 +0700
> > Bui Quang Minh wrote:
> >
> >> On 2/27/23 23:07, Igor Mammedov wrote:
> >>> On Sat, 25 Feb 2023
for q35.noacpihp use plain default Q35 DSDT table as a starting point.
Signed-off-by: Igor Mammedov
---
tests/qtest/bios-tables-test-allowed-diff.h | 2 ++
tests/data/acpi/q35/DSDT.noacpihp | Bin 0 -> 8252 bytes
2 files changed, 2 insertions(+)
create mode 100644 tests/data/a
s: 6c36ec46b0d ("pcihp: make bridge describe itself using
AcpiDevAmlIfClass:build_dev_aml")
Signed-off-by: Igor Mammedov
---
hw/acpi/pci-bridge.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/acpi/pci-bridge.c b/hw/acpi/pci-bridge.c
index 5f3ee5157f..4fbf6da6ad
ug is disabled).
Move check and related code to generic PCIDevice so it
would be independent of ACPI PCI hotplug.
Signed-off-by: Igor Mammedov
---
PS: this also one step closer to enabling acpi-index
support for microvm and virt/arm machines.
---
hw/acpi/pcihp.c
x01)
+{
+0x65
+}
+Return (EDSM (Arg0, Arg1, Arg2, Arg3, Local0))
+}
Signed-off-by: Igor Mammedov
---
tests/qtest/bios-tables-test-allowed-diff.h | 3 ---
tests/data/acpi/pc/DSDT.hpbrroot|
hotplug_handler is set on bus.
PS:
see ("pci: fix 'hotplugglable' property behavior") for details
why callback was introduced.
Signed-off-by: Igor Mammedov
---
include/hw/acpi/ich9.h | 1 +
include/hw/acpi/pcihp.h | 1 +
hw/acpi/acpi-pci-hotplug-stub.c | 4
Signed-off-by: Igor Mammedov
---
tests/qtest/bios-tables-test-allowed-diff.h | 37 --
tests/data/acpi/pc/DSDT | Bin 6360 -> 6454 bytes
tests/data/acpi/pc/DSDT.acpierst | Bin 6283 -> 6377 bytes
tests/data/acpi/pc/DSDT.ac
1 != One))
{
CDW1 |= 0x08
Signed-off-by: Igor Mammedov
---
tests/qtest/bios-tables-test-allowed-diff.h | 2 --
tests/data/acpi/pc/DSDT.hpbrroot| Bin 3081 -> 3115 bytes
tests/data/acpi/q35/DSDT.noacpihp | Bin 8252 -> 7932 bytes
3 files changed, 2 deletions(-)
diff
adds only EDSM method in DSDT
without any users. (the follow up patches will use it)
Signed-off-by: Igor Mammedov
---
hw/i386/acpi-build.c | 54
1 file changed, 54 insertions(+)
diff --git a/hw/i386/acpi-build.c b/hw/i386/acpi-build.c
index d8ec9
+{
+Local0 = Package (0x01)
+{
+0x66
+}
+Return (EDSM (Arg0, Arg1, Arg2, Arg3, Local0))
+}
+}
Signed-off-by: Igor Mammedov
---
tests/qtest/bios
)
+{
+Name (_ADR, 0x00020004) // _ADR: Address
+}
+
...
+Device (S22)
+{
+Name (_ADR, 0x00040002) // _ADR: Address
+}
Signed-off-by: Igor Mammedov
---
tests/qtest/bios-tables-test-allowed-diff.h | 37
follow up fix for missing root-port AML will affect these tests
by adding non-hotpluggable Device descriptors of colplugged
bridges when bridge hotplug is disabled.
Signed-off-by: Igor Mammedov
---
tests/qtest/bios-tables-test-allowed-diff.h | 3 +++
1 file changed, 3 insertions(+)
diff --git
escription handling.
BSEL is still used to decide if hotplug part should be called
but that will be moved out of generic code to hotplug one by
followup patches.
Signed-off-by: Igor Mammedov
---
hw/i386/acpi-build.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/i386/ac
Inject static _DSM (EDSM) if non-hotpluggable device has
acpi-index configured on it.
It lets use acpi-index non-hotpluggable devices / devices
attached to non-hotpluggable bus.
Signed-off-by: Igor Mammedov
---
hw/i386/acpi-build.c | 25 +
1 file changed, 25 insertions
Signed-off-by: Igor Mammedov
---
tests/qtest/bios-tables-test.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/tests/qtest/bios-tables-test.c b/tests/qtest/bios-tables-test.c
index 295d80740e..d7c34ba504 100644
--- a/tests/qtest/bios-tables-test.c
+++ b/tests/qtest/bios-tables-test.c
(Arg4 [One]
Signed-off-by: Igor Mammedov
---
tests/qtest/bios-tables-test-allowed-diff.h | 35 --
tests/data/acpi/pc/DSDT | Bin 6360 -> 6360 bytes
tests/data/acpi/pc/DSDT.acpierst | Bin 6283 -> 6283 bytes
tests/data/acpi/pc/DSDT.acpihmat
ilure.
Signed-off-by: Igor Mammedov
---
CC: Fabiano Rosas
tests/qtest/bios-tables-test.c | 75 ++
1 file changed, 4 insertions(+), 71 deletions(-)
diff --git a/tests/qtest/bios-tables-test.c b/tests/qtest/bios-tables-test.c
index d29a4e47af..d8c8cda58e 100644
--- a/
ns up generic code a bit from hotplug stuff
and follow up patch will remove remaining call to
build_append_pcihp_slots() from generic code, making
it possible to use without ACPI PCI hotplug dependencies.
Signed-off-by: Igor Mammedov
---
hw/i386/acpi-build.c | 15 ++-
1 file chang
ation chain that should accompany it.
Signed-off-by: Igor Mammedov
---
tests/qtest/bios-tables-test.c | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/tests/qtest/bios-tables-test.c b/tests/qtest/bios-tables-test.c
index 7828c6b7e6..295d80740e 100644
--- a/tests/qtest/bios-t
Signed-off-by: Igor Mammedov
---
include/hw/acpi/pcihp.h | 2 ++
hw/acpi/pci-bridge.c| 12 +++-
hw/i386/acpi-build.c| 9 -
3 files changed, 17 insertions(+), 6 deletions(-)
diff --git a/include/hw/acpi/pcihp.h b/include/hw/acpi/pcihp.h
index 7e268c2c9c..cd18ebdcdc
Signed-off-by: Igor Mammedov
---
tests/qtest/bios-tables-test-allowed-diff.h | 3 +++
1 file changed, 3 insertions(+)
diff --git a/tests/qtest/bios-tables-test-allowed-diff.h
b/tests/qtest/bios-tables-test-allowed-diff.h
index dfb8523c8b..70244976c9 100644
--- a/tests/qtest/bios-tables-test
Signed-off-by: Igor Mammedov
---
tests/qtest/bios-tables-test.c | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/tests/qtest/bios-tables-test.c b/tests/qtest/bios-tables-test.c
index d7c34ba504..76d5100911 100644
--- a/tests/qtest/bios-tables-test.c
+++ b/tests/qtest
has
not been properly wired and as result not worked.
1)
Fixes: ddab4d3fae4e8 ("pcihp: compose PCNT callchain right before its user
_GPE._E01")
Signed-off-by: Igor Mammedov
---
hw/i386/acpi-build.c | 22 +-
1 file changed, 13 insertions(+), 9 deletions(-)
diff --
Signed-off-by: Igor Mammedov
---
tests/qtest/bios-tables-test-allowed-diff.h | 37 +
1 file changed, 37 insertions(+)
diff --git a/tests/qtest/bios-tables-test-allowed-diff.h
b/tests/qtest/bios-tables-test-allowed-diff.h
index dfb8523c8b..8911b10650 100644
--- a/tests/qtest
it will be reused by follow up patches that will implement
static _DSM for non-hotpluggable devices.
no functional AML change, only context one, where 'cap' (Local1)
initialization is moved after UUID/revision checks.
Signed-off-by: Igor Mammedov
---
hw/i386/acpi-bu
(Introduce a new flag for i440fx to disable PCI hotplug on the
root bus)
2) df4008c9c59 (piix4: don't reserve hw resources when hotplug is off globally)
Fixes: df4008c9c59 (piix4: don't reserve hw resources when hotplug is off
globally)
Signed-off-by: Igor Mammedov
---
hw/acpi/piix
sue is to ask
hotplug_handler if bus it controls is hotpluggable
or not. To do that add is_hotpluggable_bus()
hook to HotplugHandler interface and use it in
'hotpluggable' property + teach pcie-slot to
actually look into 'hotplug' property state
before deciding if bus is ho
Signed-off-by: Igor Mammedov
---
tests/qtest/bios-tables-test-allowed-diff.h | 2 ++
1 file changed, 2 insertions(+)
diff --git a/tests/qtest/bios-tables-test-allowed-diff.h
b/tests/qtest/bios-tables-test-allowed-diff.h
index dfb8523c8b..ad2b429de8 100644
--- a/tests/qtest/bios-tables-test
Signed-off-by: Igor Mammedov
---
tests/qtest/bios-tables-test-allowed-diff.h | 35 +
1 file changed, 35 insertions(+)
diff --git a/tests/qtest/bios-tables-test-allowed-diff.h
b/tests/qtest/bios-tables-test-allowed-diff.h
index dfb8523c8b..7e7745db39 100644
--- a/tests/qtest
Signed-off-by: Igor Mammedov
---
tests/qtest/bios-tables-test-allowed-diff.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/tests/qtest/bios-tables-test-allowed-diff.h
b/tests/qtest/bios-tables-test-allowed-diff.h
index dfb8523c8b..dabc024f53 100644
--- a/tests/qtest/bios-tables-test
required).
Signed-off-by: Igor Mammedov
---
* patch is making acpi-index feature support complete in
the scope of host bridge (support of acpi-index on PXB
is out of scope of this series, and probably should follow
host-bridge clean up)
* follow up patch will add a tests case with acpi
Zero) // _ADR: Address
+}
}
Signed-off-by: Igor Mammedov
---
tests/qtest/bios-tables-test-allowed-diff.h | 4
tests/data/acpi/pc/DSDT.hpbridge| Bin 6289 -> 6323 bytes
tests/data/acpi/pc/DSDT.hpbrroot| Bin 3115 -> 3166 bytes
Signed-off-by: Igor Mammedov
---
tests/qtest/bios-tables-test-allowed-diff.h | 37 +
1 file changed, 37 insertions(+)
diff --git a/tests/qtest/bios-tables-test-allowed-diff.h
b/tests/qtest/bios-tables-test-allowed-diff.h
index dfb8523c8b..8911b10650 100644
--- a/tests/qtest
be
fixed by follow up patch
Signed-off-by: Igor Mammedov
---
tests/qtest/bios-tables-test.c | 42 +++---
1 file changed, 39 insertions(+), 3 deletions(-)
diff --git a/tests/qtest/bios-tables-test.c b/tests/qtest/bios-tables-test.c
index d8c8cda58e..7828c6b7e6 100644
... instead of duplicating them in piix4 and lpc and then
trying to pass them to pcihp routines as arguments.
it simplifies call sites and places pcihp specific in
its own structure.
Signed-off-by: Igor Mammedov
---
include/hw/acpi/pcihp.h | 8
include/hw/acpi/piix4.h
non-hotpluggable slots as I don't really want to
pull there complicated APCI PCI hotplug.
gitlab tree: https://gitlab.com/imammedo/qemu/-/tree/acpi_index_non_hotplug_v2
Igor Mammedov (34):
Revert "tests/qtest: Check for devices in bios-tables-test"
test
On Thu, 02 Mar 2023 12:43:11 -0300
Fabiano Rosas wrote:
> Igor Mammedov writes:
>
> > On Thu, 02 Mar 2023 09:55:24 -0300
> > Fabiano Rosas wrote:
> >
> >> Igor Mammedov writes:
> >>
> >> > On Wed, 1 Mar 2023 19:27:33 -0500
> &g
On Thu, 02 Mar 2023 09:55:24 -0300
Fabiano Rosas wrote:
> Igor Mammedov writes:
>
> > On Wed, 1 Mar 2023 19:27:33 -0500
> > "Michael S. Tsirkin" wrote:
> >
> >> On Fri, Feb 24, 2023 at 04:37:53PM +0100, Igor Mammedov wrote:
> >> >
On Tue, 31 Jan 2023 12:53:25 +0100
Bernhard Beschow wrote:
this lacks commit description explaining below movements
> Signed-off-by: Bernhard Beschow
> ---
> include/hw/i386/ich9.h | 8 +---
> hw/i386/acpi-build.c | 1 +
> hw/isa/lpc_ich9.c | 2 +-
> 3 files changed, 7 insertions(+)
On Mon, 6 Feb 2023 09:00:37 +0100
Philippe Mathieu-Daudé wrote:
> On 31/1/23 15:54, BALATON Zoltan wrote:
> > On Sun, 29 Jan 2023, Bernhard Beschow wrote:
> >> Adds missing functionality the real hardware supports.
> >>
> >> Signed-off-by: Bernhard Beschow
> >> ---
> >> hw/isa/vt82c686.c | 18
On Sun, 29 Jan 2023 14:55:06 +
Bernhard Beschow wrote:
> Am 25. Januar 2023 15:55:01 UTC schrieb Igor Mammedov :
> >On Sun, 22 Jan 2023 18:07:22 +0100
> >Bernhard Beschow wrote:
> >
> >> The PIIX4 datasheet defines the GPSTS register to be at offset 0x0c of
On Thu, 2 Mar 2023 05:59:16 -0500
"Michael S. Tsirkin" wrote:
> On Fri, Feb 24, 2023 at 04:37:39PM +0100, Igor Mammedov wrote:
> > Series extends acpi-index support to host-bridge(s) and bridges
> > with disabled hotplug (either explicitly or implicitly).
> > Wh
On Wed, 1 Mar 2023 19:27:33 -0500
"Michael S. Tsirkin" wrote:
> On Fri, Feb 24, 2023 at 04:37:53PM +0100, Igor Mammedov wrote:
> > only following context change:
> > - Local1 = Zero
> > If ((Arg0 != ToUUID ("e5c937d0-3553-4d7a-9117-ea4d19c3434d
On Mon, 13 Feb 2023 11:56:06 +0100
Philippe Mathieu-Daudé wrote:
> Experiment after discussing with Thomas around qdev_get_parent_bus:
> https://lore.kernel.org/qemu-devel/ad356f64-dca0-8117-d22a-a530e620d...@redhat.com/
>
> When a QDev plug on a QBus, we'll always use qdev_get_parent_bus()
> at
On Fri, 3 Feb 2023 19:09:02 +0100
Philippe Mathieu-Daudé wrote:
> No need to use the low-level QOM API when an object
> inherits from QDev. Directly use the QDev API to set
> its properties.
>
> Signed-off-by: Philippe Mathieu-Daudé
> ---
> hw/scsi/scsi-bus.c | 6 +-
> 1 file changed, 1 i
On Fri, 3 Feb 2023 19:09:00 +0100
Philippe Mathieu-Daudé wrote:
> No need to use the low-level QOM API when an object
> inherits from QDev. Directly use the QDev API to set
> its properties.
>
> Signed-off-by: Philippe Mathieu-Daudé
> ---
> hw/core/numa.c | 4 ++--
> 1 file changed, 2 inserti
On Fri, 3 Feb 2023 19:08:58 +0100
Philippe Mathieu-Daudé wrote:
> No need to use the low-level QOM API when an object
> inherits from QDev. Directly use the QDev API to set
> its properties.
>
> Signed-off-by: Philippe Mathieu-Daudé
> ---
> hw/acpi/cpu_hotplug.c | 7 +++
> hw/acpi/ich9.c
On Fri, 3 Feb 2023 19:09:07 +0100
Philippe Mathieu-Daudé wrote:
> No need to use the low-level QOM API when an object
> inherits from QDev. Directly use the QDev API to set
> its properties.
>
> Signed-off-by: Philippe Mathieu-Daudé
> ---
> hw/i386/pc.c | 8 +++-
> hw/i386/pc_q
On Fri, 3 Feb 2023 19:08:55 +0100
Philippe Mathieu-Daudé wrote:
> QEMU provides the QOM API for core objects.
> Devices are modelled on top of QOM as QDev objects.
>
> There is no point in using the lower level QOM API with
> QDev; it makes the code more complex and harder to review.
>
> I fir
On Tue, 28 Feb 2023 13:04:36 +0530
Sunil V L wrote:
> On Mon, Feb 27, 2023 at 04:41:21PM +0100, Igor Mammedov wrote:
> > On Fri, 24 Feb 2023 19:56:58 +0530
> > Sunil V L wrote:
> >
> > > Hi Igor,
> > >
> > > On Fri, Feb 24, 2023 at 01:53:43PM
On Tue, 28 Feb 2023 21:34:33 +0700
Bui Quang Minh wrote:
> On 2/27/23 23:07, Igor Mammedov wrote:
> > On Sat, 25 Feb 2023 17:15:17 +0700
> > Bui Quang Minh wrote:
> >
> >> On 2/24/23 21:29, Igor Mammedov wrote:
> >>> On Tue, 21 Feb 2023
On Fri, 24 Feb 2023 22:41:16 +0100
"Maciej S. Szmigiero" wrote:
> From: "Maciej S. Szmigiero"
>
> This driver is like virtio-balloon on steroids: it allows both changing the
> guest memory allocation via ballooning and inserting extra RAM into it by
> adding required memory backends and providi
On Tue, 28 Feb 2023 14:36:43 +0100
Philippe Mathieu-Daudé wrote:
> ping^2
please use checkpatch before pasting series.
Object -> DeviceState is a nice cleanup,
the rest is just unnecessary churn in my opinion and a matter of taste,
but I fine with it if it makes code easier to read
for someone
On Fri, 3 Feb 2023 17:30:21 +0100
Philippe Mathieu-Daudé wrote:
> ACPI CPU hotplug parent can't be any QOM object, it must be a QDev.
> Convert AcpiCpuHotplug::device field as QDev to enforce this.
> Rename 'owner' and 'device' variables as 'parent'.
>
> Signed-off-by: Philippe Mathieu-Daudé
>
On Fri, 24 Feb 2023 19:56:58 +0530
Sunil V L wrote:
> Hi Igor,
>
> On Fri, Feb 24, 2023 at 01:53:43PM +0100, Igor Mammedov wrote:
> > On Fri, 24 Feb 2023 14:06:58 +0530
> > Sunil V L wrote:
> >
> > > Add Multiple APIC Description Table (MADT) with th
On Sat, 25 Feb 2023 17:15:17 +0700
Bui Quang Minh wrote:
> On 2/24/23 21:29, Igor Mammedov wrote:
> > On Tue, 21 Feb 2023 23:04:57 +0700
> > Bui Quang Minh wrote:
> >
> >> This commit refactors APIC registers read/write function to support both
> >> MMI
has
not been properly wired and as result not worked.
1)
Fixes: ddab4d3fae4e8 ("pcihp: compose PCNT callchain right before its user
_GPE._E01")
Signed-off-by: Igor Mammedov
---
hw/i386/acpi-build.c | 22 +-
1 file changed, 13 insertions(+), 9 deletions(-)
diff --
Signed-off-by: Igor Mammedov
---
tests/qtest/bios-tables-test-allowed-diff.h | 35 +
1 file changed, 35 insertions(+)
diff --git a/tests/qtest/bios-tables-test-allowed-diff.h
b/tests/qtest/bios-tables-test-allowed-diff.h
index dfb8523c8b..7e7745db39 100644
--- a/tests/qtest
(Introduce a new flag for i440fx to disable PCI hotplug on the
root bus)
2) df4008c9c59 (piix4: don't reserve hw resources when hotplug is off globally)
Fixes: df4008c9c59 (piix4: don't reserve hw resources when hotplug is off
globally)
Signed-off-by: Igor Mammedov
---
hw/acpi/piix
... instead of duplicating them in piix4 and lpc and then
trying to pass them to pcihp routines as arguments.
it simplifies call sites and places pcihp specific in
its own structure.
Signed-off-by: Igor Mammedov
---
include/hw/acpi/pcihp.h | 8
include/hw/acpi/piix4.h
ug is disabled).
Move check and related code to generic PCIDevice so it
would be independent of ACPI PCI hotplug.
Signed-off-by: Igor Mammedov
---
PS: this also one step closer to enabling acpi-index
support for microvm and virt/arm machines.
---
hw/acpi/pcihp.c
escription handling.
BSEL is still used to decide if hotplug part should be called
but that will be moved out of generic code to hotplug one by
followup patches.
Signed-off-by: Igor Mammedov
---
hw/i386/acpi-build.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/i386/ac
ns up generic code a bit from hotplug stuff
and follow up patch will remove remaining call to
build_append_pcihp_slots() from generic code, making
it possible to use without ACPI PCI hotplug dependencies.
Signed-off-by: Igor Mammedov
---
hw/i386/acpi-build.c | 15 ++-
1 file chang
1 != One))
{
CDW1 |= 0x08
Signed-off-by: Igor Mammedov
---
tests/qtest/bios-tables-test-allowed-diff.h | 2 --
tests/data/acpi/pc/DSDT.hpbrroot| Bin 3081 -> 3115 bytes
tests/data/acpi/q35/DSDT.noacpihp | Bin 8252 -> 7932 bytes
3 files changed, 2 deletions(-)
diff
Signed-off-by: Igor Mammedov
---
tests/qtest/bios-tables-test.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/tests/qtest/bios-tables-test.c b/tests/qtest/bios-tables-test.c
index 638a193e13..88493abc40 100644
--- a/tests/qtest/bios-tables-test.c
+++ b/tests/qtest/bios-tables-test.c
Signed-off-by: Igor Mammedov
---
tests/qtest/bios-tables-test-allowed-diff.h | 37 --
tests/data/acpi/pc/DSDT | Bin 6360 -> 6454 bytes
tests/data/acpi/pc/DSDT.acpierst | Bin 6283 -> 6377 bytes
tests/data/acpi/pc/DSDT.ac
sue is to ask
hotplug_handler if bus it controls is hotpluggable
or not. To do that add is_hotpluggable_bus()
hook to HotplugHandler interface and use it in
'hotpluggable' property + teach pcie-slot to
actually look into 'hotplug' property state
before deciding if bus is ho
(Arg4 [One]
Signed-off-by: Igor Mammedov
---
tests/qtest/bios-tables-test-allowed-diff.h | 35 --
tests/data/acpi/pc/DSDT | Bin 6360 -> 6360 bytes
tests/data/acpi/pc/DSDT.acpierst | Bin 6283 -> 6283 bytes
tests/data/acpi/pc/DSDT.acpihmat
adds only EDSM method in DSDT
without any users. (the follow up patches will use it)
Signed-off-by: Igor Mammedov
---
hw/i386/acpi-build.c | 54
1 file changed, 54 insertions(+)
diff --git a/hw/i386/acpi-build.c b/hw/i386/acpi-build.c
index cb80b
it will be reused by follow up patches that will implement
static _DSM for non-hotpluggable devices.
no functional AML change, only context one, where 'cap' (Local1)
initialization is moved after UUID/revision checks.
Signed-off-by: Igor Mammedov
---
hw/i386/acpi-bu
Signed-off-by: Igor Mammedov
---
tests/qtest/bios-tables-test-allowed-diff.h | 3 +++
1 file changed, 3 insertions(+)
diff --git a/tests/qtest/bios-tables-test-allowed-diff.h
b/tests/qtest/bios-tables-test-allowed-diff.h
index dfb8523c8b..70244976c9 100644
--- a/tests/qtest/bios-tables-test
On Wed, 8 Feb 2023 09:06:48 +0800
Bin Meng wrote:
> On Wed, Feb 8, 2023 at 2:15 AM Sunil V L wrote:
> >
> > On Wed, Feb 08, 2023 at 12:10:55AM +0800, Bin Meng wrote:
> > > On Mon, Feb 6, 2023 at 9:24 PM Sunil V L
> > > wrote:
> > > >
> > > > On Mon, Feb 06, 2023 at 06:17:50PM +0800, Bin Me
follow up fix for missing root-port AML will affect these tests
by adding non-hotpluggable Device descriptors of colplugged
bridges when bridge hotplug is disabled.
Signed-off-by: Igor Mammedov
---
tests/qtest/bios-tables-test-allowed-diff.h | 3 +++
1 file changed, 3 insertions(+)
diff --git
Signed-off-by: Igor Mammedov
---
tests/qtest/bios-tables-test.c | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/tests/qtest/bios-tables-test.c b/tests/qtest/bios-tables-test.c
index 88493abc40..259405724e 100644
--- a/tests/qtest/bios-tables-test.c
+++ b/tests/qtest
s: 6c36ec46b0d ("pcihp: make bridge describe itself using
AcpiDevAmlIfClass:build_dev_aml")
Signed-off-by: Igor Mammedov
---
hw/acpi/pci-bridge.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/acpi/pci-bridge.c b/hw/acpi/pci-bridge.c
index 5f3ee5157f..4fbf6da6ad
be
fixed by follow up patch
Signed-off-by: Igor Mammedov
---
tests/qtest/bios-tables-test.c | 42 +++---
1 file changed, 39 insertions(+), 3 deletions(-)
diff --git a/tests/qtest/bios-tables-test.c b/tests/qtest/bios-tables-test.c
index d29a4e47af..5128b62634 100644
)
+{
+Name (_ADR, 0x00020004) // _ADR: Address
+}
+
...
+Device (S22)
+{
+Name (_ADR, 0x00040002) // _ADR: Address
+}
Signed-off-by: Igor Mammedov
---
tests/qtest/bios-tables-test-allowed-diff.h | 37
Signed-off-by: Igor Mammedov
---
tests/qtest/bios-tables-test-allowed-diff.h | 2 ++
1 file changed, 2 insertions(+)
diff --git a/tests/qtest/bios-tables-test-allowed-diff.h
b/tests/qtest/bios-tables-test-allowed-diff.h
index dfb8523c8b..ad2b429de8 100644
--- a/tests/qtest/bios-tables-test
required).
Signed-off-by: Igor Mammedov
---
* patch is making acpi-index feature support complete in
the scope of host bridge (support of acpi-index on PXB
is out of scope of this series, and probably should follow
host-bridge clean up)
* follow up patch will add a tests case with acpi
for q35.noacpihp use plain default Q35 DSDT table as a starting point.
Signed-off-by: Igor Mammedov
---
tests/qtest/bios-tables-test-allowed-diff.h | 2 ++
tests/data/acpi/q35/DSDT.noacpihp | Bin 0 -> 8252 bytes
2 files changed, 2 insertions(+)
create mode 100644 tests/data/a
Signed-off-by: Igor Mammedov
---
include/hw/acpi/pcihp.h | 2 ++
hw/acpi/pci-bridge.c| 12 +++-
hw/i386/acpi-build.c| 9 -
3 files changed, 17 insertions(+), 6 deletions(-)
diff --git a/include/hw/acpi/pcihp.h b/include/hw/acpi/pcihp.h
index 7e268c2c9c..cd18ebdcdc
hotplug_handler is set on bus.
PS:
see ("pci: fix 'hotplugglable' property behavior") for details
why callback was introduced.
Signed-off-by: Igor Mammedov
---
include/hw/acpi/ich9.h | 1 +
include/hw/acpi/pcihp.h | 1 +
hw/acpi/acpi-pci-hotplug-stub.c | 4
Zero) // _ADR: Address
+}
}
Signed-off-by: Igor Mammedov
---
tests/qtest/bios-tables-test-allowed-diff.h | 4
tests/data/acpi/pc/DSDT.hpbridge| Bin 6289 -> 6323 bytes
tests/data/acpi/pc/DSDT.hpbrroot| Bin 3115 -> 3166 bytes
Signed-off-by: Igor Mammedov
---
tests/qtest/bios-tables-test-allowed-diff.h | 37 +
1 file changed, 37 insertions(+)
diff --git a/tests/qtest/bios-tables-test-allowed-diff.h
b/tests/qtest/bios-tables-test-allowed-diff.h
index dfb8523c8b..8911b10650 100644
--- a/tests/qtest
Signed-off-by: Igor Mammedov
---
tests/qtest/bios-tables-test-allowed-diff.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/tests/qtest/bios-tables-test-allowed-diff.h
b/tests/qtest/bios-tables-test-allowed-diff.h
index dfb8523c8b..dabc024f53 100644
--- a/tests/qtest/bios-tables-test
Signed-off-by: Igor Mammedov
---
tests/qtest/bios-tables-test-allowed-diff.h | 37 +
1 file changed, 37 insertions(+)
diff --git a/tests/qtest/bios-tables-test-allowed-diff.h
b/tests/qtest/bios-tables-test-allowed-diff.h
index dfb8523c8b..8911b10650 100644
--- a/tests/qtest
601 - 700 of 5946 matches
Mail list logo
