* Re: [PATCH] sparc64: Fix irqtrace warnings on Ultra-S
@ 2020-09-08 16:12 Guenter Roeck
0 siblings, 0 replies; 2+ messages in thread
From: Guenter Roeck @ 2020-09-08 16:12 UTC (permalink / raw)
To: peterz
Cc: linux-kernel, mingo, will, npiggin, elver, jgross, paulmck,
rostedt, rjw, joel, svens, tglx, davem
On Tue, Sep 08, 2020 at 05:41:57PM +0200, peterz@infradead.org wrote:
> On Tue, Sep 08, 2020 at 07:40:23AM -0700, Guenter Roeck wrote:
> > qemu-system-sparc64 -M sun4u -cpu "TI UltraSparc IIi" -m 512 \
> > -initrd rootfs.cpio \
> > -kernel arch/sparc/boot/image -no-reboot \
> > -append "panic=-1 slub_debug=FZPUA rdinit=/sbin/init console=ttyS0" \
> > -nographic -monitor none
>
> Thanks I got it. Also enabling DEBUG_LOCKDEP helps (-:
>
> ---
> Subject: sparc64: Fix irqtrace warnings on Ultra-S
>
> Recent changes in Lockdep's IRQTRACE broke Ultra-S.
>
> In order avoid redundant IRQ state changes, local_irq_restore() lost the
> ability to trace a disable. Change the code to use local_irq_save() to
> disable IRQs and then use arch_local_irq_restore() to further disable
> NMIs.
>
> This result in slightly suboptimal code, but given this code uses a
> global spinlock, performance cannot be its primary purpose.
>
> Fixes: 044d0d6de9f5 ("lockdep: Only trace IRQ edges")
> Reported-by: Guenter Roeck <linux@roeck-us.net>
> Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Tested-by: Guenter Roeck <linux@roeck-us.net>
> ---
> A possible alternative would be:
>
> local_save_flags(flags);
> arch_local_irq_restore((unsigned long)PIL_NMI);
> if (IS_ENABLED(CONFIG_TRACE_IRQFLAGS))
> trace_hardirqs_off();
>
> which generates optimal code, but is more verbose.
>
> arch/sparc/prom/p1275.c | 7 +++----
> 1 file changed, 3 insertions(+), 4 deletions(-)
>
> diff --git a/arch/sparc/prom/p1275.c b/arch/sparc/prom/p1275.c
> index 889aa602f8d8..e22233fcf741 100644
> --- a/arch/sparc/prom/p1275.c
> +++ b/arch/sparc/prom/p1275.c
> @@ -37,16 +37,15 @@ void p1275_cmd_direct(unsigned long *args)
> {
> unsigned long flags;
>
> - local_save_flags(flags);
> - local_irq_restore((unsigned long)PIL_NMI);
> + local_irq_save(flags);
> + arch_local_irq_restore((unsigned long)PIL_NMI);
> raw_spin_lock(&prom_entry_lock);
>
> prom_world(1);
> prom_cif_direct(args);
> prom_world(0);
>
> - raw_spin_unlock(&prom_entry_lock);
> - local_irq_restore(flags);
> + raw_spin_unlock_irqrestore(&prom_entry_lock, flags);
> }
>
> void prom_cif_init(void *cif_handler, void *cif_stack)
^ permalink raw reply [flat|nested] 2+ messages in thread
* [PATCH] sparc64: Fix irqtrace warnings on Ultra-S
2020-09-08 14:40 ` Guenter Roeck
@ 2020-09-08 15:41 ` peterz
0 siblings, 0 replies; 2+ messages in thread
From: peterz @ 2020-09-08 15:41 UTC (permalink / raw)
To: Guenter Roeck
Cc: linux-kernel, mingo, will, npiggin, elver, jgross, paulmck,
rostedt, rjw, joel, svens, tglx, davem
On Tue, Sep 08, 2020 at 07:40:23AM -0700, Guenter Roeck wrote:
> qemu-system-sparc64 -M sun4u -cpu "TI UltraSparc IIi" -m 512 \
> -initrd rootfs.cpio \
> -kernel arch/sparc/boot/image -no-reboot \
> -append "panic=-1 slub_debug=FZPUA rdinit=/sbin/init console=ttyS0" \
> -nographic -monitor none
Thanks I got it. Also enabling DEBUG_LOCKDEP helps (-:
---
Subject: sparc64: Fix irqtrace warnings on Ultra-S
Recent changes in Lockdep's IRQTRACE broke Ultra-S.
In order avoid redundant IRQ state changes, local_irq_restore() lost the
ability to trace a disable. Change the code to use local_irq_save() to
disable IRQs and then use arch_local_irq_restore() to further disable
NMIs.
This result in slightly suboptimal code, but given this code uses a
global spinlock, performance cannot be its primary purpose.
Fixes: 044d0d6de9f5 ("lockdep: Only trace IRQ edges")
Reported-by: Guenter Roeck <linux@roeck-us.net>
Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
---
A possible alternative would be:
local_save_flags(flags);
arch_local_irq_restore((unsigned long)PIL_NMI);
if (IS_ENABLED(CONFIG_TRACE_IRQFLAGS))
trace_hardirqs_off();
which generates optimal code, but is more verbose.
arch/sparc/prom/p1275.c | 7 +++----
1 file changed, 3 insertions(+), 4 deletions(-)
diff --git a/arch/sparc/prom/p1275.c b/arch/sparc/prom/p1275.c
index 889aa602f8d8..e22233fcf741 100644
--- a/arch/sparc/prom/p1275.c
+++ b/arch/sparc/prom/p1275.c
@@ -37,16 +37,15 @@ void p1275_cmd_direct(unsigned long *args)
{
unsigned long flags;
- local_save_flags(flags);
- local_irq_restore((unsigned long)PIL_NMI);
+ local_irq_save(flags);
+ arch_local_irq_restore((unsigned long)PIL_NMI);
raw_spin_lock(&prom_entry_lock);
prom_world(1);
prom_cif_direct(args);
prom_world(0);
- raw_spin_unlock(&prom_entry_lock);
- local_irq_restore(flags);
+ raw_spin_unlock_irqrestore(&prom_entry_lock, flags);
}
void prom_cif_init(void *cif_handler, void *cif_stack)
^ permalink raw reply related [flat|nested] 2+ messages in thread
end of thread, other threads:[~2020-09-08 19:43 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-09-08 16:12 [PATCH] sparc64: Fix irqtrace warnings on Ultra-S Guenter Roeck
-- strict thread matches above, loose matches on Subject: below --
2020-08-21 8:47 [PATCH v2 00/11] TRACE_IRQFLAGS wreckage Peter Zijlstra
2020-08-21 8:47 ` [PATCH v2 10/11] lockdep: Only trace IRQ edges Peter Zijlstra
2020-09-02 4:21 ` Guenter Roeck
2020-09-02 9:09 ` peterz
2020-09-02 9:12 ` peterz
2020-09-02 13:48 ` Guenter Roeck
2020-09-08 14:22 ` peterz
2020-09-08 14:40 ` Guenter Roeck
2020-09-08 15:41 ` [PATCH] sparc64: Fix irqtrace warnings on Ultra-S peterz
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.