diff options
| author | Damien Hedde <damien.hedde@greensocs.com> | 2021-10-08 15:34:36 +0200 |
|---|---|---|
| committer | Kevin Wolf <kwolf@redhat.com> | 2021-10-15 16:08:46 +0200 |
| commit | 4a1d937796de0fecd8b22d7dbebf87f38e8282fd (patch) | |
| tree | d5a1911e87468dfd1c492f58572f73969c192cfc /hw/pci/pci.c | |
| parent | 163f384752dd9125ce7eb1b2edf00b23f0a54557 (diff) | |
| download | qemu-4a1d937796de0fecd8b22d7dbebf87f38e8282fd.zip qemu-4a1d937796de0fecd8b22d7dbebf87f38e8282fd.tar.gz qemu-4a1d937796de0fecd8b22d7dbebf87f38e8282fd.tar.bz2 | |
softmmu/qdev-monitor: add error handling in qdev_set_id
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@greensocs.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Message-Id: <20211008133442.141332-10-kwolf@redhat.com>
Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
Tested-by: Peter Krempa <pkrempa@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Diffstat (limited to 'hw/pci/pci.c')
0 files changed, 0 insertions, 0 deletions