[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH 1/2] libxl: introduce libxl__is_igd_vga_passthru



On Mon, Mar 09, 2015 at 02:27:46PM +0800, Chen, Tiejun wrote:
[...]
> >>+
> >>+    FILE *f = fopen(pci_device_vendor_path, "r");
> >>+    if (!f) {
> >>+        LOGE(ERROR,
> >>+             "pci device "PCI_BDF" does not have vendor attribute",
> >>+             pcidev->domain, pcidev->bus, pcidev->dev, pcidev->func);
> >>+        return 0xffff;
> >>+    }
> >>+    read_items = fscanf(f, "0x%lx\n", &pci_device_vendor);
> 
> we have to refactor this as well,
> 
> read_items = fscanf(f, "0x%hx\n", &pci_device_vendor);
> 
> Right?
> 

Sure. Just eliminate any warnings / errors.

Wei.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.