> -----Original Message-----
> From: Andy Shevchenko [mailto:andriy.shevche...@linux.intel.com]
> Sent: Wednesday, July 19, 2017 2:29 PM
> To: linux-a...@vger.kernel.org; de...@linuxdriverproject.org;
> sparmaintai...@unisys.com; de...@driverdev.osuosl.org; linux-
> wirel...@vger.kernel.org; linux-watch...@vger.kernel.org; linux-
> e...@vger.kernel.org; Christoph Hellwig <h...@lst.de>; linux-
> ker...@vger.kernel.org; Lukas Wunner <lu...@wunner.de>
> Cc: Andy Shevchenko <andriy.shevche...@linux.intel.com>; KY Srinivasan
> <k...@microsoft.com>; Haiyang Zhang <haiya...@microsoft.com>; Stephen
> Hemminger <sthem...@microsoft.com>
> Subject: [PATCH v1 4/6] vmbus: Switch to use new generic UUID API
>
> @@ -118,8 +116,8 @@ enum hv_fcopy_op {
>
> struct hv_fcopy_hdr {
> __u32 operation;
> - uuid_le service_id0; /* currently unused */
> - uuid_le service_id1; /* currently unused */
> + __u8 service_id0[16]; /* currently unused */
> + __u8 service_id1[16]; /* currently unused */
> } __attribute__((packed));
Even user space may not have all guid_t operations, could you still
put guid_t here for consistency? This file is also used by our kernel
mode code.
Thanks,
- Haiyang
