On 17.02.2026 23:04, Stewart Hildebrand wrote:
> On 2/10/26 05:53, Jan Beulich wrote:
>> Before adding more private stuff to xen/vpci.h, split it up. In order to
>> be able to include the private header first in a CU, the per-arch struct
>> decls also need to move (to new asm/vpci.h files).
>>
>> While adjusting the test harness'es Makefile, also switch the pre-existing
>> header symlink-ing rule to a pattern one.
>>
>> Apart from in the test harness code, things only move; no functional
>> change intended.
>>
>> Signed-off-by: Jan Beulich <[email protected]>
>> ---
>> Subsequently, at least on x86 more stuff may want moving into asm/vpci.h.
>> ---
>> v4: New.
>>
>> --- a/tools/tests/vpci/Makefile
>> +++ b/tools/tests/vpci/Makefile
>> @@ -14,8 +14,8 @@ else
>> $(warning HOSTCC != CC, will not run test)
>> endif
>>
>> -$(TARGET): vpci.c vpci.h list.h main.c emul.h
>> - $(CC) $(CFLAGS_xeninclude) -g -o $@ vpci.c main.c
>> +$(TARGET): vpci.c vpci.h list.h private.h main.c emul.h
>> + $(CC) $(CFLAGS_xeninclude) -include emul.h -g -o $@ vpci.c main.c
>>
>> .PHONY: clean
>> clean:
>
> Can you please add the generated private.h to be removed upon "make clean"?
>
> Also, can you please add tools/tests/vpci/private.h to .gitignore?
Oops, yes, of course.
>> --- /dev/null
>> +++ b/xen/arch/arm/include/asm/vpci.h
>> @@ -0,0 +1,13 @@
>> +/* SPDX-License-Identifier: GPL-2.0-only */
>> +#ifndef ARM_VPCI_H
>> +#define ARM_VPCI_H
>> +
>> +/* Arch-specific MSI data for vPCI. */
>> +struct vpci_arch_msi {
>> +};
>> +
>> +/* Arch-specific MSI-X entry data for vPCI. */
>> +struct vpci_arch_msix_entry {
>> +};
>> +
>> +#endif /* ARM_VPCI_H */
>
> Out of curiosity (not asking for any changes), why did you include an emacs
> footer on the x86 header but not here?
>
> Otherwise, the rest of the patch looks good to me.
Well - I copied the original files and then reduced them to just what I
wanted to move here.
Jan
