Re: [Qemu-devel] [PATCH v5 10/11] pci core: assert ENOSPC when add capability

[Marcel Apfelbaum <marcel@redhat.com>]

On 05/06/2016 07:20 AM, Cao jin wrote:
> ENOSPC is programming error, assert it for debugging.
>
> cc: Michael S. Tsirkin <mst@redhat.com>
> cc: Marcel Apfelbaum <marcel@redhat.com>
> cc: Markus Armbruster <armbru@redhat.com>
> Signed-off-by: Cao jin <caoj.fnst@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 <marcel@redhat.com>

Thanks,
Marcel