On Wed, Nov 23, 2016 at 3:31 AM, Paolo Bonzini <pbonz...@redhat.com> wrote: > On 23/11/2016 02:14, David Matlack wrote: >> +static bool fixed_bits_valid(u64 val, u64 fixed0, u64 fixed1) >> +{ >> + return ((val & fixed0) == fixed0) && ((~val & ~fixed1) == ~fixed1); >> +} >> + > > This is the same as vmx_control_verify (except with u64 arguments > instead of u32).
Good point. I'll remove this duplication in v3.