Re: [PATCH v3] xen/public: add comment to struct xen_mem_acquire_resource

2022-02-24 Thread Juergen Gross
On 24.02.22 17:23, Jan Beulich wrote: On 24.02.2022 16:41, Juergen Gross wrote: On 24.02.22 16:37, Jan Beulich wrote: On 24.02.2022 16:24, Juergen Gross wrote: --- a/xen/include/public/memory.h +++ b/xen/include/public/memory.h @@ -662,6 +662,13 @@ struct xen_mem_acquire_resource { * t

Re: [PATCH v3] xen/public: add comment to struct xen_mem_acquire_resource

2022-02-24 Thread Jan Beulich
On 24.02.2022 16:41, Juergen Gross wrote: > On 24.02.22 16:37, Jan Beulich wrote: >> On 24.02.2022 16:24, Juergen Gross wrote: >>> --- a/xen/include/public/memory.h >>> +++ b/xen/include/public/memory.h >>> @@ -662,6 +662,13 @@ struct xen_mem_acquire_resource { >>>* two calls. >>>*/

Re: [PATCH v3] xen/public: add comment to struct xen_mem_acquire_resource

2022-02-24 Thread Juergen Gross
On 24.02.22 16:37, Jan Beulich wrote: On 24.02.2022 16:24, Juergen Gross wrote: --- a/xen/include/public/memory.h +++ b/xen/include/public/memory.h @@ -662,6 +662,13 @@ struct xen_mem_acquire_resource { * two calls. */ uint32_t nr_frames; +/* + * Padding field, must b

Re: [PATCH v3] xen/public: add comment to struct xen_mem_acquire_resource

2022-02-24 Thread Jan Beulich
On 24.02.2022 16:24, Juergen Gross wrote: > --- a/xen/include/public/memory.h > +++ b/xen/include/public/memory.h > @@ -662,6 +662,13 @@ struct xen_mem_acquire_resource { > * two calls. > */ > uint32_t nr_frames; > +/* > + * Padding field, must be zero on input. > + * I

[PATCH v3] xen/public: add comment to struct xen_mem_acquire_resource

2022-02-24 Thread Juergen Gross
Commit 7c7f7e8fba01 changed xen/include/public/memory.h in an incompatible way. Unfortunately the changed parts were already in use in the Linux kernel, so an update of the header in the kernel would result in a build breakage. As the change of above commit was in a section originally meant to be