On Wed, Oct 01, 2014 at 07:28:39AM +0100, Chris Wilson wrote: > On Wed, Oct 01, 2014 at 01:04:19AM +0200, Daniel Vetter wrote: > > This seems to have been accidentally lost in > > > > commit be62acb4cce1389a28296852737e3917d9cc5b25 > > Author: Mika Kuoppala <mika.kuoppala@xxxxxxxxxxxxxxx> > > Date: Fri Aug 30 16:19:28 2013 +0300 > > > > drm/i915: ban badly behaving contexts > > > > Without this real gpu hangs only log output at info level, which gets > > filtered away by piglit's testrunner. > > A successful GPU hang is not an error. Might be a warn or a notice, but > it certainly isn't a driver error. Well not of the kernel driver, but might very well be a bug in the userspace driver. With this piglit marks tests that hung the gpu as dmesg-fail, without this they might even pass. Ken raised this on irc and I agree that it's a must-have feature for developers that their testsuite can tell them when stuff broke. Provding this some other way is a lot more work and imo should be done in a separate patch, this here is just the minimal fix for this regression. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
