All of lore.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: Nicholas Piggin <npiggin@gmail.com>, kvm-ppc@vger.kernel.org
Cc: linuxppc-dev@lists.ozlabs.org, kbuild-all@lists.01.org,
	Nicholas Piggin <npiggin@gmail.com>,
	Fabiano Rosas <farosas@linux.ibm.com>
Subject: Re: [PATCH v5 31/48] KVM: PPC: Book3S HV P9: inline kvmhv_load_hv_regs_and_go into __kvmhv_vcpu_entr
Date: Fri, 02 Apr 2021 20:58:05 +0000	[thread overview]
Message-ID: <202104030412.h8fIESGy-lkp@intel.com> (raw)
In-Reply-To: <20210401150325.442125-32-npiggin@gmail.com>

[-- Attachment #1: Type: text/plain, Size: 18530 bytes --]

Hi Nicholas,

I love your patch! Yet something to improve:

[auto build test ERROR on powerpc/next]
[also build test ERROR on v5.12-rc5 next-20210401]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]

url:    https://github.com/0day-ci/linux/commits/Nicholas-Piggin/KVM-PPC-Book3S-C-ify-the-P9-entry-exit-code/20210401-232743
base:   https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git next
config: powerpc64-randconfig-r033-20210402 (attached as .config)
compiler: powerpc64-linux-gcc (GCC) 9.3.0
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # https://github.com/0day-ci/linux/commit/f2a35edda3ab6cba30fbfc362e163d5bc1e086d0
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Nicholas-Piggin/KVM-PPC-Book3S-C-ify-the-P9-entry-exit-code/20210401-232743
        git checkout f2a35edda3ab6cba30fbfc362e163d5bc1e086d0
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=powerpc64 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>

All errors (new ones prefixed by >>):

   arch/powerpc/kvm/book3s_hv_interrupt.c: In function 'switch_mmu_to_guest_radix':
   arch/powerpc/kvm/book3s_hv_interrupt.c:61:46: error: 'struct kvm_vcpu_arch' has no member named 'nested'
      61 |  struct kvm_nested_guest *nested = vcpu->arch.nested;
         |                                              ^
   arch/powerpc/kvm/book3s_hv_interrupt.c:80:2: error: implicit declaration of function 'kvmppc_check_need_tlb_flush' [-Werror=implicit-function-declaration]
      80 |  kvmppc_check_need_tlb_flush(kvm, vc->pcpu, nested);
         |  ^~~~~~~~~~~~~~~~~~~~~~~~~~~
   In file included from arch/powerpc/include/asm/processor.h:11,
                    from arch/powerpc/include/asm/thread_info.h:40,
                    from include/linux/thread_info.h:58,
                    from include/asm-generic/preempt.h:5,
                    from ./arch/powerpc/include/generated/asm/preempt.h:1,
                    from include/linux/preempt.h:78,
                    from include/linux/percpu.h:6,
                    from include/linux/context_tracking_state.h:5,
                    from include/linux/hardirq.h:5,
                    from include/linux/kvm_host.h:7,
                    from arch/powerpc/kvm/book3s_hv_interrupt.c:3:
   arch/powerpc/kvm/book3s_hv_interrupt.c: In function 'switch_mmu_to_host_radix':
   arch/powerpc/kvm/book3s_hv_interrupt.c:88:28: error: 'struct kvm_arch' has no member named 'host_lpid'
      88 |  mtspr(SPRN_LPID, kvm->arch.host_lpid);
         |                            ^
   arch/powerpc/include/asm/reg.h:1393:33: note: in definition of macro 'mtspr'
    1393 |          : "r" ((unsigned long)(v)) \
         |                                 ^
   arch/powerpc/kvm/book3s_hv_interrupt.c:90:28: error: 'struct kvm_arch' has no member named 'host_lpcr'
      90 |  mtspr(SPRN_LPCR, kvm->arch.host_lpcr);
         |                            ^
   arch/powerpc/include/asm/reg.h:1393:33: note: in definition of macro 'mtspr'
    1393 |          : "r" ((unsigned long)(v)) \
         |                                 ^
   arch/powerpc/kvm/book3s_hv_interrupt.c: At top level:
   arch/powerpc/kvm/book3s_hv_interrupt.c:128:5: error: no previous prototype for 'kvmhv_vcpu_entry_p9' [-Werror=missing-prototypes]
     128 | int kvmhv_vcpu_entry_p9(struct kvm_vcpu *vcpu, u64 time_limit, unsigned long lpcr)
         |     ^~~~~~~~~~~~~~~~~~~
   arch/powerpc/kvm/book3s_hv_interrupt.c: In function 'kvmhv_vcpu_entry_p9':
>> arch/powerpc/kvm/book3s_hv_interrupt.c:170:25: error: 'struct kvmppc_host_state' has no member named 'host_purr'; did you mean 'host_r1'?
     170 |  local_paca->kvm_hstate.host_purr = mfspr(SPRN_PURR);
         |                         ^~~~~~~~~
         |                         host_r1
>> arch/powerpc/kvm/book3s_hv_interrupt.c:171:25: error: 'struct kvmppc_host_state' has no member named 'host_spurr'; did you mean 'host_r1'?
     171 |  local_paca->kvm_hstate.host_spurr = mfspr(SPRN_SPURR);
         |                         ^~~~~~~~~~
         |                         host_r1
   In file included from arch/powerpc/include/asm/processor.h:11,
                    from arch/powerpc/include/asm/thread_info.h:40,
                    from include/linux/thread_info.h:58,
                    from include/asm-generic/preempt.h:5,
                    from ./arch/powerpc/include/generated/asm/preempt.h:1,
                    from include/linux/preempt.h:78,
                    from include/linux/percpu.h:6,
                    from include/linux/context_tracking_state.h:5,
                    from include/linux/hardirq.h:5,
                    from include/linux/kvm_host.h:7,
                    from arch/powerpc/kvm/book3s_hv_interrupt.c:3:
>> arch/powerpc/kvm/book3s_hv_interrupt.c:187:31: error: 'struct kvmppc_host_state' has no member named 'fake_suspend'
     187 |        (local_paca->kvm_hstate.fake_suspend << PSSCR_FAKE_SUSPEND_LG));
         |                               ^
   arch/powerpc/include/asm/reg.h:1393:33: note: in definition of macro 'mtspr'
    1393 |          : "r" ((unsigned long)(v)) \
         |                                 ^
   arch/powerpc/kvm/book3s_hv_interrupt.c:191:31: error: 'struct kvm_vcpu_arch' has no member named 'shregs'; did you mean 'regs'?
     191 |  mtspr(SPRN_SPRG0, vcpu->arch.shregs.sprg0);
         |                               ^~~~~~
   arch/powerpc/include/asm/reg.h:1393:33: note: in definition of macro 'mtspr'
    1393 |          : "r" ((unsigned long)(v)) \
         |                                 ^
   arch/powerpc/kvm/book3s_hv_interrupt.c:192:31: error: 'struct kvm_vcpu_arch' has no member named 'shregs'; did you mean 'regs'?
     192 |  mtspr(SPRN_SPRG1, vcpu->arch.shregs.sprg1);
         |                               ^~~~~~
   arch/powerpc/include/asm/reg.h:1393:33: note: in definition of macro 'mtspr'
    1393 |          : "r" ((unsigned long)(v)) \
         |                                 ^
   arch/powerpc/kvm/book3s_hv_interrupt.c:193:31: error: 'struct kvm_vcpu_arch' has no member named 'shregs'; did you mean 'regs'?
     193 |  mtspr(SPRN_SPRG2, vcpu->arch.shregs.sprg2);
         |                               ^~~~~~
   arch/powerpc/include/asm/reg.h:1393:33: note: in definition of macro 'mtspr'
    1393 |          : "r" ((unsigned long)(v)) \
         |                                 ^
   arch/powerpc/kvm/book3s_hv_interrupt.c:194:31: error: 'struct kvm_vcpu_arch' has no member named 'shregs'; did you mean 'regs'?
     194 |  mtspr(SPRN_SPRG3, vcpu->arch.shregs.sprg3);
         |                               ^~~~~~
   arch/powerpc/include/asm/reg.h:1393:33: note: in definition of macro 'mtspr'
    1393 |          : "r" ((unsigned long)(v)) \
         |                                 ^
   arch/powerpc/kvm/book3s_hv_interrupt.c:206:30: error: 'struct kvm_vcpu_arch' has no member named 'shregs'; did you mean 'regs'?
     206 |  mtspr(SPRN_SRR0, vcpu->arch.shregs.srr0);
         |                              ^~~~~~
   arch/powerpc/include/asm/reg.h:1393:33: note: in definition of macro 'mtspr'
    1393 |          : "r" ((unsigned long)(v)) \
         |                                 ^
   arch/powerpc/kvm/book3s_hv_interrupt.c:207:30: error: 'struct kvm_vcpu_arch' has no member named 'shregs'; did you mean 'regs'?
     207 |  mtspr(SPRN_SRR1, vcpu->arch.shregs.srr1);
         |                              ^~~~~~
   arch/powerpc/include/asm/reg.h:1393:33: note: in definition of macro 'mtspr'
    1393 |          : "r" ((unsigned long)(v)) \
         |                                 ^
   In file included from arch/powerpc/include/asm/bug.h:109,
                    from include/linux/bug.h:5,
                    from include/linux/mmdebug.h:5,
                    from include/linux/percpu.h:5,
                    from include/linux/context_tracking_state.h:5,
                    from include/linux/hardirq.h:5,
                    from include/linux/kvm_host.h:7,
                    from arch/powerpc/kvm/book3s_hv_interrupt.c:3:
   arch/powerpc/kvm/book3s_hv_interrupt.c:213:26: error: 'struct kvm_vcpu_arch' has no member named 'shregs'; did you mean 'regs'?
     213 |  WARN_ON_ONCE(vcpu->arch.shregs.msr & MSR_HV);
         |                          ^~~~~~
   include/asm-generic/bug.h:102:25: note: in definition of macro 'WARN_ON_ONCE'
     102 |  int __ret_warn_on = !!(condition);   \
         |                         ^~~~~~~~~
   arch/powerpc/kvm/book3s_hv_interrupt.c:214:28: error: 'struct kvm_vcpu_arch' has no member named 'shregs'; did you mean 'regs'?
     214 |  WARN_ON_ONCE(!(vcpu->arch.shregs.msr & MSR_ME));
         |                            ^~~~~~
   include/asm-generic/bug.h:102:25: note: in definition of macro 'WARN_ON_ONCE'
     102 |  int __ret_warn_on = !!(condition);   \
         |                         ^~~~~~~~~
   In file included from arch/powerpc/include/asm/processor.h:11,
                    from arch/powerpc/include/asm/thread_info.h:40,
                    from include/linux/thread_info.h:58,
                    from include/asm-generic/preempt.h:5,
                    from ./arch/powerpc/include/generated/asm/preempt.h:1,
                    from include/linux/preempt.h:78,
                    from include/linux/percpu.h:6,
                    from include/linux/context_tracking_state.h:5,
                    from include/linux/hardirq.h:5,
                    from include/linux/kvm_host.h:7,
                    from arch/powerpc/kvm/book3s_hv_interrupt.c:3:
   arch/powerpc/kvm/book3s_hv_interrupt.c:217:32: error: 'struct kvm_vcpu_arch' has no member named 'shregs'; did you mean 'regs'?
     217 |  mtspr(SPRN_HSRR1, (vcpu->arch.shregs.msr & ~MSR_HV) | MSR_ME);
         |                                ^~~~~~
   arch/powerpc/include/asm/reg.h:1393:33: note: in definition of macro 'mtspr'
    1393 |          : "r" ((unsigned long)(v)) \
         |                                 ^
   arch/powerpc/kvm/book3s_hv_interrupt.c:244:13: error: 'struct kvm_vcpu_arch' has no member named 'shregs'; did you mean 'regs'?
     244 |  vcpu->arch.shregs.srr0 = mfspr(SPRN_SRR0);
         |             ^~~~~~
         |             regs
   arch/powerpc/kvm/book3s_hv_interrupt.c:245:13: error: 'struct kvm_vcpu_arch' has no member named 'shregs'; did you mean 'regs'?
     245 |  vcpu->arch.shregs.srr1 = mfspr(SPRN_SRR1);
         |             ^~~~~~
         |             regs
   arch/powerpc/kvm/book3s_hv_interrupt.c:246:13: error: 'struct kvm_vcpu_arch' has no member named 'shregs'; did you mean 'regs'?
     246 |  vcpu->arch.shregs.dar = mfspr(SPRN_DAR);
         |             ^~~~~~
         |             regs
   arch/powerpc/kvm/book3s_hv_interrupt.c:247:13: error: 'struct kvm_vcpu_arch' has no member named 'shregs'; did you mean 'regs'?
     247 |  vcpu->arch.shregs.dsisr = mfspr(SPRN_DSISR);
         |             ^~~~~~
         |             regs
   arch/powerpc/kvm/book3s_hv_interrupt.c:281:13: error: 'struct kvm_vcpu_arch' has no member named 'emul_inst'
     281 |   vcpu->arch.emul_inst = mfspr(SPRN_HEIR);
         |             ^
   In file included from arch/powerpc/include/asm/processor.h:11,
                    from arch/powerpc/include/asm/thread_info.h:40,
                    from include/linux/thread_info.h:58,


vim +170 arch/powerpc/kvm/book3s_hv_interrupt.c

   127	
   128	int kvmhv_vcpu_entry_p9(struct kvm_vcpu *vcpu, u64 time_limit, unsigned long lpcr)
   129	{
   130		struct kvm *kvm = vcpu->kvm;
   131		struct kvmppc_vcore *vc = vcpu->arch.vcore;
   132		s64 hdec;
   133		u64 tb, purr, spurr;
   134		u64 *exsave;
   135		unsigned long msr = mfmsr();
   136		int trap;
   137		unsigned long host_hfscr = mfspr(SPRN_HFSCR);
   138		unsigned long host_ciabr = mfspr(SPRN_CIABR);
   139		unsigned long host_dawr0 = mfspr(SPRN_DAWR0);
   140		unsigned long host_dawrx0 = mfspr(SPRN_DAWRX0);
   141		unsigned long host_psscr = mfspr(SPRN_PSSCR);
   142		unsigned long host_pidr = mfspr(SPRN_PID);
   143		unsigned long host_dawr1 = 0;
   144		unsigned long host_dawrx1 = 0;
   145	
   146		if (cpu_has_feature(CPU_FTR_DAWR1)) {
   147			host_dawr1 = mfspr(SPRN_DAWR1);
   148			host_dawrx1 = mfspr(SPRN_DAWRX1);
   149		}
   150	
   151		tb = mftb();
   152		hdec = time_limit - tb;
   153		if (hdec < 0)
   154			return BOOK3S_INTERRUPT_HV_DECREMENTER;
   155	
   156		if (vc->tb_offset) {
   157			u64 new_tb = tb + vc->tb_offset;
   158			mtspr(SPRN_TBU40, new_tb);
   159			tb = mftb();
   160			if ((tb & 0xffffff) < (new_tb & 0xffffff))
   161				mtspr(SPRN_TBU40, new_tb + 0x1000000);
   162			vc->tb_offset_applied = vc->tb_offset;
   163		}
   164	
   165		if (vc->pcr)
   166			mtspr(SPRN_PCR, vc->pcr | PCR_MASK);
   167		mtspr(SPRN_DPDES, vc->dpdes);
   168		mtspr(SPRN_VTB, vc->vtb);
   169	
 > 170		local_paca->kvm_hstate.host_purr = mfspr(SPRN_PURR);
 > 171		local_paca->kvm_hstate.host_spurr = mfspr(SPRN_SPURR);
   172		mtspr(SPRN_PURR, vcpu->arch.purr);
   173		mtspr(SPRN_SPURR, vcpu->arch.spurr);
   174	
   175		if (dawr_enabled()) {
   176			mtspr(SPRN_DAWR0, vcpu->arch.dawr0);
   177			mtspr(SPRN_DAWRX0, vcpu->arch.dawrx0);
   178			if (cpu_has_feature(CPU_FTR_DAWR1)) {
   179				mtspr(SPRN_DAWR1, vcpu->arch.dawr1);
   180				mtspr(SPRN_DAWRX1, vcpu->arch.dawrx1);
   181			}
   182		}
   183		mtspr(SPRN_CIABR, vcpu->arch.ciabr);
   184		mtspr(SPRN_IC, vcpu->arch.ic);
   185	
   186		mtspr(SPRN_PSSCR, vcpu->arch.psscr | PSSCR_EC |
 > 187		      (local_paca->kvm_hstate.fake_suspend << PSSCR_FAKE_SUSPEND_LG));
   188	
   189		mtspr(SPRN_HFSCR, vcpu->arch.hfscr);
   190	
   191		mtspr(SPRN_SPRG0, vcpu->arch.shregs.sprg0);
   192		mtspr(SPRN_SPRG1, vcpu->arch.shregs.sprg1);
   193		mtspr(SPRN_SPRG2, vcpu->arch.shregs.sprg2);
   194		mtspr(SPRN_SPRG3, vcpu->arch.shregs.sprg3);
   195	
   196		mtspr(SPRN_AMOR, ~0UL);
   197	
   198		switch_mmu_to_guest_radix(kvm, vcpu, lpcr);
   199	
   200		/*
   201		 * P9 suppresses the HDEC exception when LPCR[HDICE] = 0,
   202		 * so set guest LPCR (with HDICE) before writing HDEC.
   203		 */
   204		mtspr(SPRN_HDEC, hdec);
   205	
   206		mtspr(SPRN_SRR0, vcpu->arch.shregs.srr0);
   207		mtspr(SPRN_SRR1, vcpu->arch.shregs.srr1);
   208	
   209		start_timing(vcpu, &vcpu->arch.rm_entry);
   210	
   211		vcpu->arch.ceded = 0;
   212	
   213		WARN_ON_ONCE(vcpu->arch.shregs.msr & MSR_HV);
   214		WARN_ON_ONCE(!(vcpu->arch.shregs.msr & MSR_ME));
   215	
   216		mtspr(SPRN_HSRR0, vcpu->arch.regs.nip);
   217		mtspr(SPRN_HSRR1, (vcpu->arch.shregs.msr & ~MSR_HV) | MSR_ME);
   218	
   219		/*
   220		 * On POWER9 DD2.1 and below, sometimes on a Hypervisor Data Storage
   221		 * Interrupt (HDSI) the HDSISR is not be updated at all.
   222		 *
   223		 * To work around this we put a canary value into the HDSISR before
   224		 * returning to a guest and then check for this canary when we take a
   225		 * HDSI. If we find the canary on a HDSI, we know the hardware didn't
   226		 * update the HDSISR. In this case we return to the guest to retake the
   227		 * HDSI which should correctly update the HDSISR the second time HDSI
   228		 * entry.
   229		 *
   230		 * Just do this on all p9 processors for now.
   231		 */
   232		mtspr(SPRN_HDSISR, HDSISR_CANARY);
   233	
   234		accumulate_time(vcpu, &vcpu->arch.guest_time);
   235	
   236		local_paca->kvm_hstate.in_guest = KVM_GUEST_MODE_GUEST_HV_FAST;
   237		kvmppc_p9_enter_guest(vcpu);
   238		// Radix host and guest means host never runs with guest MMU state
   239		local_paca->kvm_hstate.in_guest = KVM_GUEST_MODE_NONE;
   240	
   241		accumulate_time(vcpu, &vcpu->arch.rm_intr);
   242	
   243		/* Get these from r11/12 and paca exsave */
   244		vcpu->arch.shregs.srr0 = mfspr(SPRN_SRR0);
   245		vcpu->arch.shregs.srr1 = mfspr(SPRN_SRR1);
   246		vcpu->arch.shregs.dar = mfspr(SPRN_DAR);
   247		vcpu->arch.shregs.dsisr = mfspr(SPRN_DSISR);
   248	
   249		/* 0x2 bit for HSRR is only used by PR and P7/8 HV paths, clear it */
   250		trap = local_paca->kvm_hstate.scratch0 & ~0x2;
   251		if (likely(trap > BOOK3S_INTERRUPT_MACHINE_CHECK)) {
   252			exsave = local_paca->exgen;
   253		} else if (trap == BOOK3S_INTERRUPT_SYSTEM_RESET) {
   254			exsave = local_paca->exnmi;
   255		} else { /* trap == 0x200 */
   256			exsave = local_paca->exmc;
   257		}
   258	
   259		vcpu->arch.regs.gpr[1] = local_paca->kvm_hstate.scratch1;
   260		vcpu->arch.regs.gpr[3] = local_paca->kvm_hstate.scratch2;
   261		vcpu->arch.regs.gpr[9] = exsave[EX_R9/sizeof(u64)];
   262		vcpu->arch.regs.gpr[10] = exsave[EX_R10/sizeof(u64)];
   263		vcpu->arch.regs.gpr[11] = exsave[EX_R11/sizeof(u64)];
   264		vcpu->arch.regs.gpr[12] = exsave[EX_R12/sizeof(u64)];
   265		vcpu->arch.regs.gpr[13] = exsave[EX_R13/sizeof(u64)];
   266		vcpu->arch.ppr = exsave[EX_PPR/sizeof(u64)];
   267		vcpu->arch.cfar = exsave[EX_CFAR/sizeof(u64)];
   268		vcpu->arch.regs.ctr = exsave[EX_CTR/sizeof(u64)];
   269	
   270		vcpu->arch.last_inst = KVM_INST_FETCH_FAILED;
   271	
   272		if (unlikely(trap == BOOK3S_INTERRUPT_MACHINE_CHECK)) {
   273			vcpu->arch.fault_dar = exsave[EX_DAR/sizeof(u64)];
   274			vcpu->arch.fault_dsisr = exsave[EX_DSISR/sizeof(u64)];
   275			kvmppc_realmode_machine_check(vcpu);
   276	
   277		} else if (unlikely(trap == BOOK3S_INTERRUPT_HMI)) {
   278			kvmppc_realmode_hmi_handler();
   279	
   280		} else if (trap == BOOK3S_INTERRUPT_H_EMUL_ASSIST) {
   281			vcpu->arch.emul_inst = mfspr(SPRN_HEIR);
   282	
   283		} else if (trap == BOOK3S_INTERRUPT_H_DATA_STORAGE) {
   284			vcpu->arch.fault_dar = exsave[EX_DAR/sizeof(u64)];
   285			vcpu->arch.fault_dsisr = exsave[EX_DSISR/sizeof(u64)];
   286			vcpu->arch.fault_gpa = mfspr(SPRN_ASDR);
   287	
   288		} else if (trap == BOOK3S_INTERRUPT_H_INST_STORAGE) {
   289			vcpu->arch.fault_gpa = mfspr(SPRN_ASDR);
   290	
   291		} else if (trap == BOOK3S_INTERRUPT_H_FAC_UNAVAIL) {
   292			vcpu->arch.hfscr = mfspr(SPRN_HFSCR);
   293	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 27150 bytes --]

WARNING: multiple messages have this Message-ID (diff)
From: kernel test robot <lkp@intel.com>
To: Nicholas Piggin <npiggin@gmail.com>, kvm-ppc@vger.kernel.org
Cc: linuxppc-dev@lists.ozlabs.org, kbuild-all@lists.01.org,
	Nicholas Piggin <npiggin@gmail.com>,
	Fabiano Rosas <farosas@linux.ibm.com>
Subject: Re: [PATCH v5 31/48] KVM: PPC: Book3S HV P9: inline kvmhv_load_hv_regs_and_go into __kvmhv_vcpu_entry_p9
Date: Sat, 3 Apr 2021 04:58:05 +0800	[thread overview]
Message-ID: <202104030412.h8fIESGy-lkp@intel.com> (raw)
In-Reply-To: <20210401150325.442125-32-npiggin@gmail.com>

[-- Attachment #1: Type: text/plain, Size: 18530 bytes --]

Hi Nicholas,

I love your patch! Yet something to improve:

[auto build test ERROR on powerpc/next]
[also build test ERROR on v5.12-rc5 next-20210401]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]

url:    https://github.com/0day-ci/linux/commits/Nicholas-Piggin/KVM-PPC-Book3S-C-ify-the-P9-entry-exit-code/20210401-232743
base:   https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git next
config: powerpc64-randconfig-r033-20210402 (attached as .config)
compiler: powerpc64-linux-gcc (GCC) 9.3.0
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # https://github.com/0day-ci/linux/commit/f2a35edda3ab6cba30fbfc362e163d5bc1e086d0
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Nicholas-Piggin/KVM-PPC-Book3S-C-ify-the-P9-entry-exit-code/20210401-232743
        git checkout f2a35edda3ab6cba30fbfc362e163d5bc1e086d0
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=powerpc64 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>

All errors (new ones prefixed by >>):

   arch/powerpc/kvm/book3s_hv_interrupt.c: In function 'switch_mmu_to_guest_radix':
   arch/powerpc/kvm/book3s_hv_interrupt.c:61:46: error: 'struct kvm_vcpu_arch' has no member named 'nested'
      61 |  struct kvm_nested_guest *nested = vcpu->arch.nested;
         |                                              ^
   arch/powerpc/kvm/book3s_hv_interrupt.c:80:2: error: implicit declaration of function 'kvmppc_check_need_tlb_flush' [-Werror=implicit-function-declaration]
      80 |  kvmppc_check_need_tlb_flush(kvm, vc->pcpu, nested);
         |  ^~~~~~~~~~~~~~~~~~~~~~~~~~~
   In file included from arch/powerpc/include/asm/processor.h:11,
                    from arch/powerpc/include/asm/thread_info.h:40,
                    from include/linux/thread_info.h:58,
                    from include/asm-generic/preempt.h:5,
                    from ./arch/powerpc/include/generated/asm/preempt.h:1,
                    from include/linux/preempt.h:78,
                    from include/linux/percpu.h:6,
                    from include/linux/context_tracking_state.h:5,
                    from include/linux/hardirq.h:5,
                    from include/linux/kvm_host.h:7,
                    from arch/powerpc/kvm/book3s_hv_interrupt.c:3:
   arch/powerpc/kvm/book3s_hv_interrupt.c: In function 'switch_mmu_to_host_radix':
   arch/powerpc/kvm/book3s_hv_interrupt.c:88:28: error: 'struct kvm_arch' has no member named 'host_lpid'
      88 |  mtspr(SPRN_LPID, kvm->arch.host_lpid);
         |                            ^
   arch/powerpc/include/asm/reg.h:1393:33: note: in definition of macro 'mtspr'
    1393 |          : "r" ((unsigned long)(v)) \
         |                                 ^
   arch/powerpc/kvm/book3s_hv_interrupt.c:90:28: error: 'struct kvm_arch' has no member named 'host_lpcr'
      90 |  mtspr(SPRN_LPCR, kvm->arch.host_lpcr);
         |                            ^
   arch/powerpc/include/asm/reg.h:1393:33: note: in definition of macro 'mtspr'
    1393 |          : "r" ((unsigned long)(v)) \
         |                                 ^
   arch/powerpc/kvm/book3s_hv_interrupt.c: At top level:
   arch/powerpc/kvm/book3s_hv_interrupt.c:128:5: error: no previous prototype for 'kvmhv_vcpu_entry_p9' [-Werror=missing-prototypes]
     128 | int kvmhv_vcpu_entry_p9(struct kvm_vcpu *vcpu, u64 time_limit, unsigned long lpcr)
         |     ^~~~~~~~~~~~~~~~~~~
   arch/powerpc/kvm/book3s_hv_interrupt.c: In function 'kvmhv_vcpu_entry_p9':
>> arch/powerpc/kvm/book3s_hv_interrupt.c:170:25: error: 'struct kvmppc_host_state' has no member named 'host_purr'; did you mean 'host_r1'?
     170 |  local_paca->kvm_hstate.host_purr = mfspr(SPRN_PURR);
         |                         ^~~~~~~~~
         |                         host_r1
>> arch/powerpc/kvm/book3s_hv_interrupt.c:171:25: error: 'struct kvmppc_host_state' has no member named 'host_spurr'; did you mean 'host_r1'?
     171 |  local_paca->kvm_hstate.host_spurr = mfspr(SPRN_SPURR);
         |                         ^~~~~~~~~~
         |                         host_r1
   In file included from arch/powerpc/include/asm/processor.h:11,
                    from arch/powerpc/include/asm/thread_info.h:40,
                    from include/linux/thread_info.h:58,
                    from include/asm-generic/preempt.h:5,
                    from ./arch/powerpc/include/generated/asm/preempt.h:1,
                    from include/linux/preempt.h:78,
                    from include/linux/percpu.h:6,
                    from include/linux/context_tracking_state.h:5,
                    from include/linux/hardirq.h:5,
                    from include/linux/kvm_host.h:7,
                    from arch/powerpc/kvm/book3s_hv_interrupt.c:3:
>> arch/powerpc/kvm/book3s_hv_interrupt.c:187:31: error: 'struct kvmppc_host_state' has no member named 'fake_suspend'
     187 |        (local_paca->kvm_hstate.fake_suspend << PSSCR_FAKE_SUSPEND_LG));
         |                               ^
   arch/powerpc/include/asm/reg.h:1393:33: note: in definition of macro 'mtspr'
    1393 |          : "r" ((unsigned long)(v)) \
         |                                 ^
   arch/powerpc/kvm/book3s_hv_interrupt.c:191:31: error: 'struct kvm_vcpu_arch' has no member named 'shregs'; did you mean 'regs'?
     191 |  mtspr(SPRN_SPRG0, vcpu->arch.shregs.sprg0);
         |                               ^~~~~~
   arch/powerpc/include/asm/reg.h:1393:33: note: in definition of macro 'mtspr'
    1393 |          : "r" ((unsigned long)(v)) \
         |                                 ^
   arch/powerpc/kvm/book3s_hv_interrupt.c:192:31: error: 'struct kvm_vcpu_arch' has no member named 'shregs'; did you mean 'regs'?
     192 |  mtspr(SPRN_SPRG1, vcpu->arch.shregs.sprg1);
         |                               ^~~~~~
   arch/powerpc/include/asm/reg.h:1393:33: note: in definition of macro 'mtspr'
    1393 |          : "r" ((unsigned long)(v)) \
         |                                 ^
   arch/powerpc/kvm/book3s_hv_interrupt.c:193:31: error: 'struct kvm_vcpu_arch' has no member named 'shregs'; did you mean 'regs'?
     193 |  mtspr(SPRN_SPRG2, vcpu->arch.shregs.sprg2);
         |                               ^~~~~~
   arch/powerpc/include/asm/reg.h:1393:33: note: in definition of macro 'mtspr'
    1393 |          : "r" ((unsigned long)(v)) \
         |                                 ^
   arch/powerpc/kvm/book3s_hv_interrupt.c:194:31: error: 'struct kvm_vcpu_arch' has no member named 'shregs'; did you mean 'regs'?
     194 |  mtspr(SPRN_SPRG3, vcpu->arch.shregs.sprg3);
         |                               ^~~~~~
   arch/powerpc/include/asm/reg.h:1393:33: note: in definition of macro 'mtspr'
    1393 |          : "r" ((unsigned long)(v)) \
         |                                 ^
   arch/powerpc/kvm/book3s_hv_interrupt.c:206:30: error: 'struct kvm_vcpu_arch' has no member named 'shregs'; did you mean 'regs'?
     206 |  mtspr(SPRN_SRR0, vcpu->arch.shregs.srr0);
         |                              ^~~~~~
   arch/powerpc/include/asm/reg.h:1393:33: note: in definition of macro 'mtspr'
    1393 |          : "r" ((unsigned long)(v)) \
         |                                 ^
   arch/powerpc/kvm/book3s_hv_interrupt.c:207:30: error: 'struct kvm_vcpu_arch' has no member named 'shregs'; did you mean 'regs'?
     207 |  mtspr(SPRN_SRR1, vcpu->arch.shregs.srr1);
         |                              ^~~~~~
   arch/powerpc/include/asm/reg.h:1393:33: note: in definition of macro 'mtspr'
    1393 |          : "r" ((unsigned long)(v)) \
         |                                 ^
   In file included from arch/powerpc/include/asm/bug.h:109,
                    from include/linux/bug.h:5,
                    from include/linux/mmdebug.h:5,
                    from include/linux/percpu.h:5,
                    from include/linux/context_tracking_state.h:5,
                    from include/linux/hardirq.h:5,
                    from include/linux/kvm_host.h:7,
                    from arch/powerpc/kvm/book3s_hv_interrupt.c:3:
   arch/powerpc/kvm/book3s_hv_interrupt.c:213:26: error: 'struct kvm_vcpu_arch' has no member named 'shregs'; did you mean 'regs'?
     213 |  WARN_ON_ONCE(vcpu->arch.shregs.msr & MSR_HV);
         |                          ^~~~~~
   include/asm-generic/bug.h:102:25: note: in definition of macro 'WARN_ON_ONCE'
     102 |  int __ret_warn_on = !!(condition);   \
         |                         ^~~~~~~~~
   arch/powerpc/kvm/book3s_hv_interrupt.c:214:28: error: 'struct kvm_vcpu_arch' has no member named 'shregs'; did you mean 'regs'?
     214 |  WARN_ON_ONCE(!(vcpu->arch.shregs.msr & MSR_ME));
         |                            ^~~~~~
   include/asm-generic/bug.h:102:25: note: in definition of macro 'WARN_ON_ONCE'
     102 |  int __ret_warn_on = !!(condition);   \
         |                         ^~~~~~~~~
   In file included from arch/powerpc/include/asm/processor.h:11,
                    from arch/powerpc/include/asm/thread_info.h:40,
                    from include/linux/thread_info.h:58,
                    from include/asm-generic/preempt.h:5,
                    from ./arch/powerpc/include/generated/asm/preempt.h:1,
                    from include/linux/preempt.h:78,
                    from include/linux/percpu.h:6,
                    from include/linux/context_tracking_state.h:5,
                    from include/linux/hardirq.h:5,
                    from include/linux/kvm_host.h:7,
                    from arch/powerpc/kvm/book3s_hv_interrupt.c:3:
   arch/powerpc/kvm/book3s_hv_interrupt.c:217:32: error: 'struct kvm_vcpu_arch' has no member named 'shregs'; did you mean 'regs'?
     217 |  mtspr(SPRN_HSRR1, (vcpu->arch.shregs.msr & ~MSR_HV) | MSR_ME);
         |                                ^~~~~~
   arch/powerpc/include/asm/reg.h:1393:33: note: in definition of macro 'mtspr'
    1393 |          : "r" ((unsigned long)(v)) \
         |                                 ^
   arch/powerpc/kvm/book3s_hv_interrupt.c:244:13: error: 'struct kvm_vcpu_arch' has no member named 'shregs'; did you mean 'regs'?
     244 |  vcpu->arch.shregs.srr0 = mfspr(SPRN_SRR0);
         |             ^~~~~~
         |             regs
   arch/powerpc/kvm/book3s_hv_interrupt.c:245:13: error: 'struct kvm_vcpu_arch' has no member named 'shregs'; did you mean 'regs'?
     245 |  vcpu->arch.shregs.srr1 = mfspr(SPRN_SRR1);
         |             ^~~~~~
         |             regs
   arch/powerpc/kvm/book3s_hv_interrupt.c:246:13: error: 'struct kvm_vcpu_arch' has no member named 'shregs'; did you mean 'regs'?
     246 |  vcpu->arch.shregs.dar = mfspr(SPRN_DAR);
         |             ^~~~~~
         |             regs
   arch/powerpc/kvm/book3s_hv_interrupt.c:247:13: error: 'struct kvm_vcpu_arch' has no member named 'shregs'; did you mean 'regs'?
     247 |  vcpu->arch.shregs.dsisr = mfspr(SPRN_DSISR);
         |             ^~~~~~
         |             regs
   arch/powerpc/kvm/book3s_hv_interrupt.c:281:13: error: 'struct kvm_vcpu_arch' has no member named 'emul_inst'
     281 |   vcpu->arch.emul_inst = mfspr(SPRN_HEIR);
         |             ^
   In file included from arch/powerpc/include/asm/processor.h:11,
                    from arch/powerpc/include/asm/thread_info.h:40,
                    from include/linux/thread_info.h:58,


vim +170 arch/powerpc/kvm/book3s_hv_interrupt.c

   127	
   128	int kvmhv_vcpu_entry_p9(struct kvm_vcpu *vcpu, u64 time_limit, unsigned long lpcr)
   129	{
   130		struct kvm *kvm = vcpu->kvm;
   131		struct kvmppc_vcore *vc = vcpu->arch.vcore;
   132		s64 hdec;
   133		u64 tb, purr, spurr;
   134		u64 *exsave;
   135		unsigned long msr = mfmsr();
   136		int trap;
   137		unsigned long host_hfscr = mfspr(SPRN_HFSCR);
   138		unsigned long host_ciabr = mfspr(SPRN_CIABR);
   139		unsigned long host_dawr0 = mfspr(SPRN_DAWR0);
   140		unsigned long host_dawrx0 = mfspr(SPRN_DAWRX0);
   141		unsigned long host_psscr = mfspr(SPRN_PSSCR);
   142		unsigned long host_pidr = mfspr(SPRN_PID);
   143		unsigned long host_dawr1 = 0;
   144		unsigned long host_dawrx1 = 0;
   145	
   146		if (cpu_has_feature(CPU_FTR_DAWR1)) {
   147			host_dawr1 = mfspr(SPRN_DAWR1);
   148			host_dawrx1 = mfspr(SPRN_DAWRX1);
   149		}
   150	
   151		tb = mftb();
   152		hdec = time_limit - tb;
   153		if (hdec < 0)
   154			return BOOK3S_INTERRUPT_HV_DECREMENTER;
   155	
   156		if (vc->tb_offset) {
   157			u64 new_tb = tb + vc->tb_offset;
   158			mtspr(SPRN_TBU40, new_tb);
   159			tb = mftb();
   160			if ((tb & 0xffffff) < (new_tb & 0xffffff))
   161				mtspr(SPRN_TBU40, new_tb + 0x1000000);
   162			vc->tb_offset_applied = vc->tb_offset;
   163		}
   164	
   165		if (vc->pcr)
   166			mtspr(SPRN_PCR, vc->pcr | PCR_MASK);
   167		mtspr(SPRN_DPDES, vc->dpdes);
   168		mtspr(SPRN_VTB, vc->vtb);
   169	
 > 170		local_paca->kvm_hstate.host_purr = mfspr(SPRN_PURR);
 > 171		local_paca->kvm_hstate.host_spurr = mfspr(SPRN_SPURR);
   172		mtspr(SPRN_PURR, vcpu->arch.purr);
   173		mtspr(SPRN_SPURR, vcpu->arch.spurr);
   174	
   175		if (dawr_enabled()) {
   176			mtspr(SPRN_DAWR0, vcpu->arch.dawr0);
   177			mtspr(SPRN_DAWRX0, vcpu->arch.dawrx0);
   178			if (cpu_has_feature(CPU_FTR_DAWR1)) {
   179				mtspr(SPRN_DAWR1, vcpu->arch.dawr1);
   180				mtspr(SPRN_DAWRX1, vcpu->arch.dawrx1);
   181			}
   182		}
   183		mtspr(SPRN_CIABR, vcpu->arch.ciabr);
   184		mtspr(SPRN_IC, vcpu->arch.ic);
   185	
   186		mtspr(SPRN_PSSCR, vcpu->arch.psscr | PSSCR_EC |
 > 187		      (local_paca->kvm_hstate.fake_suspend << PSSCR_FAKE_SUSPEND_LG));
   188	
   189		mtspr(SPRN_HFSCR, vcpu->arch.hfscr);
   190	
   191		mtspr(SPRN_SPRG0, vcpu->arch.shregs.sprg0);
   192		mtspr(SPRN_SPRG1, vcpu->arch.shregs.sprg1);
   193		mtspr(SPRN_SPRG2, vcpu->arch.shregs.sprg2);
   194		mtspr(SPRN_SPRG3, vcpu->arch.shregs.sprg3);
   195	
   196		mtspr(SPRN_AMOR, ~0UL);
   197	
   198		switch_mmu_to_guest_radix(kvm, vcpu, lpcr);
   199	
   200		/*
   201		 * P9 suppresses the HDEC exception when LPCR[HDICE] = 0,
   202		 * so set guest LPCR (with HDICE) before writing HDEC.
   203		 */
   204		mtspr(SPRN_HDEC, hdec);
   205	
   206		mtspr(SPRN_SRR0, vcpu->arch.shregs.srr0);
   207		mtspr(SPRN_SRR1, vcpu->arch.shregs.srr1);
   208	
   209		start_timing(vcpu, &vcpu->arch.rm_entry);
   210	
   211		vcpu->arch.ceded = 0;
   212	
   213		WARN_ON_ONCE(vcpu->arch.shregs.msr & MSR_HV);
   214		WARN_ON_ONCE(!(vcpu->arch.shregs.msr & MSR_ME));
   215	
   216		mtspr(SPRN_HSRR0, vcpu->arch.regs.nip);
   217		mtspr(SPRN_HSRR1, (vcpu->arch.shregs.msr & ~MSR_HV) | MSR_ME);
   218	
   219		/*
   220		 * On POWER9 DD2.1 and below, sometimes on a Hypervisor Data Storage
   221		 * Interrupt (HDSI) the HDSISR is not be updated at all.
   222		 *
   223		 * To work around this we put a canary value into the HDSISR before
   224		 * returning to a guest and then check for this canary when we take a
   225		 * HDSI. If we find the canary on a HDSI, we know the hardware didn't
   226		 * update the HDSISR. In this case we return to the guest to retake the
   227		 * HDSI which should correctly update the HDSISR the second time HDSI
   228		 * entry.
   229		 *
   230		 * Just do this on all p9 processors for now.
   231		 */
   232		mtspr(SPRN_HDSISR, HDSISR_CANARY);
   233	
   234		accumulate_time(vcpu, &vcpu->arch.guest_time);
   235	
   236		local_paca->kvm_hstate.in_guest = KVM_GUEST_MODE_GUEST_HV_FAST;
   237		kvmppc_p9_enter_guest(vcpu);
   238		// Radix host and guest means host never runs with guest MMU state
   239		local_paca->kvm_hstate.in_guest = KVM_GUEST_MODE_NONE;
   240	
   241		accumulate_time(vcpu, &vcpu->arch.rm_intr);
   242	
   243		/* Get these from r11/12 and paca exsave */
   244		vcpu->arch.shregs.srr0 = mfspr(SPRN_SRR0);
   245		vcpu->arch.shregs.srr1 = mfspr(SPRN_SRR1);
   246		vcpu->arch.shregs.dar = mfspr(SPRN_DAR);
   247		vcpu->arch.shregs.dsisr = mfspr(SPRN_DSISR);
   248	
   249		/* 0x2 bit for HSRR is only used by PR and P7/8 HV paths, clear it */
   250		trap = local_paca->kvm_hstate.scratch0 & ~0x2;
   251		if (likely(trap > BOOK3S_INTERRUPT_MACHINE_CHECK)) {
   252			exsave = local_paca->exgen;
   253		} else if (trap == BOOK3S_INTERRUPT_SYSTEM_RESET) {
   254			exsave = local_paca->exnmi;
   255		} else { /* trap == 0x200 */
   256			exsave = local_paca->exmc;
   257		}
   258	
   259		vcpu->arch.regs.gpr[1] = local_paca->kvm_hstate.scratch1;
   260		vcpu->arch.regs.gpr[3] = local_paca->kvm_hstate.scratch2;
   261		vcpu->arch.regs.gpr[9] = exsave[EX_R9/sizeof(u64)];
   262		vcpu->arch.regs.gpr[10] = exsave[EX_R10/sizeof(u64)];
   263		vcpu->arch.regs.gpr[11] = exsave[EX_R11/sizeof(u64)];
   264		vcpu->arch.regs.gpr[12] = exsave[EX_R12/sizeof(u64)];
   265		vcpu->arch.regs.gpr[13] = exsave[EX_R13/sizeof(u64)];
   266		vcpu->arch.ppr = exsave[EX_PPR/sizeof(u64)];
   267		vcpu->arch.cfar = exsave[EX_CFAR/sizeof(u64)];
   268		vcpu->arch.regs.ctr = exsave[EX_CTR/sizeof(u64)];
   269	
   270		vcpu->arch.last_inst = KVM_INST_FETCH_FAILED;
   271	
   272		if (unlikely(trap == BOOK3S_INTERRUPT_MACHINE_CHECK)) {
   273			vcpu->arch.fault_dar = exsave[EX_DAR/sizeof(u64)];
   274			vcpu->arch.fault_dsisr = exsave[EX_DSISR/sizeof(u64)];
   275			kvmppc_realmode_machine_check(vcpu);
   276	
   277		} else if (unlikely(trap == BOOK3S_INTERRUPT_HMI)) {
   278			kvmppc_realmode_hmi_handler();
   279	
   280		} else if (trap == BOOK3S_INTERRUPT_H_EMUL_ASSIST) {
   281			vcpu->arch.emul_inst = mfspr(SPRN_HEIR);
   282	
   283		} else if (trap == BOOK3S_INTERRUPT_H_DATA_STORAGE) {
   284			vcpu->arch.fault_dar = exsave[EX_DAR/sizeof(u64)];
   285			vcpu->arch.fault_dsisr = exsave[EX_DSISR/sizeof(u64)];
   286			vcpu->arch.fault_gpa = mfspr(SPRN_ASDR);
   287	
   288		} else if (trap == BOOK3S_INTERRUPT_H_INST_STORAGE) {
   289			vcpu->arch.fault_gpa = mfspr(SPRN_ASDR);
   290	
   291		} else if (trap == BOOK3S_INTERRUPT_H_FAC_UNAVAIL) {
   292			vcpu->arch.hfscr = mfspr(SPRN_HFSCR);
   293	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 27150 bytes --]

WARNING: multiple messages have this Message-ID (diff)
From: kernel test robot <lkp@intel.com>
To: kbuild-all@lists.01.org
Subject: Re: [PATCH v5 31/48] KVM: PPC: Book3S HV P9: inline kvmhv_load_hv_regs_and_go into __kvmhv_vcpu_entry_p9
Date: Sat, 03 Apr 2021 04:58:05 +0800	[thread overview]
Message-ID: <202104030412.h8fIESGy-lkp@intel.com> (raw)
In-Reply-To: <20210401150325.442125-32-npiggin@gmail.com>

[-- Attachment #1: Type: text/plain, Size: 18892 bytes --]

Hi Nicholas,

I love your patch! Yet something to improve:

[auto build test ERROR on powerpc/next]
[also build test ERROR on v5.12-rc5 next-20210401]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]

url:    https://github.com/0day-ci/linux/commits/Nicholas-Piggin/KVM-PPC-Book3S-C-ify-the-P9-entry-exit-code/20210401-232743
base:   https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git next
config: powerpc64-randconfig-r033-20210402 (attached as .config)
compiler: powerpc64-linux-gcc (GCC) 9.3.0
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # https://github.com/0day-ci/linux/commit/f2a35edda3ab6cba30fbfc362e163d5bc1e086d0
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Nicholas-Piggin/KVM-PPC-Book3S-C-ify-the-P9-entry-exit-code/20210401-232743
        git checkout f2a35edda3ab6cba30fbfc362e163d5bc1e086d0
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=powerpc64 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>

All errors (new ones prefixed by >>):

   arch/powerpc/kvm/book3s_hv_interrupt.c: In function 'switch_mmu_to_guest_radix':
   arch/powerpc/kvm/book3s_hv_interrupt.c:61:46: error: 'struct kvm_vcpu_arch' has no member named 'nested'
      61 |  struct kvm_nested_guest *nested = vcpu->arch.nested;
         |                                              ^
   arch/powerpc/kvm/book3s_hv_interrupt.c:80:2: error: implicit declaration of function 'kvmppc_check_need_tlb_flush' [-Werror=implicit-function-declaration]
      80 |  kvmppc_check_need_tlb_flush(kvm, vc->pcpu, nested);
         |  ^~~~~~~~~~~~~~~~~~~~~~~~~~~
   In file included from arch/powerpc/include/asm/processor.h:11,
                    from arch/powerpc/include/asm/thread_info.h:40,
                    from include/linux/thread_info.h:58,
                    from include/asm-generic/preempt.h:5,
                    from ./arch/powerpc/include/generated/asm/preempt.h:1,
                    from include/linux/preempt.h:78,
                    from include/linux/percpu.h:6,
                    from include/linux/context_tracking_state.h:5,
                    from include/linux/hardirq.h:5,
                    from include/linux/kvm_host.h:7,
                    from arch/powerpc/kvm/book3s_hv_interrupt.c:3:
   arch/powerpc/kvm/book3s_hv_interrupt.c: In function 'switch_mmu_to_host_radix':
   arch/powerpc/kvm/book3s_hv_interrupt.c:88:28: error: 'struct kvm_arch' has no member named 'host_lpid'
      88 |  mtspr(SPRN_LPID, kvm->arch.host_lpid);
         |                            ^
   arch/powerpc/include/asm/reg.h:1393:33: note: in definition of macro 'mtspr'
    1393 |          : "r" ((unsigned long)(v)) \
         |                                 ^
   arch/powerpc/kvm/book3s_hv_interrupt.c:90:28: error: 'struct kvm_arch' has no member named 'host_lpcr'
      90 |  mtspr(SPRN_LPCR, kvm->arch.host_lpcr);
         |                            ^
   arch/powerpc/include/asm/reg.h:1393:33: note: in definition of macro 'mtspr'
    1393 |          : "r" ((unsigned long)(v)) \
         |                                 ^
   arch/powerpc/kvm/book3s_hv_interrupt.c: At top level:
   arch/powerpc/kvm/book3s_hv_interrupt.c:128:5: error: no previous prototype for 'kvmhv_vcpu_entry_p9' [-Werror=missing-prototypes]
     128 | int kvmhv_vcpu_entry_p9(struct kvm_vcpu *vcpu, u64 time_limit, unsigned long lpcr)
         |     ^~~~~~~~~~~~~~~~~~~
   arch/powerpc/kvm/book3s_hv_interrupt.c: In function 'kvmhv_vcpu_entry_p9':
>> arch/powerpc/kvm/book3s_hv_interrupt.c:170:25: error: 'struct kvmppc_host_state' has no member named 'host_purr'; did you mean 'host_r1'?
     170 |  local_paca->kvm_hstate.host_purr = mfspr(SPRN_PURR);
         |                         ^~~~~~~~~
         |                         host_r1
>> arch/powerpc/kvm/book3s_hv_interrupt.c:171:25: error: 'struct kvmppc_host_state' has no member named 'host_spurr'; did you mean 'host_r1'?
     171 |  local_paca->kvm_hstate.host_spurr = mfspr(SPRN_SPURR);
         |                         ^~~~~~~~~~
         |                         host_r1
   In file included from arch/powerpc/include/asm/processor.h:11,
                    from arch/powerpc/include/asm/thread_info.h:40,
                    from include/linux/thread_info.h:58,
                    from include/asm-generic/preempt.h:5,
                    from ./arch/powerpc/include/generated/asm/preempt.h:1,
                    from include/linux/preempt.h:78,
                    from include/linux/percpu.h:6,
                    from include/linux/context_tracking_state.h:5,
                    from include/linux/hardirq.h:5,
                    from include/linux/kvm_host.h:7,
                    from arch/powerpc/kvm/book3s_hv_interrupt.c:3:
>> arch/powerpc/kvm/book3s_hv_interrupt.c:187:31: error: 'struct kvmppc_host_state' has no member named 'fake_suspend'
     187 |        (local_paca->kvm_hstate.fake_suspend << PSSCR_FAKE_SUSPEND_LG));
         |                               ^
   arch/powerpc/include/asm/reg.h:1393:33: note: in definition of macro 'mtspr'
    1393 |          : "r" ((unsigned long)(v)) \
         |                                 ^
   arch/powerpc/kvm/book3s_hv_interrupt.c:191:31: error: 'struct kvm_vcpu_arch' has no member named 'shregs'; did you mean 'regs'?
     191 |  mtspr(SPRN_SPRG0, vcpu->arch.shregs.sprg0);
         |                               ^~~~~~
   arch/powerpc/include/asm/reg.h:1393:33: note: in definition of macro 'mtspr'
    1393 |          : "r" ((unsigned long)(v)) \
         |                                 ^
   arch/powerpc/kvm/book3s_hv_interrupt.c:192:31: error: 'struct kvm_vcpu_arch' has no member named 'shregs'; did you mean 'regs'?
     192 |  mtspr(SPRN_SPRG1, vcpu->arch.shregs.sprg1);
         |                               ^~~~~~
   arch/powerpc/include/asm/reg.h:1393:33: note: in definition of macro 'mtspr'
    1393 |          : "r" ((unsigned long)(v)) \
         |                                 ^
   arch/powerpc/kvm/book3s_hv_interrupt.c:193:31: error: 'struct kvm_vcpu_arch' has no member named 'shregs'; did you mean 'regs'?
     193 |  mtspr(SPRN_SPRG2, vcpu->arch.shregs.sprg2);
         |                               ^~~~~~
   arch/powerpc/include/asm/reg.h:1393:33: note: in definition of macro 'mtspr'
    1393 |          : "r" ((unsigned long)(v)) \
         |                                 ^
   arch/powerpc/kvm/book3s_hv_interrupt.c:194:31: error: 'struct kvm_vcpu_arch' has no member named 'shregs'; did you mean 'regs'?
     194 |  mtspr(SPRN_SPRG3, vcpu->arch.shregs.sprg3);
         |                               ^~~~~~
   arch/powerpc/include/asm/reg.h:1393:33: note: in definition of macro 'mtspr'
    1393 |          : "r" ((unsigned long)(v)) \
         |                                 ^
   arch/powerpc/kvm/book3s_hv_interrupt.c:206:30: error: 'struct kvm_vcpu_arch' has no member named 'shregs'; did you mean 'regs'?
     206 |  mtspr(SPRN_SRR0, vcpu->arch.shregs.srr0);
         |                              ^~~~~~
   arch/powerpc/include/asm/reg.h:1393:33: note: in definition of macro 'mtspr'
    1393 |          : "r" ((unsigned long)(v)) \
         |                                 ^
   arch/powerpc/kvm/book3s_hv_interrupt.c:207:30: error: 'struct kvm_vcpu_arch' has no member named 'shregs'; did you mean 'regs'?
     207 |  mtspr(SPRN_SRR1, vcpu->arch.shregs.srr1);
         |                              ^~~~~~
   arch/powerpc/include/asm/reg.h:1393:33: note: in definition of macro 'mtspr'
    1393 |          : "r" ((unsigned long)(v)) \
         |                                 ^
   In file included from arch/powerpc/include/asm/bug.h:109,
                    from include/linux/bug.h:5,
                    from include/linux/mmdebug.h:5,
                    from include/linux/percpu.h:5,
                    from include/linux/context_tracking_state.h:5,
                    from include/linux/hardirq.h:5,
                    from include/linux/kvm_host.h:7,
                    from arch/powerpc/kvm/book3s_hv_interrupt.c:3:
   arch/powerpc/kvm/book3s_hv_interrupt.c:213:26: error: 'struct kvm_vcpu_arch' has no member named 'shregs'; did you mean 'regs'?
     213 |  WARN_ON_ONCE(vcpu->arch.shregs.msr & MSR_HV);
         |                          ^~~~~~
   include/asm-generic/bug.h:102:25: note: in definition of macro 'WARN_ON_ONCE'
     102 |  int __ret_warn_on = !!(condition);   \
         |                         ^~~~~~~~~
   arch/powerpc/kvm/book3s_hv_interrupt.c:214:28: error: 'struct kvm_vcpu_arch' has no member named 'shregs'; did you mean 'regs'?
     214 |  WARN_ON_ONCE(!(vcpu->arch.shregs.msr & MSR_ME));
         |                            ^~~~~~
   include/asm-generic/bug.h:102:25: note: in definition of macro 'WARN_ON_ONCE'
     102 |  int __ret_warn_on = !!(condition);   \
         |                         ^~~~~~~~~
   In file included from arch/powerpc/include/asm/processor.h:11,
                    from arch/powerpc/include/asm/thread_info.h:40,
                    from include/linux/thread_info.h:58,
                    from include/asm-generic/preempt.h:5,
                    from ./arch/powerpc/include/generated/asm/preempt.h:1,
                    from include/linux/preempt.h:78,
                    from include/linux/percpu.h:6,
                    from include/linux/context_tracking_state.h:5,
                    from include/linux/hardirq.h:5,
                    from include/linux/kvm_host.h:7,
                    from arch/powerpc/kvm/book3s_hv_interrupt.c:3:
   arch/powerpc/kvm/book3s_hv_interrupt.c:217:32: error: 'struct kvm_vcpu_arch' has no member named 'shregs'; did you mean 'regs'?
     217 |  mtspr(SPRN_HSRR1, (vcpu->arch.shregs.msr & ~MSR_HV) | MSR_ME);
         |                                ^~~~~~
   arch/powerpc/include/asm/reg.h:1393:33: note: in definition of macro 'mtspr'
    1393 |          : "r" ((unsigned long)(v)) \
         |                                 ^
   arch/powerpc/kvm/book3s_hv_interrupt.c:244:13: error: 'struct kvm_vcpu_arch' has no member named 'shregs'; did you mean 'regs'?
     244 |  vcpu->arch.shregs.srr0 = mfspr(SPRN_SRR0);
         |             ^~~~~~
         |             regs
   arch/powerpc/kvm/book3s_hv_interrupt.c:245:13: error: 'struct kvm_vcpu_arch' has no member named 'shregs'; did you mean 'regs'?
     245 |  vcpu->arch.shregs.srr1 = mfspr(SPRN_SRR1);
         |             ^~~~~~
         |             regs
   arch/powerpc/kvm/book3s_hv_interrupt.c:246:13: error: 'struct kvm_vcpu_arch' has no member named 'shregs'; did you mean 'regs'?
     246 |  vcpu->arch.shregs.dar = mfspr(SPRN_DAR);
         |             ^~~~~~
         |             regs
   arch/powerpc/kvm/book3s_hv_interrupt.c:247:13: error: 'struct kvm_vcpu_arch' has no member named 'shregs'; did you mean 'regs'?
     247 |  vcpu->arch.shregs.dsisr = mfspr(SPRN_DSISR);
         |             ^~~~~~
         |             regs
   arch/powerpc/kvm/book3s_hv_interrupt.c:281:13: error: 'struct kvm_vcpu_arch' has no member named 'emul_inst'
     281 |   vcpu->arch.emul_inst = mfspr(SPRN_HEIR);
         |             ^
   In file included from arch/powerpc/include/asm/processor.h:11,
                    from arch/powerpc/include/asm/thread_info.h:40,
                    from include/linux/thread_info.h:58,


vim +170 arch/powerpc/kvm/book3s_hv_interrupt.c

   127	
   128	int kvmhv_vcpu_entry_p9(struct kvm_vcpu *vcpu, u64 time_limit, unsigned long lpcr)
   129	{
   130		struct kvm *kvm = vcpu->kvm;
   131		struct kvmppc_vcore *vc = vcpu->arch.vcore;
   132		s64 hdec;
   133		u64 tb, purr, spurr;
   134		u64 *exsave;
   135		unsigned long msr = mfmsr();
   136		int trap;
   137		unsigned long host_hfscr = mfspr(SPRN_HFSCR);
   138		unsigned long host_ciabr = mfspr(SPRN_CIABR);
   139		unsigned long host_dawr0 = mfspr(SPRN_DAWR0);
   140		unsigned long host_dawrx0 = mfspr(SPRN_DAWRX0);
   141		unsigned long host_psscr = mfspr(SPRN_PSSCR);
   142		unsigned long host_pidr = mfspr(SPRN_PID);
   143		unsigned long host_dawr1 = 0;
   144		unsigned long host_dawrx1 = 0;
   145	
   146		if (cpu_has_feature(CPU_FTR_DAWR1)) {
   147			host_dawr1 = mfspr(SPRN_DAWR1);
   148			host_dawrx1 = mfspr(SPRN_DAWRX1);
   149		}
   150	
   151		tb = mftb();
   152		hdec = time_limit - tb;
   153		if (hdec < 0)
   154			return BOOK3S_INTERRUPT_HV_DECREMENTER;
   155	
   156		if (vc->tb_offset) {
   157			u64 new_tb = tb + vc->tb_offset;
   158			mtspr(SPRN_TBU40, new_tb);
   159			tb = mftb();
   160			if ((tb & 0xffffff) < (new_tb & 0xffffff))
   161				mtspr(SPRN_TBU40, new_tb + 0x1000000);
   162			vc->tb_offset_applied = vc->tb_offset;
   163		}
   164	
   165		if (vc->pcr)
   166			mtspr(SPRN_PCR, vc->pcr | PCR_MASK);
   167		mtspr(SPRN_DPDES, vc->dpdes);
   168		mtspr(SPRN_VTB, vc->vtb);
   169	
 > 170		local_paca->kvm_hstate.host_purr = mfspr(SPRN_PURR);
 > 171		local_paca->kvm_hstate.host_spurr = mfspr(SPRN_SPURR);
   172		mtspr(SPRN_PURR, vcpu->arch.purr);
   173		mtspr(SPRN_SPURR, vcpu->arch.spurr);
   174	
   175		if (dawr_enabled()) {
   176			mtspr(SPRN_DAWR0, vcpu->arch.dawr0);
   177			mtspr(SPRN_DAWRX0, vcpu->arch.dawrx0);
   178			if (cpu_has_feature(CPU_FTR_DAWR1)) {
   179				mtspr(SPRN_DAWR1, vcpu->arch.dawr1);
   180				mtspr(SPRN_DAWRX1, vcpu->arch.dawrx1);
   181			}
   182		}
   183		mtspr(SPRN_CIABR, vcpu->arch.ciabr);
   184		mtspr(SPRN_IC, vcpu->arch.ic);
   185	
   186		mtspr(SPRN_PSSCR, vcpu->arch.psscr | PSSCR_EC |
 > 187		      (local_paca->kvm_hstate.fake_suspend << PSSCR_FAKE_SUSPEND_LG));
   188	
   189		mtspr(SPRN_HFSCR, vcpu->arch.hfscr);
   190	
   191		mtspr(SPRN_SPRG0, vcpu->arch.shregs.sprg0);
   192		mtspr(SPRN_SPRG1, vcpu->arch.shregs.sprg1);
   193		mtspr(SPRN_SPRG2, vcpu->arch.shregs.sprg2);
   194		mtspr(SPRN_SPRG3, vcpu->arch.shregs.sprg3);
   195	
   196		mtspr(SPRN_AMOR, ~0UL);
   197	
   198		switch_mmu_to_guest_radix(kvm, vcpu, lpcr);
   199	
   200		/*
   201		 * P9 suppresses the HDEC exception when LPCR[HDICE] = 0,
   202		 * so set guest LPCR (with HDICE) before writing HDEC.
   203		 */
   204		mtspr(SPRN_HDEC, hdec);
   205	
   206		mtspr(SPRN_SRR0, vcpu->arch.shregs.srr0);
   207		mtspr(SPRN_SRR1, vcpu->arch.shregs.srr1);
   208	
   209		start_timing(vcpu, &vcpu->arch.rm_entry);
   210	
   211		vcpu->arch.ceded = 0;
   212	
   213		WARN_ON_ONCE(vcpu->arch.shregs.msr & MSR_HV);
   214		WARN_ON_ONCE(!(vcpu->arch.shregs.msr & MSR_ME));
   215	
   216		mtspr(SPRN_HSRR0, vcpu->arch.regs.nip);
   217		mtspr(SPRN_HSRR1, (vcpu->arch.shregs.msr & ~MSR_HV) | MSR_ME);
   218	
   219		/*
   220		 * On POWER9 DD2.1 and below, sometimes on a Hypervisor Data Storage
   221		 * Interrupt (HDSI) the HDSISR is not be updated at all.
   222		 *
   223		 * To work around this we put a canary value into the HDSISR before
   224		 * returning to a guest and then check for this canary when we take a
   225		 * HDSI. If we find the canary on a HDSI, we know the hardware didn't
   226		 * update the HDSISR. In this case we return to the guest to retake the
   227		 * HDSI which should correctly update the HDSISR the second time HDSI
   228		 * entry.
   229		 *
   230		 * Just do this on all p9 processors for now.
   231		 */
   232		mtspr(SPRN_HDSISR, HDSISR_CANARY);
   233	
   234		accumulate_time(vcpu, &vcpu->arch.guest_time);
   235	
   236		local_paca->kvm_hstate.in_guest = KVM_GUEST_MODE_GUEST_HV_FAST;
   237		kvmppc_p9_enter_guest(vcpu);
   238		// Radix host and guest means host never runs with guest MMU state
   239		local_paca->kvm_hstate.in_guest = KVM_GUEST_MODE_NONE;
   240	
   241		accumulate_time(vcpu, &vcpu->arch.rm_intr);
   242	
   243		/* Get these from r11/12 and paca exsave */
   244		vcpu->arch.shregs.srr0 = mfspr(SPRN_SRR0);
   245		vcpu->arch.shregs.srr1 = mfspr(SPRN_SRR1);
   246		vcpu->arch.shregs.dar = mfspr(SPRN_DAR);
   247		vcpu->arch.shregs.dsisr = mfspr(SPRN_DSISR);
   248	
   249		/* 0x2 bit for HSRR is only used by PR and P7/8 HV paths, clear it */
   250		trap = local_paca->kvm_hstate.scratch0 & ~0x2;
   251		if (likely(trap > BOOK3S_INTERRUPT_MACHINE_CHECK)) {
   252			exsave = local_paca->exgen;
   253		} else if (trap == BOOK3S_INTERRUPT_SYSTEM_RESET) {
   254			exsave = local_paca->exnmi;
   255		} else { /* trap == 0x200 */
   256			exsave = local_paca->exmc;
   257		}
   258	
   259		vcpu->arch.regs.gpr[1] = local_paca->kvm_hstate.scratch1;
   260		vcpu->arch.regs.gpr[3] = local_paca->kvm_hstate.scratch2;
   261		vcpu->arch.regs.gpr[9] = exsave[EX_R9/sizeof(u64)];
   262		vcpu->arch.regs.gpr[10] = exsave[EX_R10/sizeof(u64)];
   263		vcpu->arch.regs.gpr[11] = exsave[EX_R11/sizeof(u64)];
   264		vcpu->arch.regs.gpr[12] = exsave[EX_R12/sizeof(u64)];
   265		vcpu->arch.regs.gpr[13] = exsave[EX_R13/sizeof(u64)];
   266		vcpu->arch.ppr = exsave[EX_PPR/sizeof(u64)];
   267		vcpu->arch.cfar = exsave[EX_CFAR/sizeof(u64)];
   268		vcpu->arch.regs.ctr = exsave[EX_CTR/sizeof(u64)];
   269	
   270		vcpu->arch.last_inst = KVM_INST_FETCH_FAILED;
   271	
   272		if (unlikely(trap == BOOK3S_INTERRUPT_MACHINE_CHECK)) {
   273			vcpu->arch.fault_dar = exsave[EX_DAR/sizeof(u64)];
   274			vcpu->arch.fault_dsisr = exsave[EX_DSISR/sizeof(u64)];
   275			kvmppc_realmode_machine_check(vcpu);
   276	
   277		} else if (unlikely(trap == BOOK3S_INTERRUPT_HMI)) {
   278			kvmppc_realmode_hmi_handler();
   279	
   280		} else if (trap == BOOK3S_INTERRUPT_H_EMUL_ASSIST) {
   281			vcpu->arch.emul_inst = mfspr(SPRN_HEIR);
   282	
   283		} else if (trap == BOOK3S_INTERRUPT_H_DATA_STORAGE) {
   284			vcpu->arch.fault_dar = exsave[EX_DAR/sizeof(u64)];
   285			vcpu->arch.fault_dsisr = exsave[EX_DSISR/sizeof(u64)];
   286			vcpu->arch.fault_gpa = mfspr(SPRN_ASDR);
   287	
   288		} else if (trap == BOOK3S_INTERRUPT_H_INST_STORAGE) {
   289			vcpu->arch.fault_gpa = mfspr(SPRN_ASDR);
   290	
   291		} else if (trap == BOOK3S_INTERRUPT_H_FAC_UNAVAIL) {
   292			vcpu->arch.hfscr = mfspr(SPRN_HFSCR);
   293	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org

[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 27150 bytes --]

  reply	other threads:[~2021-04-02 20:58 UTC|newest]

Thread overview: 117+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-04-01 15:02 [PATCH v5 00/48] KVM: PPC: Book3S: C-ify the P9 entry/exit code Nicholas Piggin
2021-04-01 15:02 ` Nicholas Piggin
2021-04-01 15:02 ` [PATCH v5 01/48] KVM: PPC: Book3S HV: Nested move LPCR sanitising to sanitise_hv_regs Nicholas Piggin
2021-04-01 15:02   ` Nicholas Piggin
2021-04-01 15:02 ` [PATCH v5 02/48] KVM: PPC: Book3S HV: Add a function to filter guest LPCR bits Nicholas Piggin
2021-04-01 15:02   ` Nicholas Piggin
2021-04-01 15:02 ` [PATCH v5 03/48] KVM: PPC: Book3S HV: Disallow LPCR[AIL] to be set to 1 or 2 Nicholas Piggin
2021-04-01 15:02   ` Nicholas Piggin
2021-04-01 15:02 ` [PATCH v5 04/48] KVM: PPC: Book3S HV: Prevent radix guests setting LPCR[TC] Nicholas Piggin
2021-04-01 15:02   ` Nicholas Piggin
2021-04-01 15:02 ` [PATCH v5 05/48] KVM: PPC: Book3S HV: Remove redundant mtspr PSPB Nicholas Piggin
2021-04-01 15:02   ` Nicholas Piggin
2021-04-01 15:02 ` [PATCH v5 06/48] KVM: PPC: Book3S HV: remove unused kvmppc_h_protect argument Nicholas Piggin
2021-04-01 15:02   ` Nicholas Piggin
2021-04-01 15:02 ` [PATCH v5 07/48] KVM: PPC: Book3S HV: Fix CONFIG_SPAPR_TCE_IOMMU=n default hcalls Nicholas Piggin
2021-04-01 15:02   ` Nicholas Piggin
2021-04-01 15:02 ` [PATCH v5 08/48] powerpc/64s: Remove KVM handler support from CBE_RAS interrupts Nicholas Piggin
2021-04-01 15:02   ` Nicholas Piggin
2021-04-01 15:02 ` [PATCH v5 09/48] powerpc/64s: remove KVM SKIP test from instruction breakpoint handler Nicholas Piggin
2021-04-01 15:02   ` Nicholas Piggin
2021-04-01 15:02 ` [PATCH v5 10/48] KVM: PPC: Book3S HV: Ensure MSR[ME] is always set in guest MSR Nicholas Piggin
2021-04-01 15:02   ` Nicholas Piggin
2021-04-01 15:02 ` [PATCH v5 11/48] KVM: PPC: Book3S HV: Ensure MSR[HV] is always clear " Nicholas Piggin
2021-04-01 15:02   ` Nicholas Piggin
2021-04-01 15:02 ` [PATCH v5 12/48] KVM: PPC: Book3S 64: move KVM interrupt entry to a common entry point Nicholas Piggin
2021-04-01 15:02   ` Nicholas Piggin
2021-04-01 15:02 ` [PATCH v5 13/48] KVM: PPC: Book3S 64: Move GUEST_MODE_SKIP test into KVM Nicholas Piggin
2021-04-01 15:02   ` Nicholas Piggin
2021-04-01 15:02 ` [PATCH v5 14/48] KVM: PPC: Book3S 64: add hcall interrupt handler Nicholas Piggin
2021-04-01 15:02   ` Nicholas Piggin
2021-04-01 15:02 ` [PATCH v5 15/48] KVM: PPC: Book3S 64: Move hcall early register setup to KVM Nicholas Piggin
2021-04-01 15:02   ` Nicholas Piggin
2021-04-01 15:02 ` [PATCH v5 16/48] KVM: PPC: Book3S 64: Move interrupt " Nicholas Piggin
2021-04-01 15:02   ` Nicholas Piggin
2021-04-01 15:02 ` [PATCH v5 17/48] KVM: PPC: Book3S 64: move bad_host_intr check to HV handler Nicholas Piggin
2021-04-01 15:02   ` Nicholas Piggin
2021-04-01 15:02 ` [PATCH v5 18/48] KVM: PPC: Book3S 64: Minimise hcall handler calling convention differences Nicholas Piggin
2021-04-01 15:02   ` Nicholas Piggin
2021-04-01 15:02 ` [PATCH v5 19/48] KVM: PPC: Book3S HV P9: Move radix MMU switching instructions together Nicholas Piggin
2021-04-01 15:02   ` Nicholas Piggin
2021-04-01 15:02 ` [PATCH v5 20/48] KVM: PPC: Book3S HV P9: implement kvmppc_xive_pull_vcpu in C Nicholas Piggin
2021-04-01 15:02   ` Nicholas Piggin
2021-04-01 15:02 ` [PATCH v5 21/48] KVM: PPC: Book3S HV P9: Move xive vcpu context management into kvmhv_p9_guest_entry Nicholas Piggin
2021-04-01 15:02   ` Nicholas Piggin
2021-04-01 15:02 ` [PATCH v5 22/48] KVM: PPC: Book3S HV P9: Stop handling hcalls in real-mode in the P9 path Nicholas Piggin
2021-04-01 15:02   ` Nicholas Piggin
2021-04-01 15:03 ` [PATCH v5 23/48] KVM: PPC: Book3S HV P9: Move setting HDEC after switching to guest LPCR Nicholas Piggin
2021-04-01 15:03   ` Nicholas Piggin
2021-04-01 15:03 ` [PATCH v5 24/48] KVM: PPC: Book3S HV P9: Use large decrementer for HDEC Nicholas Piggin
2021-04-01 15:03   ` Nicholas Piggin
2021-04-01 15:03 ` [PATCH v5 25/48] KVM: PPC: Book3S HV P9: Use host timer accounting to avoid decrementer read Nicholas Piggin
2021-04-01 15:03   ` Nicholas Piggin
2021-04-02  4:19   ` kernel test robot
2021-04-02  4:19     ` kernel test robot
2021-04-02  4:19     ` kernel test robot
2021-04-01 15:03 ` [PATCH v5 26/48] KVM: PPC: Book3S HV P9: Reduce mftb per guest entry/exit Nicholas Piggin
2021-04-01 15:03   ` Nicholas Piggin
2021-04-01 15:03 ` [PATCH v5 27/48] KVM: PPC: Book3S HV P9: Reduce irq_work vs guest decrementer races Nicholas Piggin
2021-04-01 15:03   ` Nicholas Piggin
2021-04-01 15:03 ` [PATCH v5 28/48] KMV: PPC: Book3S HV: Use set_dec to set decrementer to host Nicholas Piggin
2021-04-01 15:03   ` Nicholas Piggin
2021-04-01 15:03 ` [PATCH v5 29/48] powerpc: add set_dec_or_work API for safely updating decrementer Nicholas Piggin
2021-04-01 15:03   ` Nicholas Piggin
2021-04-02 11:04   ` Nicholas Piggin
2021-04-02 11:04     ` Nicholas Piggin
2021-04-01 15:03 ` [PATCH v5 30/48] KVM: PPC: Book3S HV P9: Implement the rest of the P9 path in C Nicholas Piggin
2021-04-01 15:03   ` Nicholas Piggin
2021-04-01 15:03 ` [PATCH v5 31/48] KVM: PPC: Book3S HV P9: inline kvmhv_load_hv_regs_and_go into __kvmhv_vcpu_entry_p9 Nicholas Piggin
2021-04-01 15:03   ` Nicholas Piggin
2021-04-02 20:58   ` kernel test robot [this message]
2021-04-02 20:58     ` kernel test robot
2021-04-02 20:58     ` kernel test robot
2021-04-01 15:03 ` [PATCH v5 32/48] KVM: PPC: Book3S HV P9: Read machine check registers while MSR[RI] is 0 Nicholas Piggin
2021-04-01 15:03   ` Nicholas Piggin
2021-04-01 15:03 ` [PATCH v5 33/48] KVM: PPC: Book3S HV P9: Improve exit timing accounting coverage Nicholas Piggin
2021-04-01 15:03   ` Nicholas Piggin
2021-04-01 15:03 ` [PATCH v5 34/48] KVM: PPC: Book3S HV P9: Move SPR loading after expiry time check Nicholas Piggin
2021-04-01 15:03   ` Nicholas Piggin
2021-04-01 15:03 ` [PATCH v5 35/48] KVM: PPC: Book3S HV P9: Add helpers for OS SPR handling Nicholas Piggin
2021-04-01 15:03   ` Nicholas Piggin
2021-04-01 15:03 ` [PATCH v5 36/48] KVM: PPC: Book3S HV P9: Switch to guest MMU context as late as possible Nicholas Piggin
2021-04-01 15:03   ` Nicholas Piggin
2021-04-01 15:03 ` [PATCH v5 37/48] KVM: PPC: Book3S HV: Implement radix prefetch workaround by disabling MMU Nicholas Piggin
2021-04-01 15:03   ` Nicholas Piggin
2021-04-01 15:03 ` [PATCH v5 38/48] KVM: PPC: Book3S HV: Remove support for dependent threads mode on P9 Nicholas Piggin
2021-04-01 15:03   ` Nicholas Piggin
2021-04-01 15:03 ` [PATCH v5 39/48] KVM: PPC: Book3S HV: Remove radix guest support from P7/8 path Nicholas Piggin
2021-04-01 15:03   ` Nicholas Piggin
2021-04-01 15:03 ` [PATCH v5 40/48] KVM: PPC: Book3S HV: Remove virt mode checks from real mode handlers Nicholas Piggin
2021-04-01 15:03   ` Nicholas Piggin
2021-04-01 15:03 ` [PATCH v5 41/48] KVM: PPC: Book3S HV: Remove unused nested HV tests in XICS emulation Nicholas Piggin
2021-04-01 15:03   ` Nicholas Piggin
2021-04-02 15:28   ` kernel test robot
2021-04-02 15:28     ` kernel test robot
2021-04-02 15:28     ` kernel test robot
2021-04-02 16:32   ` Cédric Le Goater
2021-04-02 16:32     ` Cédric Le Goater
2021-04-04  0:48     ` Nicholas Piggin
2021-04-04  0:48       ` Nicholas Piggin
2021-04-01 15:03 ` [PATCH v5 42/48] KVM: PPC: Book3S HV: Radix guests should not have userspace hcalls reflected to the Nicholas Piggin
2021-04-01 15:03   ` [PATCH v5 42/48] KVM: PPC: Book3S HV: Radix guests should not have userspace hcalls reflected to them Nicholas Piggin
2021-04-02  0:07   ` [PATCH v5 42/48] KVM: PPC: Book3S HV: Radix guests should not have userspace hcalls reflected to Nicholas Piggin
2021-04-02  0:07     ` [PATCH v5 42/48] KVM: PPC: Book3S HV: Radix guests should not have userspace hcalls reflected to them Nicholas Piggin
2021-04-01 15:03 ` [PATCH v5 43/48] KVM: PPC: Book3S HV P9: Allow all P9 processors to enable nested HV Nicholas Piggin
2021-04-01 15:03   ` Nicholas Piggin
2021-04-01 15:03 ` [PATCH v5 44/48] KVM: PPC: Book3S HV: small pseries_do_hcall cleanup Nicholas Piggin
2021-04-01 15:03   ` Nicholas Piggin
2021-04-01 15:03 ` [PATCH v5 45/48] KVM: PPC: Book3S HV: add virtual mode handlers for HPT hcalls and page faults Nicholas Piggin
2021-04-01 15:03   ` Nicholas Piggin
2021-04-01 15:03 ` [PATCH v5 46/48] KVM: PPC: Book3S HV P9: implement hash guest support Nicholas Piggin
2021-04-01 15:03   ` Nicholas Piggin
2021-04-01 15:03 ` [PATCH v5 47/48] KVM: PPC: Book3S HV P9: implement hash host / " Nicholas Piggin
2021-04-01 15:03   ` Nicholas Piggin
2021-04-01 15:03 ` [PATCH v5 48/48] KVM: PPC: Book3S HV: remove ISA v3.0 and v3.1 support from P7/8 path Nicholas Piggin
2021-04-01 15:03   ` Nicholas Piggin
2021-04-02  0:11   ` Nicholas Piggin
2021-04-02  0:11     ` Nicholas Piggin

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=202104030412.h8fIESGy-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=farosas@linux.ibm.com \
    --cc=kbuild-all@lists.01.org \
    --cc=kvm-ppc@vger.kernel.org \
    --cc=linuxppc-dev@lists.ozlabs.org \
    --cc=npiggin@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.