Re: [PATCH 3/5] lib: Add a drm_open_any_render() that will try to use render nodes

Ben Widawsky
2013-10-24T22:17:22Z

On Thu, Oct 24, 2013 at 10:05:43PM +0100, Damien Lespiau wrote:
> On Thu, Oct 24, 2013 at 10:01:21PM +0100, Damien Lespiau wrote:
> > On Thu, Oct 24, 2013 at 01:43:22PM -0700, Ben Widawsky wrote:
> > > On Thu, Oct 24, 2013 at 03:40:54PM +0100, Damien Lespiau wrote:
> > > > I was fedup with having to run my tests as root and not being able to
> > > > use my usual setup for tests that only exercise the GT part of the GPU.
> > > >
> > > > Render nodes to the rescue!
> > >
> > > I think usually this is a bad idea except for pretty isolated proof of
> > > concept kind of things. Mostly intel-gpu-tools is testing our kernel
> > > driver, and therefore running without a reboot anyway is sort of
> > > unusual.
> >
> > Hum? I fail to see the link. It should be strickly equivalent, with the
> > added bonus that you can write and execute a test as a normal user.
>
> By "normal user" I mean with a DRM master running and without having to
> do the authentification dance.
>
> --
> Damien

You're right, this was a half baked thought. What I was thinking is,
condoning running tests on a live system is a bad idea - but forcing
rendernodes doesn't effect that.

Also, I agree with Daniel. Getting some simultaneous tests would be
nice.

Practically, requiring root won't go away I think, since certain things
like our forcewake debugfs require root. Anyway, carry on.

--
Ben Widawsky, Intel Open Source Technology Center