[PATCH 1/5] drm: add interface to get drm devices on the system v2

2015-08-14 Thread Emil Velikov
On 14/08/15 13:45, Zhou, Jammy wrote: > That's okay. Shall we get this patch merged now if no other objections? > First we should fix the funny vendor/device/etc id issue. Otherwise the API is bugged badly. -Emil

[PATCH 1/5] drm: add interface to get drm devices on the system v2

2015-08-14 Thread Zhou, Jammy
interface to get drm devices on the system v2 On 14/08/15 13:45, Zhou, Jammy wrote: > That's okay. Shall we get this patch merged now if no other objections? > First we should fix the funny vendor/device/etc id issue. Otherwise the API is bugged badly. -Emil

[PATCH 1/5] drm: add interface to get drm devices on the system v2

2015-08-14 Thread Emil Velikov
On 14 August 2015 at 13:15, Zhou, Jammy wrote: > Okay, I got it. Actually with current implementation, only the number of PCI > devices on the system is returned for drmGetDevices(NULL, 0). I extracted > related code below. I hope it can address your concern :-) > Had a blond moment and got conf

[PATCH 1/5] drm: add interface to get drm devices on the system v2

2015-08-14 Thread Zhou, Jammy
That's okay. Shall we get this patch merged now if no other objections? Regards, Jammy -Original Message- From: Emil Velikov [mailto:emil.l.veli...@gmail.com] Sent: Friday, August 14, 2015 8:29 PM To: Zhou, Jammy Cc: ML dri-devel Subject: Re: [PATCH 1/5] drm: add interface to ge

[PATCH 1/5] drm: add interface to get drm devices on the system v2

2015-08-14 Thread Zhou, Jammy
el Subject: Re: [PATCH 1/5] drm: add interface to get drm devices on the system v2 On 14 August 2015 at 10:41, Zhou, Jammy wrote: >> What is the point in claiming that you have X+Y devices, if the API does not >> provide any information about Y of them ? It seems very misleading imho

[PATCH 1/5] drm: add interface to get drm devices on the system v2

2015-08-14 Thread Emil Velikov
On 14 August 2015 at 10:41, Zhou, Jammy wrote: >> What is the point in claiming that you have X+Y devices, if the API does not >> provide any information about Y of them ? It seems very misleading imho. > > I'm not sure if I understand your question correctly. Easy - replace X with "pci" and Y wi

[PATCH 1/5] drm: add interface to get drm devices on the system v2

2015-08-14 Thread Kai Wasserbäch
Emil Velikov wrote on 14.08.2015 10:17: > On 14 August 2015 at 08:59, Kai Wasserbäch > wrote: >> Zhou, Jammy wrote on 14.08.2015 07:59: >>> We tried several different ways already for the enumeration interface >>> (libpciaccess, libudev, etc). But we ran into some problems with these >>> optio

[PATCH 1/5] drm: add interface to get drm devices on the system v2

2015-08-14 Thread Emil Velikov
On 14 August 2015 at 09:26, Kai Wasserbäch wrote: > Emil Velikov wrote on 14.08.2015 10:17: >> On 14 August 2015 at 08:59, Kai Wasserbäch >> wrote: >>> Zhou, Jammy wrote on 14.08.2015 07:59: We tried several different ways already for the enumeration interface (libpciaccess, libude

[PATCH 1/5] drm: add interface to get drm devices on the system v2

2015-08-14 Thread Kai Wasserbäch
Zhou, Jammy wrote on 14.08.2015 07:59: > We tried several different ways already for the enumeration interface > (libpciaccess, libudev, etc). But we ran into some problems with these > options for example when run Steam games which ships 32bit libraries > (including libudev) in the steam runtim

[PATCH 1/5] drm: add interface to get drm devices on the system v2

2015-08-14 Thread Zhou, Jammy
o: Kai Wasserbäch Cc: Zhou, Jammy; Daniel Vetter; ML dri-devel Subject: Re: [PATCH 1/5] drm: add interface to get drm devices on the system v2 On 14 August 2015 at 09:26, Kai Wasserbäch wrote: > Emil Velikov wrote on 14.08.2015 10:17: >> On 14 August 2015 at 08:59, Kai Wasserbäch >

[PATCH 1/5] drm: add interface to get drm devices on the system v2

2015-08-14 Thread Zhou, Jammy
: Re: [PATCH 1/5] drm: add interface to get drm devices on the system v2 On 14 August 2015 at 06:53, Zhou, Jammy wrote: > Hi Emil, > >> If there are any other devices they will still be counted when >> drmGetDevices(NULL, 0)... Is that intentional ? > Yes, I think so, so that

[PATCH 1/5] drm: add interface to get drm devices on the system v2

2015-08-14 Thread Emil Velikov
On 14 August 2015 at 06:53, Zhou, Jammy wrote: > Hi Emil, > >> If there are any other devices they will still be counted when >> drmGetDevices(NULL, 0)... Is that intentional ? > Yes, I think so, so that this interface can support different kinds of > devices in the future. For example, we have

[PATCH 1/5] drm: add interface to get drm devices on the system v2

2015-08-14 Thread Emil Velikov
On 14 August 2015 at 08:59, Kai Wasserbäch wrote: > Zhou, Jammy wrote on 14.08.2015 07:59: >> We tried several different ways already for the enumeration interface >> (libpciaccess, libudev, etc). But we ran into some problems with these >> options for example when run Steam games which ships

[PATCH 1/5] drm: add interface to get drm devices on the system v2

2015-08-14 Thread Zhou, Jammy
to avoid introducing some additional dependencies into libdrm. Regards, Jammy -Original Message- From: Emil Velikov [mailto:emil.l.veli...@gmail.com] Sent: Thursday, August 13, 2015 11:27 PM To: Daniel Vetter Cc: Zhou, Jammy; ML dri-devel Subject: Re: [PATCH 1/5] drm: add interface to

[PATCH 1/5] drm: add interface to get drm devices on the system v2

2015-08-14 Thread Zhou, Jammy
ri-devel Subject: Re: [PATCH 1/5] drm: add interface to get drm devices on the system v2 On 13 August 2015 at 04:33, Jammy Zhou wrote: > From: Emil Velikov > > For mutiple GPU support, the devices on the system should be > enumerated to get necessary information about each device, and t

[PATCH 1/5] drm: add interface to get drm devices on the system v2

2015-08-13 Thread Daniel Vetter
On Thu, Aug 13, 2015 at 11:33:41AM +0800, Jammy Zhou wrote: > From: Emil Velikov > > 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 suppor

[PATCH 1/5] drm: add interface to get drm devices on the system v2

2015-08-13 Thread Emil Velikov
On 13 August 2015 at 16:07, Daniel Vetter wrote: > On Thu, Aug 13, 2015 at 11:33:41AM +0800, Jammy Zhou wrote: >> From: Emil Velikov >> >> For mutiple GPU support, the devices on the system should be enumerated >> to get necessary information about each device, and the drmGetDevices >> interface

[PATCH 1/5] drm: add interface to get drm devices on the system v2

2015-08-13 Thread Emil Velikov
On 13 August 2015 at 04:33, Jammy Zhou wrote: > From: Emil Velikov > > 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

[PATCH 1/5] drm: add interface to get drm devices on the system v2

2015-08-13 Thread Jammy Zhou
From: Emil Velikov 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. Typical usage: int count; drmDevicePtr *foo;