Re: [Intel-gfx] [PATCH 2/5] [v2] drm/i915: cleanup map&fence in bind

2013-08-13 Thread Chris Wilson
On Mon, Aug 12, 2013 at 05:34:17PM -0700, Ben Widawsky wrote: > Cleanup the map and fenceable setting during bind to make more sense, > and not check i915_is_ggtt() 2 unnecessary times > > v2: Move the bools into the if block (Chris) - There are ways to tidy > this function (fence calculations for

[Intel-gfx] [PATCH 2/5] [v2] drm/i915: cleanup map&fence in bind

2013-08-12 Thread Ben Widawsky
Cleanup the map and fenceable setting during bind to make more sense, and not check i915_is_ggtt() 2 unnecessary times v2: Move the bools into the if block (Chris) - There are ways to tidy this function (fence calculations for instance) even further, but they are quite invasive, so I am punting on