Re: [PATCH v3 RESEND] selftests: filesystems: Add functional test for the abort file in fusectl

2025-06-13 Thread Chen Linxuan
On Thu, Jun 12, 2025 at 4:56 PM Miklos Szeredi wrote: > > On Tue, 10 Jun 2025 at 04:10, Chen Linxuan wrote: > > > > This patch add a simple functional test for the "abort" file > > in fusectlfs (/sys/fs/fuse/connections/ID/about). > > >

[PATCH v4] selftests: filesystems: Add functional test for the abort file in fusectl

2025-06-12 Thread Chen Linxuan
g_OBhmSY=q...@mail.gmail.com/ Signed-off-by: Chen Linxuan Acked-by: Shuah Khan Reviewed-by: Amir Goldstein Co-developed-by: Miklos Szeredi Reviewed-by: Miklos Szeredi --- Changes in v4: - Apply patch suggested by Miklos Szeredi - Setting up a userns environment for testing - Fix a EBUSY on um

[PATCH v3 RESEND] selftests: filesystems: Add functional test for the abort file in fusectl

2025-06-09 Thread Chen Linxuan
g_OBhmSY=q...@mail.gmail.com/ Signed-off-by: Chen Linxuan Acked-by: Shuah Khan Reviewed-by: Amir Goldstein --- Changes in v3: - Apply changes suggested by Amir Goldstein - Rename the test subdir to filesystems/fuse - Verify errno when connection is aborted - Apply changes suggested by Shuah Khan

[PATCH RESEND] selftests: add missing include

2025-06-09 Thread Chen Linxuan
I failed to build this test on Ubuntu 24.04. Compiler complained about undefined functions mount, umount and some mount related flags. Signed-off-by: Chen Linxuan --- tools/testing/selftests/pid_namespace/pid_max.c | 1 + 1 file changed, 1 insertion(+) diff --git a/tools/testing/selftests

[PATCH RESEND] selftests: Suppress unused variable warning

2025-06-09 Thread Chen Linxuan
("/"); | ^~ This patch addresses the warning by explicitly suppressing the unused result of the `chdir` function. Signed-off-by: Chen Linxuan --- .../selftests/filesystems/mount-notify/mount-notify_test.c| 4 +++- 1 file changed, 3 insertions(+),

[PATCH RESEND] selftests/memfd: clean Makefile

2025-06-09 Thread Chen Linxuan
takes effect. The reason fuse_mnt compiles successfully is because CFLAGS is set at the very beginning of the file. Signed-off-by: Chen Linxuan --- tools/testing/selftests/memfd/Makefile | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/tools/testing/selftests/memfd/Makefile b

Re: [PATCH v2] selftests: Add functional test for the abort file in fusectl

2025-06-02 Thread Chen Linxuan
V3 has been sent. See https://lore.kernel.org/all/20250526014226.14192-1-chenlinx...@uniontech.com/ On Tue, Jun 3, 2025 at 7:02 AM Shuah Khan wrote: > > On 5/25/25 19:41, Chen Linxuan wrote: > > On Fri, May 23, 2025 at 6:50 AM Shuah Khan > > wrote: > > > >>

[PATCH v3] selftests: filesystems: Add functional test for the abort file in fusectl

2025-05-25 Thread Chen Linxuan
g_OBhmSY=q...@mail.gmail.com/ Cc: Amir Goldstein Signed-off-by: Chen Linxuan Acked-by: Shuah Khan --- Changes in v3: - Apply changes suggested by Amir Goldstein - Rename the test subdir to filesystems/fuse - Verify errno when connection is aborted - Apply changes suggested by Shuah Khan - Upd

Re: [PATCH v2] selftests: Add functional test for the abort file in fusectl

2025-05-25 Thread Chen Linxuan
On Fri, May 23, 2025 at 6:50 AM Shuah Khan wrote: > Also if this test requires root previlege, add check for it. Currently, this test does not require root privileges. Thanks, Chen Linxuan

Re: [PATCH] selftests: Add functional test for the abort file in fusectl

2025-05-18 Thread Chen Linxuan
On Thu, May 15, 2025 at 6:27 PM Amir Goldstein wrote: > > On Thu, May 15, 2025 at 9:35 AM Chen Linxuan > wrote: > > > > + ret = read(test_fd, path_buf, sizeof(path_buf)); > > + ASSERT_LT(ret, 0); > > Nice! > I guess you could also verify errno ==

[PATCH v2] selftests: Add functional test for the abort file in fusectl

2025-05-16 Thread Chen Linxuan
g_OBhmSY=q...@mail.gmail.com/ Cc: Amir Goldstein Signed-off-by: Chen Linxuan --- Changes in v2: - Apply changes suggested by Amir Goldstein - Check errno - Link to v1: https://lore.kernel.org/all/20250515073449.346774-2-chenlinx...@uniontech.com/ --- MAINTAINERS

[PATCH] selftests/memfd: clean Makefile

2025-05-15 Thread Chen Linxuan
takes effect. The reason fuse_mnt compiles successfully is because CFLAGS is set at the very beginning of the file. Signed-off-by: Chen Linxuan --- tools/testing/selftests/memfd/Makefile | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/tools/testing/selftests/memfd/Makefile b

[PATCH] selftests: Add functional test for the abort file in fusectl

2025-05-15 Thread Chen Linxuan
g_OBhmSY=q...@mail.gmail.com/ Cc: Amir Goldstein Signed-off-by: Chen Linxuan --- MAINTAINERS | 1 + tools/testing/selftests/Makefile | 1 + .../selftests/filesystems/fusectl/.gitignore | 3 + .../selftests/filesystems/fusectl/Makefil

Re: [PATCH] selftests: Add functional test for the abort file in fusectl

2025-05-15 Thread Chen Linxuan
On Thu, May 15, 2025 at 3:35 PM Chen Linxuan wrote: > This patch add a simple functional test for the "about" file Sorry for the typo, it should be "abort".

[PATCH] selftests: remove duplicated function definition

2025-05-13 Thread Chen Linxuan
I failed to build this test on Ubuntu 24.04. Compiler complains that function sys_open_tree has already been defined in "../filesystems/overlayfs/wrappers.h". Signed-off-by: Chen Linxuan --- tools/testing/selftests/mount_setattr/mount_setattr_test.c | 5 - 1 file changed, 5

[PATCH] selftests: add missing include

2025-05-13 Thread Chen Linxuan
I failed to build this test on Ubuntu 24.04. Compiler complained about undefined functions mount, umount and some mount related flags. Signed-off-by: Chen Linxuan --- tools/testing/selftests/pid_namespace/pid_max.c | 1 + 1 file changed, 1 insertion(+) diff --git a/tools/testing/selftests

[PATCH] selftests: Suppress unused variable warning

2025-05-13 Thread Chen Linxuan
("/"); | ^~ This patch addresses the warning by explicitly suppressing the unused result of the `chdir` function. Signed-off-by: Chen Linxuan --- .../selftests/filesystems/mount-notify/mount-notify_test.c| 4 +++- 1 file changed, 3 insertions(+),

[RFC PATCH 3/7] vfio/virtio: add __always_inline for virtiovf_get_device_config_size

2025-04-11 Thread Chen Linxuan
e: in expansion of macro 'BUILD_BUG_ON' 401 | BUILD_BUG_ON(!is_power_of_2(virtvdev->bar0_virtual_buf_size)); | ^~~~ Signed-off-by: Winston Wen Co-Developed-by: Chen Linxuan Signed-off-by: Chen Linxuan --- drivers/vfio/pci/virtio/legacy_io.c | 2 +-