On Wed, Oct 01, 2014 at 10:13:00AM +0200, Daniel Vetter wrote: > 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. I strongly disagree that we should be working around self-imposed limitations of the test suite by making users believe their kernel is broken. -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
