[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH v7 0/6] Allow Valgrind checking all QEMU process
From: |
Andrey Shinkevich |
Subject: |
Re: [Qemu-block] [PATCH v7 0/6] Allow Valgrind checking all QEMU processes |
Date: |
Tue, 3 Sep 2019 16:36:05 +0000 |
On 03/09/2019 17:28, Kevin Wolf wrote:
> Am 03.09.2019 um 16:22 hat Andrey Shinkevich geschrieben:
>>
>>
>> On 03/09/2019 13:02, Kevin Wolf wrote:
>>> Am 01.09.2019 um 13:53 hat Andrey Shinkevich geschrieben:
>>>> In the current implementation of the QEMU bash iotests, only qemu-io
>>>> processes may be run under the Valgrind with the switch '-valgrind'.
>>>> Let's allow the common.rc bash script running all other QEMU processes,
>>>> such as qemu-kvm, qemu-img, qemu-ndb and qemu-vxhs, under the Valgrind.
>>>
>>> Thanks, applied to the block branch.
>>>
>>> Kevin
>>>
>>
>> Kevin!
>> Please postpone the pull request!
>> The last optimization in the patch 1/6 broke the logic in the patch 2/3.
>> So, the test 039 hangs under the Valgrind, as it was.
>> The patch 2/6 must be optimized too.
>> I am about to make a little change in the patch 2/6 and will send v8
>> today...
>
> Ok, I'll unstage v7.
>
> Kevin
>
The v8 is ready to be sent. The test 039 passes now being run under the
Valgrind. The iotests pass with the v8 series applied being run without
the Valgrind:
./check -qcow2
and
./check -nbd
Now, I am waiting for all the iotests to pass with the switch
'-valgrind'. It takes much more time actually.
Few more hours are needed to complete running the iotests under the
Valgrind. I could send the v8 now or better wait until tomorrow for
assurance.
Andrey
--
With the best regards,
Andrey Shinkevich
- [Qemu-block] [PATCH v7 3/6] iotests: Add casenotrun report to bash tests, (continued)
- [Qemu-block] [PATCH v7 3/6] iotests: Add casenotrun report to bash tests, Andrey Shinkevich, 2019/09/01
- [Qemu-block] [PATCH v7 4/6] iotests: Valgrind fails with nonexistent directory, Andrey Shinkevich, 2019/09/01
- [Qemu-block] [PATCH v7 5/6] iotests: extended timeout under Valgrind, Andrey Shinkevich, 2019/09/01
- [Qemu-block] [PATCH v7 6/6] iotests: extend sleeping time under Valgrind, Andrey Shinkevich, 2019/09/01
- [Qemu-block] [PATCH v7 2/6] iotests: exclude killed processes from running under Valgrind, Andrey Shinkevich, 2019/09/01
- [Qemu-block] [PATCH v7 1/6] iotests: allow Valgrind checking all QEMU processes, Andrey Shinkevich, 2019/09/01
- Re: [Qemu-block] [PATCH v7 0/6] Allow Valgrind checking all QEMU processes, Kevin Wolf, 2019/09/03