On 2017-12-11 09:46 AM, Oded Gabbay wrote: >> + >> + /* Fill in IOLINK subtype. >> + * TODO: Fill-in other fields of iolink subtype >> + */ >> + sub_type_hdr->io_interface_type = CRAT_IOLINK_TYPE_PCIEXPRESS; >> + sub_type_hdr->proximity_domain_from = proximity_domain; >> +#ifdef CONFIG_NUMA >> + if (kdev->pdev->dev.numa_node == NUMA_NO_NODE) > Had to add #include <linux/pci.h> at the head of the file to make this > line compile Thanks for catching that. I removed some "unnecessary" includes as I was splitting a larger patch into CPU and dGPU support for topology. I think the kernel config I used for testing that didn't have NUMA enable for some reason, so I didn't catch this. Regards, Â Felix > > Oded > >