powerpc/nvram: Add spinlock to oops_to_nvram to prevent oops in compression code.
authorAnton Blanchard <anton@samba.org>
Wed, 30 Nov 2011 15:46:45 +0000 (15:46 +0000)
committerBenjamin Herrenschmidt <benh@kernel.crashing.org>
Thu, 8 Dec 2011 03:22:54 +0000 (14:22 +1100)
When issuing a system reset we almost always oops in the oops_to_nvram
code because multiple CPUs are using the deflate work area. Add a
spinlock to protect it.

To play it safe I'm using trylock to avoid locking up if the NVRAM
code oopses. This means we might miss multiple CPUs oopsing at exactly
the same time but I think it's best to play it safe for now. Once we
are happy with the reliability we can change it to a full spinlock.

Signed-off-by: Anton Blanchard <anton@samba.org>
Acked-by: Jim Keniston <jkenisto@us.ibm.com>
Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
arch/powerpc/platforms/pseries/nvram.c

index a76b22844d18b63cc87f33720009ea3e567f28e9..330a57b7c17ca48ee997fa2508c9ec29716efd5d 100644 (file)
@@ -625,6 +625,8 @@ static void oops_to_nvram(struct kmsg_dumper *dumper,
 {
        static unsigned int oops_count = 0;
        static bool panicking = false;
+       static DEFINE_SPINLOCK(lock);
+       unsigned long flags;
        size_t text_len;
        unsigned int err_type = ERR_TYPE_KERNEL_PANIC_GZ;
        int rc = -1;
@@ -655,6 +657,9 @@ static void oops_to_nvram(struct kmsg_dumper *dumper,
        if (clobbering_unread_rtas_event())
                return;
 
+       if (!spin_trylock_irqsave(&lock, flags))
+               return;
+
        if (big_oops_buf) {
                text_len = capture_last_msgs(old_msgs, old_len,
                        new_msgs, new_len, big_oops_buf, big_oops_buf_sz);
@@ -670,4 +675,6 @@ static void oops_to_nvram(struct kmsg_dumper *dumper,
 
        (void) nvram_write_os_partition(&oops_log_partition, oops_buf,
                (int) (sizeof(*oops_len) + *oops_len), err_type, ++oops_count);
+
+       spin_unlock_irqrestore(&lock, flags);
 }