Re: [PATCH v3 4/6] ALSA: xen-front: Implement handling of shared buffers

[Takashi Iwai <tiwai@suse.de>]

On Mon, 14 May 2018 08:27:40 +0200,
Oleksandr Andrushchenko wrote:
> --- /dev/null
> +++ b/sound/xen/xen_snd_front_shbuf.c
> @@ -0,0 +1,193 @@
> +// SPDX-License-Identifier: GPL-2.0 OR MIT
> +
> +/*
> + * Xen para-virtual sound device
> + *
> + * Copyright (C) 2016-2018 EPAM Systems Inc.
> + *
> + * Author: Oleksandr Andrushchenko <oleksandr_andrushchenko@epam.com>
> + */
> +
> +#include <xen/xen.h>
> +#include <xen/xenbus.h>
> +
> +#include "xen_snd_front_shbuf.h"

Hm, with the local build test, I get the following error:

  CC [M]  sound/xen/xen_snd_front_shbuf.o
  In file included from sound/xen/xen_snd_front_shbuf.c:11:0:
  ./include/xen/xen.h:18:8: error: unknown type name ‘bool’
   extern bool xen_pvh;
           ^~~~
   In file included from ./include/xen/interface/xen.h:30:0,
                    from ./include/xen/xen.h:29,
                    from sound/xen/xen_snd_front_shbuf.c:11:
  ./arch/x86/include/asm/xen/interface.h:92:21: error: unknown type name ‘uint64_t’
   DEFINE_GUEST_HANDLE(uint64_t);
                       ^
			
Adding #include <linux/kernel.h> fixed the issue.

Did you really test your patches with the latest Linus tree?


thanks,

Takashi