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

Tejun Heo wrote:

Robert Hancock wrote:

This adds human-readable decoding of the ATA status and error registers
(similar
to what drivers/ide does) as well as the SATA Serror register to libata

...

I'm not really convinced whether this is necessary.  The human readable
form is also a bit cryptic and can get quite long.  So, mild NACK from me.

Same here, but I would like to see it in there under a CONFIG_DEBUG_LIBATA
kernel build option or something.  Kind of like the "FANCY_STATUS_DUMPS" flag
that drivers/ide used to have for this kind of stuff.

Cheers

-
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: Jeff Garzik <[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]>

• Prev by Date: Re: [PATCH 7/12] i386 sg: add support for chaining scatterlists
• Next by Date: Re: [PATCH 2.6.21] m32r: Fix switch_to macro to push/pop frame pointer if needed
• 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]