> On 15 Jul 2019, at 21:28, Paolo Bonzini <pbonzini@xxxxxxxxxx> wrote: > > On 15/07/19 19:21, Sean Christopherson wrote: >>>> + if ((seg_reg != VCPU_SREG_FS) && (seg_reg != VCPU_SREG_GS)) >> I'm pretty sure the internal parantheses are unnecessary. >> > > Indeed, that's so Pascal! :) I'll apply Vitaly's suggestion and queue it. > > Paolo I like parentheses as it makes ordering of expression a no-brainer. But that’s just a matter of taste I guess. I don’t mind you will change it according to given suggestions. :) -Liran