motorhead/qemu
1
0
Fork 0
mirror of https://github.com/Motorhead1991/qemu.git synced 2025-08-05 08:43:55 -06:00
Code Issues Projects Releases Packages Wiki Activity Actions

hw/acpi: Make storage of node id uint32_t to reduce fragility

Browse source 
>From review of generic port introduction.

The value is handled as a uint32_t so store it in that type.
The value cannot in reality exceed MAX_NODES which is currently
128 but if the types are matched there is no need to rely on that
restriction.

Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
Message-Id: <20240916174237.1843213-1-Jonathan.Cameron@huawei.com>
Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
This commit is contained in:
Jonathan Cameron 2024-09-16 18:42:37 +01:00 • committed by Michael S. Tsirkin
parent a82fe82916
commit cf2181aef2
3 changed files with 3 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

2
hw/acpi/aml-build.c
View file

@ -1972,7 +1972,7 @@ static void build_append_srat_acpi_device_handle(GArray *table_data,
* 5.2.16.6 Generic Initiator Affinity Structure * 5.2.16.6 Generic Initiator Affinity Structure
* With PCI Device Handle. * With PCI Device Handle.
*/ */
void build_srat_pci_generic_initiator(GArray *table_data, int node, void build_srat_pci_generic_initiator(GArray *table_data, uint32_t node,
uint16_t segment, uint8_t bus, uint16_t segment, uint8_t bus,
uint8_t devfn) uint8_t devfn)
{ {

2
hw/acpi/pci.c
View file

@ -72,7 +72,7 @@ typedef struct AcpiGenericInitiator {
/* public */ /* public */
char *pci_dev; char *pci_dev;
uint16_t node; uint32_t node;
} AcpiGenericInitiator; } AcpiGenericInitiator;
typedef struct AcpiGenericInitiatorClass { typedef struct AcpiGenericInitiatorClass {

2
include/hw/acpi/aml-build.h
View file

@ -486,7 +486,7 @@ Aml *build_crs(PCIHostState *host, CrsRangeSet *range_set, uint32_t io_offset,
void build_srat_memory(GArray *table_data, uint64_t base, void build_srat_memory(GArray *table_data, uint64_t base,
uint64_t len, int node, MemoryAffinityFlags flags); uint64_t len, int node, MemoryAffinityFlags flags);
void build_srat_pci_generic_initiator(GArray *table_data, int node, void build_srat_pci_generic_initiator(GArray *table_data, uint32_t node,
uint16_t segment, uint8_t bus, uint16_t segment, uint8_t bus,
uint8_t devfn); uint8_t devfn);