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

2021-10-13 Thread Damien Hedde
On 10/13/21 09:10, Alistair Francis wrote: On Thu, Sep 23, 2021 at 2:29 AM Damien Hedde 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

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

2021-10-13 Thread Alistair Francis
On Thu, Sep 23, 2021 at 2:29 AM Damien Hedde 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 an

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

2021-09-22 Thread Damien Hedde
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/failur