All of lore.kernel.org
 help / color / mirror / Atom feed
From: Hanna Czenczek <hreitz@redhat.com>
To: Fiona Ebner <f.ebner@proxmox.com>, qemu-block@nongnu.org
Cc: qemu-devel@nongnu.org, kwolf@redhat.com, eblake@redhat.com,
	armbru@redhat.com
Subject: Re: [PATCH] block: include 'file' child node name in block device info
Date: Tue, 1 Jul 2025 15:04:14 +0200	[thread overview]
Message-ID: <81cb7080-cf5e-4049-82f8-e009aac93cdc@redhat.com> (raw)
In-Reply-To: <20250630141421.1558724-1-f.ebner@proxmox.com>

On 30.06.25 16:06, Fiona Ebner wrote:
> In combination with using a throttle filter to enforce IO limits for
> a guest device, knowing the 'file' child of a block device can be
> useful. If the throttle filter is only intended for guest IO, block
> jobs should not also be limited by the throttle filter, so the
> block operations need to be done with the 'file' child of the top
> throttle node as the target. In combination with mirroring, the name
> of that child is not fixed.
>
> Another scenario is when unplugging a guest device after mirroring
> below a top throttle node, where the mirror target is added explicitly
> via blockdev-add. After mirroring, the target becomes the new 'file'
> child of the throttle node. For unplugging, both the top throttle node
> and the mirror target need to be deleted, because only implicitly
> added child nodes are deleted automatically, and the current 'file'
> child of the throttle node was explicitly added (as the mirror
> target).

Why not return a map of all children?

Hanna

> Signed-off-by: Fiona Ebner <f.ebner@proxmox.com>
> ---
>   block/qapi.c               | 4 ++++
>   qapi/block-core.json       | 4 ++++
>   tests/qemu-iotests/184.out | 1 +
>   tests/qemu-iotests/191.out | 8 ++++++++
>   tests/qemu-iotests/273.out | 2 ++
>   5 files changed, 19 insertions(+)



  reply	other threads:[~2025-07-01 13:05 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-06-30 14:06 [PATCH] block: include 'file' child node name in block device info Fiona Ebner
2025-07-01 13:04 ` Hanna Czenczek [this message]
2025-07-01 14:18   ` Fiona Ebner
2025-07-01 15:53   ` 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=81cb7080-cf5e-4049-82f8-e009aac93cdc@redhat.com \
    --to=hreitz@redhat.com \
    --cc=armbru@redhat.com \
    --cc=eblake@redhat.com \
    --cc=f.ebner@proxmox.com \
    --cc=kwolf@redhat.com \
    --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.