>>> On 06.09.18 at 11:26, <paul.durr...@citrix.com> wrote: >> From: Andrew Cooper [mailto:andrew.coop...@citrix.com] >> Sent: 05 September 2018 19:12 >> >> The parameter marshalling and xsm checks are common to both the set and >> get >> paths. Lift all of the common code out into do_hvm_op() and let >> hvmop_{get,set}_param() operate on struct xen_hvm_param directly. >> >> This is somewhat noisy in the functions as each a. reference has to change to >> a-> instead. >> >> In addition, drop an empty default statement, insert newlines as appropriate >> between cases, and there is no need to update the IDENT_PT on the >> fastpath, >> because the common path after the switch will make the update. >> >> No functional change, but a net shrink of -328 to do_hvm_op(). >> >> Signed-off-by: Andrew Cooper <andrew.coop...@citrix.com> > > Reviewed-by: Paul Durrant <paul.durr...@citrix.com>
Acked-by: Jan Beulich <jbeul...@suse.com> _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel
