From: Claudio Fontana <cfontana@suse.de>
To: Roman Bolshakov <r.bolshakov@yadro.com>
Cc: qemu-devel@nongnu.org
Subject: Re: [PATCH for-6.0 0/6] Add HMP/QMP commands to query accelerator
Date: Thu, 19 Nov 2020 16:54:45 +0100 [thread overview]
Message-ID: <e7892cfc-ca17-42e0-94fe-974f642b146c@suse.de> (raw)
In-Reply-To: <20201119154650.GG81070@SPB-NB-133.local>
On 11/19/20 4:46 PM, Roman Bolshakov wrote:
> On Thu, Nov 19, 2020 at 03:41:53PM +0100, Claudio Fontana wrote:
>> On 11/16/20 2:10 PM, Roman Bolshakov wrote:
>>> Management applications have no way to determine if certain accelerator
>>> is available. That complicates discovery of non-KVM accelerators.
>>
>> are we thinking about how to make this future-proof when it comes to
>> modularization efforts, ie, when we get to fully modularized
>> accelerator plugins?
>>
>> Maybe too soon to consider, but still worth mentioning on my side I think.
>>
>
> Hi Claudio,
>
> I'd be happy to do it future-proof if you have something on the mind.
> As far as I understand from the discussion, if we have /machine/accel
> container, we can use QOM to query properties of the container including
> accel name:
> qom-get /machine/accel/type
>
> Thanks,
> Roman
>
My understanding is very limited here, especially when it comes then to how libvirt f.e. uses this,
I wonder how to make sure that libvirt does not query the currently selected accelerator "too early",
ie before it has been finally selected (what in the other series I called the INIT_ACCEL_CPU time),
and how to query "available/supported" accelerators in a way that accounts for the fact that the plugin might not be loaded or available.
I guess this is a larger question about how libvirt detects modularized features in QEMU, when those features are (or are not) present only in an external binary plugin.
Ciao,
Claudio
prev parent reply other threads:[~2020-11-19 15:56 UTC|newest]
Thread overview: 45+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-11-16 13:10 [PATCH for-6.0 0/6] Add HMP/QMP commands to query accelerator Roman Bolshakov
2020-11-16 13:10 ` [PATCH for-6.0 1/6] qapi: Add query-accel command Roman Bolshakov
2020-11-16 16:20 ` Eric Blake
2020-11-16 18:56 ` Roman Bolshakov
2020-11-16 21:13 ` Eduardo Habkost
2020-11-17 8:51 ` Markus Armbruster
2020-11-18 1:19 ` Roman Bolshakov
2020-11-18 8:36 ` Markus Armbruster
2020-11-18 9:21 ` Paolo Bonzini
2020-11-18 13:08 ` Markus Armbruster
2020-11-18 13:46 ` Paolo Bonzini
2020-11-18 14:45 ` Markus Armbruster
2020-11-18 14:54 ` Paolo Bonzini
2020-11-18 14:00 ` Roman Bolshakov
2020-11-18 11:28 ` Kevin Wolf
2020-11-18 11:56 ` Daniel P. Berrangé
2020-11-18 13:53 ` Markus Armbruster
2020-11-18 15:45 ` Eduardo Habkost
2020-11-18 15:56 ` Eric Blake
2020-11-18 16:23 ` Eduardo Habkost
2020-11-19 13:17 ` Markus Armbruster
2020-11-30 17:05 ` Philippe Mathieu-Daudé
2020-11-16 13:10 ` [PATCH for-6.0 2/6] qapi: Rename KvmInfo to AccelInfo Roman Bolshakov
2020-11-27 10:40 ` Dr. David Alan Gilbert
2020-11-27 12:08 ` Markus Armbruster
2020-11-16 13:10 ` [PATCH for-6.0 3/6] qapi: Use qmp_query_accel() in qmp_query_kvm() Roman Bolshakov
2020-11-16 13:10 ` [PATCH for-6.0 4/6] softmmu: Remove kvm_available() Roman Bolshakov
2020-11-16 13:10 ` [PATCH for-6.0 5/6] hmp: Add 'info accel' command Roman Bolshakov
2020-11-27 10:39 ` Dr. David Alan Gilbert
2020-11-16 13:10 ` [PATCH for-6.0 6/6] qapi: Deprecate 'query-kvm' Roman Bolshakov
2020-11-27 10:50 ` Daniel P. Berrangé
2020-11-27 11:21 ` Peter Krempa
2020-11-27 11:45 ` Roman Bolshakov
2020-11-27 12:18 ` Peter Krempa
2020-11-27 15:44 ` Markus Armbruster
2020-11-27 16:30 ` Peter Krempa
2020-11-30 9:21 ` Markus Armbruster
2020-11-30 10:09 ` Peter Krempa
2020-11-30 16:03 ` Markus Armbruster
2020-11-30 15:30 ` Eric Blake
2020-11-27 15:53 ` Daniel P. Berrangé
2020-11-27 16:35 ` Peter Krempa
2020-11-19 14:41 ` [PATCH for-6.0 0/6] Add HMP/QMP commands to query accelerator Claudio Fontana
2020-11-19 15:46 ` Roman Bolshakov
2020-11-19 15:54 ` Claudio Fontana [this message]
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=e7892cfc-ca17-42e0-94fe-974f642b146c@suse.de \
--to=cfontana@suse.de \
--cc=qemu-devel@nongnu.org \
--cc=r.bolshakov@yadro.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 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).