Re: [PATCH] libata: add human-readable error value decoding

Robert Hancock wrote:

I don't think this is as big of a deal here as in other cases, like oops output. With libata errors, if they're at the console (which they'd have to be to see these messages), unless something has actually caused a panic the scrollback buffer should still be functional and they'd be able to see the entire output..

Scrollback rarely works as planned, for me. Overall, a balance must be found.

More information is more helpful.  But.

There are downsides to spewing everything possible, upon error. You cause logging to the possibly problematic disk, you push older messages out of the printk ring buffer, etc., etc.

	Jeff


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

---

• Follow-Ups:
  • Re: [PATCH] libata: add human-readable error value decoding
    • From: Alan Cox <[email protected]>

• References:
  • [PATCH] libata: add human-readable error value decoding
    • From: Robert Hancock <[email protected]>
  • Re: [PATCH] libata: add human-readable error value decoding
    • From: Tejun Heo <[email protected]>
  • Re: [PATCH] libata: add human-readable error value decoding
    • From: Mark Lord <[email protected]>
  • Re: [PATCH] libata: add human-readable error value decoding
    • From: Jeff Garzik <[email protected]>
  • Re: [PATCH] libata: add human-readable error value decoding
    • From: Mark Lord <[email protected]>
  • Re: [PATCH] libata: add human-readable error value decoding
    • From: Jeff Garzik <[email protected]>
  • Re: [PATCH] libata: add human-readable error value decoding
    • From: Robert Hancock <[email protected]>

• Prev by Date: Re: 2.6.21-mm2 -- powerpc missing kset
• Next by Date: Re: [PATCH] use defines in sys_getpriority/sys_setpriority
• Previous by thread: Re: [PATCH] libata: add human-readable error value decoding
• Next by thread: Re: [PATCH] libata: add human-readable error value decoding
• Index(es):
  • Date
  • Thread

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]