Hi Jan, Thanks for the patch.
> -----Original Message----- > From: Jan Beulich <[email protected]> > Subject: [PATCH][4.17] kexec: restore hypercall 1st arg's type > > This reverts a small part of 7e21b25059ed ("xen: harmonize return types > of hypercall handlers"). The change from "unsigned long" to "unsigned > int" for the native handler function meant that previously invalid > values became valid. While perhaps not a significant issue, strictly > speaking that's still a change to the ABI. Don't go as far as restoring > the compat entry point's type though: That one can't have values passed > which don't fit in 32 bits. > > Note that as a side effect this fixes the invocation of > hypercall_create_continuation(), which by mistake wasn't adjusted by the > earlier change. > > Also take the opportunity and correct the respective comment in the > public header. (The way it was it really supports that it probably was > pointless to use "long", but that's the way the hypercall was > introduced.) > > Requested-by: Andrew Cooper <[email protected]> > Signed-off-by: Jan Beulich <[email protected]> Release-acked-by: Henry Wang <[email protected]> Kind regards, Henry
