Re: [PATCH] drm/vblank: Fix delta_ns to an absolute value

2017-09-04 Thread Hoegeun Kwon
On 09/04/2017 04:36 PM, Daniel Vetter wrote: On Fri, Sep 01, 2017 at 04:36:25PM +0300, Ville Syrjälä wrote: On Fri, Sep 01, 2017 at 04:07:16PM +0900, Hoegeun Kwon wrote: If scanout started, we should reduce etime by delta_ns. But delta_ns is negative if scanout has not started. If delta_ns is n

Re: [PATCH] drm/vblank: Fix delta_ns to an absolute value

2017-09-04 Thread Daniel Vetter
On Fri, Sep 01, 2017 at 04:36:25PM +0300, Ville Syrjälä wrote: > On Fri, Sep 01, 2017 at 04:07:16PM +0900, Hoegeun Kwon wrote: > > If scanout started, we should reduce etime by delta_ns. But delta_ns > > is negative if scanout has not started. If delta_ns is negative, > > subtraction of delta_ns fr

Re: [PATCH] drm/vblank: Fix delta_ns to an absolute value

2017-09-01 Thread Ville Syrjälä
On Fri, Sep 01, 2017 at 04:07:16PM +0900, Hoegeun Kwon wrote: > If scanout started, we should reduce etime by delta_ns. But delta_ns > is negative if scanout has not started. If delta_ns is negative, > subtraction of delta_ns from etime increases etime. This is wrong, the > etime should not be incr