Re: [PATCH v5 05/13] KVM: Extend the memslot to support fd-based private memory

From: Sean Christopherson
Date: Fri Apr 08 2022 - 13:45:33 EST


On Fri, Apr 08, 2022, Chao Peng wrote:
> On Mon, Mar 28, 2022 at 09:56:33PM +0000, Sean Christopherson wrote:
> > struct kvm_userspace_memory_region_ext {
> > #ifdef __KERNEL__
>
> Is this #ifndef? As I think anonymous struct is only for kernel?

Doh, yes, I inverted that.

> Thanks,
> Chao
>
> > struct kvm_userspace_memory_region region;
> > #else
> > struct kvm_userspace_memory_region;
> > #endif
> > __u64 private_offset;
> > __u32 private_fd;
> > __u32 padding[5];
> > };
> >
> > #ifdef __KERNEL__
> > #define kvm_user_mem_region kvm_userspace_memory_region_ext
> > #endif
> >
> > [*] https://lore.kernel.org/all/20220301145233.3689119-1-arnd@xxxxxxxxxx
> >
> > > + __u64 private_offset;
> > > + __u32 private_fd;
> > > + __u32 padding[5];
> > > +};