[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH v2 for-2.10 1/1] qemu-iotests: _cleanup_qemu mus
From: |
Jeff Cody |
Subject: |
Re: [Qemu-block] [PATCH v2 for-2.10 1/1] qemu-iotests: _cleanup_qemu must be called on exit |
Date: |
Tue, 18 Apr 2017 15:42:23 -0400 |
User-agent: |
Mutt/1.5.24 (2015-08-30) |
On Tue, Apr 18, 2017 at 02:39:12PM -0500, Eric Blake wrote:
> On 04/18/2017 02:31 PM, Jeff Cody wrote:
> > On Tue, Apr 18, 2017 at 01:44:43PM -0500, Eric Blake wrote:
> >> On 04/18/2017 12:45 PM, Jeff Cody wrote:
> >>> For the tests that use the common.qemu functions for running a QEMU
> >>> process, _cleanup_qemu must be called in the exit function.
> >>>
> >>> If it is not, if the qemu process aborts, then not all of the droppings
> >>> are cleaned up (e.g. pidfile, fifos).
> >>>
>
> >>
> >> Looks like you missed (at least?) 063
> >>
> >
> > I did miss 156. But unless I am missing something, 063 does not use
> > common.qemu...
>
> Good point. 063 does litter, even on success, however:
>
> $ rm -rf scratch/
> $ ./check -raw 063
> ...
> $ ls scratch/
> t.raw.raw1
>
> One idea presented in an earlier thread was that common.rc should create
> a subdirectory per test (rather than all tests sharing scratch/), and
> then common.rc itself install the cleanup hook that wipes out the entire
> subdirectory (or maybe even add a command-line option to ./check to
> suppress wiping when it is desirable to debug a test failure by seeing
> the droppings left behind).
>
Yep, good ideas. I'll leave that for another series though; there are a few
other changes based on some suggestions from Daniel that I'd like to
implement as well, and I can fold your suggestions above into a new series
(unless someone else does it first).
-Jeff
- [Qemu-block] [PATCH v2 for-2.10 1/1] qemu-iotests: _cleanup_qemu must be called on exit, Jeff Cody, 2017/04/18
- Re: [Qemu-block] [PATCH v2 for-2.10 1/1] qemu-iotests: _cleanup_qemu must be called on exit, Eric Blake, 2017/04/18
- Re: [Qemu-block] [PATCH v2 for-2.10 1/1] qemu-iotests: _cleanup_qemu must be called on exit, Jeff Cody, 2017/04/18
- Re: [Qemu-block] [PATCH v2 for-2.10 1/1] qemu-iotests: _cleanup_qemu must be called on exit, Eric Blake, 2017/04/18
- Re: [Qemu-block] [PATCH v2 for-2.10 1/1] qemu-iotests: _cleanup_qemu must be called on exit,
Jeff Cody <=
- Re: [Qemu-block] [Qemu-devel] [PATCH v2 for-2.10 1/1] qemu-iotests: _cleanup_qemu must be called on exit, Fam Zheng, 2017/04/19
- Re: [Qemu-block] [Qemu-devel] [PATCH v2 for-2.10 1/1] qemu-iotests: _cleanup_qemu must be called on exit, Max Reitz, 2017/04/19
- Re: [Qemu-block] [Qemu-devel] [PATCH v2 for-2.10 1/1] qemu-iotests: _cleanup_qemu must be called on exit, Jeff Cody, 2017/04/19
- Re: [Qemu-block] [Qemu-devel] [PATCH v2 for-2.10 1/1] qemu-iotests: _cleanup_qemu must be called on exit, Max Reitz, 2017/04/19