[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 1/8] bootindex: add modify_boot_device_path f
From: |
Gonglei (Arei) |
Subject: |
Re: [Qemu-devel] [PATCH v4 1/8] bootindex: add modify_boot_device_path function |
Date: |
Mon, 4 Aug 2014 07:02:27 +0000 |
Hi,
>
> > On Thu, Jul 31, 2014 at 05:47:26PM +0800, address@hidden wrote:
> > [...]
> > > +void modify_boot_device_path(int32_t bootindex, DeviceState *dev,
> > > + const char *suffix)
> > > +{
> > > + FWBootEntry *i, *old_entry = NULL;
> > > +
> > > + assert(dev != NULL || suffix != NULL);
> > > +
> > > + if (bootindex >= 0) {
> > > + QTAILQ_FOREACH(i, &fw_boot_order, link) {
> > > + if (i->bootindex == bootindex) {
> > > + qerror_report(ERROR_CLASS_GENERIC_ERROR,
> > > + "The bootindex %d has already been
> used",
> > > + bootindex);
> >
> > Isn't an Error** parameter preferable here, instead of using
> > qerror_report()?
>
> Good catch. I'm not following this series, but using qerror_report() is
> almost always wrong these days.
>
Would you give me some advice? Thanks.
Using error_report() instead of qerror_report()?
Best regards,
-Gonglei