On 18/06/2021 11:23, Jan Beulich wrote: > Some hypercalls, memory-op in particular, can return values requiring > more than 31 bits to represent. Hence the underlying layers need to make > sure they won't truncate such values. (Note that for Solaris the > function also gets renamed, to match the other OSes.)
Spot the environment which obviously hasn't been compiled since the 4.5(?) timeframe... Also, I'm fairly sure the comment in the NetBSD version is false when it says it is copying the Linux way of doing things - I'm pretty sure it means copying the FreeBSD way. > Signed-off-by: Jan Beulich <jbeul...@suse.com> I think the commit message needs to state that this doesn't fix truncation in the Linux or Solaris, nor the truncation in the xencall{0..5}() public APIs. It only fixes truncation issues for FreeBSD, MiniOS and NetBSD. With a suitable adjustment, and ideally a fix to the NetBSD comment, Acked-by: Andrew Cooper <andrew.coop...@citrix.com>