qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Kevin Wolf <kwolf@redhat.com>
To: Michael Tokarev <mjt@tls.msk.ru>
Cc: Andrey Drobyshev <andrey.drobyshev@virtuozzo.com>,
	qemu-block@nongnu.org, qemu-devel@nongnu.org, hreitz@redhat.com,
	eblake@redhat.com, den@virtuozzo.com
Subject: Re: [PATCH v3 1/8] qemu-img: rebase: stop when reaching EOF of old backing file
Date: Thu, 2 Nov 2023 11:46:57 +0100	[thread overview]
Message-ID: <ZUN+IYdAGZg2D7pN@redhat.com> (raw)
In-Reply-To: <0834e809-6255-42e7-b72c-e0a21dd15901@tls.msk.ru>

Am 01.11.2023 um 18:51 hat Michael Tokarev geschrieben:
> 01.11.2023 18:38, Andrey Drobyshev wrote:
> > Hi Michael,
> > 
> > Since this series is already merged in master, I'm not sure whether it's
> > necessary to forward this particular patch to qemu-stable, or it should
> > rather be cherry-picked to -stable by one of the block maintainers.
> 
> It's been my job lately to pick something up for stable, once it gets
> applied to master.  But it is usually the patch author or subsystem
> maintainer to mark a change as a candidate for -stable, - I can't decide
> about every change out there, since I don't have enough expertise in
> every area.  You Cc: qemu-stable@ and I pick it up.  I do look at stuff
> being applied to master from time to time though, and ask if I see
> something which might be worth to pick, as in this case.

I didn't even think about it for this patch, but yes, it sounds like a
candidate for stable to me.

> BTW, there's another change in this series which might be a good candidate
> too, - "qemu-img: rebase: use backing files' BlockBackend for buffer
> alignment".  Once again, I dunno if it's worth to pick it up or not,
> it's basically up to you to decide.  Basically, you understand much
> better what the implications are and if the change fixes actual bug.

It's an optimisation, I wouldn't pick that one.

Kevin



  reply	other threads:[~2023-11-02 10:47 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-09-19 16:57 [PATCH v3 0/8] qemu-img: rebase: add compression support Andrey Drobyshev via
2023-09-19 16:57 ` [PATCH v3 1/8] qemu-img: rebase: stop when reaching EOF of old backing file Andrey Drobyshev via
2023-11-01  9:50   ` Michael Tokarev
2023-11-01 15:38     ` Andrey Drobyshev
2023-11-01 17:51       ` Michael Tokarev
2023-11-02 10:46         ` Kevin Wolf [this message]
2023-09-19 16:57 ` [PATCH v3 2/8] qemu-iotests: 024: add rebasing test case for overlay_size > backing_size Andrey Drobyshev via
2023-09-19 16:57 ` [PATCH v3 3/8] qemu-img: rebase: use backing files' BlockBackend for buffer alignment Andrey Drobyshev via
2023-09-19 16:58 ` [PATCH v3 4/8] qemu-img: add chunk size parameter to compare_buffers() Andrey Drobyshev via
2023-10-17 19:30   ` Kevin Wolf
2023-09-19 16:58 ` [PATCH v3 5/8] qemu-img: rebase: avoid unnecessary COW operations Andrey Drobyshev via
2023-09-19 16:58 ` [PATCH v3 6/8] iotests/{024, 271}: add testcases for qemu-img rebase Andrey Drobyshev via
2023-09-19 16:58 ` [PATCH v3 7/8] qemu-img: add compression option to rebase subcommand Andrey Drobyshev via
2023-09-19 16:58 ` [PATCH v3 8/8] iotests: add tests for "qemu-img rebase" with compression Andrey Drobyshev via
2023-10-02  6:35 ` [PATCH v3 0/8] qemu-img: rebase: add compression support Andrey Drobyshev
2023-10-16  8:34   ` Andrey Drobyshev
2023-10-17 20:08 ` Kevin Wolf

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=ZUN+IYdAGZg2D7pN@redhat.com \
    --to=kwolf@redhat.com \
    --cc=andrey.drobyshev@virtuozzo.com \
    --cc=den@virtuozzo.com \
    --cc=eblake@redhat.com \
    --cc=hreitz@redhat.com \
    --cc=mjt@tls.msk.ru \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).