Re: [LTP] LTP: syscalls: regression on mainline - ioctl_loop01 mknod07 setns01

From: Yang Xu
Date: Fri Jun 05 2020 - 04:59:56 EST

Next message: Neal Liu: "Re: Security Random Number Generator support"
Previous message: Sudeep Holla: "Re: [RFC] dt-bindings: mailbox: add doorbell support to ARM MHU"
In reply to: Yang Xu: "Re: LTP: syscalls: regression on mainline - ioctl_loop01 mknod07 setns01"
Next in thread: Martijn Coenen: "Re: [LTP] LTP: syscalls: regression on mainline - ioctl_loop01 mknod07 setns01"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

Hi Martijn

Sorry for noise. I see your patch in here[1] . I will modify ioctl_loop01 to test that LO_FLAGS_PARTSCAN can not clear and LO_FLAGS_AUTOCLEAR can be clear.

ps: Giving the url of patch is better so that other people doesn't need to investigate it again.
[1]https://patchwork.kernel.org/patch/11588321/

Best Regards
Yang Xu

Hi Martijn

Hi Naresh,

I just sent a patch and cc'd you. I verified all the loop tests pass
again with that patch.

I think you want to say "without".Â I verified the ioctl_loop01 fails with faf1d25440 ("loop: Clean up LOOP_SET_STATUS lo_flags handling").

This kernel commit breaks old behaviour(if old flag all 0, new flag is always 0 regradless your flag setting).

I think we should modify code as below:
diff --git a/drivers/block/loop.c b/drivers/block/loop.c
index 13518ba191f5..c6ba8cf486ce 100644
--- a/drivers/block/loop.c
+++ b/drivers/block/loop.c
@@ -1364,11 +1364,9 @@ loop_set_status(struct loop_device *lo, const struct loop_info64 *info)
ÂÂÂÂÂÂÂ if (err)
ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ goto out_unfreeze;

-ÂÂÂÂÂÂ /* Mask out flags that can't be set using LOOP_SET_STATUS. */
-ÂÂÂÂÂÂ lo->lo_flags &= ~LOOP_SET_STATUS_SETTABLE_FLAGS;
-ÂÂÂÂÂÂ /* For those flags, use the previous values instead */
-ÂÂÂÂÂÂ lo->lo_flags |= prev_lo_flags & ~LOOP_SET_STATUS_SETTABLE_FLAGS;
-ÂÂÂÂÂÂ /* For flags that can't be cleared, use previous values too */
+ÂÂÂÂÂÂ /* Mask out flags that can be set using LOOP_SET_STATUS. */
+ÂÂÂÂÂÂ lo->lo_flags &= LOOP_SET_STATUS_SETTABLE_FLAGS;
+ÂÂÂÂÂÂ /* For flags that can't be cleared, use previous values. */
ÂÂÂÂÂÂÂ lo->lo_flags |= prev_lo_flags &~LOOP_SET_STATUS_CLEARABLE_FLAGS;

Best Regards
Yang Xu

Thanks,
Martijn

On Thu, Jun 4, 2020 at 9:10 PM Martijn Coenen <maco@xxxxxxxxxxx> wrote:

Hi Naresh,

