| Message ID | 20241014070650.14296-1-jgross@suse.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | xen/public: increment domctl interface version | expand |
On 14.10.2024 09:06, Juergen Gross wrote: > The recent addition of the XEN_DOMCTL_dt_overlay function was missing > the related update of XEN_DOMCTL_INTERFACE_VERSION, as it has been the > first interface change of the 4.20 release cycle. > > Fixes: 4c733873b5c2 ("xen/arm: Add XEN_DOMCTL_dt_overlay and device attachment to domains") I'm confused: That change (a) pre-dates the branching of 4.20 and (b) bumped the version ... > --- a/xen/include/public/domctl.h > +++ b/xen/include/public/domctl.h > @@ -21,7 +21,7 @@ > #include "hvm/save.h" > #include "memory.h" > > -#define XEN_DOMCTL_INTERFACE_VERSION 0x00000017 > +#define XEN_DOMCTL_INTERFACE_VERSION 0x00000018 ... from 0x16 to 0x17. And for no apparent reason, as plain additions don't require a bump. Didi you maybe mean to reference a different commit? Jan
On 14.10.2024 09:14, Jan Beulich wrote: > On 14.10.2024 09:06, Juergen Gross wrote: >> The recent addition of the XEN_DOMCTL_dt_overlay function was missing >> the related update of XEN_DOMCTL_INTERFACE_VERSION, as it has been the >> first interface change of the 4.20 release cycle. >> >> Fixes: 4c733873b5c2 ("xen/arm: Add XEN_DOMCTL_dt_overlay and device attachment to domains") > > I'm confused: That change (a) pre-dates the branching of 4.20 and (b) > bumped the version ... > >> --- a/xen/include/public/domctl.h >> +++ b/xen/include/public/domctl.h >> @@ -21,7 +21,7 @@ >> #include "hvm/save.h" >> #include "memory.h" >> >> -#define XEN_DOMCTL_INTERFACE_VERSION 0x00000017 >> +#define XEN_DOMCTL_INTERFACE_VERSION 0x00000018 > > ... from 0x16 to 0x17. And for no apparent reason, as plain additions don't > require a bump. Didi you maybe mean to reference a different commit? Albeit looking at the history of domctl.h I also can't spot any candidate. Jan
On 14.10.24 09:14, Jan Beulich wrote: > On 14.10.2024 09:06, Juergen Gross wrote: >> The recent addition of the XEN_DOMCTL_dt_overlay function was missing >> the related update of XEN_DOMCTL_INTERFACE_VERSION, as it has been the >> first interface change of the 4.20 release cycle. >> >> Fixes: 4c733873b5c2 ("xen/arm: Add XEN_DOMCTL_dt_overlay and device attachment to domains") > > I'm confused: That change (a) pre-dates the branching of 4.20 and (b) > bumped the version ... > >> --- a/xen/include/public/domctl.h >> +++ b/xen/include/public/domctl.h >> @@ -21,7 +21,7 @@ >> #include "hvm/save.h" >> #include "memory.h" >> >> -#define XEN_DOMCTL_INTERFACE_VERSION 0x00000017 >> +#define XEN_DOMCTL_INTERFACE_VERSION 0x00000018 > > ... from 0x16 to 0x17. And for no apparent reason, as plain additions don't > require a bump. Didi you maybe mean to reference a different commit? Oh, indeed. I wanted to reference d6e9a2aab39e. And regarding to "plain additions don't require a bump": 4c733873b5c2 did a plain addition and bumped the version. Juergen
On 14.10.2024 09:36, Jürgen Groß wrote: > On 14.10.24 09:14, Jan Beulich wrote: >> On 14.10.2024 09:06, Juergen Gross wrote: >>> The recent addition of the XEN_DOMCTL_dt_overlay function was missing >>> the related update of XEN_DOMCTL_INTERFACE_VERSION, as it has been the >>> first interface change of the 4.20 release cycle. >>> >>> Fixes: 4c733873b5c2 ("xen/arm: Add XEN_DOMCTL_dt_overlay and device attachment to domains") >> >> I'm confused: That change (a) pre-dates the branching of 4.20 and (b) >> bumped the version ... >> >>> --- a/xen/include/public/domctl.h >>> +++ b/xen/include/public/domctl.h >>> @@ -21,7 +21,7 @@ >>> #include "hvm/save.h" >>> #include "memory.h" >>> >>> -#define XEN_DOMCTL_INTERFACE_VERSION 0x00000017 >>> +#define XEN_DOMCTL_INTERFACE_VERSION 0x00000018 >> >> ... from 0x16 to 0x17. And for no apparent reason, as plain additions don't >> require a bump. Did you maybe mean to reference a different commit? > > Oh, indeed. I wanted to reference d6e9a2aab39e. > > And regarding to "plain additions don't require a bump": 4c733873b5c2 did > a plain addition and bumped the version. Right, hence why I said "for no apparent reason". Jan
On 14.10.24 09:46, Jan Beulich wrote: > On 14.10.2024 09:36, Jürgen Groß wrote: >> On 14.10.24 09:14, Jan Beulich wrote: >>> On 14.10.2024 09:06, Juergen Gross wrote: >>>> The recent addition of the XEN_DOMCTL_dt_overlay function was missing >>>> the related update of XEN_DOMCTL_INTERFACE_VERSION, as it has been the >>>> first interface change of the 4.20 release cycle. >>>> >>>> Fixes: 4c733873b5c2 ("xen/arm: Add XEN_DOMCTL_dt_overlay and device attachment to domains") >>> >>> I'm confused: That change (a) pre-dates the branching of 4.20 and (b) >>> bumped the version ... >>> >>>> --- a/xen/include/public/domctl.h >>>> +++ b/xen/include/public/domctl.h >>>> @@ -21,7 +21,7 @@ >>>> #include "hvm/save.h" >>>> #include "memory.h" >>>> >>>> -#define XEN_DOMCTL_INTERFACE_VERSION 0x00000017 >>>> +#define XEN_DOMCTL_INTERFACE_VERSION 0x00000018 >>> >>> ... from 0x16 to 0x17. And for no apparent reason, as plain additions don't >>> require a bump. Did you maybe mean to reference a different commit? >> >> Oh, indeed. I wanted to reference d6e9a2aab39e. >> >> And regarding to "plain additions don't require a bump": 4c733873b5c2 did >> a plain addition and bumped the version. > > Right, hence why I said "for no apparent reason". There seems to be a lack of documentation in this regard. Julien explicitly asked for the bump for that addition. I'm fine with dropping my patch if others agree that the bump isn't needed. In that case I'll send another one adding a comment for the mechanics of interface version bump in domctl.h and sysctl.h. Juergen
On 14.10.2024 10:00, Jürgen Groß wrote: > On 14.10.24 09:46, Jan Beulich wrote: >> On 14.10.2024 09:36, Jürgen Groß wrote: >>> On 14.10.24 09:14, Jan Beulich wrote: >>>> On 14.10.2024 09:06, Juergen Gross wrote: >>>>> The recent addition of the XEN_DOMCTL_dt_overlay function was missing >>>>> the related update of XEN_DOMCTL_INTERFACE_VERSION, as it has been the >>>>> first interface change of the 4.20 release cycle. >>>>> >>>>> Fixes: 4c733873b5c2 ("xen/arm: Add XEN_DOMCTL_dt_overlay and device attachment to domains") >>>> >>>> I'm confused: That change (a) pre-dates the branching of 4.20 and (b) >>>> bumped the version ... >>>> >>>>> --- a/xen/include/public/domctl.h >>>>> +++ b/xen/include/public/domctl.h >>>>> @@ -21,7 +21,7 @@ >>>>> #include "hvm/save.h" >>>>> #include "memory.h" >>>>> >>>>> -#define XEN_DOMCTL_INTERFACE_VERSION 0x00000017 >>>>> +#define XEN_DOMCTL_INTERFACE_VERSION 0x00000018 >>>> >>>> ... from 0x16 to 0x17. And for no apparent reason, as plain additions don't >>>> require a bump. Did you maybe mean to reference a different commit? >>> >>> Oh, indeed. I wanted to reference d6e9a2aab39e. >>> >>> And regarding to "plain additions don't require a bump": 4c733873b5c2 did >>> a plain addition and bumped the version. >> >> Right, hence why I said "for no apparent reason". > > There seems to be a lack of documentation in this regard. > > Julien explicitly asked for the bump for that addition. Julien - why was that? Bumps are needed only for backwards incompatible changes. Plain additions therefore never require a bump. As as we get better with properly checking e.g. padding fields, the frequency of required bumps should also further reduce. > I'm fine with dropping my patch if others agree that the bump isn't needed. > In that case I'll send another one adding a comment for the mechanics of > interface version bump in domctl.h and sysctl.h. Oh, yes, please feel free to do so. Jan
Hi Jan, On 14/10/2024 11:19, Jan Beulich wrote: > On 14.10.2024 10:00, Jürgen Groß wrote: >> On 14.10.24 09:46, Jan Beulich wrote: >>> On 14.10.2024 09:36, Jürgen Groß wrote: >>>> On 14.10.24 09:14, Jan Beulich wrote: >>>>> On 14.10.2024 09:06, Juergen Gross wrote: >>>>>> The recent addition of the XEN_DOMCTL_dt_overlay function was missing >>>>>> the related update of XEN_DOMCTL_INTERFACE_VERSION, as it has been the >>>>>> first interface change of the 4.20 release cycle. >>>>>> >>>>>> Fixes: 4c733873b5c2 ("xen/arm: Add XEN_DOMCTL_dt_overlay and device attachment to domains") >>>>> >>>>> I'm confused: That change (a) pre-dates the branching of 4.20 and (b) >>>>> bumped the version ... >>>>> >>>>>> --- a/xen/include/public/domctl.h >>>>>> +++ b/xen/include/public/domctl.h >>>>>> @@ -21,7 +21,7 @@ >>>>>> #include "hvm/save.h" >>>>>> #include "memory.h" >>>>>> >>>>>> -#define XEN_DOMCTL_INTERFACE_VERSION 0x00000017 >>>>>> +#define XEN_DOMCTL_INTERFACE_VERSION 0x00000018 >>>>> >>>>> ... from 0x16 to 0x17. And for no apparent reason, as plain additions don't >>>>> require a bump. Did you maybe mean to reference a different commit? >>>> >>>> Oh, indeed. I wanted to reference d6e9a2aab39e. >>>> >>>> And regarding to "plain additions don't require a bump": 4c733873b5c2 did >>>> a plain addition and bumped the version. >>> >>> Right, hence why I said "for no apparent reason". >> >> There seems to be a lack of documentation in this regard. >> >> Julien explicitly asked for the bump for that addition. > > Julien - why was that? I can't exactly remember why... I possibly just assumed that we updated the version every release... Cheers,
diff --git a/xen/include/public/domctl.h b/xen/include/public/domctl.h index e1028fc524..54cc9a06db 100644 --- a/xen/include/public/domctl.h +++ b/xen/include/public/domctl.h @@ -21,7 +21,7 @@ #include "hvm/save.h" #include "memory.h" -#define XEN_DOMCTL_INTERFACE_VERSION 0x00000017 +#define XEN_DOMCTL_INTERFACE_VERSION 0x00000018 /* * NB. xen_domctl.domain is an IN/OUT parameter for this operation.
The recent addition of the XEN_DOMCTL_dt_overlay function was missing the related update of XEN_DOMCTL_INTERFACE_VERSION, as it has been the first interface change of the 4.20 release cycle. Fixes: 4c733873b5c2 ("xen/arm: Add XEN_DOMCTL_dt_overlay and device attachment to domains") Signed-off-by: Juergen Gross <jgross@suse.com> --- xen/include/public/domctl.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)