All of lore.kernel.org
 help / color / mirror / Atom feed
From: Eduardo Habkost <ehabkost@redhat.com>
To: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Cc: "Daniel P. Berrangé" <berrange@redhat.com>,
	"Richard Henderson" <richard.henderson@linaro.org>,
	"Markus Armbruster" <armbru@redhat.com>,
	qemu-devel@nongnu.org,
	"Valeriy Vdovin" <valeriy.vdovin@virtuozzo.com>,
	"Denis Lunev" <den@openvz.org>,
	"Paolo Bonzini" <pbonzini@redhat.com>
Subject: Re: [PATCH v3] qapi: introduce 'query-cpu-model-cpuid' action
Date: Mon, 19 Apr 2021 16:23:36 -0400	[thread overview]
Message-ID: <20210419202336.shf3yo7lmr7tmzvp@habkost.net> (raw)
In-Reply-To: <15924dba-1618-0b7e-fbc3-42e4f02d8176@virtuozzo.com>

On Mon, Mar 29, 2021 at 03:41:34PM +0300, Vladimir Sementsov-Ogievskiy wrote:
> 29.03.2021 14:48, Daniel P. Berrangé wrote:
[...]
> > > > There's feels like there's a lot of conceptual overlap with the
> > > > query-cpu-model-expansion command. That reports in a arch independant
> > > > format, but IIUC the property data it returns can be mapped into
> > > > CPUID leaf values. Is it not possible for you to use this existing
> > > > command and maintain a mapping of property names -> CPUID leaves ?
> > > As already stated in the use-case description above, having this method
> > > around, helps us in a way that we can just take values and return them
> > > to containers. QEMU code already does a great job, generating CPUID
> > > responses, we don't want to do the same in our own code.
> > 
> > This is asking QEMU to maintain a new QAPI command which does not appear
> > to have a use case / benefit for QEMU mgmt. It isn't clear to me that
> > this should be considered in scope for QMP.
> > 
> 
> Hmm. On the other hand,
> 
> 1. The command just exports some information, like a lot of other qmp query- commands, it doesn't look as something alien in the QEMU interface.
> 
> 2. We do have a use-case. Not a VM use-case, but a use-case of cpu handling subsystem.
> 
> Isn't it enough?
> 
> We want to handle cpu configurations in a compatible with QEMU way. The simplest thing for it is just generate needed information with help of QEMU. Note, that's not the only usage of QEMU binary for not-VM-running. QEMU binary may be used for different block-jobs and manipulating bitmaps in disk images (yes, now we also have qemu-storage-daemon, but still).
> 
> Do you have an idea how our task could be solved an a better way?

The new command would also be useful for writing automated tests
for the x86 CPUID compatibility code.  I don't object to its
inclusion.

-- 
Eduardo



  reply	other threads:[~2021-04-19 20:47 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-03-26 17:30 [PATCH v3] qapi: introduce 'query-cpu-model-cpuid' action Valeriy Vdovin
2021-03-29  9:20 ` Daniel P. Berrangé
2021-03-29 11:21   ` Valeriy Vdovin
2021-03-29 11:48     ` Daniel P. Berrangé
2021-03-29 12:41       ` Vladimir Sementsov-Ogievskiy
2021-04-19 20:23         ` Eduardo Habkost [this message]
2021-04-14 13:56 ` Vladimir Sementsov-Ogievskiy

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=20210419202336.shf3yo7lmr7tmzvp@habkost.net \
    --to=ehabkost@redhat.com \
    --cc=armbru@redhat.com \
    --cc=berrange@redhat.com \
    --cc=den@openvz.org \
    --cc=pbonzini@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=richard.henderson@linaro.org \
    --cc=valeriy.vdovin@virtuozzo.com \
    --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.