[PATCH v2 0/2] block/export: Conditionally ignore set-context error

Max Reitz posted 2 patches 4 years, 7 months ago
Test checkpatch passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20210624083825.29224-1-mreitz@redhat.com
Maintainers: Kevin Wolf <kwolf@redhat.com>, Max Reitz <mreitz@redhat.com>
block/export/export.c      |  5 ++++-
tests/qemu-iotests/307     | 15 +++++++++++++++
tests/qemu-iotests/307.out |  8 ++++++++
3 files changed, 27 insertions(+), 1 deletion(-)
[PATCH v2 0/2] block/export: Conditionally ignore set-context error
Posted by Max Reitz 4 years, 7 months ago
Hi,

I had this v2 lying around for quite some time but for some reason never
sent it.  I probably just forgot.  Sorry.

v1:
https://lists.nongnu.org/archive/html/qemu-block/2021-04/msg00584.html

Changes in v2:
- Letting `bdrv_try_set_aio_context()` return an error and then just
  discarding it conditionally is kind of not right.
  If we want to ignore the error, decide so from the start: Depending on
  `fixed_iothread`, pass either `errp` or `NULL`.

- Patch 2: Reference output has changed because of 30ebb9aa920.


git-backport-diff against v1:

Key:
[----] : patches are identical
[####] : number of functional differences between upstream/downstream patch
[down] : patch is downstream-only
The flags [FC] indicate (F)unctional and (C)ontextual differences, respectively

001/2:[0009] [FC] 'block/export: Conditionally ignore set-context error'
002/2:[0002] [FC] 'iotests/307: Test iothread conflict for exports'


Max Reitz (2):
  block/export: Conditionally ignore set-context error
  iotests/307: Test iothread conflict for exports

 block/export/export.c      |  5 ++++-
 tests/qemu-iotests/307     | 15 +++++++++++++++
 tests/qemu-iotests/307.out |  8 ++++++++
 3 files changed, 27 insertions(+), 1 deletion(-)

-- 
2.31.1


Re: [PATCH v2 0/2] block/export: Conditionally ignore set-context error
Posted by Kevin Wolf 4 years, 6 months ago
Am 24.06.2021 um 10:38 hat Max Reitz geschrieben:
> Max Reitz (2):
>   block/export: Conditionally ignore set-context error
>   iotests/307: Test iothread conflict for exports

Thanks, applied to the block branch.

Kevin