Re: [PATCH] Fix compilation when UFFDIO_REGISTER is not set.

[Michael Tokarev]

28.09.2023 15:56, Pierre Labatut wrote:
> Signed-off-by: Pierre Labatut <plabatut@google.com>
> ---
>   subprojects/libvhost-user/libvhost-user.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)

I'm not sure how subprojects are handled here.  Should we commit things
to a subproject directly? This one does not seem to be wrapped at least,
and has been in contrib/ before moving to subprojects/, so seems to be
ok.

> diff --git a/subprojects/libvhost-user/libvhost-user.c 
> b/subprojects/libvhost-user/libvhost-user.c
> index 0469a50101..d4a2eab883 100644
> --- a/subprojects/libvhost-user/libvhost-user.c
> +++ b/subprojects/libvhost-user/libvhost-user.c
> @@ -629,9 +629,9 @@ static bool
>   generate_faults(VuDev *dev) {
>       unsigned int i;
>       for (i = 0; i < dev->nregions; i++) {
> +#ifdef UFFDIO_REGISTER
>           VuDevRegion *dev_region = &dev->regions[i];
>           int ret;
> -#ifdef UFFDIO_REGISTER
>           struct uffdio_register reg_struct;
>   
>           /*

Applied to my trivial-patches tree, thank you!

/mjt