* David Miller | 2014-03-04 14:10:03 [-0500]: >> This should be fixed like that's done for 8250 >> [patch drivers-serial-cleanup-locking-for-rt.patch] > >Like the patch below? The patch looks good. >Is this transformation legitimate outside of the -rt tree? I wish the commit >messages for the 8250 change was more verbose, explaining exactly why this >is needed. Looking at it, it looks like the transformation is legitimate in vanila, too. In -RT the spin_lock_irqsave() does not spin but sleep if the lock is taken. Before that, local_irq_save() is invoked which disables interrupts even on -RT. That is why you might_sleep() triggers a warning here. In the ->sysrq and oops_in_progress case it is save to trylock the lock i.e. this is what we do now anyway except for ->sysrq where we assume that the lock is already taken. The spin_lock_irqsave() grabs the lock and disables the interrupts on vanila (the same behavior) and on -RT it won't disable interrupts. >And if it's needed, all the sparc serial/console drivers need this too. Let me forward the 8250 to GregKH and see how he reacts. Would be nice if it is another patch that gets removed from that queue… Sebastian -- To unsubscribe from this list: send the line "unsubscribe sparclinux" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
