[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-stable] [PATCH v2 0/5] vpc: Return 0 from vpc_co_create() on s
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-stable] [PATCH v2 0/5] vpc: Return 0 from vpc_co_create() on success |
Date: |
Tue, 3 Sep 2019 11:06:22 +0200 |
User-agent: |
Mutt/1.12.0 (2019-05-25) |
Am 02.09.2019 um 21:33 hat Max Reitz geschrieben:
> (v2 for “block: Let blockdev-create return 0 on success”)
>
> Jobs are expected to return 0 on success, so this extends to
> .bdrv_co_create(). After some inspection, it turns out that vpc is the
> only block driver that may return a positive value instead (to indicate
> success). Fix that.
>
> Without this patch, blockdev-create is likely to fail for VPC images.
> Hence patch 5.
>
> John indicated his preference for me to use iotests.script_main(). I
> did that; but I still wanted to retain some form of verify_protocol().
> Patch 2 adds @supported_protocols to execute_test() (and thus to
> iotests.script_main() and iotests.main()). Then I noticed we should
> probably make all Python tests (that use either script_main() or main())
> pass something for that parameter, because it’s a bit silly to run all
> Python tests for raw when you just want to run the nbd tests (which are
> five or so). Enter patches 3 and 4.
>
> (There are two Python tests (093 and 136) which I didn’t change to pass
> supported_protocols, because they use null-{co,aio} as their protocol.
> As these are not actually testee protocols for the iotests, I decided to
> just keep running these tests for any protocol.)
Thanks, applied to the block branch.
- [Qemu-stable] [PATCH v2 0/5] vpc: Return 0 from vpc_co_create() on success, Max Reitz, 2019/09/02
- [Qemu-stable] [PATCH v2 1/5] vpc: Return 0 from vpc_co_create() on success, Max Reitz, 2019/09/02
- [Qemu-stable] [PATCH v2 2/5] iotests: Add supported protocols to execute_test(), Max Reitz, 2019/09/02
- [Qemu-stable] [PATCH v2 4/5] iotests: Restrict nbd Python tests to nbd, Max Reitz, 2019/09/02
- [Qemu-stable] [PATCH v2 3/5] iotests: Restrict file Python tests to file, Max Reitz, 2019/09/02
- [Qemu-stable] [PATCH v2 5/5] iotests: Test blockdev-create for vpc, Max Reitz, 2019/09/02
- Re: [Qemu-stable] [PATCH v2 0/5] vpc: Return 0 from vpc_co_create() on success,
Kevin Wolf <=