[PATCH] drm/i915: Limit the number of loops for reading a split 64bit register

2015-09-08 Thread Chris Wilson
In I915_READ64_2x32 we attempt to read a 64bit register using 2 32bit reads. Due to the nature of the registers we try to read in this manner, they may increment between the two instruction (e.g. a timestamp counter). To keep the result accurate, we repeat the read if we detect an overflow (i.e. th

Re: [PATCH] drm/i915: Limit the number of loops for reading a split 64bit register

2015-09-08 Thread Chris Wilson
On Tue, Sep 08, 2015 at 08:24:19AM +0100, Chris Wilson wrote: > In I915_READ64_2x32 we attempt to read a 64bit register using 2 32bit > reads. Due to the nature of the registers we try to read in this manner, > they may increment between the two instruction (e.g. a timestamp > counter). To keep the

Re: [PATCH] drm/i915: Limit the number of loops for reading a split 64bit register

2015-09-08 Thread Daniel Vetter
On Tue, Sep 08, 2015 at 08:51:50AM +0100, Chris Wilson wrote: > On Tue, Sep 08, 2015 at 08:24:19AM +0100, Chris Wilson wrote: > > In I915_READ64_2x32 we attempt to read a 64bit register using 2 32bit > > reads. Due to the nature of the registers we try to read in this manner, > > they may increment

Re: [Intel-gfx] [PATCH] drm/i915: Limit the number of loops for reading a split 64bit register

2015-09-08 Thread Jani Nikula
On Tue, 08 Sep 2015, Chris Wilson wrote: > In I915_READ64_2x32 we attempt to read a 64bit register using 2 32bit > reads. Due to the nature of the registers we try to read in this manner, > they may increment between the two instruction (e.g. a timestamp > counter). To keep the result accurate, we

Re: [Intel-gfx] [PATCH] drm/i915: Limit the number of loops for reading a split 64bit register

2015-09-08 Thread Chris Wilson
On Tue, Sep 08, 2015 at 03:36:32PM +0300, Jani Nikula wrote: > On Tue, 08 Sep 2015, Chris Wilson wrote: > > In I915_READ64_2x32 we attempt to read a 64bit register using 2 32bit > > reads. Due to the nature of the registers we try to read in this manner, > > they may increment between the two inst

Patch "drm/i915: Limit the number of loops for reading a split 64bit register" has been added to the 4.2-stable tree

2015-09-15 Thread gregkh
This is a note to let you know that I've just added the patch titled drm/i915: Limit the number of loops for reading a split 64bit register to the 4.2-stable tree which can be found at: http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary The filename of the

Patch "drm/i915: Limit the number of loops for reading a split 64bit register" has been added to the 4.1-stable tree

2015-09-15 Thread gregkh
This is a note to let you know that I've just added the patch titled drm/i915: Limit the number of loops for reading a split 64bit register to the 4.1-stable tree which can be found at: http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary The filename of the