linux-fsdevel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Amir Goldstein <amir73il@gmail.com>
To: Christian Brauner <brauner@kernel.org>
Cc: Jan Kara <jack@suse.cz>, John Hubbard <jhubbard@nvidia.com>,
	Shuah Khan <skhan@linuxfoundation.org>,
	linux-fsdevel@vger.kernel.org
Subject: [PATCH 0/5] filesystems selftests cleanups
Date: Wed,  7 May 2025 22:42:57 +0200	[thread overview]
Message-ID: <20250507204302.460913-1-amir73il@gmail.com> (raw)

Christian,

I was cleaning up my test env today to prepare for the test
of fanotify mount ns notifications inside userns and tried to
disolve the headers_install depenency.

These patches got rid of the dependency for my kvm setup for the
affected filesystems tests.

Building with TOOLS_INCLUDES dir was recommended by John Hubbard [1].

NOTE #1: these patches are based on a merge of vfs-6.16.mount
(changes wrappers.h) into v6.15-rc5 (changes mount-notify_test.c),
so if this cleanup is acceptable, we should probably setup a stable
branch for 6.16, so that I can base my fanotify patches on it.

NOTE #2: some of the defines in wrappers.h are only left for
mount_setattr, which was not converted to use TOOLS_INCLUDES.
I did not want to mess with it.

Thanks,
Amir.

[1] https://lore.kernel.org/linux-fsdevel/6dd57f0e-34b4-4456-854b-a8abdba9163b@nvidia.com/

Amir Goldstein (5):
  selftests/filesystems: move wrapper.h out of overlayfs subdir
  selftests/fs/statmount: build with tools include dir
  selftests/fs/mount-notify: build with tools include dir
  selftests/filesystems: create get_unique_mnt_id() helper
  selftests/filesystems: create setup_userns() helper

 tools/include/uapi/linux/fanotify.h           | 274 ++++++++++++++++++
 tools/include/uapi/linux/mount.h              | 235 +++++++++++++++
 tools/include/uapi/linux/nsfs.h               |  45 +++
 .../filesystems/mount-notify/Makefile         |   6 +-
 .../mount-notify/mount-notify_test.c          |  38 +--
 .../selftests/filesystems/overlayfs/Makefile  |   2 +-
 .../filesystems/overlayfs/dev_in_maps.c       |   2 +-
 .../overlayfs/set_layers_via_fds.c            |   2 +-
 .../selftests/filesystems/statmount/Makefile  |   6 +-
 .../filesystems/statmount/statmount.h         |  12 +
 .../filesystems/statmount/statmount_test_ns.c |  88 +-----
 tools/testing/selftests/filesystems/utils.c   |  85 ++++++
 tools/testing/selftests/filesystems/utils.h   |   3 +
 .../filesystems/{overlayfs => }/wrappers.h    |   0
 .../testing/selftests/mount_setattr/Makefile  |   2 +
 .../mount_setattr/mount_setattr_test.c        |   2 +-
 16 files changed, 684 insertions(+), 118 deletions(-)
 create mode 100644 tools/include/uapi/linux/fanotify.h
 create mode 100644 tools/include/uapi/linux/mount.h
 create mode 100644 tools/include/uapi/linux/nsfs.h
 rename tools/testing/selftests/filesystems/{overlayfs => }/wrappers.h (100%)

-- 
2.34.1


             reply	other threads:[~2025-05-07 20:43 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-05-07 20:42 Amir Goldstein [this message]
2025-05-07 20:42 ` [PATCH 1/5] selftests/filesystems: move wrapper.h out of overlayfs subdir Amir Goldstein
2025-05-08  7:31   ` John Hubbard
2025-05-07 20:42 ` [PATCH 2/5] selftests/fs/statmount: build with tools include dir Amir Goldstein
2025-05-08  7:30   ` John Hubbard
2025-05-08 11:36     ` Amir Goldstein
2025-05-08 19:30       ` John Hubbard
2025-05-08 20:29         ` Amir Goldstein
2025-05-09 10:57       ` Christian Brauner
2025-05-09 11:08         ` Amir Goldstein
2025-05-09 13:34         ` Amir Goldstein
2025-05-08  7:33   ` John Hubbard
2025-05-07 20:43 ` [PATCH 3/5] selftests/fs/mount-notify: " Amir Goldstein
2025-05-08  7:38   ` John Hubbard
2025-05-08 11:53     ` Amir Goldstein
2025-05-08 19:13       ` John Hubbard
2025-05-07 20:43 ` [PATCH 4/5] selftests/filesystems: create get_unique_mnt_id() helper Amir Goldstein
2025-05-08  7:43   ` John Hubbard
2025-05-08 11:44     ` Amir Goldstein
2025-05-08 19:35       ` John Hubbard
2025-05-09  2:52         ` John Hubbard
2025-05-07 20:43 ` [PATCH 5/5] selftests/filesystems: create setup_userns() helper Amir Goldstein
2025-05-08  7:52   ` John Hubbard
2025-05-08 12:08     ` Amir Goldstein
2025-05-09 10:54       ` Christian Brauner
2025-05-08 12:40   ` Amir Goldstein
2025-05-08 19:32     ` John Hubbard

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=20250507204302.460913-1-amir73il@gmail.com \
    --to=amir73il@gmail.com \
    --cc=brauner@kernel.org \
    --cc=jack@suse.cz \
    --cc=jhubbard@nvidia.com \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=skhan@linuxfoundation.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 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).