OSDN Git Service

qdev: Make netdev properties work as list elements
authorKevin Wolf <kwolf@redhat.com>
Thu, 9 Nov 2023 17:42:39 +0000 (18:42 +0100)
committerKevin Wolf <kwolf@redhat.com>
Fri, 10 Nov 2023 17:19:15 +0000 (18:19 +0100)
commit3257b854d81ca3ebe6f14375e83a0ed2db3c7562
treea3c6d4c395858fe7cc5519031c316e726e49e921
parent125062e791258c68109f3a59cb7aca3dadbdb5a3
qdev: Make netdev properties work as list elements

The 'name' parameter of QOM setters is primarily used to specify the name
of the currently parsed input element in the visitor interface. For
top-level qdev properties, this is always set and matches 'prop->name'.

However, for list elements it is NULL, because each element of a list
doesn't have a separate name. Passing a non-NULL value runs into
assertion failures in the visitor code.

Therefore, using 'name' in error messages is not right for property
types that are used in lists, because "(null)" (or even a segfault)
isn't very helpful to identify what QEMU is complaining about.

Change netdev properties to use 'prop->name' instead, which will contain
the name of the array property after switching array properties to lists
in the external interface. (This is still not perfect, as it doesn't
identify which element in the list caused the error, but strictly better
than before.)

Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Message-ID: <20231109174240.72376-11-kwolf@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
hw/core/qdev-properties-system.c