qdev/core: fix qbus_is_full()
commit12b2e9f30f978f26f35f9df5c2ef96fbc019bab6
authorTony Krowiak <akrowiak@linux.ibm.com>
Mon, 17 Dec 2018 15:57:30 +0000 (17 10:57 -0500)
committerEduardo Habkost <ehabkost@redhat.com>
Wed, 6 Mar 2019 14:51:08 +0000 (6 11:51 -0300)
tree9fe1c09e733beecb3164acc6406e200a79fb17fa
parent9b748c5e061b1202fba59afd857e16a693743d90
qdev/core: fix qbus_is_full()

The qbus_is_full(BusState *bus) function (qdev_monitor.c) compares the max_index
value of the BusState structure with the max_dev value of the BusClass structure
to determine whether the maximum number of children has been reached for the
bus. The problem is, the max_index field of the BusState structure does not
necessarily reflect the number of devices that have been plugged into
the bus.

Whenever a child device is plugged into the bus, the bus's max_index value is
assigned to the child device and then incremented. If the child is subsequently
unplugged, the value of the max_index does not change and no longer reflects the
number of children.

When the bus's max_index value reaches the maximum number of devices
allowed for the bus (i.e., the max_dev field in the BusClass structure),
attempts to plug another device will be rejected claiming that the bus is
full -- even if the bus is actually empty.

To resolve the problem, a new 'num_children' field is being added to the
BusState structure to keep track of the number of children plugged into the
bus. It will be incremented when a child is plugged, and decremented when a
child is unplugged.

Signed-off-by: Tony Krowiak <akrowiak@linux.ibm.com>
Reviewed-by: Pierre Morel<pmorel@linux.ibm.com>
Reviewed-by: Halil Pasic <pasic@linux.ibm.com>
Message-Id: <1545062250-7573-1-git-send-email-akrowiak@linux.ibm.com>
Reviewed-by: Igor Mammedov <imammedo@redhat.com>
Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
hw/core/qdev.c
include/hw/qdev-core.h
qdev-monitor.c