[Intel-gfx] [PATCH 6/7] drm/i915: Only grab timestamps when needed

Tvrtko Ursulin tvrtko.ursulin at linux.intel.com
Mon Jan 11 07:04:00 PST 2016


On 11/01/16 14:36, Chris Wilson wrote:
> On Mon, Jan 11, 2016 at 02:08:40PM +0000, Tvrtko Ursulin wrote:
>> From: Tvrtko Ursulin <tvrtko.ursulin at intel.com>
>>
>> No need to call ktime_get_raw_ns twice per unlimited wait and can
>> also elimate a local variable.
>
> But we could eliminate both, and the unsightly pointless assignment only
> required to shut gcc up.
>
> Still preferring my patch.

Ah I remember it now.. you were storing it in the pointer provided by 
the caller. I think that is significantly worse, sorry cannot approve that.

Regards,

Tvrtko


More information about the Intel-gfx mailing list