Re: [PATCH v2 0/3] block: align CoR requests to subclusters

[Stefan Hajnoczi <stefanha@redhat.com>]

[-- Attachment #1: Type: text/plain, Size: 2256 bytes --]

On Wed, Aug 23, 2023 at 03:50:55PM +0300, Andrey Drobyshev wrote:
> On 8/22/23 22:58, John Snow wrote:
> > On Tue, Aug 22, 2023 at 1:33 PM Andrey Drobyshev
> > <andrey.drobyshev@virtuozzo.com> wrote:
> >>
> >> On 8/16/23 12:22, Andrey Drobyshev wrote:
> >>> On 7/31/23 17:51, Andrey Drobyshev wrote:
> >>>> On 7/24/23 16:11, Andrey Drobyshev wrote:
> >>>>> On 7/11/23 20:25, Andrey Drobyshev wrote:
> >>>>>> v1 --> v2:
> >>>>>>  * Fixed line indentation;
> >>>>>>  * Fixed wording in a comment;
> >>>>>>  * Added R-b.
> >>>>>>
> >>>>>> v1: https://lists.nongnu.org/archive/html/qemu-block/2023-06/msg00606.html
> >>>>>>
> >>>>>> Andrey Drobyshev (3):
> >>>>>>   block: add subcluster_size field to BlockDriverInfo
> >>>>>>   block/io: align requests to subcluster_size
> >>>>>>   tests/qemu-iotests/197: add testcase for CoR with subclusters
> >>>>>>
> >>>>>>  block.c                      |  7 +++++
> >>>>>>  block/io.c                   | 50 ++++++++++++++++++------------------
> >>>>>>  block/mirror.c               |  8 +++---
> >>>>>>  block/qcow2.c                |  1 +
> >>>>>>  include/block/block-common.h |  5 ++++
> >>>>>>  include/block/block-io.h     |  8 +++---
> >>>>>>  tests/qemu-iotests/197       | 29 +++++++++++++++++++++
> >>>>>>  tests/qemu-iotests/197.out   | 24 +++++++++++++++++
> >>>>>>  8 files changed, 99 insertions(+), 33 deletions(-)
> >>>>>>
> >>>>>
> >>>>> Ping
> >>>>
> >>>> Another ping
> >>>
> >>> Yet another friendly ping
> >>
> >> One more friendly ping
> > 
> > Looks like Stefan gave you an R-B for the series; do we just need an
> > ACK by the block maintainers at this point or is there someone
> > specific you're hoping will review this?
> > 
> > --js
> > 
> 
> Hi John,
> 
> I figure a maintainer's R-b doesn't imply the patches being merged into
> the tree.  Hence I'm waiting for the notice that they actually are merged.
> 
> Please let me know if the process should be different.

Hi Andrey,
Kevin is away right now but seemed happy enough when I mentioned this
series to him, so I have merged this into my own tree:

  https://gitlab.com/stefanha/qemu block

Sorry that your patch series have not been merged in a timely manner.

Stefan

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]