All of lore.kernel.org
 help / color / mirror / Atom feed
From: Eric Blake <eblake@redhat.com>
To: qemu-devel@nongnu.org
Cc: Kevin Wolf <kwolf@redhat.com>,
	Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>,
	"open list:raw" <qemu-block@nongnu.org>,
	Max Reitz <mreitz@redhat.com>
Subject: [PULL 07/14] block/raw-format: implement .bdrv_cancel_in_flight handler
Date: Fri, 12 Feb 2021 14:16:12 -0600	[thread overview]
Message-ID: <20210212201619.1388255-8-eblake@redhat.com> (raw)
In-Reply-To: <20210212201619.1388255-1-eblake@redhat.com>

From: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>

We are going to cancel in-flight requests on mirror nbd target on job
cancel. Still nbd is often used not directly but as raw-format child.
So, add pass-through handler here.

Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-Id: <20210205163720.887197-4-vsementsov@virtuozzo.com>
Signed-off-by: Eric Blake <eblake@redhat.com>
---
 block/raw-format.c | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/block/raw-format.c b/block/raw-format.c
index 42ec50802bcd..7717578ed6ab 100644
--- a/block/raw-format.c
+++ b/block/raw-format.c
@@ -575,6 +575,11 @@ static const char *const raw_strong_runtime_opts[] = {
     NULL
 };

+static void raw_cancel_in_flight(BlockDriverState *bs)
+{
+    bdrv_cancel_in_flight(bs->file->bs);
+}
+
 BlockDriver bdrv_raw = {
     .format_name          = "raw",
     .instance_size        = sizeof(BDRVRawState),
@@ -608,6 +613,7 @@ BlockDriver bdrv_raw = {
     .bdrv_has_zero_init   = &raw_has_zero_init,
     .strong_runtime_opts  = raw_strong_runtime_opts,
     .mutable_opts         = mutable_opts,
+    .bdrv_cancel_in_flight = raw_cancel_in_flight,
 };

 static void bdrv_raw_init(void)
-- 
2.30.1



  parent reply	other threads:[~2021-02-12 20:28 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-02-12 20:16 [PULL 00/14] NBD patches through 2021-02-12 Eric Blake
2021-02-12 20:16 ` [PULL 01/14] qemu-nbd: Use SOMAXCONN for socket listen() backlog Eric Blake
2021-02-12 20:16 ` [PULL 02/14] qemu-nbd: Permit --shared=0 for unlimited clients Eric Blake
2021-02-12 20:16 ` [PULL 03/14] iotests/210: Fix reference output Eric Blake
2021-02-12 20:16 ` [PULL 04/14] io: error_prepend() in qio_channel_readv_full_all() causes segfault Eric Blake
2021-02-12 20:16 ` [PULL 05/14] block: add new BlockDriver handler: bdrv_cancel_in_flight Eric Blake
2021-02-12 20:16 ` [PULL 06/14] block/nbd: implement .bdrv_cancel_in_flight Eric Blake
2021-02-12 20:16 ` Eric Blake [this message]
2021-02-12 20:16 ` [PULL 08/14] job: add .cancel handler for the driver Eric Blake
2021-02-12 20:16 ` [PULL 09/14] block/mirror: implement .cancel job handler Eric Blake
2021-02-12 20:16 ` [PULL 10/14] iotests/264: move to python unittest Eric Blake
2021-02-12 20:16 ` [PULL 11/14] iotests.py: qemu_nbd_popen: remove pid file after use Eric Blake
2021-02-12 20:16 ` [PULL 12/14] iotests/264: add mirror-cancel test-case Eric Blake
2021-02-12 20:16 ` [PULL 13/14] block/backup: implement .cancel job handler Eric Blake
2021-02-12 20:16 ` [PULL 14/14] iotests/264: add backup-cancel test-case Eric Blake
2021-02-13 18:16 ` [PULL 00/14] NBD patches through 2021-02-12 Peter Maydell

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=20210212201619.1388255-8-eblake@redhat.com \
    --to=eblake@redhat.com \
    --cc=kwolf@redhat.com \
    --cc=mreitz@redhat.com \
    --cc=qemu-block@nongnu.org \
    --cc=qemu-devel@nongnu.org \
    --cc=vsementsov@virtuozzo.com \
    /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.