[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2] hmp: allow cpu index for "info lapic"
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH v2] hmp: allow cpu index for "info lapic" |
Date: |
Wed, 19 Jul 2017 10:17:36 -0500 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1 |
On 07/19/2017 10:07 AM, Daniel P. Berrange wrote:
>> It doesn't. Perhaps we should add that as a future libvirt-qemu.so API
>> addition, although it's probably easier to just use QMP than HMP when
>> using 'virsh qemu-monitor-command' if HMP doesn't do what you want.
>
> Or special case the "cpu 1" command - ie notice that it is being
> requested and don't execute 'human-montor-command'. Instead just
> record the CPU index, and use that for future "human-monitor-command"
> invokations, so we get full compat with the (dubious) stateful HMP
> semantics that traditionally existed.
Is 'cpu' (and the followup commands affected by it) the only stateful
HMP command pairing? Is there a way to specify multiple HMP commands in
a single human-monitor-command QMP call?
Indeed, tweaking qemu's human-monitor-command call to track the state
might be cleaner than having libvirt have to tweak API to work around
this wart of HMP.
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3266
Virtualization: qemu.org | libvirt.org
signature.asc
Description: OpenPGP digital signature
- Re: [Qemu-devel] [PATCH v2] hmp: allow cpu index for "info lapic", (continued)
- Re: [Qemu-devel] [PATCH v2] hmp: allow cpu index for "info lapic", Eduardo Habkost, 2017/07/18
- Re: [Qemu-devel] [PATCH v2] hmp: allow cpu index for "info lapic", wang.yi59, 2017/07/19
- Re: [Qemu-devel] [PATCH v2] hmp: allow cpu index for "info lapic", wang.yi59, 2017/07/19
- Re: [Qemu-devel] [PATCH v2] hmp: allow cpu index for "info lapic", Dr. David Alan Gilbert, 2017/07/19
- Re: [Qemu-devel] [PATCH v2] hmp: allow cpu index for "info lapic", Eduardo Habkost, 2017/07/19
- Re: [Qemu-devel] [PATCH v2] hmp: allow cpu index for "info lapic", Dr. David Alan Gilbert, 2017/07/19
- Re: [Qemu-devel] [PATCH v2] hmp: allow cpu index for "info lapic", Eduardo Habkost, 2017/07/19
- Re: [Qemu-devel] [PATCH v2] hmp: allow cpu index for "info lapic", Eric Blake, 2017/07/19
- Re: [Qemu-devel] [PATCH v2] hmp: allow cpu index for "info lapic", Daniel P. Berrange, 2017/07/19
- Re: [Qemu-devel] [PATCH v2] hmp: allow cpu index for "info lapic",
Eric Blake <=
- Re: [Qemu-devel] [PATCH v2] hmp: allow cpu index for "info lapic", Eduardo Habkost, 2017/07/19
- Re: [Qemu-devel] [PATCH v2] hmp: allow cpu index for "info lapic", Dr. David Alan Gilbert, 2017/07/19
- Re: [Qemu-devel] [PATCH v2] hmp: allow cpu index for "info lapic", Eduardo Habkost, 2017/07/19
Re: [Qemu-devel] [PATCH v2] hmp: allow cpu index for "info lapic", wang.yi59, 2017/07/19
Re: [Qemu-devel] [PATCH v2] hmp: allow cpu index for "info lapic", wang.yi59, 2017/07/20