<p dir="ltr">Yes.<br>
On Oct 12, 2013 1:18 AM, "Daniel Vetter" <<a href="mailto:daniel@ffwll.ch">daniel@ffwll.ch</a>> wrote:<br>
><br>
> On Fri, Oct 11, 2013 at 04:31:38PM +0200, Mario Kleiner wrote:<br>
> > Daniel, Ville,<br>
> ><br>
> > i tested Ville's patch series for the scanoutpos improvements on a<br>
> > GMA-950, on top of airlied's current drm-next branch.<br>
> ><br>
> > There's one issue: The variable "position" in<br>
> > i915_get_crtc_scanoutpos() must be turned from a u32 into a int,<br>
> > otherwise funny sign errors happen and we end up with *vpos being<br>
> > off by multiple million scanlines and timestamps being off by over<br>
> > 60 seconds.<br>
> ><br>
> > Other than that looks good. Execution time is now better:<br>
> ><br>
> > Before uncore.lock addition: 3-4 usecs execution time for the<br>
> > scanoutpos query on my machine. After uncore.lock addition<br>
> > (3.12.0-rc3) 9-20 usecs, sometimes repetition of the timing loop<br>
> > triggered. After Ville's patches down to typically 3-8 usecs,<br>
> > occassionally spiking to almost 20 usecs.<br>
> ><br>
> > I'll make my patches for the realtime kernel + increased accuracy on<br>
> > top of drm-next + Ville's patches.<br>
><br>
> So official reviewed-by/tested-by from you on Ville's latest patches in<br>
> this thread?</p>
<p dir="ltr">Yes.<br>
-mario</p>
<p dir="ltr">> -Daniel<br>
> --<br>
> Daniel Vetter<br>
> Software Engineer, Intel Corporation<br>
> +41 (0) 79 365 57 48 - <a href="http://blog.ffwll.ch">http://blog.ffwll.ch</a><br>
</p>