From: Kevin Wolf <kwolf@redhat.com>
To: Anton Kuchin <antonkuchin@yandex-team.ru>
Cc: qemu-devel@nongnu.org,
"Dr. David Alan Gilbert" <dgilbert@redhat.com>,
qemu-block@nongnu.org, Max Reitz <mreitz@redhat.com>,
Eric Blake <eblake@redhat.com>,
Markus Armbruster <armbru@redhat.com>,
Evgeny Yakovlev <wrfsh@yandex-team.ru>
Subject: Re: [Qemu-devel] [PATCH 0/2] block: add blk_lookup() for getting device by node_name
Date: Mon, 28 Jan 2019 15:47:20 +0100 [thread overview]
Message-ID: <20190128144720.GB5756@localhost.localdomain> (raw)
In-Reply-To: <20190128142748.29140-1-antonkuchin@yandex-team.ru>
Am 28.01.2019 um 15:27 hat Anton Kuchin geschrieben:
> Some HMP and QMP commands are targeting BlockBackend but
> for hotplugged devices name of BB is deprecated, instead
> name of root BlockDriverState is set. These patches add
> functions to search BB by attached root BDS name.
>
> This approach isn't perfect, but I couldn't invent a better
> one and I belive it's more convinient than accessing BB
> by QOM path.
There could be more than one BlockBackend attached to a single node, so
this approach is simply wrong.
I think the qdev ID is actually not only a pretty convenient way, but in
fact the only logical way to address a guest device (and BlockBackends
that can be accessed by the monitor are essentially a part of the guest
device).
Does your series allow to perform any operation that isn't possible
currently? If so, it's probably this operation that needs to be fixed to
either accept node names (if it's a backend operation) or a device ID
(if it's a frontend operation).
Kevin
next prev parent reply other threads:[~2019-01-28 14:47 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-01-28 14:27 [Qemu-devel] [PATCH 0/2] block: add blk_lookup() for getting device by node_name Anton Kuchin
2019-01-28 14:27 ` [Qemu-devel] [PATCH 1/2] block: add functions to search BlockBackend by root BDS name Anton Kuchin
2019-01-28 14:27 ` [Qemu-devel] [PATCH 2/2] block: migrate callers from blk_by_name to blk_lookup Anton Kuchin
2019-01-28 14:47 ` Kevin Wolf [this message]
2019-01-28 16:53 ` [Qemu-devel] [PATCH 0/2] block: add blk_lookup() for getting device by node_name Anton Kuchin
2019-01-28 18:02 ` 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=20190128144720.GB5756@localhost.localdomain \
--to=kwolf@redhat.com \
--cc=antonkuchin@yandex-team.ru \
--cc=armbru@redhat.com \
--cc=dgilbert@redhat.com \
--cc=eblake@redhat.com \
--cc=mreitz@redhat.com \
--cc=qemu-block@nongnu.org \
--cc=qemu-devel@nongnu.org \
--cc=wrfsh@yandex-team.ru \
/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).