From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:46054) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bDhgb-0005ZC-9o for qemu-devel@nongnu.org; Thu, 16 Jun 2016 20:31:02 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bDhgV-0006OE-1J for qemu-devel@nongnu.org; Thu, 16 Jun 2016 20:31:00 -0400 Received: from mx1.redhat.com ([209.132.183.28]:49204) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bDhgU-0006O3-Rv for qemu-devel@nongnu.org; Thu, 16 Jun 2016 20:30:54 -0400 Date: Fri, 17 Jun 2016 03:30:50 +0300 From: "Michael S. Tsirkin" Message-ID: <20160617033050-mutt-send-email-mst@redhat.com> References: <1466123354-22896-1-git-send-email-mst@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1466123354-22896-1-git-send-email-mst@redhat.com> Subject: [Qemu-devel] [PULL v3 12/15] pci core: assert ENOSPC when add capability List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Peter Maydell , Cao jin , Marcel Apfelbaum , Markus Armbruster From: Cao jin ENOSPC is programming error, assert it for debugging. cc: Michael S. Tsirkin cc: Marcel Apfelbaum cc: Markus Armbruster Reviewed-by: Markus Armbruster Reviewed-by: Marcel Apfelbaum Signed-off-by: Cao jin Reviewed-by: Markus Armbruster Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin --- 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 87bea47..4b585f4 100644 --- a/hw/pci/pci.c +++ b/hw/pci/pci.c @@ -2228,10 +2228,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 is programming error */ + assert(offset); } else { /* Verify that capabilities don't overlap. Note: device assignment * depends on this check to verify that the device is not broken. -- MST