Re: [PATCH v13 00/10] Apply COR-filter to the block-stream permanently

2020-12-04 Thread Vladimir Sementsov-Ogievskiy

I still think we should keep dropped iotest by introducing "bottom" interface 
for stream job. And deprecate old interfaces.
Patch is better than arguing, so I decided to try it myself. Now I'm close to 
complete v14, so, will send it soon.

02.12.2020 21:30, Andrey Shinkevich wrote:

The previous version 12 was discussed in the email thread:
Message-Id: <1603390423-980205-1-git-send-email-andrey.shinkev...@virtuozzo.com>

v13:
   02: The bdrv_remove_node() was dropped.
   05: Three patches with fixes were merged into one.
   06: Minor changes based on Vladimir's suggestions.
   08: Three patches with fixes were merged into one.
   09: The search for format_name of backing file was added.
   10: The flag BLK_PERM_GRAPH_MOD was removed.

Andrey Shinkevich (10):
   copy-on-read: support preadv/pwritev_part functions
   block: add API function to insert a node
   copy-on-read: add filter drop function
   qapi: add filter-node-name to block-stream
   qapi: create BlockdevOptionsCor structure for COR driver
   iotests: add #310 to test bottom node in COR driver
   block: include supported_read_flags into BDS structure
   copy-on-read: skip non-guest reads if no copy needed
   stream: skip filters when writing backing file name to QCOW2 header
   block: apply COR-filter to block-stream jobs

  block.c|  25 +++
  block/copy-on-read.c   | 143 +
  block/copy-on-read.h   |  32 +
  block/io.c |  12 +++-
  block/monitor/block-hmp-cmds.c |   4 +-
  block/stream.c | 120 +++---
  blockdev.c |  12 ++--
  include/block/block.h  |  10 ++-
  include/block/block_int.h  |  11 +++-
  qapi/block-core.json   |  27 +++-
  tests/qemu-iotests/030 |  51 ++-
  tests/qemu-iotests/030.out |   4 +-
  tests/qemu-iotests/141.out |   2 +-
  tests/qemu-iotests/245 |  22 +--
  tests/qemu-iotests/310 | 114 
  tests/qemu-iotests/310.out |  15 +
  tests/qemu-iotests/group   |   1 +
  17 files changed, 484 insertions(+), 121 deletions(-)
  create mode 100644 block/copy-on-read.h
  create mode 100755 tests/qemu-iotests/310
  create mode 100644 tests/qemu-iotests/310.out




--
Best regards,
Vladimir



[PATCH v13 00/10] Apply COR-filter to the block-stream permanently

2020-12-02 Thread Andrey Shinkevich via
The previous version 12 was discussed in the email thread:
Message-Id: <1603390423-980205-1-git-send-email-andrey.shinkev...@virtuozzo.com>

v13:
  02: The bdrv_remove_node() was dropped.
  05: Three patches with fixes were merged into one.
  06: Minor changes based on Vladimir's suggestions.
  08: Three patches with fixes were merged into one.
  09: The search for format_name of backing file was added.
  10: The flag BLK_PERM_GRAPH_MOD was removed.

Andrey Shinkevich (10):
  copy-on-read: support preadv/pwritev_part functions
  block: add API function to insert a node
  copy-on-read: add filter drop function
  qapi: add filter-node-name to block-stream
  qapi: create BlockdevOptionsCor structure for COR driver
  iotests: add #310 to test bottom node in COR driver
  block: include supported_read_flags into BDS structure
  copy-on-read: skip non-guest reads if no copy needed
  stream: skip filters when writing backing file name to QCOW2 header
  block: apply COR-filter to block-stream jobs

 block.c|  25 +++
 block/copy-on-read.c   | 143 +
 block/copy-on-read.h   |  32 +
 block/io.c |  12 +++-
 block/monitor/block-hmp-cmds.c |   4 +-
 block/stream.c | 120 +++---
 blockdev.c |  12 ++--
 include/block/block.h  |  10 ++-
 include/block/block_int.h  |  11 +++-
 qapi/block-core.json   |  27 +++-
 tests/qemu-iotests/030 |  51 ++-
 tests/qemu-iotests/030.out |   4 +-
 tests/qemu-iotests/141.out |   2 +-
 tests/qemu-iotests/245 |  22 +--
 tests/qemu-iotests/310 | 114 
 tests/qemu-iotests/310.out |  15 +
 tests/qemu-iotests/group   |   1 +
 17 files changed, 484 insertions(+), 121 deletions(-)
 create mode 100644 block/copy-on-read.h
 create mode 100755 tests/qemu-iotests/310
 create mode 100644 tests/qemu-iotests/310.out

-- 
1.8.3.1