Messages in this thread | | | From | "Michael Kerrisk (man-pages)" <> | Date | Tue, 3 Feb 2015 10:21:53 +0100 | Subject | Re: [PATCH v3 1/1] ioctl-fat.2: new manpage for the ioctl fat API |
| |
Hi Andreas,
Thanks for checking over the page.
On 3 February 2015 at 09:49, Andreas Dilger <adilger@dilger.ca> wrote: > On Jan 23, 2015, at 12:54 PM, Heinrich Schuchardt <xypron.glpk@gmx.de> wrote: >> >> The ioctl(2) system call may be used to retrieve information about >> the fat file system and to set file attributes. >> >> This new manpage describes the details. >> >> Michael Kerrisk suggested to CC linux-fsdevel@vger.kernel.org and >> linux-kernel@vger.kernel.org for review. >> >> version 3: correct typos >> version 2: consider comments by Michael Kerrisk >> verison 1: original patch >> >> Signed-by: Heinrich Schuchardt <xypron.glpk@gmx.de> >> --- >> man2/ioctl-fat.2 | 442 +++++++++++++++++++++++++++++++++++++++++++++++++++++++ >> 1 file changed, 442 insertions(+) >> create mode 100644 man2/ioctl-fat.2 >> >> diff --git a/man2/ioctl-fat.2 b/man2/ioctl-fat.2 >> new file mode 100644 >> index 0000000..80891d8 >> --- /dev/null >> +++ b/man2/ioctl-fat.2 >> @@ -0,0 +1,442 @@ >> +.\" Copyright (C) 2014, Heinrich Schuchardt <xypron.glpk@gmx.de> >> +.\" >> +.\" %%%LICENSE_START(VERBATIM) >> +.\" Permission is granted to make and distribute verbatim copies of this >> +.\" manual provided the copyright notice and this permission notice are >> +.\" preserved on all copies. >> +.\" >> +.\" Permission is granted to copy and distribute modified versions of >> +.\" this manual under the conditions for verbatim copying, provided that >> +.\" the entire resulting derived work is distributed under the terms of >> +.\" a permission notice identical to this one. >> +.\" >> +.\" Since the Linux kernel and libraries are constantly changing, this >> +.\" manual page may be incorrect or out-of-date. The author(s) assume. >> +.\" no responsibility for errors or omissions, or for damages resulting. >> +.\" from the use of the information contained herein. The author(s) may. >> +.\" not have taken the same level of care in the production of this. >> +.\" manual, which is licensed free of charge, as they might when working. > > There are extraneous periods at the end of each of these lines.
Fixed.
[...]
>> +The bits of the bit mask are >> +.TP >> +.B ATTR_RO >> +This bit specifies that the file or directory is read-only. > > It's too bad that these constants have such generic names. It would > be better to use MSDOS_ATTR_* or FAT_ATTR_*, since these are also > exposed to userspace, but are only used by FAT.
Agreed. Too late now, I guess, though.
Cheers,
Michael -- Michael Kerrisk Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/ Linux/UNIX System Programming Training: http://man7.org/training/
| |