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 enable render nodes > ether by adding drm.rnodes=1 to you kernel command line arguments in > grub, or by adding: > options drm rnodes=1 > to a config file in /etc/modprobe.d/
turns out I wasn't in the video group, sry for the noise. these patches work OK. regards, Jan > > -Tom > > > 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 of messages. > > > > regards, > > Jan > > > > PS: $ ls /dev/dri/ > > card0 controlD64 renderD128 > > > > On Thu, 2014-01-30 at 11:34 -0500, Tom Stellard wrote: > > > 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 <t...@stellard.net> wrote: > > > > > From: Tom Stellard <thomas.stell...@amd.com> > > > > > > > > > > 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 like to get a feedback whether there's still any open > > > > issues with render-nodes. If not, I will fix the minor-allocation-race > > > > and then try to make rnodes the default in 3.15. > > > > > > > > > > I am in favor of enabling rnodes by default for 3.15. I haven't notice > > > any problems with them so far. > > > > > > -Tom > > > > > > > Thanks > > > > David > > > > > > > > > --- > > > > > > > > > > For reference, version 1 of this patch: > > > > > http://lists.freedesktop.org/archives/mesa-dev/2013-October/047296.html > > > > > > > > > > .../auxiliary/pipe-loader/pipe_loader_drm.c | 80 > > > > > +++++++++++++++++++++- > > > > > 1 file changed, 77 insertions(+), 3 deletions(-) > > > > > > > > > > diff --git a/src/gallium/auxiliary/pipe-loader/pipe_loader_drm.c > > > > > b/src/gallium/auxiliary/pipe-loader/pipe_loader_drm.c > > > > > index 9484db8..7a1af91 100644 > > > > > --- a/src/gallium/auxiliary/pipe-loader/pipe_loader_drm.c > > > > > +++ b/src/gallium/auxiliary/pipe-loader/pipe_loader_drm.c > > > > > @@ -49,6 +49,11 @@ > > > > > #include "util/u_dl.h" > > > > > #include "util/u_debug.h" > > > > > > > > > > +#define DRM_RENDER_NODE_DEV_NAME_FORMAT "%s/renderD%d" > > > > > +#define DRM_RENDER_NODE_MAX_NODES 63 > > > > > +#define DRM_RENDER_NODE_MIN_MINOR 128 > > > > > +#define DRM_RENDER_NODE_MAX_MINOR (DRM_RENDER_NODE_MIN_MINOR + > > > > > DRM_RENDER_NODE_MAX_NODES) > > > > > + > > > > > struct pipe_loader_drm_device { > > > > > struct pipe_loader_device base; > > > > > struct util_dl_library *lib; > > > > > @@ -152,18 +157,87 @@ open_drm_minor(int minor) > > > > > return open(path, O_RDWR, 0); > > > > > } > > > > > > > > > > +static int > > > > > +open_drm_render_node_minor(int minor) > > > > > +{ > > > > > + char path[PATH_MAX]; > > > > > + snprintf(path, sizeof(path), DRM_RENDER_NODE_DEV_NAME_FORMAT, > > > > > DRM_DIR_NAME, > > > > > + minor); > > > > > + return open(path, O_RDWR, 0); > > > > > +} > > > > > + > > > > > int > > > > > pipe_loader_drm_probe(struct pipe_loader_device **devs, int ndev) > > > > > { > > > > > - int i, j, fd; > > > > > + int i, k, fd, num_render_node_devs; > > > > > + int j = 0; > > > > > + > > > > > + struct { > > > > > + unsigned vendor_id; > > > > > + unsigned chip_id; > > > > > + } render_node_devs[DRM_RENDER_NODE_MAX_NODES]; > > > > > + > > > > > + /* Look for render nodes first */ > > > > > + for (i = DRM_RENDER_NODE_MIN_MINOR, j = 0; > > > > > + i <= DRM_RENDER_NODE_MAX_MINOR; i++) { > > > > > + fd = open_drm_render_node_minor(i); > > > > > + struct pipe_loader_device *dev; > > > > > + if (fd < 0) > > > > > + continue; > > > > > > > > > > - for (i = 0, j = 0; i < DRM_MAX_MINOR; i++) { > > > > > + if (!pipe_loader_drm_probe_fd(&dev, fd, false)) { > > > > > + close(fd); > > > > > + continue; > > > > > + } > > > > > + > > > > > + render_node_devs[j].vendor_id = dev->u.pci.vendor_id; > > > > > + render_node_devs[j].chip_id = dev->u.pci.chip_id; > > > > > + > > > > > + if (j < ndev) { > > > > > + devs[j] = dev; > > > > > + } else { > > > > > + close(fd); > > > > > + dev->ops->release(&dev); > > > > > + } > > > > > + j++; > > > > > + } > > > > > + > > > > > + num_render_node_devs = j; > > > > > + > > > > > + /* Next look for drm devices. */ > > > > > + for (i = 0; i < DRM_MAX_MINOR; i++) { > > > > > + struct pipe_loader_device *dev; > > > > > + boolean duplicate = FALSE; > > > > > fd = open_drm_minor(i); > > > > > if (fd < 0) > > > > > continue; > > > > > > > > > > - if (j >= ndev || !pipe_loader_drm_probe_fd(&devs[j], fd, true)) > > > > > + if (!pipe_loader_drm_probe_fd(&dev, fd, true)) { > > > > > close(fd); > > > > > + continue; > > > > > + } > > > > > + > > > > > + /* Check to make sure we aren't already accessing this device > > > > > via > > > > > + * render nodes. > > > > > + */ > > > > > + for (k = 0; k < num_render_node_devs; k++) { > > > > > + if (dev->u.pci.vendor_id == render_node_devs[k].vendor_id && > > > > > + dev->u.pci.chip_id == render_node_devs[k].chip_id) { > > > > > + close(fd); > > > > > + dev->ops->release(&dev); > > > > > + duplicate = TRUE; > > > > > + break; > > > > > + } > > > > > + } > > > > > + > > > > > + if (duplicate) > > > > > + continue; > > > > > + > > > > > + if (j < ndev) { > > > > > + devs[j] = dev; > > > > > + } else { > > > > > + dev->ops->release(&dev); > > > > > + } > > > > > > > > > > j++; > > > > > } > > > > > -- > > > > > 1.8.1.5 > > > > > > > > > > _______________________________________________ > > > > > mesa-dev mailing list > > > > > mesa-dev@lists.freedesktop.org > > > > > http://lists.freedesktop.org/mailman/listinfo/mesa-dev > > > _______________________________________________ > > > mesa-dev mailing list > > > mesa-dev@lists.freedesktop.org > > > http://lists.freedesktop.org/mailman/listinfo/mesa-dev > > > > -- > > Jan Vesely <jan.ves...@rutgers.edu> > > -- Jan Vesely <jan.ves...@rutgers.edu>
signature.asc
Description: This is a digitally signed message part
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
