[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 0/3] block: align CoR requests to subclusters
From: |
Stefan Hajnoczi |
Subject: |
Re: [PATCH v2 0/3] block: align CoR requests to subclusters |
Date: |
Thu, 24 Aug 2023 10:32:47 -0400 |
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
signature.asc
Description: PGP signature