Re: [Mesa-dev] [PATCH 2/2] pipe-loader: Add support for render nodes v2

2014-02-03 Thread David Herrmann
Hi On Fri, Jan 31, 2014 at 9:58 PM, Tom Stellard wrote: > On Thu, Jan 30, 2014 at 10:16:41PM +, Emil Velikov wrote: >> On 27/01/14 16:13, Tom Stellard wrote: >> > From: Tom Stellard >> > >> > v2: >> >- Add missing call to pipe_loader_drm_release() >> >- Fix render node macros >> >

Re: [Mesa-dev] [PATCH 2/2] pipe-loader: Add support for render nodes v2

2014-01-31 Thread Tom Stellard
On Thu, Jan 30, 2014 at 10:16:41PM +, Emil Velikov wrote: > On 27/01/14 16:13, Tom Stellard wrote: > > From: Tom Stellard > > > > v2: > >- Add missing call to pipe_loader_drm_release() > >- Fix render node macros > >- Drop render-node configure option > > --- > > > > For referenc

Re: [Mesa-dev] [PATCH 2/2] pipe-loader: Add support for render nodes v2

2014-01-30 Thread Jan Vesely
On Thu, 2014-01-30 at 15:52 -0500, Tom Stellard wrote: > On Thu, Jan 30, 2014 at 01:55:40PM -0500, Jan Vesely wrote: > > Hi, > > > > are these patches the last thing necessary to get CL programs running > > without needing X permission? > > You also need a 3.12 or newer kernel and you must enabl

Re: [Mesa-dev] [PATCH 2/2] pipe-loader: Add support for render nodes v2

2014-01-30 Thread Emil Velikov
On 27/01/14 16:13, Tom Stellard wrote: > From: Tom Stellard > > v2: >- Add missing call to pipe_loader_drm_release() >- Fix render node macros >- Drop render-node configure option > --- > > For reference, version 1 of this patch: > http://lists.freedesktop.org/archives/mesa-dev/2013-

Re: [Mesa-dev] [PATCH 2/2] pipe-loader: Add support for render nodes v2

2014-01-30 Thread Tom Stellard
On Thu, Jan 30, 2014 at 01:55:40PM -0500, Jan Vesely wrote: > Hi, > > are these patches the last thing necessary to get CL programs running > without needing X permission? You also need a 3.12 or newer kernel and you must enable render nodes ether by adding drm.rnodes=1 to you kernel command lin

Re: [Mesa-dev] [PATCH 2/2] pipe-loader: Add support for render nodes v2

2014-01-30 Thread Jan Vesely
Hi, are these patches the last thing necessary to get CL programs running without needing X permission? I tried to test them, but got: pci id for fd 3: 1002:675d, driver r600 pci id for fd 3: 1002:675d, driver r600 radeon: Failed to get PCI ID, error number -13 which is kind of funny combination

Re: [Mesa-dev] [PATCH 2/2] pipe-loader: Add support for render nodes v2

2014-01-30 Thread Tom Stellard
On Tue, Jan 28, 2014 at 03:04:00PM +0100, David Herrmann wrote: > Hi Tom > > On Mon, Jan 27, 2014 at 5:13 PM, Tom Stellard wrote: > > From: Tom Stellard > > > > v2: > >- Add missing call to pipe_loader_drm_release() > >- Fix render node macros > >- Drop render-node configure option >

Re: [Mesa-dev] [PATCH 2/2] pipe-loader: Add support for render nodes v2

2014-01-28 Thread David Herrmann
Hi Tom On Mon, Jan 27, 2014 at 5:13 PM, Tom Stellard wrote: > From: Tom Stellard > > v2: >- Add missing call to pipe_loader_drm_release() >- Fix render node macros >- Drop render-node configure option Patch looks fine to me, but I lack mesa-insights to really comment on it. But I'd

[Mesa-dev] [PATCH 2/2] pipe-loader: Add support for render nodes v2

2014-01-27 Thread Tom Stellard
From: Tom Stellard v2: - Add missing call to pipe_loader_drm_release() - Fix render node macros - Drop render-node configure option --- For reference, version 1 of this patch: http://lists.freedesktop.org/archives/mesa-dev/2013-October/047296.html .../auxiliary/pipe-loader/pipe_loader