Re: [PATCH RESEND 1/3] printk: convert byte-buffer tovariable-length record buffer

From: Greg Kroah-Hartmann
Date: Thu May 03 2012 - 16:18:42 EST


On Thu, May 03, 2012 at 10:11:02PM +0200, Peter Zijlstra wrote:
> On Thu, 2012-05-03 at 13:09 -0700, Linus Torvalds wrote:
> > On Thu, May 3, 2012 at 1:02 PM, Peter Zijlstra <peterz@xxxxxxxxxxxxx> wrote:
> > >
> > > Thing is, with bonkers stuff like usb-console and kms/drm that's a _lot_
> > > of code running under the logbuf/console locks.
> >
> > The top-level console lock shouldn't be a problem - we use trylock and
> > delay if it is held.
> >
> > It's the lower-level driver-specific locks that screw us up. And quite
> > frankly, I am *not* willing to say that that is a printk() problem.
> > That is purely a "USB serial console is damn well broken" issue, and
> > should not be considered a limitation of printk.
>
> I'm happy to call all that broken ;-)

USB serial console was a drunken bet gone wrong. I'm amazed that it
even works at times, and it should never be considered critical enough
to affect any core kernel code.

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/