Re: [PATCH 3/4] KVM: x86: Omit VCPU_REGS_RIP from emulator's _regs array

From: Sean Christopherson
Date: Thu May 26 2022 - 11:47:24 EST


On Thu, May 26, 2022, kernel test robot wrote:
> Hi Sean,
>
> I love your patch! Yet something to improve:
>
> [auto build test ERROR on 90bde5bea810d766e7046bf5884f2ccf76dd78e9]
>
> url: https://github.com/intel-lab-lkp/linux/commits/Sean-Christopherson/KVM-x86-Emulator-_regs-fixes-and-cleanups/20220526-062734
> base: 90bde5bea810d766e7046bf5884f2ccf76dd78e9
> config: i386-debian-10.3-kselftests (https://download.01.org/0day-ci/archive/20220526/202205261040.m1luL6IW-lkp@xxxxxxxxx/config)

Doh, forgot to build and test 32-bit KVM. For this patch, I think it makes sense
to just hardcode the number of registers to 16. But in a follow-up, that can be
reduced to 8 for 32-bit KVM, and then rsm_load_state_32() can use NR_EMULATOR_GPRS
too.