[Intel-gfx] The latest Display Status

2011-08-12 Thread Sun, Yi
All, After a new round of display testing, some new bugs filed is as following table. The kernel is: The Linux Kernel, the operating system core itself Kernel: (drm-intel-fixes)d7d39e20cc5e44f915d6c5687f65839632c44381 Some additional commit info: Author: Jesse Barnes Date: Wed Aug 3

Re: [Intel-gfx] i915 native backlight never got merged

2011-08-12 Thread Michel Alexandre Salim
Hi Keith & all, On Thu, 2011-08-11 at 15:28 -0700, Keith Packard wrote: > On Thu, 11 Aug 2011 21:36:38 +0200, Michel Alexandre Salim > wrote: > > > Since there's no known regression introduced by Matthew's patch, could > > it be merged? Feel free to add a > > I've had to amend the patch a bit t

Re: [Intel-gfx] missing acpi backlight bisected to 9661e92c10

2011-08-12 Thread Matthew Garrett
On Mon, Aug 22, 2011 at 10:25:24AM +0430, Ali Gholami Rudi wrote: > Yet, /sys/devices/virtual/backlight/acpi_video0/brightness only > appears after the revert. Seems something changes its behavior > if new_bd->dev.parent is not NULL in backlight_device_register(). Well, yes, if it has a parent i

Re: [Intel-gfx] i915 native backlight never got merged

2011-08-12 Thread Keith Packard
On Fri, 12 Aug 2011 12:11:33 +0200, Michel Alexandre Salim wrote: > Matthew's last patch from July 16th applies without modification on top > of Linux 3.0 and 3.1-rc1, and applies with some offsets on top of > drm-intel-fixes. I must have skipped right over it somehow. > I've eyeballed the cod

Re: [Intel-gfx] The latest Display Status

2011-08-12 Thread Keith Packard
On Fri, 12 Aug 2011 15:58:52 +0800, "Sun, Yi" wrote: Non-text part: multipart/mixed Non-text part: multipart/alternative > drm/i915: enable/disable spread spectrum as needed on mode set > > 40031 > > [Regression] testdisplay can't display

Re: [Intel-gfx] i915 native backlight never got merged

2011-08-12 Thread Kamal Mostafa
On Fri, 2011-08-12 at 06:52 -0700, Keith Packard wrote: > On Fri, 12 Aug 2011 12:11:33 +0200, Michel Alexandre Salim > wrote: > > > Matthew's last patch from July 16th applies without modification on top > > of Linux 3.0 and 3.1-rc1 [...] >> From fa7419eee713b989e2c268c7b06ec9a544a2b647 Mon Sep

[Intel-gfx] [RFC] drm/i915: use PIPE_CONTROL for flushing on gen6+

2011-08-12 Thread Jesse Barnes
I'm trying to figure out why this doesn't work. Anyone have ideas? On gen6+ (well probably since Cantiga actually) we're supposed to use PIPE_CONTROL rather than MI_FLUSH for flushing the pipeline and caches. This patch doesn't cause hangs or crashes in my testing, but does prevent glxgears from

[Intel-gfx] [PATCH 1/2] drm/i915: clear GFX_MODE on IVB at init time

2011-08-12 Thread Jesse Barnes
GFX_MODE controls important behavior like PPGTT, run lists, and TLB invalidate behavior. On the SDV I'm using, the TLB invalidation mode was defaulting to "pipe control only" which meant regular MI_FLUSHes wouldn't actually flush the TLB, leading to all sorts of stale data getting used. So initia

[Intel-gfx] [PATCH 2/2] drm/i915: suspend/resume rps and ring freq state on IVB

2011-08-12 Thread Jesse Barnes
These are enabled at init time, but we need to save/restore them as well. Signed-off-by: Jesse Barnes --- drivers/gpu/drm/i915/i915_suspend.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/i915/i915_suspend.c b/drivers/gpu/drm/i915/i915_suspend.c ind

Re: [Intel-gfx] [PATCH 1/2] drm/i915: clear GFX_MODE on IVB at init time

2011-08-12 Thread Jesse Barnes
On Fri, 12 Aug 2011 14:55:32 -0700 Jesse Barnes wrote: > GFX_MODE controls important behavior like PPGTT, run lists, and TLB > invalidate behavior. On the SDV I'm using, the TLB invalidation mode > was defaulting to "pipe control only" which meant regular MI_FLUSHes > wouldn't actually flush the

Re: [Intel-gfx] [PATCH 1/2] drm/i915: clear GFX_MODE on IVB at init time

2011-08-12 Thread Jesse Barnes
On Fri, 12 Aug 2011 15:18:09 -0700 Jesse Barnes wrote: > On Fri, 12 Aug 2011 14:55:32 -0700 > Jesse Barnes wrote: > > > GFX_MODE controls important behavior like PPGTT, run lists, and TLB > > invalidate behavior. On the SDV I'm using, the TLB invalidation mode > > was defaulting to "pipe contr

Re: [Intel-gfx] [PATCH 1/2] drm/i915: clear GFX_MODE on IVB at init time

2011-08-12 Thread Chris Wilson
On Fri, 12 Aug 2011 15:28:32 -0700, Jesse Barnes wrote: > I915_WRITE(MI_MODE, mode); > + if (IS_GEN7(dev)) > + I915_WRITE(GFX_MODE_GEN7, ((GFX_TLB_INVALIDATE_ALWAYS | > + GFX_REPLAY_MODE) << 16) | > +

Re: [Intel-gfx] [RFC] non-blocking map

2011-08-12 Thread Eric Anholt
On Thu, 11 Aug 2011 18:19:40 -0700, Ben Widawsky wrote: Non-text part: multipart/mixed Non-text part: multipart/signed > On Thu, Aug 11, 2011 at 05:56:21PM -0700, Keith Packard wrote: > > On Thu, 11 Aug 2011 17:47:24 -0700, Ben Widawsky wrote: > > > > > These are dirty hacks just to enable testi

Re: [Intel-gfx] [RFC] drm/i915: use PIPE_CONTROL for flushing on gen6+

2011-08-12 Thread Eric Anholt
On Fri, 12 Aug 2011 11:18:45 -0700, Jesse Barnes wrote: > I'm trying to figure out why this doesn't work. Anyone have ideas? > > On gen6+ (well probably since Cantiga actually) we're supposed to use > PIPE_CONTROL rather than MI_FLUSH for flushing the pipeline and > caches. This patch doesn't

Re: [Intel-gfx] [RFC] non-blocking map

2011-08-12 Thread Ben Widawsky
On Fri, Aug 12, 2011 at 04:28:17PM -0700, Eric Anholt wrote: > On Thu, 11 Aug 2011 18:19:40 -0700, Ben Widawsky wrote: > Non-text part: multipart/mixed > Non-text part: multipart/signed > > On Thu, Aug 11, 2011 at 05:56:21PM -0700, Keith Packard wrote: > > > On Thu, 11 Aug 2011 17:47:24 -0700, Ben