On Wed, 05/14 15:12, Markus Armbruster wrote: > The shell script attempts to suppress core dumps like this: > > old_ulimit=$(ulimit -c) > ulimit -c 0 > $QEMU_IO arg... > ulimit -c "$old_ulimit" > > This breaks the test hard unless the limit was zero to begin with! > ulimit sets both hard and soft limit by default, and (re-)raising the > hard limit requires privileges. Broken since it was added in commit > dc68afe. > > Could be fixed by adding -S to set only the soft limit, but I'm not > sure how portable that is in practice. Simply do it in a subshell > instead, like this: > > (ulimit -c 0; exec $QEMU_IO arg...) > > Signed-off-by: Markus Armbruster <arm...@redhat.com>
Reviewed-by: Fam Zheng <f...@redhat.com>