All of lore.kernel.org
 help / color / mirror / Atom feed
From: Petr Vorel <pvorel@suse.cz>
To: cel@kernel.org
Cc: Jan Kara <jack@suse.cz>,
	stable@vger.kernel.org, Chuck Lever <chuck.lever@oracle.com>,
	ltp@lists.linux.it, Martin Doucha <martin.doucha@suse.com>
Subject: Re: [LTP] [PATCH RFC] syscalls/fanotify09: Note backport of commit e730558adffb
Date: Sat, 31 Aug 2024 20:00:10 +0200	[thread overview]
Message-ID: <20240831180010.GA173943@pevik> (raw)
In-Reply-To: <20240831160900.173809-1-cel@kernel.org>

Hi Chuck, Amir, all,

[ Add some enterprise folks just to notify ]

> From: Chuck Lever <chuck.lever@oracle.com>

Thanks for the fix!

> I backported commit e730558adffb ("fsnotify: consistent behavior for
> parent not watching children") to v5.15.y and v5.10.y. Update
> fanotify09 to test older LTS kernels containing that commit.

> Suggested-by: Amir Goldstein <amir73il@gmail.com>
> Signed-off-by: Chuck Lever <chuck.lever@oracle.com>

For others, suggested here:
https://lore.kernel.org/all/CAOQ4uxiUwSiRQ9tLPw6FPDB05rRLFdjxruFM4Lk=HcZfP2tfqA@mail.gmail.com/

Reviewed-by: Petr Vorel <pvorel@suse.cz>

NOTE: we might need to add check for enterprise kernels, but that should be
trivial. I'll test it on Monday on SLES and maybe add follow up for it.

Kind regards,
Petr
> ---
>  testcases/kernel/syscalls/fanotify/fanotify09.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)

> Untested.

> diff --git a/testcases/kernel/syscalls/fanotify/fanotify09.c b/testcases/kernel/syscalls/fanotify/fanotify09.c
> index f61c4e45a88c..48b198b9415a 100644
> --- a/testcases/kernel/syscalls/fanotify/fanotify09.c
> +++ b/testcases/kernel/syscalls/fanotify/fanotify09.c
> @@ -29,7 +29,6 @@
>   *      7372e79c9eb9 fanotify: fix logic of reporting name info with watched parent
>   *
>   * Test cases #6-#7 are regression tests for commit:
> - * (from v5.19, unlikely to be backported thus not in .tags):
>   *
>   *      e730558adffb fanotify: consistent behavior for parent not watching children
>   */
> @@ -380,9 +379,9 @@ static void test_fanotify(unsigned int n)
>  		return;
>  	}

> -	if (tc->ignore && tst_kvercmp(5, 19, 0) < 0) {
> +	if (tc->ignore && tst_kvercmp(5, 10, 0) < 0) {
>  		tst_res(TCONF, "ignored mask on parent dir has undefined "
> -				"behavior on kernel < 5.19");
> +				"behavior on kernel < 5.10");
>  		return;
>  	}

> @@ -520,6 +519,7 @@ static struct tst_test test = {
>  		{"linux-git", "b469e7e47c8a"},
>  		{"linux-git", "55bf882c7f13"},
>  		{"linux-git", "7372e79c9eb9"},
> +		{"linux-git", "e730558adffb"},
>  		{}
>  	}
>  };

-- 
Mailing list info: https://lists.linux.it/listinfo/ltp

WARNING: multiple messages have this Message-ID (diff)
From: Petr Vorel <pvorel@suse.cz>
To: cel@kernel.org
Cc: ltp@lists.linux.it, Amir Goldstein <amir73il@gmail.com>,
	stable@vger.kernel.org, Chuck Lever <chuck.lever@oracle.com>,
	Li Wang <liwang@redhat.com>,
	Martin Doucha <martin.doucha@suse.com>, Jan Kara <jack@suse.cz>
Subject: Re: [PATCH RFC] syscalls/fanotify09: Note backport of commit e730558adffb
Date: Sat, 31 Aug 2024 20:00:10 +0200	[thread overview]
Message-ID: <20240831180010.GA173943@pevik> (raw)
In-Reply-To: <20240831160900.173809-1-cel@kernel.org>

Hi Chuck, Amir, all,

[ Add some enterprise folks just to notify ]

> From: Chuck Lever <chuck.lever@oracle.com>

Thanks for the fix!

> I backported commit e730558adffb ("fsnotify: consistent behavior for
> parent not watching children") to v5.15.y and v5.10.y. Update
> fanotify09 to test older LTS kernels containing that commit.

> Suggested-by: Amir Goldstein <amir73il@gmail.com>
> Signed-off-by: Chuck Lever <chuck.lever@oracle.com>

For others, suggested here:
https://lore.kernel.org/all/CAOQ4uxiUwSiRQ9tLPw6FPDB05rRLFdjxruFM4Lk=HcZfP2tfqA@mail.gmail.com/

Reviewed-by: Petr Vorel <pvorel@suse.cz>

NOTE: we might need to add check for enterprise kernels, but that should be
trivial. I'll test it on Monday on SLES and maybe add follow up for it.

Kind regards,
Petr
> ---
>  testcases/kernel/syscalls/fanotify/fanotify09.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)

> Untested.

> diff --git a/testcases/kernel/syscalls/fanotify/fanotify09.c b/testcases/kernel/syscalls/fanotify/fanotify09.c
> index f61c4e45a88c..48b198b9415a 100644
> --- a/testcases/kernel/syscalls/fanotify/fanotify09.c
> +++ b/testcases/kernel/syscalls/fanotify/fanotify09.c
> @@ -29,7 +29,6 @@
>   *      7372e79c9eb9 fanotify: fix logic of reporting name info with watched parent
>   *
>   * Test cases #6-#7 are regression tests for commit:
> - * (from v5.19, unlikely to be backported thus not in .tags):
>   *
>   *      e730558adffb fanotify: consistent behavior for parent not watching children
>   */
> @@ -380,9 +379,9 @@ static void test_fanotify(unsigned int n)
>  		return;
>  	}

> -	if (tc->ignore && tst_kvercmp(5, 19, 0) < 0) {
> +	if (tc->ignore && tst_kvercmp(5, 10, 0) < 0) {
>  		tst_res(TCONF, "ignored mask on parent dir has undefined "
> -				"behavior on kernel < 5.19");
> +				"behavior on kernel < 5.10");
>  		return;
>  	}

> @@ -520,6 +519,7 @@ static struct tst_test test = {
>  		{"linux-git", "b469e7e47c8a"},
>  		{"linux-git", "55bf882c7f13"},
>  		{"linux-git", "7372e79c9eb9"},
> +		{"linux-git", "e730558adffb"},
>  		{}
>  	}
>  };

  reply	other threads:[~2024-08-31 18:00 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-08-31 16:09 [LTP] [PATCH RFC] syscalls/fanotify09: Note backport of commit e730558adffb cel--- via ltp
2024-08-31 16:09 ` cel
2024-08-31 18:00 ` Petr Vorel [this message]
2024-08-31 18:00   ` Petr Vorel
2024-09-03  7:31 ` [LTP] " Petr Vorel
2024-09-03  7:31   ` Petr Vorel

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=20240831180010.GA173943@pevik \
    --to=pvorel@suse.cz \
    --cc=cel@kernel.org \
    --cc=chuck.lever@oracle.com \
    --cc=jack@suse.cz \
    --cc=ltp@lists.linux.it \
    --cc=martin.doucha@suse.com \
    --cc=stable@vger.kernel.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.