On Wed, Nov 08, 2023 at 12:30:39AM +0530, ank...@nvidia.com wrote: > + for (l = gi->nodelist; l; l = l->next) { > + PCIDeviceHandle dev_handle = {0}; > + PCIDevice *pci_dev = PCI_DEVICE(o); > + dev_handle.bdf = PCI_BUILD_BDF(pci_bus_num(pci_get_bus(pci_dev)), > + pci_dev->devfn); > + build_srat_generic_pci_initiator_affinity(table_data, > + l->value, &dev_handle); > + } > + }
if you never initialize segment then I don't see why have it. It's just the bdf, just pass that as parameter no need for a struct. -- MST