[Mesa-dev] [PATCH 0/8] Render node only opencl and pipe-loader cleanups

[Emil Velikov]

On 07/07/15 19:42, Tom Stellard wrote:
> On Tue, Jul 07, 2015 at 05:43:19PM +0100, Emil Velikov wrote:
>> On 30/06/15 16:09, Emil Velikov wrote:
>>> Hello all,
>>>
>>> As mentioned over IRC a few weeks back, here is a series that removes 
>>> support for non-render node devices.
>>>
>>> The two main motivations being:
>>>  - Currently we force X/xcb onto everyone that wants to use OpenCL
>>> (headless OpenCL systems/farms anyone ?)
> 
> Is this really true?  I don't see where lack of xcb prevents users from
> building OpenCL.
> 
Ouch just realised how silly the wording is. Sorry about that.

Currently if you have xcb at build time it will get picked, regardless
if xcb is present at runtime or not. Something nasty which I refer to as
"hidden dependency".


> -Tom
> 
>>>  - Nice overall cleanup - 43 insertions(+), 279 deletions(-)
>>>
>>>
>>> Note that the final patches touch related code - from removing a unused 
>>> function (pipe_loader_sw_probe_xlib) to using loader_open_device() over 
>>> open(), with the former caring about CLOEXEC.
>>>
>> Francisco, Tom,
>>
>> Can you guys please take a look at the series. Even an Ack would be
>> greatly appreciated.
>>
> 
> I have no problems with merging these.
> 
If you'd like to take a look I can give it another week. Alternatively
I'll fix patch 1/8 summary (same "force" mistake) and will push these in
a day or so.

Thanks
Emil