On 15/03/2019 15:59, Boris Ostrovsky wrote: > On 3/15/19 11:23 AM, Andrew Cooper wrote: >> Taking svm_feature_flags by pointer and using test_bit() results in generated >> code which loads svm_feature_flags into a 32bit register, then does a bitwise >> operation. >> >> The logic can be expressed in terms of a straight bitwise operation, >> resulting >> in the following minor improvement. >> >> add/remove: 0/0 grow/shrink: 0/4 up/down: 0/-136 (-136) >> Function old new delta >> svm_nested_features_on_efer_update 281 273 -8 >> svm_create_vmcb 1404 1388 -16 >> svm_vmexit_handler 6271 6239 -32 >> start_svm 818 738 -80 >> Total: Before=3347569, After=3347433, chg -0.00% >> >> Signed-off-by: Andrew Cooper <andrew.coop...@citrix.com> > Reviewed-by: Boris Ostrovsky <boris.ostrov...@oracel.com> >
I suppose you'd like me to fix up Oracle on commit? ;) ~Andrew _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel