* [PATCH] hw/acpi/pcihp: Fix typo in function name
@ 2025-05-04 21:56 Gustavo Romero
2025-05-05 7:13 ` Eric Auger
0 siblings, 1 reply; 6+ messages in thread
From: Gustavo Romero @ 2025-05-04 21:56 UTC (permalink / raw)
To: qemu-devel; +Cc: qemu-trivial, philmd, alex.bennee, gustavo.romero
Fix typo in QEMU's ACPI PCI hotplug API function name that checks
whether a given bus is hotplug-capable.
Signed-off-by: Gustavo Romero <gustavo.romero@linaro.org>
---
hw/acpi/acpi-pci-hotplug-stub.c | 2 +-
hw/acpi/ich9.c | 2 +-
hw/acpi/pcihp.c | 2 +-
hw/acpi/piix4.c | 2 +-
include/hw/acpi/pcihp.h | 2 +-
5 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/hw/acpi/acpi-pci-hotplug-stub.c b/hw/acpi/acpi-pci-hotplug-stub.c
index b67b4a92da..b7bc6e40a1 100644
--- a/hw/acpi/acpi-pci-hotplug-stub.c
+++ b/hw/acpi/acpi-pci-hotplug-stub.c
@@ -34,7 +34,7 @@ void acpi_pcihp_reset(AcpiPciHpState *s)
{
}
-bool acpi_pcihp_is_hotpluggbale_bus(AcpiPciHpState *s, BusState *bus)
+bool acpi_pcihp_is_hotpluggable_bus(AcpiPciHpState *s, BusState *bus)
{
return true;
}
diff --git a/hw/acpi/ich9.c b/hw/acpi/ich9.c
index c7a735bf64..a07c9d115b 100644
--- a/hw/acpi/ich9.c
+++ b/hw/acpi/ich9.c
@@ -570,7 +570,7 @@ void ich9_pm_device_unplug_cb(HotplugHandler *hotplug_dev, DeviceState *dev,
bool ich9_pm_is_hotpluggable_bus(HotplugHandler *hotplug_dev, BusState *bus)
{
ICH9LPCState *lpc = ICH9_LPC_DEVICE(hotplug_dev);
- return acpi_pcihp_is_hotpluggbale_bus(&lpc->pm.acpi_pci_hotplug, bus);
+ return acpi_pcihp_is_hotpluggable_bus(&lpc->pm.acpi_pci_hotplug, bus);
}
void ich9_pm_ospm_status(AcpiDeviceIf *adev, ACPIOSTInfoList ***list)
diff --git a/hw/acpi/pcihp.c b/hw/acpi/pcihp.c
index 5f79c9016b..aac90013d4 100644
--- a/hw/acpi/pcihp.c
+++ b/hw/acpi/pcihp.c
@@ -371,7 +371,7 @@ void acpi_pcihp_device_unplug_request_cb(HotplugHandler *hotplug_dev,
acpi_send_event(DEVICE(hotplug_dev), ACPI_PCI_HOTPLUG_STATUS);
}
-bool acpi_pcihp_is_hotpluggbale_bus(AcpiPciHpState *s, BusState *bus)
+bool acpi_pcihp_is_hotpluggable_bus(AcpiPciHpState *s, BusState *bus)
{
Object *o = OBJECT(bus->parent);
diff --git a/hw/acpi/piix4.c b/hw/acpi/piix4.c
index b16d45f03e..d98b80df6d 100644
--- a/hw/acpi/piix4.c
+++ b/hw/acpi/piix4.c
@@ -406,7 +406,7 @@ static bool piix4_is_hotpluggable_bus(HotplugHandler *hotplug_dev,
BusState *bus)
{
PIIX4PMState *s = PIIX4_PM(hotplug_dev);
- return acpi_pcihp_is_hotpluggbale_bus(&s->acpi_pci_hotplug, bus);
+ return acpi_pcihp_is_hotpluggable_bus(&s->acpi_pci_hotplug, bus);
}
static void piix4_pm_machine_ready(Notifier *n, void *opaque)
diff --git a/include/hw/acpi/pcihp.h b/include/hw/acpi/pcihp.h
index ac21a95913..a97904bada 100644
--- a/include/hw/acpi/pcihp.h
+++ b/include/hw/acpi/pcihp.h
@@ -58,7 +58,7 @@ typedef struct AcpiPciHpState {
void acpi_pcihp_init(Object *owner, AcpiPciHpState *, PCIBus *root,
MemoryRegion *io, uint16_t io_base);
-bool acpi_pcihp_is_hotpluggbale_bus(AcpiPciHpState *s, BusState *bus);
+bool acpi_pcihp_is_hotpluggable_bus(AcpiPciHpState *s, BusState *bus);
void acpi_pcihp_device_pre_plug_cb(HotplugHandler *hotplug_dev,
DeviceState *dev, Error **errp);
void acpi_pcihp_device_plug_cb(HotplugHandler *hotplug_dev, AcpiPciHpState *s,
--
2.34.1
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [PATCH] hw/acpi/pcihp: Fix typo in function name
2025-05-04 21:56 [PATCH] hw/acpi/pcihp: Fix typo in function name Gustavo Romero
@ 2025-05-05 7:13 ` Eric Auger
2025-05-05 12:29 ` Gustavo Romero
0 siblings, 1 reply; 6+ messages in thread
From: Eric Auger @ 2025-05-05 7:13 UTC (permalink / raw)
To: Gustavo Romero, qemu-devel; +Cc: qemu-trivial, philmd, alex.bennee
On 5/4/25 11:56 PM, Gustavo Romero wrote:
> Fix typo in QEMU's ACPI PCI hotplug API function name that checks
> whether a given bus is hotplug-capable.
>
> Signed-off-by: Gustavo Romero <gustavo.romero@linaro.org>
Reviewed-by: Eric Auger <eric.auger@redhat.com>
For awareness, there is a similar typo in pci-port.
see
https://lore.kernel.org/all/20250428102628.378046-2-eric.auger@redhat.com/
another trivial indentation fix in generic_event_device.c
https://lore.kernel.org/all/20250428102628.378046-3-eric.auger@redhat.com/
Eric
> ---
> hw/acpi/acpi-pci-hotplug-stub.c | 2 +-
> hw/acpi/ich9.c | 2 +-
> hw/acpi/pcihp.c | 2 +-
> hw/acpi/piix4.c | 2 +-
> include/hw/acpi/pcihp.h | 2 +-
> 5 files changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/hw/acpi/acpi-pci-hotplug-stub.c b/hw/acpi/acpi-pci-hotplug-stub.c
> index b67b4a92da..b7bc6e40a1 100644
> --- a/hw/acpi/acpi-pci-hotplug-stub.c
> +++ b/hw/acpi/acpi-pci-hotplug-stub.c
> @@ -34,7 +34,7 @@ void acpi_pcihp_reset(AcpiPciHpState *s)
> {
> }
>
> -bool acpi_pcihp_is_hotpluggbale_bus(AcpiPciHpState *s, BusState *bus)
> +bool acpi_pcihp_is_hotpluggable_bus(AcpiPciHpState *s, BusState *bus)
> {
> return true;
> }
> diff --git a/hw/acpi/ich9.c b/hw/acpi/ich9.c
> index c7a735bf64..a07c9d115b 100644
> --- a/hw/acpi/ich9.c
> +++ b/hw/acpi/ich9.c
> @@ -570,7 +570,7 @@ void ich9_pm_device_unplug_cb(HotplugHandler *hotplug_dev, DeviceState *dev,
> bool ich9_pm_is_hotpluggable_bus(HotplugHandler *hotplug_dev, BusState *bus)
> {
> ICH9LPCState *lpc = ICH9_LPC_DEVICE(hotplug_dev);
> - return acpi_pcihp_is_hotpluggbale_bus(&lpc->pm.acpi_pci_hotplug, bus);
> + return acpi_pcihp_is_hotpluggable_bus(&lpc->pm.acpi_pci_hotplug, bus);
> }
>
> void ich9_pm_ospm_status(AcpiDeviceIf *adev, ACPIOSTInfoList ***list)
> diff --git a/hw/acpi/pcihp.c b/hw/acpi/pcihp.c
> index 5f79c9016b..aac90013d4 100644
> --- a/hw/acpi/pcihp.c
> +++ b/hw/acpi/pcihp.c
> @@ -371,7 +371,7 @@ void acpi_pcihp_device_unplug_request_cb(HotplugHandler *hotplug_dev,
> acpi_send_event(DEVICE(hotplug_dev), ACPI_PCI_HOTPLUG_STATUS);
> }
>
> -bool acpi_pcihp_is_hotpluggbale_bus(AcpiPciHpState *s, BusState *bus)
> +bool acpi_pcihp_is_hotpluggable_bus(AcpiPciHpState *s, BusState *bus)
> {
> Object *o = OBJECT(bus->parent);
>
> diff --git a/hw/acpi/piix4.c b/hw/acpi/piix4.c
> index b16d45f03e..d98b80df6d 100644
> --- a/hw/acpi/piix4.c
> +++ b/hw/acpi/piix4.c
> @@ -406,7 +406,7 @@ static bool piix4_is_hotpluggable_bus(HotplugHandler *hotplug_dev,
> BusState *bus)
> {
> PIIX4PMState *s = PIIX4_PM(hotplug_dev);
> - return acpi_pcihp_is_hotpluggbale_bus(&s->acpi_pci_hotplug, bus);
> + return acpi_pcihp_is_hotpluggable_bus(&s->acpi_pci_hotplug, bus);
> }
>
> static void piix4_pm_machine_ready(Notifier *n, void *opaque)
> diff --git a/include/hw/acpi/pcihp.h b/include/hw/acpi/pcihp.h
> index ac21a95913..a97904bada 100644
> --- a/include/hw/acpi/pcihp.h
> +++ b/include/hw/acpi/pcihp.h
> @@ -58,7 +58,7 @@ typedef struct AcpiPciHpState {
> void acpi_pcihp_init(Object *owner, AcpiPciHpState *, PCIBus *root,
> MemoryRegion *io, uint16_t io_base);
>
> -bool acpi_pcihp_is_hotpluggbale_bus(AcpiPciHpState *s, BusState *bus);
> +bool acpi_pcihp_is_hotpluggable_bus(AcpiPciHpState *s, BusState *bus);
> void acpi_pcihp_device_pre_plug_cb(HotplugHandler *hotplug_dev,
> DeviceState *dev, Error **errp);
> void acpi_pcihp_device_plug_cb(HotplugHandler *hotplug_dev, AcpiPciHpState *s,
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH] hw/acpi/pcihp: Fix typo in function name
2025-05-05 7:13 ` Eric Auger
@ 2025-05-05 12:29 ` Gustavo Romero
2025-05-05 14:19 ` Eric Auger
0 siblings, 1 reply; 6+ messages in thread
From: Gustavo Romero @ 2025-05-05 12:29 UTC (permalink / raw)
To: eric.auger, qemu-devel, Alex Bennée; +Cc: qemu-trivial, philmd
Hi Eric,
On 5/5/25 04:13, Eric Auger wrote:
>
>
> On 5/4/25 11:56 PM, Gustavo Romero wrote:
>> Fix typo in QEMU's ACPI PCI hotplug API function name that checks
>> whether a given bus is hotplug-capable.
>>
>> Signed-off-by: Gustavo Romero <gustavo.romero@linaro.org>
> Reviewed-by: Eric Auger <eric.auger@redhat.com>
>
> For awareness, there is a similar typo in pci-port.
> see
> https://lore.kernel.org/all/20250428102628.378046-2-eric.auger@redhat.com/
hmm it's unfortunate that I was about to publish a series to exactly
enable the APCI PCI hotplug on ARM. Next time I should try to sync better
with Red Hat folks on the work fronts...
I've skimmed through your RFC and it seems to be doing mostly of what I
was doing for my series, so I'll reviewed it formally today, it should be quick.
Anyways, thanks for the heads up.
Cheers,
Gustavo
> another trivial indentation fix in generic_event_device.c
> https://lore.kernel.org/all/20250428102628.378046-3-eric.auger@redhat.com/
>
> Eric
>
>
>
>
>> ---
>> hw/acpi/acpi-pci-hotplug-stub.c | 2 +-
>> hw/acpi/ich9.c | 2 +-
>> hw/acpi/pcihp.c | 2 +-
>> hw/acpi/piix4.c | 2 +-
>> include/hw/acpi/pcihp.h | 2 +-
>> 5 files changed, 5 insertions(+), 5 deletions(-)
>>
>> diff --git a/hw/acpi/acpi-pci-hotplug-stub.c b/hw/acpi/acpi-pci-hotplug-stub.c
>> index b67b4a92da..b7bc6e40a1 100644
>> --- a/hw/acpi/acpi-pci-hotplug-stub.c
>> +++ b/hw/acpi/acpi-pci-hotplug-stub.c
>> @@ -34,7 +34,7 @@ void acpi_pcihp_reset(AcpiPciHpState *s)
>> {
>> }
>>
>> -bool acpi_pcihp_is_hotpluggbale_bus(AcpiPciHpState *s, BusState *bus)
>> +bool acpi_pcihp_is_hotpluggable_bus(AcpiPciHpState *s, BusState *bus)
>> {
>> return true;
>> }
>> diff --git a/hw/acpi/ich9.c b/hw/acpi/ich9.c
>> index c7a735bf64..a07c9d115b 100644
>> --- a/hw/acpi/ich9.c
>> +++ b/hw/acpi/ich9.c
>> @@ -570,7 +570,7 @@ void ich9_pm_device_unplug_cb(HotplugHandler *hotplug_dev, DeviceState *dev,
>> bool ich9_pm_is_hotpluggable_bus(HotplugHandler *hotplug_dev, BusState *bus)
>> {
>> ICH9LPCState *lpc = ICH9_LPC_DEVICE(hotplug_dev);
>> - return acpi_pcihp_is_hotpluggbale_bus(&lpc->pm.acpi_pci_hotplug, bus);
>> + return acpi_pcihp_is_hotpluggable_bus(&lpc->pm.acpi_pci_hotplug, bus);
>> }
>>
>> void ich9_pm_ospm_status(AcpiDeviceIf *adev, ACPIOSTInfoList ***list)
>> diff --git a/hw/acpi/pcihp.c b/hw/acpi/pcihp.c
>> index 5f79c9016b..aac90013d4 100644
>> --- a/hw/acpi/pcihp.c
>> +++ b/hw/acpi/pcihp.c
>> @@ -371,7 +371,7 @@ void acpi_pcihp_device_unplug_request_cb(HotplugHandler *hotplug_dev,
>> acpi_send_event(DEVICE(hotplug_dev), ACPI_PCI_HOTPLUG_STATUS);
>> }
>>
>> -bool acpi_pcihp_is_hotpluggbale_bus(AcpiPciHpState *s, BusState *bus)
>> +bool acpi_pcihp_is_hotpluggable_bus(AcpiPciHpState *s, BusState *bus)
>> {
>> Object *o = OBJECT(bus->parent);
>>
>> diff --git a/hw/acpi/piix4.c b/hw/acpi/piix4.c
>> index b16d45f03e..d98b80df6d 100644
>> --- a/hw/acpi/piix4.c
>> +++ b/hw/acpi/piix4.c
>> @@ -406,7 +406,7 @@ static bool piix4_is_hotpluggable_bus(HotplugHandler *hotplug_dev,
>> BusState *bus)
>> {
>> PIIX4PMState *s = PIIX4_PM(hotplug_dev);
>> - return acpi_pcihp_is_hotpluggbale_bus(&s->acpi_pci_hotplug, bus);
>> + return acpi_pcihp_is_hotpluggable_bus(&s->acpi_pci_hotplug, bus);
>> }
>>
>> static void piix4_pm_machine_ready(Notifier *n, void *opaque)
>> diff --git a/include/hw/acpi/pcihp.h b/include/hw/acpi/pcihp.h
>> index ac21a95913..a97904bada 100644
>> --- a/include/hw/acpi/pcihp.h
>> +++ b/include/hw/acpi/pcihp.h
>> @@ -58,7 +58,7 @@ typedef struct AcpiPciHpState {
>> void acpi_pcihp_init(Object *owner, AcpiPciHpState *, PCIBus *root,
>> MemoryRegion *io, uint16_t io_base);
>>
>> -bool acpi_pcihp_is_hotpluggbale_bus(AcpiPciHpState *s, BusState *bus);
>> +bool acpi_pcihp_is_hotpluggable_bus(AcpiPciHpState *s, BusState *bus);
>> void acpi_pcihp_device_pre_plug_cb(HotplugHandler *hotplug_dev,
>> DeviceState *dev, Error **errp);
>> void acpi_pcihp_device_plug_cb(HotplugHandler *hotplug_dev, AcpiPciHpState *s,
>
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH] hw/acpi/pcihp: Fix typo in function name
2025-05-05 12:29 ` Gustavo Romero
@ 2025-05-05 14:19 ` Eric Auger
2025-05-06 0:26 ` Gustavo Romero
0 siblings, 1 reply; 6+ messages in thread
From: Eric Auger @ 2025-05-05 14:19 UTC (permalink / raw)
To: Gustavo Romero, qemu-devel, Alex Bennée; +Cc: qemu-trivial, philmd
Hi Gustavo,
On 5/5/25 2:29 PM, Gustavo Romero wrote:
> Hi Eric,
>
> On 5/5/25 04:13, Eric Auger wrote:
>>
>>
>> On 5/4/25 11:56 PM, Gustavo Romero wrote:
>>> Fix typo in QEMU's ACPI PCI hotplug API function name that checks
>>> whether a given bus is hotplug-capable.
>>>
>>> Signed-off-by: Gustavo Romero <gustavo.romero@linaro.org>
>> Reviewed-by: Eric Auger <eric.auger@redhat.com>
>>
>> For awareness, there is a similar typo in pci-port.
>> see
>> https://lore.kernel.org/all/20250428102628.378046-2-eric.auger@redhat.com/
>>
>
> hmm it's unfortunate that I was about to publish a series to exactly
> enable the APCI PCI hotplug on ARM. Next time I should try to sync better
> with Red Hat folks on the work fronts...
Sorry to hear that. That's always a frustrating situation :-(
>
> I've skimmed through your RFC and it seems to be doing mostly of what I
> was doing for my series, so I'll reviewed it formally today, it should
> be quick.
I guess we may need to add some additional qtests for hotplug (just as
they have on x86). Maybe we can collaborate on this or you already have
some stuff under work.
Thanks
Eric
>
> Anyways, thanks for the heads up.
>
>
> Cheers,
> Gustavo
>
>> another trivial indentation fix in generic_event_device.c
>> https://lore.kernel.org/all/20250428102628.378046-3-eric.auger@redhat.com/
>>
>>
>> Eric
>>
>>
>>
>>
>>> ---
>>> hw/acpi/acpi-pci-hotplug-stub.c | 2 +-
>>> hw/acpi/ich9.c | 2 +-
>>> hw/acpi/pcihp.c | 2 +-
>>> hw/acpi/piix4.c | 2 +-
>>> include/hw/acpi/pcihp.h | 2 +-
>>> 5 files changed, 5 insertions(+), 5 deletions(-)
>>>
>>> diff --git a/hw/acpi/acpi-pci-hotplug-stub.c
>>> b/hw/acpi/acpi-pci-hotplug-stub.c
>>> index b67b4a92da..b7bc6e40a1 100644
>>> --- a/hw/acpi/acpi-pci-hotplug-stub.c
>>> +++ b/hw/acpi/acpi-pci-hotplug-stub.c
>>> @@ -34,7 +34,7 @@ void acpi_pcihp_reset(AcpiPciHpState *s)
>>> {
>>> }
>>> -bool acpi_pcihp_is_hotpluggbale_bus(AcpiPciHpState *s, BusState
>>> *bus)
>>> +bool acpi_pcihp_is_hotpluggable_bus(AcpiPciHpState *s, BusState *bus)
>>> {
>>> return true;
>>> }
>>> diff --git a/hw/acpi/ich9.c b/hw/acpi/ich9.c
>>> index c7a735bf64..a07c9d115b 100644
>>> --- a/hw/acpi/ich9.c
>>> +++ b/hw/acpi/ich9.c
>>> @@ -570,7 +570,7 @@ void ich9_pm_device_unplug_cb(HotplugHandler
>>> *hotplug_dev, DeviceState *dev,
>>> bool ich9_pm_is_hotpluggable_bus(HotplugHandler *hotplug_dev,
>>> BusState *bus)
>>> {
>>> ICH9LPCState *lpc = ICH9_LPC_DEVICE(hotplug_dev);
>>> - return
>>> acpi_pcihp_is_hotpluggbale_bus(&lpc->pm.acpi_pci_hotplug, bus);
>>> + return
>>> acpi_pcihp_is_hotpluggable_bus(&lpc->pm.acpi_pci_hotplug, bus);
>>> }
>>> void ich9_pm_ospm_status(AcpiDeviceIf *adev, ACPIOSTInfoList
>>> ***list)
>>> diff --git a/hw/acpi/pcihp.c b/hw/acpi/pcihp.c
>>> index 5f79c9016b..aac90013d4 100644
>>> --- a/hw/acpi/pcihp.c
>>> +++ b/hw/acpi/pcihp.c
>>> @@ -371,7 +371,7 @@ void
>>> acpi_pcihp_device_unplug_request_cb(HotplugHandler *hotplug_dev,
>>> acpi_send_event(DEVICE(hotplug_dev), ACPI_PCI_HOTPLUG_STATUS);
>>> }
>>> -bool acpi_pcihp_is_hotpluggbale_bus(AcpiPciHpState *s, BusState
>>> *bus)
>>> +bool acpi_pcihp_is_hotpluggable_bus(AcpiPciHpState *s, BusState *bus)
>>> {
>>> Object *o = OBJECT(bus->parent);
>>> diff --git a/hw/acpi/piix4.c b/hw/acpi/piix4.c
>>> index b16d45f03e..d98b80df6d 100644
>>> --- a/hw/acpi/piix4.c
>>> +++ b/hw/acpi/piix4.c
>>> @@ -406,7 +406,7 @@ static bool
>>> piix4_is_hotpluggable_bus(HotplugHandler *hotplug_dev,
>>> BusState *bus)
>>> {
>>> PIIX4PMState *s = PIIX4_PM(hotplug_dev);
>>> - return acpi_pcihp_is_hotpluggbale_bus(&s->acpi_pci_hotplug, bus);
>>> + return acpi_pcihp_is_hotpluggable_bus(&s->acpi_pci_hotplug, bus);
>>> }
>>> static void piix4_pm_machine_ready(Notifier *n, void *opaque)
>>> diff --git a/include/hw/acpi/pcihp.h b/include/hw/acpi/pcihp.h
>>> index ac21a95913..a97904bada 100644
>>> --- a/include/hw/acpi/pcihp.h
>>> +++ b/include/hw/acpi/pcihp.h
>>> @@ -58,7 +58,7 @@ typedef struct AcpiPciHpState {
>>> void acpi_pcihp_init(Object *owner, AcpiPciHpState *, PCIBus *root,
>>> MemoryRegion *io, uint16_t io_base);
>>> -bool acpi_pcihp_is_hotpluggbale_bus(AcpiPciHpState *s, BusState
>>> *bus);
>>> +bool acpi_pcihp_is_hotpluggable_bus(AcpiPciHpState *s, BusState *bus);
>>> void acpi_pcihp_device_pre_plug_cb(HotplugHandler *hotplug_dev,
>>> DeviceState *dev, Error **errp);
>>> void acpi_pcihp_device_plug_cb(HotplugHandler *hotplug_dev,
>>> AcpiPciHpState *s,
>>
>
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH] hw/acpi/pcihp: Fix typo in function name
2025-05-05 14:19 ` Eric Auger
@ 2025-05-06 0:26 ` Gustavo Romero
2025-05-06 8:58 ` Eric Auger
0 siblings, 1 reply; 6+ messages in thread
From: Gustavo Romero @ 2025-05-06 0:26 UTC (permalink / raw)
To: eric.auger, qemu-devel, Alex Bennée; +Cc: qemu-trivial, philmd
Hi Eric,
On 5/5/25 11:19, Eric Auger wrote:
> Hi Gustavo,
>
> On 5/5/25 2:29 PM, Gustavo Romero wrote:
>> Hi Eric,
>>
>> On 5/5/25 04:13, Eric Auger wrote:
>>>
>>>
>>> On 5/4/25 11:56 PM, Gustavo Romero wrote:
>>>> Fix typo in QEMU's ACPI PCI hotplug API function name that checks
>>>> whether a given bus is hotplug-capable.
>>>>
>>>> Signed-off-by: Gustavo Romero <gustavo.romero@linaro.org>
>>> Reviewed-by: Eric Auger <eric.auger@redhat.com>
>>>
>>> For awareness, there is a similar typo in pci-port.
>>> see
>>> https://lore.kernel.org/all/20250428102628.378046-2-eric.auger@redhat.com/
>>>
>>
>> hmm it's unfortunate that I was about to publish a series to exactly
>> enable the APCI PCI hotplug on ARM. Next time I should try to sync better
>> with Red Hat folks on the work fronts...
> Sorry to hear that. That's always a frustrating situation :-(
Yes...
>> I've skimmed through your RFC and it seems to be doing mostly of what I
>> was doing for my series, so I'll reviewed it formally today, it should
>> be quick.
> I guess we may need to add some additional qtests for hotplug (just as
> they have on x86). Maybe we can collaborate on this or you already have
> some stuff under work.
Sure, the new test I'm baking is the one I've used in the smoke test. I'll
contribute it on top of your series so.
For the x86 "parity", which tests you're thinking of to add for arm64?
Cheers,
Gustavo
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] hw/acpi/pcihp: Fix typo in function name
2025-05-06 0:26 ` Gustavo Romero
@ 2025-05-06 8:58 ` Eric Auger
0 siblings, 0 replies; 6+ messages in thread
From: Eric Auger @ 2025-05-06 8:58 UTC (permalink / raw)
To: Gustavo Romero, qemu-devel, Alex Bennée; +Cc: qemu-trivial, philmd
Hi Gustavo,
On 5/6/25 2:26 AM, Gustavo Romero wrote:
> Hi Eric,
>
> On 5/5/25 11:19, Eric Auger wrote:
>> Hi Gustavo,
>>
>> On 5/5/25 2:29 PM, Gustavo Romero wrote:
>>> Hi Eric,
>>>
>>> On 5/5/25 04:13, Eric Auger wrote:
>>>>
>>>>
>>>> On 5/4/25 11:56 PM, Gustavo Romero wrote:
>>>>> Fix typo in QEMU's ACPI PCI hotplug API function name that checks
>>>>> whether a given bus is hotplug-capable.
>>>>>
>>>>> Signed-off-by: Gustavo Romero <gustavo.romero@linaro.org>
>>>> Reviewed-by: Eric Auger <eric.auger@redhat.com>
>>>>
>>>> For awareness, there is a similar typo in pci-port.
>>>> see
>>>> https://lore.kernel.org/all/20250428102628.378046-2-eric.auger@redhat.com/
>>>>
>>>>
>>>
>>> hmm it's unfortunate that I was about to publish a series to exactly
>>> enable the APCI PCI hotplug on ARM. Next time I should try to sync
>>> better
>>> with Red Hat folks on the work fronts...
>> Sorry to hear that. That's always a frustrating situation :-(
>
> Yes...
>
>
>>> I've skimmed through your RFC and it seems to be doing mostly of what I
>>> was doing for my series, so I'll reviewed it formally today, it should
>>> be quick.
>> I guess we may need to add some additional qtests for hotplug (just as
>> they have on x86). Maybe we can collaborate on this or you already have
>> some stuff under work.
>
> Sure, the new test I'm baking is the one I've used in the smoke test.
> I'll
> contribute it on top of your series so.
sure
>
> For the x86 "parity", which tests you're thinking of to add for arm64?
I found
https://lore.kernel.org/all/20200918084111.15339-1-ani@anisinha.ca/#r
so in tests/qtest/bios-tables-test.c
tests featuring various values of
acpi-root-pci-hotplug/acpi-pci-hotplug-with-bridge-support
Cheers
Eric
>
>
> Cheers,
> Gustavo
>
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2025-05-06 8:59 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-05-04 21:56 [PATCH] hw/acpi/pcihp: Fix typo in function name Gustavo Romero
2025-05-05 7:13 ` Eric Auger
2025-05-05 12:29 ` Gustavo Romero
2025-05-05 14:19 ` Eric Auger
2025-05-06 0:26 ` Gustavo Romero
2025-05-06 8:58 ` Eric Auger
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).