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

Reply via email to