[PATCH] xlat: update ST_NOSYMFOLLOW constant

Dmitry V. Levin ldv at altlinux.org
Fri Mar 26 00:52:23 UTC 2021


Hi,

Since the change itself is trivial, I'll be extra picky about details
so you could get a taste of patch review process.

On Wed, Mar 24, 2021 at 09:26:33PM -0700, Deborah Brouwer wrote:
> Linux 5.10 introduced new constant ST_NOSYMFOLLOW.
> 
> Signed-off-by: Deborah Brouwer <3563deb6753 at gmail.com>

As we try to be more specific nowadays, we usually mention the Linux
kernel commit introduced the new constant, see e.g. strace commit
v5.11~31.

We also try to list user visible changes in the NEWS file, see e.g.
the same commit.

Sometimes we also update tests to check printing of new constants, but
that is not mandatory.

The first line of the commit message says "xlat: update ST_NOSYMFOLLOW
constant" which is a bit misleading because the change doesn't update
ST_NOSYMFOLLOW constant.

> ---
>  src/xlat/statfs_flags.in | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/src/xlat/statfs_flags.in b/src/xlat/statfs_flags.in
> index f791a60e..90a06138 100644
> --- a/src/xlat/statfs_flags.in
> +++ b/src/xlat/statfs_flags.in
> @@ -8,3 +8,4 @@ ST_MANDLOCK	0x0040
>  ST_NOATIME	0x0400
>  ST_NODIRATIME	0x0800
>  ST_RELATIME	0x1000
> +ST_NOSYMFOLLOW	0x2000

This part is fine.


-- 
ldv


More information about the Strace-devel mailing list