On Tue, 1 Feb 2011 15:07:55 +0100 Francesco Allertsen fallertsen@gmail.com wrote:
On Tue, Feb 01, 2011 at 01:16:01PM +0000, Chris Wilson wrote:
Useful for me. Just confirms that you have an equivalent machine to this Lenovo x201s, on which those patches were tested after receiving from Jesse. :|
Yes, I have a Lenovo x201s :-P.
The dmesg would have been useful to know whether the i915.ko module is loaded during X startup, and so reveal some information about the sequence of function calls (i.e. whether the modified routines were running at the time of the hang, or whether they had completed much earlier during boot). The drm.debug dmesg is usually a gold mine of information.
Attacched there is the full dmesg log.
Another useful test is whether i915.powersave=0 also prevents the hang?
No, it doesn't have any effect.
Yeah I don't think we use that flag for rc6, though we probably should.
The bisect is interesting, I'd have expected a failure when we re-enabled rc6 on ILK or when we fixed up the ring buffer init.
The cleanup patch should be just that, but at least reverting it shouldn't cause any trouble.