All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Andreas Färber" <afaerber@suse.de>
To: "Gonglei (Arei)" <arei.gonglei@huawei.com>,
	"Li, Guang" <lig.fnst@cn.fujitsu.com>,
	"qemu-devel@nongnu.org" <qemu-devel@nongnu.org>
Cc: "kwolf@redhat.com" <kwolf@redhat.com>,
	"chenliang (T)" <chenliang88@huawei.com>,
	"peter.crosthwaite@xilinx.com" <peter.crosthwaite@xilinx.com>,
	"Huangweidong (C)" <weidong.huang@huawei.com>,
	"ehabkost@redhat.com" <ehabkost@redhat.com>,
	"mst@redhat.com" <mst@redhat.com>,
	"aik@ozlabs.ru" <aik@ozlabs.ru>, "Hu, Tao" <hutao@cn.fujitsu.com>,
	Luonengjun <luonengjun@huawei.com>,
	"armbru@redhat.com" <armbru@redhat.com>,
	"agraf@suse.de" <agraf@suse.de>,
	"hani@linux.com" <hani@linux.com>,
	"kraxel@redhat.com" <kraxel@redhat.com>,
	"aliguori@amazon.com" <aliguori@amazon.com>,
	"imammedo@redhat.com" <imammedo@redhat.com>,
	"lcapitulino@redhat.com" <lcapitulino@redhat.com>,
	"pbonzini@redhat.com" <pbonzini@redhat.com>,
	"Huangpeng (Peter)" <peter.huangpeng@huawei.com>,
	"akong@redhat.com" <akong@redhat.com>,
	"stefanha@redhat.com" <stefanha@redhat.com>
Subject: Re: [Qemu-devel] [PATCH v8 00/30] modify boot order of guest, and take effect after rebooting
Date: Thu, 11 Sep 2014 14:46:33 +0200	[thread overview]
Message-ID: <541199A9.8020304@suse.de> (raw)
In-Reply-To: <33183CC9F5247A488A2544077AF1902086DC95A2@SZXEMA503-MBS.china.huawei.com>

Am 11.09.2014 um 07:58 schrieb Gonglei (Arei):
>>> ./qemu-system-x86_64 -enable-kvm -m 4096 -smp 4 -name redhat6.2 -drive \
>>> file=/home/win7_32_2U,if=none,id=drive-ide0-0-0 -device ide-hd,bus=ide.0,\
>>> unit=0,drive=drive-ide0-0-0,id=ide0-0-0,bootindex=1 -drive \
>>> file=/home/rhel-server-7.0-x86_64-dvd.iso,if=none,id=drive-ide0-0-1 \ -device
>>> ide-cd,bus=ide.0,unit=1,drive=drive-ide0-0-1,id=ide0-0-1,bootindex=4 \ -vnc
>>> 0.0.0.0:10 -netdev type=user,id=net0 -device
>>> virtio-net-pci,netdev=net0,bootindex=3,id=nic1 \ -drive
>>> file=/mnt/sdb/gonglei/image/virtio-win-1.5.3.vfd,if=none,id=drive-fdc0-0-0,for
>>> mat=raw \ -device isa-fdc,driveA=drive-fdc0-0-0,bootindexA=5,id=floppy1
>> -qmp
>>> unix:/tmp/qmp,server,nowait \ -monitor stdio -netdev type=user,id=net1
>>> -device e1000,netdev=net1,bootindex=2,id=nic \ -boot menu=on -device
>>> virtio-scsi-pci,id=scsi0 -drive file=/home/suse11_sp3_32,if=none,\
>>> id=drive-scsi0-0-0-0,format=raw,cache=none,aio=native \ -device
>>>
>> scsi-hd,bus=scsi0.0,channel=0,scsi-id=0,lun=0,drive=drive-scsi0-0-0-0,id=scsi0-
>>> 0-0-0,bootindex=8
>>> QEMU 2.1.50 monitor - type 'help' for more information
>>> (qemu) qom-get nic1 bootindex
>>> 3 (0x3)
>>> (qemu) qom-set nic1 bootindex 3
>>> The bootindex 3 has already been used
>>> (qemu) qom-set nic1 bootindex 0
>>> (qemu) qom-set floppy1 bootindexA 3
>>> (qemu) system_reset
>>> (qemu) qom-get nic1 bootindex
>>> 0 (0x0)
>>> (qemu) qom-get scsi0-0-0-0 bootindex
>>> 8 (0x8)
>>> (qemu) qom-set scsi0-0-0-0 bootindex 0
>>> The bootindex 0 has already been used
>>> (qemu) qom-set nic1 bootindex -1
>>> (qemu) qom-set scsi0-0-0-0 bootindex 0
>>> (qemu) qom-get scsi0-0-0-0 bootindex
>>> 0 (0x0)
>>> (qemu)
>>>
>>
>> Hmm..., seems we also need something like this:
>> (qemu) qom-get bootindex
>> dev0 bootindex 0
>> dev1 bootindex 1
>> dev2 bootindex 2
>> ...
> I don't think so. Qom-get interface is ready-to-wear, we must
> provide both QOM path and QOM property name. This interface
> is not added by me. Thanks. :)

