Re: [PATCH] xfs: Document error handling behavior

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Jul 19, 2016 at 11:18:01PM -0700, Eric Sandeen wrote:
> 
> 
> On 7/19/16 2:15 PM, Eric Sandeen wrote:
> > On 7/19/16 3:04 AM, Carlos Maiolino wrote:
> >> This is the first try to document the implementation of error handlers into
> >> sysfs.
> >>
> >> Reviews and comments are appreciated, please also notice I'm not english-native,
> >> so, spelling corrections are also appreciated :)
> > 
> > Thanks for doing this! 
> > 
> > There seems to be a specific sysfs documentation format, see for example
> > Documentation/ABI/testing/sysfs-fs-ext4 
> > 
> > It might be better to follow that format, and refer to it after a brief
> > explanation of the functionality in the xfs.txt file?
> 
> Or not; Dave doesn't like this location, so perhaps best not to take
> my suggestion.  ;)

Oh, I can see now why he doesn't like that, I've never seen such directory until
you mentioned it, why should it be so hidden, and why should we split filesystem
information into different locations.

IMHO, if someone want to take a look into filesystem documentation, the person
goes directly to Documentation/filesystems, I honestly think splitting
information into two different directories are wrong, and, even though you point
to there in some other place, it is still bad, sounds like a RPG book... Start
here...now go to page X...now go to page Y...now go to page Z.

I can re-format the documentation to the same format from sysfs-fs-ext4, but I
believe keeping it under Documentation/filesystems is still the best to do. To
be honest, I actually think we should create an XFS directory under it and put
everything xfs related there.

Cheers
> 
> -Eric
> 
> _______________________________________________
> xfs mailing list
> xfs@xxxxxxxxxxx
> http://oss.sgi.com/mailman/listinfo/xfs

-- 
Carlos

_______________________________________________
xfs mailing list
xfs@xxxxxxxxxxx
http://oss.sgi.com/mailman/listinfo/xfs



[Index of Archives]     [Linux XFS Devel]     [Linux Filesystem Development]     [Filesystem Testing]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux