Commit 95cb1b02 authored by Daniel Vetter's avatar Daniel Vetter

drm/i915: don't frob the vblank ts in finish_page_flip

Now that we correctly generate it, this hack is no longer required (and
might actually paper over a serious bug).

pageflip timestamps are sanity check in the latest version of the flip-test
in intel-gpu-tools.

v2: Also remove the gettimeofday(&now) which is no longer used.
Noticed by Mario Kleiner.

Reviewed-by: mario.kleiner@tuebingen.mpg.de
Tested-by: default avatarImre Deak <imre.deak@intel.com>
Signed-off-by: default avatarDaniel Vetter <daniel.vetter@ffwll.ch>
parent 74d44445
...@@ -6181,15 +6181,13 @@ static void do_intel_finish_page_flip(struct drm_device *dev, ...@@ -6181,15 +6181,13 @@ static void do_intel_finish_page_flip(struct drm_device *dev,
struct intel_unpin_work *work; struct intel_unpin_work *work;
struct drm_i915_gem_object *obj; struct drm_i915_gem_object *obj;
struct drm_pending_vblank_event *e; struct drm_pending_vblank_event *e;
struct timeval tnow, tvbl; struct timeval tvbl;
unsigned long flags; unsigned long flags;
/* Ignore early vblank irqs */ /* Ignore early vblank irqs */
if (intel_crtc == NULL) if (intel_crtc == NULL)
return; return;
do_gettimeofday(&tnow);
spin_lock_irqsave(&dev->event_lock, flags); spin_lock_irqsave(&dev->event_lock, flags);
work = intel_crtc->unpin_work; work = intel_crtc->unpin_work;
if (work == NULL || !work->pending) { if (work == NULL || !work->pending) {
...@@ -6203,25 +6201,6 @@ static void do_intel_finish_page_flip(struct drm_device *dev, ...@@ -6203,25 +6201,6 @@ static void do_intel_finish_page_flip(struct drm_device *dev,
e = work->event; e = work->event;
e->event.sequence = drm_vblank_count_and_time(dev, intel_crtc->pipe, &tvbl); e->event.sequence = drm_vblank_count_and_time(dev, intel_crtc->pipe, &tvbl);
/* Called before vblank count and timestamps have
* been updated for the vblank interval of flip
* completion? Need to increment vblank count and
* add one videorefresh duration to returned timestamp
* to account for this. We assume this happened if we
* get called over 0.9 frame durations after the last
* timestamped vblank.
*
* This calculation can not be used with vrefresh rates
* below 5Hz (10Hz to be on the safe side) without
* promoting to 64 integers.
*/
if (10 * (timeval_to_ns(&tnow) - timeval_to_ns(&tvbl)) >
9 * crtc->framedur_ns) {
e->event.sequence++;
tvbl = ns_to_timeval(timeval_to_ns(&tvbl) +
crtc->framedur_ns);
}
e->event.tv_sec = tvbl.tv_sec; e->event.tv_sec = tvbl.tv_sec;
e->event.tv_usec = tvbl.tv_usec; e->event.tv_usec = tvbl.tv_usec;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment