All of lore.kernel.org
 help / color / mirror / Atom feed
From: Stefano Garzarella <sgarzare@redhat.com>
To: Peter Lieven <pl@kamp.de>
Cc: "Kevin Wolf" <kwolf@redhat.com>,
	"Daniel P. Berrangé" <berrange@redhat.com>,
	"qemu block" <qemu-block@nongnu.org>,
	"Christian Theune" <ct@flyingcircus.io>,
	qemu-devel@nongnu.org, "Paolo Bonzini" <pbonzini@redhat.com>,
	"Ilya Dryomov" <idryomov@gmail.com>,
	"Max Reitz" <mreitz@redhat.com>
Subject: Re: [PATCH V2 for-6.2 0/2] fixes for bdrv_co_block_status
Date: Tue, 25 Jan 2022 11:09:42 +0100	[thread overview]
Message-ID: <20220125100942.caiweqllt53szoto@steredhat> (raw)
In-Reply-To: <cc75ea76-fbc6-1f88-6d07-09cc65a06d60@kamp.de>

On Thu, Jan 20, 2022 at 10:19:27AM +0100, Peter Lieven wrote:
>Am 19.01.22 um 15:57 schrieb Stefano Garzarella:
>> On Fri, Jan 14, 2022 at 11:58:40AM +0100, Ilya Dryomov wrote:
>>> On Thu, Jan 13, 2022 at 3:44 PM Peter Lieven <pl@kamp.de> wrote:
>>>>
>>>> V1->V2:
>>>>  Patch 1: Treat a hole just like an unallocated area. [Ilya]
>>>>  Patch 2: Apply workaround only for pre-Quincy librbd versions and
>>>>           ensure default striping and non child images. [Ilya]
>>>>
>>>> Peter Lieven (2):
>>>>   block/rbd: fix handling of holes in .bdrv_co_block_status
>>>>   block/rbd: workaround for ceph issue #53784
>>>>
>>>>  block/rbd.c | 52 +++++++++++++++++++++++++++++++++++++++++++++-------
>>>>  1 file changed, 45 insertions(+), 7 deletions(-)
>>>>
>>>> --
>>>> 2.25.1
>>>>
>>>>
>>>
>>> These patches have both "for-6.2" in the subject and
>>> Cc: qemu-stable@nongnu.org in the description, which is a little
>>> confusing.  Just want to clarify that they should go into master
>>> and be backported to 6.2.
>>
>> Yeah, a bit confusing. These are for 7.0, so @Kevin can these patches go with your tree?
>
>
>Yes, sorry, my fault. It should be 7.0

Don't worry :-)

What about sending a v3 fixing the version tag (I think you can just 
remove for-6.2), the extra space in the comment, and the Fixes tag on 
patch 2?

If you will send v3, remember to report the R-b/T-b tags received in 
this version from me and Ilya.

Thanks,
Stefano



  reply	other threads:[~2022-01-25 10:16 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-01-13 14:44 [PATCH V2 for-6.2 0/2] fixes for bdrv_co_block_status Peter Lieven
2022-01-13 14:44 ` [PATCH V2 for-6.2 1/2] block/rbd: fix handling of holes in .bdrv_co_block_status Peter Lieven
2022-01-13 14:44 ` [PATCH V2 for-6.2 2/2] block/rbd: workaround for ceph issue #53784 Peter Lieven
2022-01-14 10:57   ` Ilya Dryomov
2022-01-17 14:23   ` Stefano Garzarella
2022-01-14 10:58 ` [PATCH V2 for-6.2 0/2] fixes for bdrv_co_block_status Ilya Dryomov
2022-01-19 14:57   ` Stefano Garzarella
2022-01-20  9:19     ` Peter Lieven
2022-01-25 10:09       ` Stefano Garzarella [this message]
2022-02-01 14:39 ` Kevin Wolf
2022-02-03 11:42   ` Peter Lieven
2022-02-03 12:14     ` Stefano Garzarella

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20220125100942.caiweqllt53szoto@steredhat \
    --to=sgarzare@redhat.com \
    --cc=berrange@redhat.com \
    --cc=ct@flyingcircus.io \
    --cc=idryomov@gmail.com \
    --cc=kwolf@redhat.com \
    --cc=mreitz@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=pl@kamp.de \
    --cc=qemu-block@nongnu.org \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.