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@xxxxxxxxx> > > --- > > 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 _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
