Messages in this thread | | | Date | Fri, 11 May 2007 12:48:25 -0400 | From | Chuck Ebbert <> | Subject | Re: [PATCH] libata: add human-readable error value decoding |
| |
Robert Hancock wrote: >>> + ehc->i.serror & SERR_TRANS_ST_ERROR ? "TransStatTransErr " >>> : "", >>> + ehc->i.serror & SERR_UNRECOG_FIS ? "UnrecogFIS " : "", >>> + ehc->i.serror & SERR_DEV_XCHG ? "DevExchanged " : "" ); >> >> 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. >> > > It certainly seems useful when debugging hotplug issues or random SATA > problems which end up being caused by communication problems. Without > this output, Joe User stands no chance of figuring out what's going on, > and neither does Joe libata Developer unless they really care to dig > through the spec and count bits to figure out what they mean. At least > with this you can see that there was a CRC error, etc. and go from that.. >
Why not just document the error messages?
And the scsi ones too, I can't seem to find what the sense codes mean.
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |