On 17/08/2017 14:33, Yu Zhang wrote: >>>> >>>> +enum { >>>> + NO_CHECK_LIMIT = 0, >>>> + CHECK_LIMIT = 1, >>>> +}; >>> emulate.c should not include cpuid.h. The argument can be simply a >>> bool, though. >> >> Thanks, Paolo. >> So we just use true/false in emulate.c & svm.c, is this OK? I would use true/false everywhere. >> BTW could you please > Sorry for the unfinished line. I was wondering, why can't emulate.c > include cpuid.h? The emulator should be separate from the rest of KVM, in principle it could be used by userspace too. So its interface should be as limited as possible. Paolo