On Wed, Apr 24, 2013 at 11:04:40AM +0100, Chris Wilson wrote: > On Tue, Apr 23, 2013 at 11:15:37PM -0700, Ben Widawsky wrote: > > - i915_gem_context_init(dev); > > + if (!dev_priv->hw_contexts_disabled && > > + i915_gem_context_enable(dev_priv)) { > > Move the hw_contexts_disabled test into i915_gem_context_enable(), just > makes this bit of code cleaner and less incestrous. > -Chris Got it locally, thanks for the idea. I'll wait a bit to resend in case more review comes in. > > -- > Chris Wilson, Intel Open Source Technology Centre -- Ben Widawsky, Intel Open Source Technology Center