<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Mar 18, 2019 at 8:29 PM Cyril Hrubis <<a href="mailto:chrubis@suse.cz" target="_blank">chrubis@suse.cz</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi!<br>
> LTP liabary should invoke do_cleanup function promptly when killing<br>
> test process doesn't work, especially useful to testcase do mount or<br>
> format devices in their setup phase. Otherwise, that abnormal broken<br>
> will effect the testcase which LTP is preparing to execute in next.<br>
<br>
Actually this code should be executed only if the test process got stuck<br>
in the kernel and in such case I doubt that there is a value in doing<br>
the cleanup as the cleanup code will likely get stuck there as well.<br>
<br>
The rationale is that if process that uses particular mount point is<br>
stuck in the kernel the umount() will likely get stuck there as well and<br>
there is no point in doing anything else than rebooting the machine.<br></blockquote><div><div class="gmail_default" style="font-size:small"></div><div class="gmail_default" style="font-size:small"><span style="color:rgb(0,0,0);white-space:pre-wrap"><br></span></div><div class="gmail_default" style="font-size:small"><span style="color:rgb(0,0,0);white-space:pre-wrap">I think your analysis is reasonable. But I hit a strange situation in the LTP automation task. Things like that readahead02 failed to kill its children processes and exit abnormally, system maybe get stuck there a bit time and then recover, so that readahead02 abort without invoke do_cleanup leads to the remaining tests cann't mount/format correctly, so we get a lot of same failures in next. More test logs in below, if anything I was wrong please correct me. Thanks!</span></div><div class="gmail_default" style="font-size:small"><span style="color:rgb(0,0,0);white-space:pre-wrap"><br></span></div>----<span class="gmail_default" style="font-size:small">SYSTEM </span>DMESG LOG-----<br>[ 1740.404393] LTP: starting readahead02 <br>[ 1742.927684] EXT4-fs (loop0): mounting ext2 file system using the ext4 subsystem <br>[ 1743.350353] EXT4-fs (loop0): mounted filesystem without journal. Opts: (null) <br>[ 1746.433393] readahead02 (27927): drop_caches: 1 <br>[ 1771.274691] restraintd[4152]: *** Current Time: Thu Mar 14 14:41:50 2019 Localwatchdog at: Thu Mar 14 19:15:49 2019 <br>[ 1790.819807] readahead02 (27927): drop_caches: 1 <br>[ 1830.514187] restraintd[4152]: *** Current Time: Thu Mar 14 14:42:49 2019 Localwatchdog at: Thu Mar 14 19:15:49 2019 <br>[ 1849.500644] readahead02 (27927): drop_caches: 1 <br>[ 1865.206921] readahead02 (27927): drop_caches: 1 <br>[ 1891.885486] restraintd[4152]: *** Current Time: Thu Mar 14 14:43:50 2019 Localwatchdog at: Thu Mar 14 19:15:49 2019 <br>[ 1898.643303] readahead02 (27927): drop_caches: 1 <br>[ 1952.598108] restraintd[4152]: *** Current Time: Thu Mar 14 14:44:51 2019 Localwatchdog at: Thu Mar 14 19:15:49 2019 <br>[-- MARK -- Thu Mar 14 18:45:00 2019] <br>[ 1996.672750] readahead02 (27927): drop_caches: 1 <br>[ 2011.717543] restraintd[4152]: *** Current Time: Thu Mar 14 14:45:50 2019 Localwatchdog at: Thu Mar 14 19:15:49 2019 <br>[ 2073.364241] restraintd[4152]: *** Current Time: Thu Mar 14 14:46:52 2019 Localwatchdog at: Thu Mar 14 19:15:49 2019 <br>[ 2093.992180] LTP: starting readdir01 <br>[ 2096.539432] LTP: starting readdir02 <br>[ 2096.868280] LTP: starting readdir21 <br>[ 2096.995509] LTP: starting readlink01A (symlink01 -T readlink01) <br>---------------------------<div class="gmail_default" style="font-size:small"><span style="color:rgb(0,0,0);white-space:pre-wrap"><br></span></div><div class="gmail_default" style="font-size:small"><span style="color:rgb(0,0,0);white-space:pre-wrap"><br></span></div><div class="gmail_default" style="font-size:small"><span style="color:rgb(0,0,0);white-space:pre-wrap">------LTP TEST LOG----------</span></div><<<test_output>>><br>tst_device.c:230: INFO: Using test device LTP_DEV='/dev/loop0'<br>tst_mkfs.c:90: INFO: Formatting /dev/loop0 with ext2 opts='' extra opts=''<br>mke2fs 1.44.3 (10-July-2018)<br>tst_test.c:1085: INFO: Timeout per run is 0h 05m 00s<br>readahead02.c:414: INFO: readahead length: 2097152 </div><div><...snip...></div><div>readahead02.c:191: PASS: offset is still at 0 as expected<br>readahead02.c:295: INFO: read_testfile(0) took: 97472667 usec<br>readahead02.c:296: INFO: read_testfile(1) took: 40059201 usec<br>readahead02.c:298: INFO: read_testfile(0) read: 67108864 bytes<br>readahead02.c:300: INFO: read_testfile(1) read: 0 bytes<br>readahead02.c:303: PASS: readahead saved some I/O<br>readahead02.c:311: INFO: cache can hold at least: 3173632 kB<br>readahead02.c:312: INFO: read_testfile(0) used cache: 132352 kB<br>readahead02.c:313: INFO: read_testfile(1) used cache: 65280 kB<br>readahead02.c:321: PASS: using cache as expected<br>readahead02.c:231: INFO: Test #3: POSIX_FADV_WILLNEED on overlayfs file<br>readahead02.c:136: INFO: creating test file of size: 67108864<br>Test timeouted, sending SIGKILL!<br>Test timeouted, sending SIGKILL!<br>Test timeouted, sending SIGKILL!<br>Test timeouted, sending SIGKILL!<br>Test timeouted, sending SIGKILL!<br>Test timeouted, sending SIGKILL!<br>Test timeouted, sending SIGKILL!<br>Test timeouted, sending SIGKILL!<br>Test timeouted, sending SIGKILL!<br>Test timeouted, sending SIGKILL!<br>Test timeouted, sending SIGKILL!<br>Cannot kill test processes!<br>Congratulation, likely test hit a kernel bug. <br>Exitting uncleanly...</div></div><div><br></div><div><div class="gmail_default" style="font-size:small"><...snip...></div><br>mke2fs 1.44.3 (10-July-2018)<br>/dev/loop0 is mounted; will not make a filesystem here!<br>renameat01    0  TINFO  :  Using test device LTP_DEV='/dev/loop0'<br>renameat01    0  TINFO  :  Formatting /dev/loop0 with ext2 opts='' extra opts=''<br>renameat01    1  TBROK  :  tst_mkfs.c:101: mkfs.ext2:1: renameat01.c failed with 185 <br>renameat01 2 TBROK : tst_mkfs.c:101: Remaining cases broken</div><div><br></div><div><div class="gmail_default" style=""><...snip...><br><br><<<test_output>>><br>tst_device.c:230: INFO: Using test device LTP_DEV='/dev/loop0'<br>tst_supported_fs_types.c:72: INFO: Kernel supports ext2<br>tst_supported_fs_types.c:56: INFO: mkfs.ext2 does exist<br>tst_supported_fs_types.c:72: INFO: Kernel supports ext3<br>tst_supported_fs_types.c:56: INFO: mkfs.ext3 does exist<br>tst_supported_fs_types.c:72: INFO: Kernel supports ext4<br>tst_supported_fs_types.c:56: INFO: mkfs.ext4 does exist<br>tst_supported_fs_types.c:72: INFO: Kernel supports xfs<br>tst_supported_fs_types.c:56: INFO: mkfs.xfs does exist<br>tst_supported_fs_types.c:95: INFO: Filesystem btrfs is not supported<br>tst_supported_fs_types.c:72: INFO: Kernel supports vfat<br>tst_supported_fs_types.c:52: INFO: mkfs.vfat does not exist<br>tst_supported_fs_types.c:95: INFO: Filesystem exfat is not supported<br>tst_supported_fs_types.c:95: INFO: Filesystem ntfs is not supported<br>tst_test.c:1146: INFO: Testing on ext2<br>tst_mkfs.c:90: INFO: Formatting /dev/loop0 with ext2 opts='' extra opts=''<br>mke2fs 1.44.3 (10-July-2018)<br>/dev/loop0 is mounted; will not make a filesystem here! <br>tst_mkfs.c:101: BROK: mkfs.ext2:1: tst_test.c failed with 749<br></div><div class="gmail_default" style="font-size:small"></div><...snip...><br><br>tag=umount01 stime=1552589616<br>cmdline="umount01"<br>contacts=""<br>analysis=exit<br><<<test_output>>><br>tst_device.c:230: INFO: Using test device LTP_DEV='/dev/loop0'<br>tst_mkfs.c:90: INFO: Formatting /dev/loop0 with ext2 opts='' extra opts=''<br>mke2fs 1.44.3 (10-July-2018)<br>/dev/loop0 is mounted; will not make a filesystem here! <br>tst_mkfs.c:101: BROK: mkfs.ext2:1: tst_test.c failed with 749<div class="gmail_default" style="font-size:small">----------------------------------------</div><br></div>-- <br><div dir="ltr" class="m_-4010038514094474794gmail_signature"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>