Subject: [to-be-updated] printk-re-add-irqsave-restore-in-printk_sched.patch removed from -mm tree
To: john.stultz@xxxxxxxxxx,jack@xxxxxxx,jbohac@xxxxxxx,mingo@xxxxxxxxxx,peterz@xxxxxxxxxxxxx,rostedt@xxxxxxxxxxx,tglx@xxxxxxxxxxxxx,mm-commits@xxxxxxxxxxxxxxx
From: akpm@xxxxxxxxxxxxxxxxxxxx
Date: Mon, 05 May 2014 15:04:58 -0700
The patch titled
Subject: printk: re-add irqsave/restore in printk_sched
has been removed from the -mm tree. Its filename was
printk-re-add-irqsave-restore-in-printk_sched.patch
This patch was dropped because an updated version will be merged
------------------------------------------------------
From: John Stultz <john.stultz@xxxxxxxxxx>
Subject: printk: re-add irqsave/restore in printk_sched
Recently, Jiri pointed out a potential deadlock when calling printk while
holding the timekeeping seqlock.
Annoyingly, the seqlock lockdep enablement doesn't catch this, as printk
disables lockdep.
When looking for possible solutions, one idea was to use a local buffer
and defer the printk to later. Ends up there is already similar
functionality in printk_sched() to avoid similar style deadlocks w/ the
scheduler.
Thus this patchset renames printk_sched to printk_deferred and then moves
the affected timekeeping printks to make use of it.
There were some points in the discussion between Jan and Peter that made
it seem that there may still be problems lurking in the console layer, and
I'm not sure I fully understand their point, so this solution may be
incomplete.
Additionally, the same issue likely affects any WARN_ONs as well, but I
wanted to get some thoughts on this approach before trying to remove or
convert affected WARN_ONS.
This patch (of 4)
A commit in akpm's tree (printk: remove separate printk_sched buffers...),
removed the printk_sched irqsave/restore lines since it was safe for
current users. Since we may be expanding usage of printk_sched(), re-add
the irqsave/restore logic to make the functionality more generally safe.
Signed-off-by: John Stultz <john.stultz@xxxxxxxxxx>
Cc: Jan Kara <jack@xxxxxxx>
Cc: Peter Zijlstra <peterz@xxxxxxxxxxxxx>
Cc: Jiri Bohac <jbohac@xxxxxxx>
Cc: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
Cc: Ingo Molnar <mingo@xxxxxxxxxx>
Cc: Steven Rostedt <rostedt@xxxxxxxxxxx>
Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
---
kernel/printk/printk.c | 3 +++
1 file changed, 3 insertions(+)
diff -puN kernel/printk/printk.c~printk-re-add-irqsave-restore-in-printk_sched kernel/printk/printk.c
--- a/kernel/printk/printk.c~printk-re-add-irqsave-restore-in-printk_sched
+++ a/kernel/printk/printk.c
@@ -2586,15 +2586,18 @@ void wake_up_klogd(void)
int printk_sched(const char *fmt, ...)
{
+ unsigned long flags;
va_list args;
int r;
+ local_irq_save(flags);
va_start(args, fmt);
r = vprintk_emit(0, SCHED_MESSAGE_LOGLEVEL, NULL, 0, fmt, args);
va_end(args);
__this_cpu_or(printk_pending, PRINTK_PENDING_OUTPUT);
irq_work_queue(&__get_cpu_var(wake_up_klogd_work));
+ local_irq_restore(flags);
return r;
}
_
Patches currently in -mm which might be from john.stultz@xxxxxxxxxx are
printk-rename-printk_sched-to-printk_deferred.patch
printk-add-printk_once_deferred.patch
timekeeping-use-printk_deferred-when-holding-timekeeping-seqlock.patch
--
To unsubscribe from this list: send the line "unsubscribe mm-commits" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
