On Tue, Sep 20, 2016 at 10:59:59AM -0700, Liu Bo wrote: > On Tue, Sep 20, 2016 at 10:03:43AM +0200, David Sterba wrote: > > On Mon, Sep 19, 2016 at 04:11:44PM -0700, Liu Bo wrote: > > > > > That's EIO. Sometimes the EIO is big enough we have to abort, but > > > > > really the abort is just adding bonus. > > > > > > > > I think we misuse the EIO where we should really return EFSCORRUPTED > > > > that's an alias for EUCLEAN, looking at xfs or ext4. EIO should be > > > > really a message that the hardware is bad. > > > > > > I love this idea, but one quick question, when returning EUCLEAN, what > > > message do users get? > > > > > > "#define EUCLEAN 117 /* Structure needs cleaning */" > > > > strerror(EUCLEAN) -> "Structure needs cleaning" > > Hmm, if I was the user, I'm not sure how to deal with "Structure needs > cleaning", so still need to take a glance at dmesg log.
I understand that, it's not descriptive at all. We could remap it to EIO once it goes outside of btrfs module, so it would be only internal error. The commit that introduces it to xfs is 10 years old, da2f4d679c8070ba5b6a920281e495917b293aa0 and mentions something like that. -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
