mbox series

[v6,00/11] 64bit block-layer: part II

Message ID 20210903102807.27127-1-vsementsov@virtuozzo.com
Headers show
Series 64bit block-layer: part II | expand

Message

Vladimir Sementsov-Ogievskiy Sept. 3, 2021, 10:27 a.m. UTC
Hi all!

Sorry for a long delay :( Finally, here is v6.

v6: rebase on new rbd handlers and backup-top renamed to copy-before-write. Also:

01: add Eric's r-b
    tweak commit msg to not mention sheepdog
02: add Eric's r-b
03: tweak commit msg
    drop extra type conversion in raw_co_pwrite_zeroes

04: vvfat: no write_target_commit() anymore
    nvme: fix over-80 line
    type conversion in raw_co_pwrite_zeroes() now dropped in 03

05: tweak commit msg
    add Eric's r-b

06: update comment

07: tweak commit msg, grammar and typos in comments
    don't add extra empty line
    handle also qemu_rbd_co_pwrite_zeroes

08: tweak commit msg, add Eric's r-b
09: tweak commit msg, add Eric's r-b
10: add rbd, drop sheepdog
    add assertion to iscsi_co_pdiscard()
11: tweak commit msg, add Eric's r-b

Vladimir Sementsov-Ogievskiy (11):
  block/io: bring request check to bdrv_co_(read,write)v_vmstate
  qcow2: check request on vmstate save/load path
  block: use int64_t instead of uint64_t in driver read handlers
  block: use int64_t instead of uint64_t in driver write handlers
  block: use int64_t instead of uint64_t in copy_range driver handlers
  block: make BlockLimits::max_pwrite_zeroes 64bit
  block: use int64_t instead of int in driver write_zeroes handlers
  block/io: allow 64bit write-zeroes requests
  block: make BlockLimits::max_pdiscard 64bit
  block: use int64_t instead of int in driver discard handlers
  block/io: allow 64bit discard requests

 include/block/block_int.h        | 66 ++++++++++++++++--------------
 block/blkdebug.c                 | 12 +++---
 block/blklogwrites.c             | 16 ++++----
 block/blkreplay.c                |  8 ++--
 block/blkverify.c                |  8 ++--
 block/bochs.c                    |  4 +-
 block/cloop.c                    |  4 +-
 block/commit.c                   |  2 +-
 block/copy-before-write.c        | 15 +++----
 block/copy-on-read.c             | 19 +++++----
 block/crypto.c                   |  8 ++--
 block/curl.c                     |  3 +-
 block/dmg.c                      |  4 +-
 block/file-posix.c               | 35 ++++++++--------
 block/file-win32.c               |  8 ++--
 block/filter-compress.c          | 15 +++----
 block/gluster.c                  | 13 +++---
 block/io.c                       | 44 +++++++++++++++-----
 block/iscsi.c                    | 58 ++++++++++++++++----------
 block/mirror.c                   |  8 ++--
 block/nbd.c                      | 22 ++++++----
 block/nfs.c                      | 12 +++---
 block/null.c                     | 18 ++++----
 block/nvme.c                     | 48 ++++++++++++++++++----
 block/preallocate.c              | 14 +++----
 block/qcow.c                     | 16 ++++----
 block/qcow2-cluster.c            | 14 ++++++-
 block/qcow2.c                    | 70 +++++++++++++++++++++++---------
 block/qed.c                      |  9 +++-
 block/quorum.c                   | 11 ++---
 block/raw-format.c               | 36 ++++++++--------
 block/rbd.c                      | 20 ++++-----
 block/throttle.c                 | 18 ++++----
 block/vdi.c                      |  8 ++--
 block/vmdk.c                     | 14 +++----
 block/vpc.c                      |  8 ++--
 block/vvfat.c                    |  8 ++--
 tests/unit/test-bdrv-drain.c     | 16 ++++----
 tests/unit/test-block-iothread.c | 21 +++++++---
 block/trace-events               | 10 ++---
 slirp                            |  2 +-
 ui/keycodemapdb                  |  2 +-
 42 files changed, 452 insertions(+), 295 deletions(-)

Comments

Vladimir Sementsov-Ogievskiy Sept. 22, 2021, 7:52 a.m. UTC | #1
Ping)

Not reviewed: 6,7,10

03.09.2021 13:27, Vladimir Sementsov-Ogievskiy wrote:
> Hi all!
> 
> Sorry for a long delay:(  Finally, here is v6.
> 
> v6: rebase on new rbd handlers and backup-top renamed to copy-before-write. Also:
> 
> 01: add Eric's r-b
>      tweak commit msg to not mention sheepdog
> 02: add Eric's r-b
> 03: tweak commit msg
>      drop extra type conversion in raw_co_pwrite_zeroes
> 
> 04: vvfat: no write_target_commit() anymore
>      nvme: fix over-80 line
>      type conversion in raw_co_pwrite_zeroes() now dropped in 03
> 
> 05: tweak commit msg
>      add Eric's r-b
> 
> 06: update comment
> 
> 07: tweak commit msg, grammar and typos in comments
>      don't add extra empty line
>      handle also qemu_rbd_co_pwrite_zeroes
> 
> 08: tweak commit msg, add Eric's r-b
> 09: tweak commit msg, add Eric's r-b
> 10: add rbd, drop sheepdog
>      add assertion to iscsi_co_pdiscard()
> 11: tweak commit msg, add Eric's r-b
> 
> Vladimir Sementsov-Ogievskiy (11):
>    block/io: bring request check to bdrv_co_(read,write)v_vmstate
>    qcow2: check request on vmstate save/load path
>    block: use int64_t instead of uint64_t in driver read handlers
>    block: use int64_t instead of uint64_t in driver write handlers
>    block: use int64_t instead of uint64_t in copy_range driver handlers
>    block: make BlockLimits::max_pwrite_zeroes 64bit
>    block: use int64_t instead of int in driver write_zeroes handlers
>    block/io: allow 64bit write-zeroes requests
>    block: make BlockLimits::max_pdiscard 64bit
>    block: use int64_t instead of int in driver discard handlers
>    block/io: allow 64bit discard requests
Eric Blake Sept. 22, 2021, 1:13 p.m. UTC | #2
On Wed, Sep 22, 2021 at 10:52:20AM +0300, Vladimir Sementsov-Ogievskiy wrote:
> Ping)
> 
> Not reviewed: 6,7,10

Also on my list to see what I can get in this week's NBD pull request.

> 
> 03.09.2021 13:27, Vladimir Sementsov-Ogievskiy wrote:
> > Hi all!
> > 
> > Sorry for a long delay:(  Finally, here is v6.
> > 

> > Vladimir Sementsov-Ogievskiy (11):
> >    block/io: bring request check to bdrv_co_(read,write)v_vmstate
> >    qcow2: check request on vmstate save/load path
> >    block: use int64_t instead of uint64_t in driver read handlers
> >    block: use int64_t instead of uint64_t in driver write handlers
> >    block: use int64_t instead of uint64_t in copy_range driver handlers
> >    block: make BlockLimits::max_pwrite_zeroes 64bit
> >    block: use int64_t instead of int in driver write_zeroes handlers
> >    block/io: allow 64bit write-zeroes requests
> >    block: make BlockLimits::max_pdiscard 64bit
> >    block: use int64_t instead of int in driver discard handlers
> >    block/io: allow 64bit discard requests
> 
> 
> -- 
> Best regards,
> Vladimir
>