<br><br><div class="gmail_quote">On Wed, Jun 22, 2011 at 10:00 AM, Chris Wilson <span dir="ltr"><<a href="mailto:chris@chris-wilson.co.uk">chris@chris-wilson.co.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="im">On Wed, 22 Jun 2011 09:55:00 -0700, Ben Widawsky <<a href="mailto:ben@bwidawsk.net">ben@bwidawsk.net</a>> wrote:<br>
> Get some more useful info for debugging backtraces on forcewake<br>
> problems. Appropriate debug flags will be required for the drm module.<br>
<br>
</div>Wouldn't it be neater to pass the reg down to<br>
gen6_gt_force_wake_get() so that we only have a single debug message and<br>
not an additional one for every inlined read/write?<br>
-Chris<br></blockquote><div><br></div><div>Urg... having some trouble with my email right now, so you have to deal with my gmail/webmail...</div><div><br></div><div>1. Yes, but I was lazy.</div><div>2. People call force_wake_get() without actually specifying a register, so I didn't want to enforce people entering bogus info on those calls.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<font color="#888888"><br>
--<br>
Chris Wilson, Intel Open Source Technology Centre<br>
</font><div><div></div><div class="h5">_______________________________________________<br>
Intel-gfx mailing list<br>
<a href="mailto:Intel-gfx@lists.freedesktop.org">Intel-gfx@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/intel-gfx" target="_blank">http://lists.freedesktop.org/mailman/listinfo/intel-gfx</a><br>
</div></div></blockquote></div><br>