[Intel-gfx] [PATCH 16/23] drm_lib.sh: Tune the DRM master message a bit

Damien Lespiau damien.lespiau at intel.com
Fri Nov 15 18:17:39 CET 2013


On Fri, Nov 15, 2013 at 06:13:48PM +0100, Daniel Vetter wrote:
> On Fri, Nov 15, 2013 at 04:33:33PM +0000, Damien Lespiau wrote:
> > check_drm_clients can be used to make sure there's no master running.
> > Tune a bit the error message as this is really what we want to test.
> > 
> > Signed-off-by: Damien Lespiau <damien.lespiau at intel.com>
> > ---
> >  tests/drm_lib.sh | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/tests/drm_lib.sh b/tests/drm_lib.sh
> > index 97f6f92..92b379f 100755
> > --- a/tests/drm_lib.sh
> > +++ b/tests/drm_lib.sh
> > @@ -31,7 +31,7 @@ fi
> >  # read everything we can
> >  if [ `cat $i915_dfs_path/clients | wc -l` -gt "2" ] ; then
> >  	[ -n "$DRM_LIB_ALLOW_NO_MASTER" ] || \
> > -		die "ERROR: other drm clients running"
> > +		die "ERROR: a drm master is already running"
> 
> Imo the current line is clear enough and we do actually check for any drm
> client, not just master ... But I don't really care.

Me neither, I'm fine with dropping the patch.

-- 
Damien



More information about the Intel-gfx mailing list