On 05/06/2016 07:20 AM, Cao jin wrote:
ENOSPC is programming error, assert it for debugging.
cc: Michael S. Tsirkin <m...@redhat.com>
cc: Marcel Apfelbaum <mar...@redhat.com>
cc: Markus Armbruster <arm...@redhat.com>
Signed-off-by: Cao jin <caoj.f...@cn.fujitsu.com>
---
hw/pci/pci.c | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/hw/pci/pci.c b/hw/pci/pci.c
index f0f41dc..fc8b377 100644
--- a/hw/pci/pci.c
+++ b/hw/pci/pci.c
@@ -2151,10 +2151,8 @@ int pci_add_capability2(PCIDevice *pdev, uint8_t cap_id,
if (!offset) {
offset = pci_find_space(pdev, size);
- if (!offset) {
- error_setg(errp, "out of PCI config space");
- return -ENOSPC;
- }
+ /* out of PCI config space should be programming error */
'is', not 'should be'
+ assert(offset);
} else {
/* Verify that capabilities don't overlap. Note: device assignment
* depends on this check to verify that the device is not broken.
Reviewed-by: Marcel Apfelbaum <mar...@redhat.com>
Thanks,
Marcel