Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754443AbbBCJWU (ORCPT ); Tue, 3 Feb 2015 04:22:20 -0500 Received: from mail-la0-f49.google.com ([209.85.215.49]:44951 "EHLO mail-la0-f49.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750843AbbBCJWQ (ORCPT ); Tue, 3 Feb 2015 04:22:16 -0500 MIME-Version: 1.0 Reply-To: mtk.manpages@gmail.com In-Reply-To: <5984550D-1518-490C-9A3F-3F9B1FA2EA00@dilger.ca> References: <1422042884-10824-1-git-send-email-xypron.glpk@gmx.de> <5984550D-1518-490C-9A3F-3F9B1FA2EA00@dilger.ca> From: "Michael Kerrisk (man-pages)" Date: Tue, 3 Feb 2015 10:21:53 +0100 Message-ID: Subject: Re: [PATCH v3 1/1] ioctl-fat.2: new manpage for the ioctl fat API To: Andreas Dilger Cc: Heinrich Schuchardt , Ogawa Hirofumi , Linux Filesystem Development List , LKML , linux-man Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2886 Lines: 79 Hi Andreas, Thanks for checking over the page. On 3 February 2015 at 09:49, Andreas Dilger wrote: > On Jan 23, 2015, at 12:54 PM, Heinrich Schuchardt 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 >> --- >> 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 >> +.\" >> +.\" %%%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/ -- 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/