On 13 August 2015 at 04:33, Jammy Zhou <Jammy.Zhou@xxxxxxx> wrote:
> From: Emil Velikov <emil.l.velikov@xxxxxxxxx>
>
> For mutiple GPU support, the devices on the system should be enumerated
> to get necessary information about each device, and the drmGetDevices
> interface is added for this. Currently only PCI devices are supported for
> the enumeration.
>
If there are any other devices they will still be counted when
drmGetDevices(NULL, 0)... Is that intentional ?
> +static int drmParsePciDeviceInfo(const char *config,
> + drmPciDeviceInfoPtr device)
> +{
> + if (config == NULL)
> + return -EINVAL;
> +
> + device->vendor_id = config[0] | (config[1] << 8);
> + device->device_id = config[2] | (config[3] << 8);
> + device->revision_id = config[8];
> + device->subvendor_id = config[44] | (config[45] << 8);
> + device->subdevice_id = config[46] | (config[47] << 8);
> +
Something funny is happening here - on my intel system vendor_id is
reported as 0xff86, instead of 0x8086. Subvendor/device are also
messed up - ffaa and ffda instead of 17aa + 21da.
One could bikeshed on the de-duplication error path(s), but
considering that things work and there are no leaks we can leave that
for some other day.
-Emil
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel
