On 4/14/25 10:54 AM, Thomas Huth wrote:
On 14/04/2025 16.37, Rorie Reyes wrote:
On 4/11/25 2:45 AM, Thomas Huth wrote:

#include CONFIG_DEVICES

at the beginning of the ioinst.c file. Then you should be able to do:

#ifdef CONFIG_VFIO_AP
    if (s390_has_feat(S390_FEAT_AP)) {
       return ap_chsc_sei_nt0_get_event(res);
    }
#endif
This worked

(or whatever the code should look like).

Alternatively, and this might even be the nicer variant, add a file hw/ vfio/ap-stub.c and include a dummy ap_chsc_sei_nt0_get_event() function there. Then in hw/vfio/meson.build add this line:

vfio_ss.add(when: 'CONFIG_VFIO_AP', if_false: files('ap-stub.c'))
This worked as well. Since you mentioned that this is a nicer variant, I'll go with this change. What do you recommend I do for my patches? Should I do an interactive rebase to add the new file hw/vfio/ap-stub.c and updating hw/ vfio/meson.build? Or should I make two new commits for each file (ap-stub.c and meson.build)

Please do a "git rebase -i ..." to fix up the corresponding patch (that's what we're doing in the QEMU development workflow).

 Thanks
  Thomas

Thank you. I'll make those changes and send my patches later today

Reply via email to