]> git.dujemihanovic.xyz Git - u-boot.git/commit
test/py: Wait for guestmount worker to exit after running guestunmount
authorAlper Nebi Yasak <alpernebiyasak@gmail.com>
Fri, 4 Jun 2021 19:04:46 +0000 (22:04 +0300)
committerTom Rini <trini@konsulko.com>
Mon, 5 Jul 2021 19:28:32 +0000 (15:28 -0400)
commit99f5303c8b857320b00a63dca81d1f6f7c75c636
tree058d1cda7bd76682fa4bd59f203173be00cd3830
parent8f5f5d3a4593f26f0a755f5ecfaa7299e42edeeb
test/py: Wait for guestmount worker to exit after running guestunmount

Some filesystem tests are failing when their image is prepared with
guestmount, but succeeding if loop mounts are used instead. The reason
seems to be a race condition the guestmount(1) manual page explains:

    When guestunmount(1)/fusermount(1) exits, guestmount may still be
    running and cleaning up the mountpoint.  The disk image will not be
    fully finalized.

    This means that scripts like the following have a nasty race condition:

     guestmount -a disk.img -i /mnt
     # copy things into /mnt
     guestunmount /mnt
     # immediately try to use 'disk.img' ** UNSAFE **

    The solution is to use the --pid-file option to write the guestmount
    PID to a file, then after guestunmount spin waiting for this PID to
    exit.

The Python standard library has an os.waitpid() function for waiting a
child to terminate, but it cannot wait on non-child processes. Implement
a utility function that can do this by polling the process repeatedly
for a given duration, optionally killing the process if it won't
terminate on its own. Apply the suggested solution with this utility
function, which makes the failing tests succeed again.

Signed-off-by: Alper Nebi Yasak <alpernebiyasak@gmail.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
test/py/tests/test_fs/conftest.py
test/py/u_boot_utils.py