Skip to content

Commit 0affda0

Browse files
anisinha-ntnxmstsirkin
authored andcommitted
Rename use_acpi_pci_hotplug to more appropriate use_acpi_hotplug_bridge
Currently, the option use_acpi_pci_hotplug is being used to control device hotplug capability using ACPI for slots of cold plugged bridges. Hence, we are renaming this option to better reflect what it actually does. Signed-off-by: Ani Sinha <[email protected]> Message-Id: <[email protected]> Reviewed-by: Igor Mammedov <[email protected]> Signed-off-by: Ani Sinha <[email protected]> Reviewed-by: Michael S. Tsirkin <[email protected]> Signed-off-by: Michael S. Tsirkin <[email protected]>
1 parent 8d19371 commit 0affda0

File tree

1 file changed

+11
-10
lines changed

1 file changed

+11
-10
lines changed

hw/acpi/piix4.c

Lines changed: 11 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,7 @@ typedef struct PIIX4PMState {
7777
Notifier powerdown_notifier;
7878

7979
AcpiPciHpState acpi_pci_hotplug;
80-
bool use_acpi_pci_hotplug;
80+
bool use_acpi_hotplug_bridge;
8181

8282
uint8_t disable_s3;
8383
uint8_t disable_s4;
@@ -204,16 +204,17 @@ static const VMStateDescription vmstate_pci_status = {
204204
}
205205
};
206206

207-
static bool vmstate_test_use_acpi_pci_hotplug(void *opaque, int version_id)
207+
static bool vmstate_test_use_acpi_hotplug_bridge(void *opaque, int version_id)
208208
{
209209
PIIX4PMState *s = opaque;
210-
return s->use_acpi_pci_hotplug;
210+
return s->use_acpi_hotplug_bridge;
211211
}
212212

213-
static bool vmstate_test_no_use_acpi_pci_hotplug(void *opaque, int version_id)
213+
static bool vmstate_test_no_use_acpi_hotplug_bridge(void *opaque,
214+
int version_id)
214215
{
215216
PIIX4PMState *s = opaque;
216-
return !s->use_acpi_pci_hotplug;
217+
return !s->use_acpi_hotplug_bridge;
217218
}
218219

219220
static bool vmstate_test_use_memhp(void *opaque)
@@ -290,11 +291,11 @@ static const VMStateDescription vmstate_acpi = {
290291
VMSTATE_STRUCT_TEST(
291292
acpi_pci_hotplug.acpi_pcihp_pci_status[ACPI_PCIHP_BSEL_DEFAULT],
292293
PIIX4PMState,
293-
vmstate_test_no_use_acpi_pci_hotplug,
294+
vmstate_test_no_use_acpi_hotplug_bridge,
294295
2, vmstate_pci_status,
295296
struct AcpiPciHpPciStatus),
296297
VMSTATE_PCI_HOTPLUG(acpi_pci_hotplug, PIIX4PMState,
297-
vmstate_test_use_acpi_pci_hotplug),
298+
vmstate_test_use_acpi_hotplug_bridge),
298299
VMSTATE_END_OF_LIST()
299300
},
300301
.subsections = (const VMStateDescription*[]) {
@@ -530,7 +531,7 @@ I2CBus *piix4_pm_init(PCIBus *bus, int devfn, uint32_t smb_io_base,
530531
s->smi_irq = smi_irq;
531532
s->smm_enabled = smm_enabled;
532533
if (xen_enabled()) {
533-
s->use_acpi_pci_hotplug = false;
534+
s->use_acpi_hotplug_bridge = false;
534535
}
535536

536537
pci_realize_and_unref(pci_dev, bus, &error_fatal);
@@ -595,7 +596,7 @@ static void piix4_acpi_system_hot_add_init(MemoryRegion *parent,
595596
memory_region_add_subregion(parent, GPE_BASE, &s->io_gpe);
596597

597598
acpi_pcihp_init(OBJECT(s), &s->acpi_pci_hotplug, bus, parent,
598-
s->use_acpi_pci_hotplug);
599+
s->use_acpi_hotplug_bridge);
599600

600601
s->cpu_hotplug_legacy = true;
601602
object_property_add_bool(OBJECT(s), "cpu-hotplug-legacy",
@@ -633,7 +634,7 @@ static Property piix4_pm_properties[] = {
633634
DEFINE_PROP_UINT8(ACPI_PM_PROP_S4_DISABLED, PIIX4PMState, disable_s4, 0),
634635
DEFINE_PROP_UINT8(ACPI_PM_PROP_S4_VAL, PIIX4PMState, s4_val, 2),
635636
DEFINE_PROP_BOOL("acpi-pci-hotplug-with-bridge-support", PIIX4PMState,
636-
use_acpi_pci_hotplug, true),
637+
use_acpi_hotplug_bridge, true),
637638
DEFINE_PROP_BOOL("memory-hotplug-support", PIIX4PMState,
638639
acpi_memory_hotplug.is_enabled, true),
639640
DEFINE_PROP_END_OF_LIST(),

0 commit comments

Comments
 (0)