[git pull] i915 regressions

[git pull] i915 regressions

[Chris Wilson]

Hi Linus,

  still a fair few regressions popping up in the i915 driver. Julia
spotted an old bug in our error handling that could cause the aperture to
be consumed with unpinnable buffers, Keith found that the wait-for-vblank
handling changes were not correct for DP, Jesse failed at basic math (and
nobody spotted the mistake!) and I introduced a refleak during eviction if
we were interrupted by a signal.
-Chris

The following changes since commit c6ea21e35bf3691cad59647c771e6606067f627d:

  Merge git://git.kernel.org/pub/scm/linux/kernel/git/sfrench/cifs-2.6 (2010-10-01 15:03:37 -0700)

are available in the git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/ickle/drm-intel.git drm-intel-fixes

Chris Wilson (1):
      drm/i915: Fix refleak during eviction.

Jesse Barnes (1):
      drm/i915: fix GMCH power reporting

Julia Lawall (1):
      drivers/gpu/drm/i915/i915_gem.c: Add missing error handling code

Keith Packard (2):
      drm/i915: vblank status not valid while training display port
      drm/i915: Use pipe state to tell when pipe is off

 drivers/gpu/drm/i915/i915_dma.c       |    6 ++--
 drivers/gpu/drm/i915/i915_gem.c       |    2 +
 drivers/gpu/drm/i915/i915_gem_evict.c |   45 ++++++++++-------------
 drivers/gpu/drm/i915/intel_display.c  |   62 ++++++++++++++++++++-------------
 drivers/gpu/drm/i915/intel_dp.c       |   19 +++++-----
 drivers/gpu/drm/i915/intel_drv.h      |    1 -
 6 files changed, 72 insertions(+), 63 deletions(-)

-- 
Chris Wilson, Intel Open Source Technology Centre

Re: [git pull] i915 regressions

[Jesse Barnes]

On Sun, 03 Oct 2010 11:33:36 +0100
Chris Wilson <chris@chris-wilson.co.uk> wrote:

> Hi Linus,
> 
>   still a fair few regressions popping up in the i915 driver. Julia
> spotted an old bug in our error handling that could cause the
> aperture to be consumed with unpinnable buffers, Keith found that the
> wait-for-vblank handling changes were not correct for DP, Jesse
> failed at basic math (and nobody spotted the mistake!) and I

I think I'm innocent on this one!  I think Eric just introduced a merge
error when he pulled in the i915 changes for IPS.  I was using do_div
(which modifies the first arg), but Eric changed to to div_u64 to fix a
build error of some kind, but forgot to fix things up due to the
changed semantics.

At any rate, it's a good fix and gives the IPS driver good GMCH power
values again.

-- 
Jesse Barnes, Intel Open Source Technology Center