On 27 February 2015 at 18:07, Jan Vesely <jan.vesely@xxxxxxxxxxx> wrote: > Signed-off-by: Jan Vesely <jan.vesely@xxxxxxxxxxx> > --- > tests/exynos/exynos_fimg2d_test.c | 4 +--- > xf86drm.c | 9 ++++++--- > 2 files changed, 7 insertions(+), 6 deletions(-) > ... > @@ -333,7 +336,6 @@ static int drmOpenDevice(dev_t dev, int minor, int type) > drm_server_info->get_perms(&serv_group, &serv_mode); > devmode = serv_mode ? serv_mode : DRM_DEV_MODE; > devmode &= ~(S_IXUSR|S_IXGRP|S_IXOTH); > - group = (serv_group >= 0) ? serv_group : DRM_DEV_GID; > } > > #if !defined(UDEV) > @@ -354,6 +356,7 @@ static int drmOpenDevice(dev_t dev, int minor, int type) > } > > if (drm_server_info) { > + group = (serv_group >= 0) ? serv_group : DRM_DEV_GID; I think you can leave this back where it was. It doesn't seem to bring anything beneficial. -Emil _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel