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>

Reply via email to