[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:34:43 UTC 2017


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.

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.
  */
-- 
2.13.2



More information about the devel mailing list