On 18.02.2026 22:58, Stewart Hildebrand wrote:
> On 2/10/26 05:53, Jan Beulich wrote:
>> --- a/xen/include/xen/vpci.h
>> +++ b/xen/include/xen/vpci.h
>> @@ -30,20 +19,6 @@ typedef struct {
>> */
>> #define VPCI_MAX_VIRT_DEV (PCI_SLOT(~0) + 1)
>>
>> -#define REGISTER_VPCI_CAPABILITY(cap, name, finit, fclean, ext) \
>> - static const vpci_capability_t name##_entry \
>> - __used_section(".data.rel.ro.vpci") = { \
>> - .id = (cap), \
>> - .init = (finit), \
>> - .cleanup = (fclean), \
>> - .is_ext = (ext), \
>> - }
>> -
>> -#define REGISTER_VPCI_CAP(name, finit, fclean) \
>> - REGISTER_VPCI_CAPABILITY(PCI_CAP_ID_##name, name, finit, fclean, false)
>> -#define REGISTER_VPCI_EXTCAP(name, finit, fclean) \
>> - REGISTER_VPCI_CAPABILITY(PCI_EXT_CAP_ID_##name, name, finit, fclean,
>> true)
>> -
>> int __must_check vpci_init_header(struct pci_dev *pdev);
> Nit: I suppose vpci_init_header() could also move to the new private header
> file
Hmm, yes, will do. I wonder why I didn't, as I went by being pretty aggressive
in moving, to then move back what needs exposing to the outside.
Jan