I suspect the loop failures are due to
faf1d25440d6ad06d509dada4b6fe62fea844370 ("loop: Clean up
LOOP_SET_STATUS lo_flags handling"), I will investigate and get back
to you.

Thanks,
Martijn

On Thu, Jun 4, 2020 at 7:19 PM Naresh Kamboju <naresh.kamboju@xxxxxxxxxx> wrote:

+ linux-block@xxxxxxxxxxxxxxx

On Thu, 4 Jun 2020 at 22:47, Naresh Kamboju <naresh.kamboju@xxxxxxxxxx> wrote:

Following three test cases reported as regression on Linux mainline kernel
on x86_64, arm64, arm and i386

ÂÂ ltp-syscalls-tests:
ÂÂÂÂ * ioctl_loop01
ÂÂÂÂ * mknod07
ÂÂÂÂ * setns01

git repo: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
git branch: master
GOOD:
ÂÂ git commit: b23c4771ff62de8ca9b5e4a2d64491b2fb6f8f69
ÂÂ git describe: v5.7-1230-gb23c4771ff62
BAD:
ÂÂ git commit: 1ee08de1e234d95b5b4f866878b72fceb5372904
ÂÂ git describe: v5.7-3523-g1ee08de1e234

kernel-config: https://builds.tuxbuild.com/U3bU0dMA62OVHb4DvZIVuw/kernel.config

We are investigating these failures.

tst_test.c:906: CONF: btrfs driver not available
tst_test.c:1246: INFO: Timeout per run is 0h 15m 00s
tst_device.c:88: INFO: Found free device 1 '/dev/loop1'
ioctl_loop01.c:49: PASS: /sys/block/loop1/loop/partscan = 0
[ 1073.639677] loop_set_status: loop1 () has still dirty pages (nrpages=1)
ioctl_loop01.c:50: PASS: /sys/block/loop1/loop/autoclear = 0
ioctl_loop01.c:51: PASS: /sys/block/loop1/loop/backing_file =
'/scratch/ltp-mnIdulzriQ/9cPtLQ/test.img'
ioctl_loop01.c:63: FAIL: expect 12 but got 17
ioctl_loop01.c:67: FAIL: /sys/block/loop1/loop/partscan != 1 got 0
ioctl_loop01.c:68: FAIL: /sys/block/loop1/loop/autoclear != 1 got 0
ioctl_loop01.c:79: FAIL: access /dev/loop1p1 fails
[ 1073.679678] loop_set_status: loop1 () has still dirty pages (nrpages=1)
ioctl_loop01.c:85: FAIL: access /sys/block/loop1/loop1p1 fails

HINT: You _MAY_ be missing kernel fixes, see:
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=10c70d95c0f2

mke2fs 1.43.8 (1-Jan-2018)
[ 1264.711379] EXT4-fs (loop0): mounting ext2 file system using the
ext4 subsystem
[ 1264.716642] EXT4-fs (loop0): mounted filesystem without journal. Opts: (null)
mknod07ÂÂÂÂ 0Â TINFOÂ :Â Using test device LTP_DEV='/dev/loop0'
mknod07ÂÂÂÂ 0Â TINFOÂ :Â Formatting /dev/loop0 with ext2 opts='' extra opts=''
mknod07ÂÂÂÂ 1Â TPASSÂ :Â mknod failed as expected:
TEST_ERRNO=EACCES(13): Permission denied
mknod07ÂÂÂÂ 2Â TPASSÂ :Â mknod failed as expected:
TEST_ERRNO=EACCES(13): Permission denied
mknod07ÂÂÂÂ 3Â TFAILÂ :Â mknod07.c:155: mknod succeeded unexpectedly
mknod07ÂÂÂÂ 4Â TPASSÂ :Â mknod failed as expected:
TEST_ERRNO=EPERM(1): Operation not permitted
mknod07ÂÂÂÂ 5Â TPASSÂ :Â mknod failed as expected:
TEST_ERRNO=EROFS(30): Read-only file system
mknod07ÂÂÂÂ 6Â TPASSÂ :Â mknod failed as expected:
TEST_ERRNO=ELOOP(40): Too many levels of symbolic links

setns01ÂÂÂÂ 0Â TINFOÂ :Â ns_name=ipc, ns_fds[0]=6, ns_types[0]=0x8000000
setns01ÂÂÂÂ 0Â TINFOÂ :Â ns_name=mnt, ns_fds[1]=7, ns_types[1]=0x20000
setns01ÂÂÂÂ 0Â TINFOÂ :Â ns_name=net, ns_fds[2]=8, ns_types[2]=0x40000000
setns01ÂÂÂÂ 0Â TINFOÂ :Â ns_name=pid, ns_fds[3]=9, ns_types[3]=0x20000000
setns01ÂÂÂÂ 0Â TINFOÂ :Â ns_name=uts, ns_fds[4]=10, ns_types[4]=0x4000000
setns01ÂÂÂÂ 0Â TINFOÂ :Â setns(-1, 0x8000000)
setns01ÂÂÂÂ 1Â TPASSÂ :Â invalid fd exp_errno=9
setns01ÂÂÂÂ 0Â TINFOÂ :Â setns(-1, 0x20000)
setns01ÂÂÂÂ 2Â TPASSÂ :Â invalid fd exp_errno=9
setns01ÂÂÂÂ 0Â TINFOÂ :Â setns(-1, 0x40000000)
setns01ÂÂÂÂ 3Â TPASSÂ :Â invalid fd exp_errno=9
setns01ÂÂÂÂ 0Â TINFOÂ :Â setns(-1, 0x20000000)
setns01ÂÂÂÂ 4Â TPASSÂ :Â invalid fd exp_errno=9
setns01ÂÂÂÂ 0Â TINFOÂ :Â setns(-1, 0x4000000)
setns01ÂÂÂÂ 5Â TPASSÂ :Â invalid fd exp_errno=9
setns01ÂÂÂÂ 0Â TINFOÂ :Â setns(11, 0x8000000)
setns01ÂÂÂÂ 6Â TFAILÂ :Â setns01.c:176: regular file fd exp_errno=22:
errno=EBADF(9): Bad file descriptor
setns01ÂÂÂÂ 0Â TINFOÂ :Â setns(11, 0x20000)
setns01ÂÂÂÂ 7Â TFAILÂ :Â setns01.c:176: regular file fd exp_errno=22:
errno=EBADF(9): Bad file descriptor
setns01ÂÂÂÂ 0Â TINFOÂ :Â setns(11, 0x40000000)
setns01ÂÂÂÂ 8Â TFAILÂ :Â setns01.c:176: regular file fd exp_errno=22:
errno=EBADF(9): Bad file descriptor
setns01ÂÂÂÂ 0Â TINFOÂ :Â setns(11, 0x20000000)
setns01ÂÂÂÂ 9Â TFAILÂ :Â setns01.c:176: regular file fd exp_errno=22:
errno=EBADF(9): Bad file descriptor
setns01ÂÂÂÂ 0Â TINFOÂ :Â setns(11, 0x4000000)
setns01ÂÂÂ 10Â TFAILÂ :Â setns01.c:176: regular file fd exp_errno=22:
errno=EBADF(9): Bad file descriptor

Full test log link,
https://lkft.validation.linaro.org/scheduler/job/1467931#L8047

test results comparison shows this test case started failing from June-2-2020
https://qa-reports.linaro.org/lkft/linux-mainline-oe/build/v5.7-4092-g38696e33e2bd/testrun/2779586/suite/ltp-syscalls-tests/test/ioctl_loop01/history/

https://qa-reports.linaro.org/lkft/linux-mainline-oe/build/v5.7-4092-g38696e33e2bd/testrun/2779586/suite/ltp-syscalls-tests/test/setns01/history/

https://qa-reports.linaro.org/lkft/linux-mainline-oe/build/v5.7-4092-g38696e33e2bd/testrun/2779586/suite/ltp-syscalls-tests/test/mknod07/history/

--
Linaro LKFT
https://lkft.linaro.org

Next message: Neal Liu: "Re: Security Random Number Generator support"
Previous message: Sudeep Holla: "Re: [RFC] dt-bindings: mailbox: add doorbell support to ARM MHU"
In reply to: Yang Xu: "Re: LTP: syscalls: regression on mainline - ioctl_loop01 mknod07 setns01"
Next in thread: Martijn Coenen: "Re: [LTP] LTP: syscalls: regression on mainline - ioctl_loop01 mknod07 setns01"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]