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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Sun, 13 Aug 2017 16:39:18 +0300
From:   Andy Shevchenko <andriy.shevchenko@...ux.intel.com>
To:     Thomas Gleixner <tglx@...utronix.de>,
        Ingo Molnar <mingo@...nel.org>, devel@...uxdriverproject.org,
        linux-kernel@...r.kernel.org
Cc:     Vitaly Kuznetsov <vkuznets@...hat.com>,
        "K . Y . Srinivasan" <kys@...rosoft.com>,
        Haiyang Zhang <haiyangz@...rosoft.com>,
        Stephen Hemminger <sthemmin@...rosoft.com>
Subject: Re: [PATCH v1 1/2] hyperv: Include linux/types.h to avoid build
 break

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@...hat.com>
> Cc: K. Y. Srinivasan <kys@...rosoft.com>
> Cc: Haiyang Zhang <haiyangz@...rosoft.com>
> Cc: Stephen Hemminger <sthemmin@...rosoft.com>
> Signed-off-by: Andy Shevchenko <andriy.shevchenko@...ux.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@...ux.intel.com>
Intel Finland Oy