On Fri, Aug 14, 2020 at 08:38:53AM -0400, Sasha Levin wrote:
> Add support for a Hyper-V based vGPU implementation that exposes the
> DirectX API to Linux userspace.

Api questions:

> +struct d3dkmthandle {
> +     union {
> +             struct {
> +                     u32 instance    :  6;
> +                     u32 index       : 24;
> +                     u32 unique      : 2;

What is the endian of this?

> +             };
> +             u32 v;
> +     };
> +};
> +
> +extern const struct d3dkmthandle zerohandle;
> +
> +struct ntstatus {
> +     union {
> +             struct {
> +                     int code        : 16;
> +                     int facility    : 13;
> +                     int customer    : 1;
> +                     int severity    : 2;

Same here.

Are these things that cross the user/kernel boundry?

And why int on one and u32 on the other?

> +             };
> +             int v;
> +     };
> +};
> +
> +struct winluid {
> +     uint a;
> +     uint b;

And now uint?  Come on, be consistent please :)

thanks,

greg k-h

Reply via email to