>>> On 18.10.17 at 13:40, <[email protected]> wrote: > + ret = pci_size_mem_bar(sbdf, idx, NULL, &pdev->vf_rlen[i], > + PCI_BAR_VF | > + (i == PCI_SRIOV_NUM_BARS - 1) ? > + PCI_BAR_LAST : 0);
There are parentheses missing around the conditional expression here. > + if ( ret < 0 ) > + break; I don't think you should bail out here, or report the error you've got back to your caller. Otherwise this is latent (for the former) and actual (for the latter) change in behavior. Jan _______________________________________________ Xen-devel mailing list [email protected] https://lists.xenproject.org/mailman/listinfo/xen-devel
