On 03/08/16 16:07, Jan Beulich wrote: >>>> On 18.07.16 at 11:51, <andrew.coop...@citrix.com> wrote: >> For the same reason as c/s 33a231e3f "x86/HVM: fold hypercall tables", this >> removes the risk of accidentally updating only one of the tables. >> >> Signed-off-by: Andrew Cooper <andrew.coop...@citrix.com> > Reviewed-by: Jan Beulich <jbeul...@suse.com> > > But having come here I still can't see why this can't be folded with > patch 5 without also folding in patch 6. Anyway - as long as they're > going to get committed without too big of a time gap in between, > the final result is what matters most.
References to hypercall_table and compat_hypercall_table are buried in the multicall inline assembler. Folding the tables first involves complicated changes, just to be taken out one patch later. The risk of accidentally breaking bisectability is greater than the downside of splitting the patches up a bit. (Either way, they will be committed together.) ~Andrew _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel
