Messages in this thread | | | From | Thiago Jung Bauermann <> | Subject | Re: [PATCH] powerpc/pseries/svm: Remove unwanted check for shared_lppaca_size | Date | Wed, 10 Jun 2020 01:15:21 -0300 |
| |
Satheesh Rajendran <sathnaga@linux.vnet.ibm.com> writes:
> Early secure guest boot hits the below crash while booting with > vcpus numbers aligned with page boundary for PAGE size of 64k > and LPPACA size of 1k i.e 64, 128 etc, due to the BUG_ON assert > for shared_lppaca_total_size equal to shared_lppaca_size, > > [ 0.000000] Partition configured for 64 cpus. > [ 0.000000] CPU maps initialized for 1 thread per core > [ 0.000000] ------------[ cut here ]------------ > [ 0.000000] kernel BUG at arch/powerpc/kernel/paca.c:89! > [ 0.000000] Oops: Exception in kernel mode, sig: 5 [#1] > [ 0.000000] LE PAGE_SIZE=64K MMU=Radix SMP NR_CPUS=2048 NUMA pSeries > > which is not necessary, let's remove it. > > Cc: linux-kernel@vger.kernel.org > Cc: Thiago Jung Bauermann <bauerman@linux.ibm.com> > Cc: Ram Pai <linuxram@us.ibm.com> > Cc: Sukadev Bhattiprolu <sukadev@linux.vnet.ibm.com> > Cc: Laurent Dufour <ldufour@linux.ibm.com> > Signed-off-by: Satheesh Rajendran <sathnaga@linux.vnet.ibm.com> > --- > arch/powerpc/kernel/paca.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-)
Thanks for fixing this bug! I would only add:
Fixes: bd104e6db6f0 ("powerpc/pseries/svm: Use shared memory for LPPACA structures")
In any case:
Reviewed-by: Thiago Jung Bauermann <bauerman@linux.ibm.com>
-- Thiago Jung Bauermann IBM Linux Technology Center
| |