Re: [Intel-gfx] Performance regression using Firefox between 2.14 and 2.15

I was able to bisect the performance regression, the commit which introduced it was: commit 686018f283f1d131073ef5917213e6a8ac013f26 Author: Chris Wilson ch...@chris-wilson.co.uk Date: Tue Apr 12 08:23:04 2011 +0100 Turn relaxed-fencing off by default for older (pre-G33) chipset Are there

Re: [Intel-gfx] Performance regression using Firefox between 2.14 and 2.15

Hi, We think we've fixed all the related bugs in the kernel, so you can try flipping the switch and see what happens. Manuelle re-enabling relaxedfencing brings speed back to normal, quite a few pages that scrolled slow in firefox are now as snappy as before 2.15 :) However, I think your

Re: [Intel-gfx] Memory corruption on hibernate/thaw with KMS

On Tue, 2011-09-27 at 16:12 +1000, Bojan Smojver wrote: Does anyone have any idea what's causing this? Or better, how to fix it? I can confirm that as of kernel 3.1.0-rc7 this is still a problem. Anyone? -- Bojan ___ Intel-gfx mailing list

[Intel-gfx] [PATCH 00/24] MacBook Air patch sequence (v2)

Ok, so I've split all of the changes into bite-sized pieces so that they should make sense individually now. I've also added the same asynchronous power control to the panel power, this reduces the module load time down to about 700ms on my MacBook Air, which is pretty nice. Given the length of

[Intel-gfx] [PATCH 04/21] drm/i915: Only use VBT panel mode on eDP if no EDID is found

We're going to assume that EDID is more reliable than the VBT tables for eDP panels, which is notably true on MacBook machines where the VBT contains completely bogus data. Signed-off-by: Keith Packard kei...@keithp.com --- drivers/gpu/drm/i915/intel_dp.c | 20 ++-- 1 files

[Intel-gfx] [PATCH 01/21] drm/i915: Enable digital port hotplug on PCH systems

We were relying on the BIOS to set these bits, which doesn't always happen. Signed-off-by: Keith Packard kei...@keithp.com --- drivers/gpu/drm/i915/i915_irq.c | 24 drivers/gpu/drm/i915/i915_reg.h |5 - 2 files changed, 28 insertions(+), 1 deletions(-) diff

[Intel-gfx] [PATCH 03/21] drm/i915: Remove extra 300ms delay during eDP mode setting

There's no reason to enforce a 300ms delay during eDP mode setting. Signed-off-by: Keith Packard kei...@keithp.com --- drivers/gpu/drm/i915/intel_dp.c |7 --- 1 files changed, 0 insertions(+), 7 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c

[Intel-gfx] [PATCH 02/21] drm/i915: Shut down PCH interrupts during irq_uninstall

This masks out all interrupts and ack's any pending ones at IRQ uninstall time to make sure we don't receive any unexpected interrupts later on. Signed-off-by: Keith Packard kei...@keithp.com --- drivers/gpu/drm/i915/i915_irq.c |4 1 files changed, 4 insertions(+), 0 deletions(-) diff

[Intel-gfx] [PATCH 05/21] drm/i915: Check eDP power when doing aux channel communications

Verify that the eDP VDD is on, either with the panel being on or with the VDD force-on bit being set. This demonstrates that in many instances, VDD is not on when needed, which leads to failed EDID communications. Signed-off-by: Keith Packard kei...@keithp.com ---

[Intel-gfx] [PATCH 06/21] drm/i915: Unlock PCH_PP_CONTROL always

Avoid any question about locked registers by just writing the unlock pattern with every write to the register. Signed-off-by: Keith Packard kei...@keithp.com --- drivers/gpu/drm/i915/i915_reg.h |1 + drivers/gpu/drm/i915/intel_dp.c | 14 +- 2 files changed, 14 insertions(+), 1

[Intel-gfx] [PATCH 15/21] drm/i915: Move eDP panel fixed mode from dev_priv to intel_dp

This value doesn't come directly from the VBT, and so is rather specific to the particular DP output. Signed-off-by: Keith Packard kei...@keithp.com --- drivers/gpu/drm/i915/i915_drv.h |1 - drivers/gpu/drm/i915/intel_dp.c | 35 --- 2 files changed, 16

[Intel-gfx] [PATCH 17/21] drm/i915: Create helper functions to determine eDP power state

We need to check eDP VDD force and panel on in several places, so create some simple helper functions to avoid duplicating code. Signed-off-by: Keith Packard kei...@keithp.com --- drivers/gpu/drm/i915/intel_dp.c | 39 ++- 1 files changed, 26 insertions(+),

[Intel-gfx] [PATCH 07/21] drm/i915: Check for eDP inside intel_edp_panel_vdd_on/off

Cleans up code dealing with eDP VDD a bit. Remove redundant checks in callers Signed-off-by: Keith Packard kei...@keithp.com --- drivers/gpu/drm/i915/intel_dp.c | 20 ++-- 1 files changed, 10 insertions(+), 10 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_dp.c

[Intel-gfx] [PATCH 13/21] drm/i915: Place long delays after each eDP VDD operation

This ensures that the panel power sequencing is complete before attempting to communicate over the aux channel. Signed-off-by: Keith Packard kei...@keithp.com --- drivers/gpu/drm/i915/intel_dp.c | 12 +++- 1 files changed, 11 insertions(+), 1 deletions(-) diff --git

[Intel-gfx] [PATCH 12/21] drm/i915: Ensure eDP powered up during DP_SET_POWER operation in dp_prepare

Any call to intel_dp_sink_dpms must ensure that the panel has power so that the DP_SET_POWER operation will be correctly received. The only one missing this was in intel_dp_prepare. Signed-off-by: Keith Packard kei...@keithp.com --- drivers/gpu/drm/i915/intel_dp.c |2 ++ 1 files changed, 2

[Intel-gfx] [PATCH 16/21] drm/i915: edp_panel_on does not need to return a bool

The return value was unused, so just stop doing that. Signed-off-by: Keith Packard kei...@keithp.com --- drivers/gpu/drm/i915/intel_dp.c |6 ++ 1 files changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c index

[Intel-gfx] [PATCH 11/21] drm/i915: Enable eDP panel power during I2C initialization sequence

The DP i2c initialization code does a couple of i2c transactions, which means that an eDP panel must be powered up. Signed-off-by: Keith Packard kei...@keithp.com --- drivers/gpu/drm/i915/intel_dp.c | 10 +- 1 files changed, 9 insertions(+), 1 deletions(-) diff --git

Re: [Intel-gfx] [PATCH 00/24] MacBook Air patch sequence (v2)

On Thu, Sep 29, 2011 at 06:09:32PM -0700, Keith Packard wrote: Ok, so I've split all of the changes into bite-sized pieces so that they should make sense individually now. I've also added the same asynchronous power control to the panel power, this reduces the module load time down to about