>>> On 06.09.18 at 11:26, <[email protected]> wrote: >> From: Andrew Cooper [mailto:[email protected]] >> 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 <[email protected]> > > Reviewed-by: Paul Durrant <[email protected]>
Acked-by: Jan Beulich <[email protected]> _______________________________________________ Xen-devel mailing list [email protected] https://lists.xenproject.org/mailman/listinfo/xen-devel