Yeah, I don't think we need wildcards for qom-get here. I wouldn't
oppose a new HMP command though if someone really sees a need.

> BTW, you can look at with Andreas's patch series:
> http://thread.gmane.org/gmane.comp.emulators.qemu/271513
> 
> which haven't applied in mater tree.

Sigh, many things on the to-do list... :(

Regards,
Andreas

-- 
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg

  parent reply	other threads:[~2014-09-11 12:46 UTC|newest]

Thread overview: 35+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-09-10 12:03 [Qemu-devel] [PATCH v8 00/30] modify boot order of guest, and take effect after rebooting arei.gonglei
2014-09-10 12:03 ` [Qemu-devel] [PATCH v8 01/30] bootdevice: move bootdevice related code to new file bootdevice.c arei.gonglei
2014-09-10 12:03 ` [Qemu-devel] [PATCH v8 02/30] bootindex: add check bootindex function arei.gonglei
2014-09-10 12:03 ` [Qemu-devel] [PATCH v8 03/30] bootindex: add del_boot_device_path function arei.gonglei
2014-09-10 12:03 ` [Qemu-devel] [PATCH v8 04/30] fw_cfg: add fw_cfg_machine_reset function arei.gonglei
2014-09-10 12:03 ` [Qemu-devel] [PATCH v8 05/30] bootindex: rework add_boot_device_path function arei.gonglei
2014-09-10 12:04 ` [Qemu-devel] [PATCH v8 06/30] bootindex: support to set a existent device's bootindex to -1 arei.gonglei
2014-09-10 12:04 ` [Qemu-devel] [PATCH v8 07/30] bootindex: add a setter/getter functions wrapper for bootindex property arei.gonglei
2014-09-10 12:04 ` [Qemu-devel] [PATCH v8 08/30] virtio-net: add bootindex to qom property arei.gonglei
2014-09-10 12:04 ` [Qemu-devel] [PATCH v8 09/30] e1000: " arei.gonglei
2014-09-10 12:04 ` [Qemu-devel] [PATCH v8 10/30] eepro100: " arei.gonglei
2014-09-10 12:04 ` [Qemu-devel] [PATCH v8 11/30] ne2000: " arei.gonglei
2014-09-10 12:04 ` [Qemu-devel] [PATCH v8 12/30] pcnet: " arei.gonglei
2014-09-10 12:04 ` [Qemu-devel] [PATCH v8 13/30] rtl8139: " arei.gonglei
2014-09-10 12:04 ` [Qemu-devel] [PATCH v8 14/30] spapr_lian: " arei.gonglei
2014-09-10 12:04 ` [Qemu-devel] [PATCH v8 15/30] vmxnet3: " arei.gonglei
2014-09-10 12:04 ` [Qemu-devel] [PATCH v8 16/30] usb-net: " arei.gonglei
2014-09-10 12:04 ` [Qemu-devel] [PATCH v8 17/30] net: remove bootindex property from qdev to qom arei.gonglei
2014-09-10 12:04 ` [Qemu-devel] [PATCH v8 18/30] virtio-net: alias bootindex property explicitly for virt-net-pci/ccw/s390 arei.gonglei
2014-09-10 12:04 ` [Qemu-devel] [PATCH v8 19/30] host-libusb: remove bootindex property from qdev to qom arei.gonglei
2014-09-10 12:04 ` [Qemu-devel] [PATCH v8 20/30] pci-assign: " arei.gonglei
2014-09-10 12:04 ` [Qemu-devel] [PATCH v8 21/30] vfio: " arei.gonglei
2014-09-10 12:04 ` [Qemu-devel] [PATCH v8 22/30] redirect: " arei.gonglei
2014-09-10 12:04 ` [Qemu-devel] [PATCH v8 23/30] isa-fdc: remove bootindexA/B " arei.gonglei
2014-09-10 12:04 ` [Qemu-devel] [PATCH v8 24/30] scsi: add bootindex to qom property arei.gonglei
2014-09-10 12:04 ` [Qemu-devel] [PATCH v8 25/30] ide: " arei.gonglei
2014-09-10 12:04 ` [Qemu-devel] [PATCH v8 26/30] virtio-blk: " arei.gonglei
2014-09-10 12:04 ` [Qemu-devel] [PATCH v8 27/30] block: remove bootindex property from qdev to qom arei.gonglei
2014-09-10 12:04 ` [Qemu-devel] [PATCH v8 28/30] ide: add calling add_boot_device_patch in bootindex setter function arei.gonglei
2014-09-10 12:04 ` [Qemu-devel] [PATCH v8 29/30] bootindex: move calling add_boot_device_patch to " arei.gonglei
2014-09-10 12:04 ` [Qemu-devel] [PATCH v8 30/30] bootindex: delete bootindex when device is removed arei.gonglei
     [not found] ` <534C5107B281854290643468BBAAD41F1CC845@G08CNEXMBPEKD01.g08.fujitsu.local>
2014-09-11  5:58   ` [Qemu-devel] [PATCH v8 00/30] modify boot order of guest, and take effect after rebooting Gonglei (Arei)
     [not found]     ` <534C5107B281854290643468BBAAD41F1CD4AA@G08CNEXMBPEKD01.g08.fujitsu.local>
2014-09-11  7:55       ` Gonglei (Arei)
2014-09-11 12:46     ` Andreas Färber [this message]
2014-09-12  6:42       ` Gonglei (Arei)

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=541199A9.8020304@suse.de \
    --to=afaerber@suse.de \
    --cc=agraf@suse.de \
    --cc=aik@ozlabs.ru \
    --cc=akong@redhat.com \
    --cc=aliguori@amazon.com \
    --cc=arei.gonglei@huawei.com \
    --cc=armbru@redhat.com \
    --cc=chenliang88@huawei.com \
    --cc=ehabkost@redhat.com \
    --cc=hani@linux.com \
    --cc=hutao@cn.fujitsu.com \
    --cc=imammedo@redhat.com \
    --cc=kraxel@redhat.com \
    --cc=kwolf@redhat.com \
    --cc=lcapitulino@redhat.com \
    --cc=lig.fnst@cn.fujitsu.com \
    --cc=luonengjun@huawei.com \
    --cc=mst@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=peter.crosthwaite@xilinx.com \
    --cc=peter.huangpeng@huawei.com \
    --cc=qemu-devel@nongnu.org \
    --cc=stefanha@redhat.com \
    --cc=weidong.huang@huawei.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.