Hi Sean, On Mon, Aug 03, 2020 at 11:45:45AM -0700, Sean Christopherson wrote: > On Mon, Aug 03, 2020 at 02:27:07PM +0200, Joerg Roedel wrote: > > + return test_bit(GHCB_BITMAP_IDX(field), \ > > + (unsigned long *)&(ghcb)->save.valid_bitmap); \ > > 'ghcb' doesn't need to be wrapped in (), it's a parameter to a function. Thanks for your reviews, I addressed them and sent v4. Regards, Joerg