From: "Darrick J. Wong" <darrick.wong@oracle.com>
To: Tyson Nottingham <tgnottingham@gmail.com>
Cc: linux-ext4@vger.kernel.org, tytso@mit.edu
Subject: Re: [PATCH] chattr: update man page attribute descriptions.
Date: Tue, 23 Jan 2018 21:07:06 -0800 [thread overview]
Message-ID: <20180124050706.GE5594@magnolia> (raw)
In-Reply-To: <1516764214-27295-1-git-send-email-tgnottingham@gmail.com>
On Tue, Jan 23, 2018 at 07:23:34PM -0800, Tyson Nottingham wrote:
> * Remove descriptions of X and Z, whose support was removed in
> 4a05268cf86f713 (Remove compression support).
>
> * Update a brief description of E, whose meaning was changed in
> 22f22ab1d2bc5ae (Reserve the codepoints for the new INCOMPAT
> feature ENCRYPT).
>
> * Add P to symbolic mode format string.
>
> Signed-off-by: Tyson Nottingham <tgnottingham@gmail.com>
Looks ok,
Reviewed-by: Darrick J. Wong <darrick.wong@oracle.com>
> ---
> misc/chattr.1.in | 21 +++------------------
> 1 file changed, 3 insertions(+), 18 deletions(-)
>
> diff --git a/misc/chattr.1.in b/misc/chattr.1.in
> index 6a1d446..cf9cb11 100644
> --- a/misc/chattr.1.in
> +++ b/misc/chattr.1.in
> @@ -23,7 +23,7 @@ chattr \- change file attributes on a Linux file system
> .B chattr
> changes the file attributes on a Linux file system.
> .PP
> -The format of a symbolic mode is +-=[aAcCdDeijsStTu].
> +The format of a symbolic mode is +-=[aAcCdDeijPsStTu].
> .PP
> The operator '+' causes the selected attributes to be added to the
> existing attributes of the files; '-' causes them to be removed; and '='
> @@ -49,12 +49,10 @@ and undeletable (u).
> The following attributes are read-only, and may be listed by
> .BR lsattr (1)
> but not modified by chattr:
> -compression error (E),
> +encrypted (E),
> huge file (h),
> indexed directory (I),
> -inline data (N),
> -compression raw access (X),
> -and compressed dirty file (Z).
> +and inline data (N).
> .PP
> Not all flags are supported or utilized by all filesystems; refer to
> filesystem-specific man pages such as
> @@ -192,19 +190,6 @@ saved. This allows the user to ask for its undeletion. Note: please
> make sure to read the bugs and limitations section at the end of this
> document.
> .PP
> -The 'X' attribute is used by the experimental compression patches to
> -indicate that the raw contents of a compressed file can be accessed
> -directly. It currently may not be set or reset using
> -.BR chattr (1),
> -although it can be displayed by
> -.BR lsattr (1).
> -.PP
> -The 'Z' attribute is used by the experimental compression patches to
> -indicate a compressed file is dirty. It may not be set or reset using
> -.BR chattr (1),
> -although it can be displayed by
> -.BR lsattr (1).
> -.PP
> .SH AUTHOR
> .B chattr
> was written by Remy Card <Remy.Card@linux.org>. It is currently being
> --
> 2.7.4
>
prev parent reply other threads:[~2018-01-24 5:07 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-01-24 3:23 [PATCH] chattr: update man page attribute descriptions Tyson Nottingham
2018-01-24 5:07 ` Darrick J. Wong [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20180124050706.GE5594@magnolia \
--to=darrick.wong@oracle.com \
--cc=linux-ext4@vger.kernel.org \
--cc=tgnottingham@gmail.com \
--cc=tytso@mit.edu \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).