Re: [PATCH v2] drm/i915/bdw: BDW Software Turbo

Sun, Daisy
2014-07-15T06:35:22Z

Hi Daniel, Chris

The concern for traditional X and media server do make sense. I'll update the patch with RP_UP_EI_INTERRUPT as trigger instead of the page flip.
Thanks for the valuable input.

- Daisy

-----Original Message-----
From: daniel.vetter@ffwll.ch [mailto:daniel.vetter@ffwll.ch] On Behalf Of Daniel Vetter
Sent: Monday, July 14, 2014 12:04 AM
To: Sun, Daisy
Cc: Chris Wilson; intel-gfx@lists.freedesktop.org
Subject: Re: [Intel-gfx] [PATCH v2] drm/i915/bdw: BDW Software Turbo

On Mon, Jul 14, 2014 at 8:59 AM, Daniel Vetter <daniel@ffwll.ch> wrote:
> On Mon, Jul 14, 2014 at 04:22:44AM +0000, Sun, Daisy wrote:
>> 3) The function will be called when flip happened, this should cover
>> most of the cases. One exception is background media process without
>> any display output, it's relatively rare. Please let me know if you
>> have concern on other cases, I will try to cover it definitely.
>
> Traditional X never flips. And we kinda have to keep this working.
> Instead of checking when flipping we need to check at regular time
> intervals I guess, for as long as the gt is busy.

Oh and transcode servers are a real thing apparently. They also never flip, and we actually care from a business pov ...
-Daniel
--
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch