* [PATCH AUTOSEL 5.15 081/188] s390/nmi: add missing __pa/__va address conversion of extended save area [not found] <20220118023152.1948105-1-sashal@kernel.org> @ 2022-01-18 2:30 ` Sasha Levin 2022-01-18 8:18 ` Christian Borntraeger 0 siblings, 1 reply; 3+ messages in thread From: Sasha Levin @ 2022-01-18 2:30 UTC (permalink / raw) To: linux-kernel, stable Cc: Heiko Carstens, Vineeth Vijayan, Sasha Levin, gor, borntraeger, egorenar, agordeev, svens, ebiederm, valentin.schneider, rppt, iii, linux-s390 From: Heiko Carstens <hca@linux.ibm.com> [ Upstream commit 402ff5a3387dc8ec6987a80d3ce26b0c25773622 ] Add missing __pa/__va address conversion of machine check extended save area designation, which is an absolute address. Note: this currently doesn't fix a real bug, since virtual addresses are indentical to physical ones. Reported-by: Vineeth Vijayan <vneethv@linux.ibm.com> Tested-by: Vineeth Vijayan <vneethv@linux.ibm.com> Signed-off-by: Heiko Carstens <hca@linux.ibm.com> Signed-off-by: Sasha Levin <sashal@kernel.org> --- arch/s390/kernel/machine_kexec.c | 2 +- arch/s390/kernel/nmi.c | 10 +++++----- arch/s390/kernel/smp.c | 2 +- 3 files changed, 7 insertions(+), 7 deletions(-) diff --git a/arch/s390/kernel/machine_kexec.c b/arch/s390/kernel/machine_kexec.c index 0505e55a62979..a16467b3825ec 100644 --- a/arch/s390/kernel/machine_kexec.c +++ b/arch/s390/kernel/machine_kexec.c @@ -86,7 +86,7 @@ static noinline void __machine_kdump(void *image) continue; } /* Store status of the boot CPU */ - mcesa = (struct mcesa *)(S390_lowcore.mcesad & MCESA_ORIGIN_MASK); + mcesa = __va(S390_lowcore.mcesad & MCESA_ORIGIN_MASK); if (MACHINE_HAS_VX) save_vx_regs((__vector128 *) mcesa->vector_save_area); if (MACHINE_HAS_GS) { diff --git a/arch/s390/kernel/nmi.c b/arch/s390/kernel/nmi.c index 20f8e1868853f..3f18c1412eba3 100644 --- a/arch/s390/kernel/nmi.c +++ b/arch/s390/kernel/nmi.c @@ -68,7 +68,7 @@ void __init nmi_alloc_boot_cpu(struct lowcore *lc) { if (!nmi_needs_mcesa()) return; - lc->mcesad = (unsigned long) &boot_mcesa; + lc->mcesad = __pa(&boot_mcesa); if (MACHINE_HAS_GS) lc->mcesad |= ilog2(MCESA_MAX_SIZE); } @@ -94,7 +94,7 @@ static int __init nmi_init(void) __ctl_store(cr0, 0, 0); __ctl_clear_bit(0, 28); /* disable lowcore protection */ /* Replace boot_mcesa on the boot CPU */ - S390_lowcore.mcesad = origin | mcesa_origin_lc; + S390_lowcore.mcesad = __pa(origin) | mcesa_origin_lc; __ctl_load(cr0, 0, 0); return 0; } @@ -111,7 +111,7 @@ int nmi_alloc_per_cpu(struct lowcore *lc) return -ENOMEM; /* The pointer is stored with mcesa_bits ORed in */ kmemleak_not_leak((void *) origin); - lc->mcesad = origin | mcesa_origin_lc; + lc->mcesad = __pa(origin) | mcesa_origin_lc; return 0; } @@ -119,7 +119,7 @@ void nmi_free_per_cpu(struct lowcore *lc) { if (!nmi_needs_mcesa()) return; - kmem_cache_free(mcesa_cache, (void *)(lc->mcesad & MCESA_ORIGIN_MASK)); + kmem_cache_free(mcesa_cache, __va(lc->mcesad & MCESA_ORIGIN_MASK)); } static notrace void s390_handle_damage(void) @@ -246,7 +246,7 @@ static int notrace s390_validate_registers(union mci mci, int umode) : "Q" (S390_lowcore.fpt_creg_save_area)); } - mcesa = (struct mcesa *)(S390_lowcore.mcesad & MCESA_ORIGIN_MASK); + mcesa = __va(S390_lowcore.mcesad & MCESA_ORIGIN_MASK); if (!MACHINE_HAS_VX) { /* Validate floating point registers */ asm volatile( diff --git a/arch/s390/kernel/smp.c b/arch/s390/kernel/smp.c index 1a04e5bdf6555..5c3d3d8f6b5d8 100644 --- a/arch/s390/kernel/smp.c +++ b/arch/s390/kernel/smp.c @@ -622,7 +622,7 @@ int smp_store_status(int cpu) return -EIO; if (!MACHINE_HAS_VX && !MACHINE_HAS_GS) return 0; - pa = __pa(lc->mcesad & MCESA_ORIGIN_MASK); + pa = lc->mcesad & MCESA_ORIGIN_MASK; if (MACHINE_HAS_GS) pa |= lc->mcesad & MCESA_LC_MASK; if (__pcpu_sigp_relax(pcpu->address, SIGP_STORE_ADDITIONAL_STATUS, -- 2.34.1 ^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH AUTOSEL 5.15 081/188] s390/nmi: add missing __pa/__va address conversion of extended save area 2022-01-18 2:30 ` [PATCH AUTOSEL 5.15 081/188] s390/nmi: add missing __pa/__va address conversion of extended save area Sasha Levin @ 2022-01-18 8:18 ` Christian Borntraeger 2022-01-22 19:18 ` Sasha Levin 0 siblings, 1 reply; 3+ messages in thread From: Christian Borntraeger @ 2022-01-18 8:18 UTC (permalink / raw) To: Sasha Levin, linux-kernel, stable Cc: Heiko Carstens, Vineeth Vijayan, gor, egorenar, agordeev, svens, ebiederm, valentin.schneider, rppt, iii, linux-s390 Am 18.01.22 um 03:30 schrieb Sasha Levin: > From: Heiko Carstens <hca@linux.ibm.com> > > [ Upstream commit 402ff5a3387dc8ec6987a80d3ce26b0c25773622 ] > > Add missing __pa/__va address conversion of machine check extended > save area designation, which is an absolute address. > vv > Note: this currently doesn't fix a real bug, since virtual addresses > are indentical to physical ones. ^^ Sasha, please note the disclaimer above. There will be plenty of such fixes in s390 code and there is no point in backporting single fixes to stable. It will provide no benefit on its own but adds a risk of regression. > > Reported-by: Vineeth Vijayan <vneethv@linux.ibm.com> > Tested-by: Vineeth Vijayan <vneethv@linux.ibm.com> > Signed-off-by: Heiko Carstens <hca@linux.ibm.com> > Signed-off-by: Sasha Levin <sashal@kernel.org> > --- > arch/s390/kernel/machine_kexec.c | 2 +- > arch/s390/kernel/nmi.c | 10 +++++----- > arch/s390/kernel/smp.c | 2 +- > 3 files changed, 7 insertions(+), 7 deletions(-) > > diff --git a/arch/s390/kernel/machine_kexec.c b/arch/s390/kernel/machine_kexec.c > index 0505e55a62979..a16467b3825ec 100644 > --- a/arch/s390/kernel/machine_kexec.c > +++ b/arch/s390/kernel/machine_kexec.c > @@ -86,7 +86,7 @@ static noinline void __machine_kdump(void *image) > continue; > } > /* Store status of the boot CPU */ > - mcesa = (struct mcesa *)(S390_lowcore.mcesad & MCESA_ORIGIN_MASK); > + mcesa = __va(S390_lowcore.mcesad & MCESA_ORIGIN_MASK); > if (MACHINE_HAS_VX) > save_vx_regs((__vector128 *) mcesa->vector_save_area); > if (MACHINE_HAS_GS) { > diff --git a/arch/s390/kernel/nmi.c b/arch/s390/kernel/nmi.c > index 20f8e1868853f..3f18c1412eba3 100644 > --- a/arch/s390/kernel/nmi.c > +++ b/arch/s390/kernel/nmi.c > @@ -68,7 +68,7 @@ void __init nmi_alloc_boot_cpu(struct lowcore *lc) > { > if (!nmi_needs_mcesa()) > return; > - lc->mcesad = (unsigned long) &boot_mcesa; > + lc->mcesad = __pa(&boot_mcesa); > if (MACHINE_HAS_GS) > lc->mcesad |= ilog2(MCESA_MAX_SIZE); > } > @@ -94,7 +94,7 @@ static int __init nmi_init(void) > __ctl_store(cr0, 0, 0); > __ctl_clear_bit(0, 28); /* disable lowcore protection */ > /* Replace boot_mcesa on the boot CPU */ > - S390_lowcore.mcesad = origin | mcesa_origin_lc; > + S390_lowcore.mcesad = __pa(origin) | mcesa_origin_lc; > __ctl_load(cr0, 0, 0); > return 0; > } > @@ -111,7 +111,7 @@ int nmi_alloc_per_cpu(struct lowcore *lc) > return -ENOMEM; > /* The pointer is stored with mcesa_bits ORed in */ > kmemleak_not_leak((void *) origin); > - lc->mcesad = origin | mcesa_origin_lc; > + lc->mcesad = __pa(origin) | mcesa_origin_lc; > return 0; > } > > @@ -119,7 +119,7 @@ void nmi_free_per_cpu(struct lowcore *lc) > { > if (!nmi_needs_mcesa()) > return; > - kmem_cache_free(mcesa_cache, (void *)(lc->mcesad & MCESA_ORIGIN_MASK)); > + kmem_cache_free(mcesa_cache, __va(lc->mcesad & MCESA_ORIGIN_MASK)); > } > > static notrace void s390_handle_damage(void) > @@ -246,7 +246,7 @@ static int notrace s390_validate_registers(union mci mci, int umode) > : "Q" (S390_lowcore.fpt_creg_save_area)); > } > > - mcesa = (struct mcesa *)(S390_lowcore.mcesad & MCESA_ORIGIN_MASK); > + mcesa = __va(S390_lowcore.mcesad & MCESA_ORIGIN_MASK); > if (!MACHINE_HAS_VX) { > /* Validate floating point registers */ > asm volatile( > diff --git a/arch/s390/kernel/smp.c b/arch/s390/kernel/smp.c > index 1a04e5bdf6555..5c3d3d8f6b5d8 100644 > --- a/arch/s390/kernel/smp.c > +++ b/arch/s390/kernel/smp.c > @@ -622,7 +622,7 @@ int smp_store_status(int cpu) > return -EIO; > if (!MACHINE_HAS_VX && !MACHINE_HAS_GS) > return 0; > - pa = __pa(lc->mcesad & MCESA_ORIGIN_MASK); > + pa = lc->mcesad & MCESA_ORIGIN_MASK; > if (MACHINE_HAS_GS) > pa |= lc->mcesad & MCESA_LC_MASK; > if (__pcpu_sigp_relax(pcpu->address, SIGP_STORE_ADDITIONAL_STATUS, ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH AUTOSEL 5.15 081/188] s390/nmi: add missing __pa/__va address conversion of extended save area 2022-01-18 8:18 ` Christian Borntraeger @ 2022-01-22 19:18 ` Sasha Levin 0 siblings, 0 replies; 3+ messages in thread From: Sasha Levin @ 2022-01-22 19:18 UTC (permalink / raw) To: Christian Borntraeger Cc: linux-kernel, stable, Heiko Carstens, Vineeth Vijayan, gor, egorenar, agordeev, svens, ebiederm, valentin.schneider, rppt, iii, linux-s390 On Tue, Jan 18, 2022 at 09:18:40AM +0100, Christian Borntraeger wrote: >Am 18.01.22 um 03:30 schrieb Sasha Levin: >>From: Heiko Carstens <hca@linux.ibm.com> >> >>[ Upstream commit 402ff5a3387dc8ec6987a80d3ce26b0c25773622 ] >> >>Add missing __pa/__va address conversion of machine check extended >>save area designation, which is an absolute address. >> > >vv >>Note: this currently doesn't fix a real bug, since virtual addresses >>are indentical to physical ones. >^^ > >Sasha, >please note the disclaimer above. There will be plenty of such fixes >in s390 code and there is no point in backporting single fixes to stable. >It will provide no benefit on its own but adds a risk of regression. Sure, I'll drop it. Thanks! -- Thanks, Sasha ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2022-01-22 19:18 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20220118023152.1948105-1-sashal@kernel.org>
2022-01-18 2:30 ` [PATCH AUTOSEL 5.15 081/188] s390/nmi: add missing __pa/__va address conversion of extended save area Sasha Levin
2022-01-18 8:18 ` Christian Borntraeger
2022-01-22 19:18 ` Sasha Levin
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox