Re: [RFC PATCH v2 04/16] softmmu/qdev-monitor: add error handling in qdev_set_id

[Damien Hedde <damien.hedde@xxxxxxxxxxxxx>]

On 10/13/21 09:10, Alistair Francis wrote:
> On Thu, Sep 23, 2021 at 2:29 AM Damien Hedde <damien.hedde@xxxxxxxxxxxxx> wrote:
> > qdev_set_id() is mostly used when the user adds a device (using
> > -device cli option or device_add qmp command). This commit adds
> > an error parameter to handle the case where the given id is
> > already taken.
> >
> > Also document the function and add a return value in order to
> > be able to capture success/failure: the function now returns the
> > id in case of success, or NULL in case of failure.
> >
> > The commit modifies the 2 calling places (qdev-monitor and
> > xen-legacy-backend) to add the error object parameter.
> >
> > Note that the id is, right now, guaranteed to be unique because
> > all ids came from the "device" QemuOptsList where the id is used
> > as key. This addition is a preparation for a future commit which
> > will relax the uniqueness.
> >
> > Signed-off-by: Damien Hedde <damien.hedde@xxxxxxxxxxxxx>
> Reviewed-by: Alistair Francis <alistair.francis@xxxxxxx>
>
> Alistair
CC'ing Kevin who integrated this patch into his series about
"qdev: Add JSON -device"
https://lists.gnu.org/archive/html/qemu-devel/2021-10/msg01826.html

Thanks,
--
Damien