[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC PATCH 1/5] bootindex: add *_boot_device_path funct
From: |
Gonglei (Arei) |
Subject: |
Re: [Qemu-devel] [RFC PATCH 1/5] bootindex: add *_boot_device_path function |
Date: |
Wed, 9 Jul 2014 01:03:49 +0000 |
> -----Original Message-----
> From: Amos Kong [mailto:address@hidden
> Sent: Tuesday, July 08, 2014 10:55 PM
> To: Gonglei (Arei)
> Cc: chenliang (T); address@hidden; address@hidden;
> address@hidden; address@hidden; address@hidden;
> address@hidden; address@hidden; address@hidden;
> address@hidden; address@hidden; address@hidden;
> address@hidden; address@hidden; address@hidden;
> address@hidden; address@hidden; address@hidden;
> address@hidden; address@hidden; address@hidden;
> address@hidden; Huangweidong (C); Luonengjun; Huangpeng (Peter)
> Subject: Re: [RFC PATCH 1/5] bootindex: add *_boot_device_path function
>
> On Tue, Jul 08, 2014 at 01:22:53PM +0000, Gonglei (Arei) wrote:
> > > -----Original Message-----
> > > From: chenliang (T)
> > > Sent: Tuesday, July 08, 2014 7:03 PM
> > > To: Amos Kong
> > > Cc: Gonglei (Arei); address@hidden; address@hidden;
> > > address@hidden; address@hidden; address@hidden;
> > > address@hidden; address@hidden; address@hidden;
> > > address@hidden; address@hidden; address@hidden;
> > > address@hidden; address@hidden; address@hidden;
> > > address@hidden; address@hidden; address@hidden;
> > > address@hidden; address@hidden; address@hidden;
> > > address@hidden; Huangweidong (C); Luonengjun; Huangpeng (Peter)
> > > Subject: Re: [RFC PATCH 1/5] bootindex: add *_boot_device_path function
> > >
> > > On 2014/7/8 16:33, Amos Kong wrote:
> > >
> > > > On Mon, Jul 07, 2014 at 05:10:57PM +0800, address@hidden
> > > wrote:
> > > >> From: Chenliang <address@hidden>
> > > >>
> > > >> Add del_boot_device_path and modify_boot_device_path. Device should
> > > >> be removed from boot device list by del_boot_device_path when
> device
> > > >> hotplug. modify_boot_device_path is used to modify deviceboot order.
> > > >
> > > > s/hotplug/is unhotplugged/
> > > >
> > > > same issue in commitlog of patch 3/5
> > > >
> >
> > Yep, thanks!
> >
> > > >> Signed-off-by: Chenliang <address@hidden>
> > > >> Signed-off-by: Gonglei <address@hidden>
> > > >> ---
> > > >> include/sysemu/sysemu.h | 4 ++++
> > > >> vl.c | 55
> > > +++++++++++++++++++++++++++++++++++++++++++++++++
> > > >> 2 files changed, 59 insertions(+)
> > > >>
> > > >> diff --git a/include/sysemu/sysemu.h b/include/sysemu/sysemu.h
> > > >> index 285c45b..38ef1cd 100644
> > > >> --- a/include/sysemu/sysemu.h
> > > >> +++ b/include/sysemu/sysemu.h
> > > >> @@ -204,6 +204,10 @@ void usb_info(Monitor *mon, const QDict
> *qdict);
> > > >>
> > > >> void add_boot_device_path(int32_t bootindex, DeviceState *dev,
> > > >> const char *suffix);
> > > >> +void del_boot_device_path(int32_t bootindex, DeviceState *dev,
> > > >> + const char *suffix);
> > > >> +void modify_boot_device_path(int32_t bootindex, DeviceState *dev,
> > > >> + const char *suffix);
> > > >> char *get_boot_devices_list(size_t *size, bool ignore_suffixes);
> > > >>
> > > >> DeviceState *get_boot_device(uint32_t position);
> > > >> diff --git a/vl.c b/vl.c
> > > >> index a1686ef..6264e11 100644
> > > >> --- a/vl.c
> > > >> +++ b/vl.c
> > > >> @@ -1247,6 +1247,61 @@ void add_boot_device_path(int32_t
> bootindex,
> > > DeviceState *dev,
> > > >> QTAILQ_INSERT_TAIL(&fw_boot_order, node, link);
> > > >> }
> > > >>
> > > >> +static bool is_same_fw_dev_path(DeviceState *src, DeviceState
> *target)
> > > >> +{
> > > >> + bool ret = false;
> > > >> + char *devpath_src = qdev_get_fw_dev_path(src);
> > > >> + char *devpath_target = qdev_get_fw_dev_path(target);
> > > >> +
> > > >> + if (!strcmp(devpath_src, devpath_target)) {
> > > >> + ret = true;
> > > >> + }
> > > >> +
> > > >> + g_free(devpath_src);
> > > >> + g_free(devpath_target);
> > > >> + return ret;
> > > >> +}
> > > >> +
> > > >> +void del_boot_device_path(int32_t bootindex, DeviceState *dev,
> > > >> + const char *suffix)
> > > >> +{
> > > >> + FWBootEntry *i;
> > > >> +
> > > >> + assert(dev != NULL);
> > > >> +
> > > >
> > > > assert(booindex >= 0 || suffix != NULL);
> > > >
> >
> > suffix call be NULL.
> >
> > > >> + QTAILQ_FOREACH(i, &fw_boot_order, link) {
> > > >> + if (is_same_fw_dev_path(i->dev, dev)) {
> > > >
> > > > if (!suffix) {
> > > > break;
> > > > }
>
> If suffix is NULL, at least we should do nothing in the loop.
>
> > > >
> > > >> + if (suffix && i->suffix && strcmp(i->suffix, suffix)) {
> > > >> + continue;
> > > >> + }
>
> How about this one?
>
> if (!suffix) {
> break;
> } else (i->suffix && strcmp(i->suffix, suffix)) {
> continue;
> }
>
>
Agreed.
It's more better, which will reduce useless loop, thanks!
> > > >
> > > > If suffix is NULL, then all the entries will be removed?
> >
> > If suffix is NULL, the entry will be checked by the bootindex as below
> > QTAILQ_FOREACH loop. If suffix and bootindex are all not match,
> > no entry will not be deleted from the global fw_boot_order list.
>
> This is why I added "assert(booindex >= 0 || suffix != NULL);" on
> above.
>
> ^^^^
OK. You are right. Thanks, Amos.
Best regards,
-Gonglei
[Qemu-devel] [RFC PATCH 2/5] bootindex: reset bootindex when vm reset, arei.gonglei, 2014/07/07
[Qemu-devel] [RFC PATCH 3/5] bootindex: delete boot index when device is removed, arei.gonglei, 2014/07/07
[Qemu-devel] [RFC PATCH 5/5] bootindex: fix memory leak when set boot index, arei.gonglei, 2014/07/07
Re: [Qemu-devel] [RFC PATCH 0/5] modify boot order when vm is running, Michael S. Tsirkin, 2014/07/07
Re: [Qemu-devel] [RFC PATCH 0/5] modify boot order when vm is running, Gonglei (Arei), 2014/07/07