[PATCH v1 1/2] hyperv: Include linux/types.h to avoid build break

Andy Shevchenko andriy.shevchenko at linux.intel.com
Sun Aug 13 13:39:18 UTC 2017


On Sun, 2017-08-13 at 16:34 +0300, Andy Shevchenko wrote:
> The commit 2ffd9e33ce4a
> 
>     ("x86/hyper-v: Use hypercall for remote TLB flush")
> 
> introduced a new C-file where uapi/linux/hyperv.h is included first.
> 
> This makes build failed since UAPI headers are heavily dependent to
> special
> types of variables, e.g. __u8.
> 
>   CC      arch/x86/hyperv/mmu.o
> In file included from include/linux/hyperv.h:28:0,
>                  from arch/x86/hyperv/mmu.c:3:
> include/uapi/linux/hyperv.h:71:2: error: unknown type name ‘__u8’
>   __u8 operation;
>   ^~~~
> 
> Include linux/types.h to the UAPI header like many others do.
> 

Oops, it's because I have an update to this file in my local uuid
branch.

In any case this is right thing to do independently of uuid.h inclusion.

Please, ignore this patch for now, but note the second one is still
valid.

> Cc: Vitaly Kuznetsov <vkuznets at redhat.com>
> Cc: K. Y. Srinivasan <kys at microsoft.com>
> Cc: Haiyang Zhang <haiyangz at microsoft.com>
> Cc: Stephen Hemminger <sthemmin at microsoft.com>
> Signed-off-by: Andy Shevchenko <andriy.shevchenko at linux.intel.com>
> ---
>  include/uapi/linux/hyperv.h | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/include/uapi/linux/hyperv.h b/include/uapi/linux/hyperv.h
> index 7560c0aa60e3..aaf32131b556 100644
> --- a/include/uapi/linux/hyperv.h
> +++ b/include/uapi/linux/hyperv.h
> @@ -25,6 +25,8 @@
>  #ifndef _UAPI_HYPERV_H
>  #define _UAPI_HYPERV_H
>  
> +#include <linux/types.h>
> +
>  /*
>   * Framework version for util services.
>   */

-- 
Andy Shevchenko <andriy.shevchenko at linux.intel.com>
Intel Finland Oy


More information about the devel mailing list