Re: [Intel-gfx] [PATCH 09/13] drm/i915: add reset_state for hw_contexts

2013-02-27 Thread Chris Wilson
On Tue, Feb 26, 2013 at 05:47:12PM -0800, Ian Romanick wrote: > On 02/26/2013 03:05 AM, Mika Kuoppala wrote: > >For arb-robustness, every context needs to have it's own > >reset state tracking. Default context will be handled in a identical > >way as the no-context case in further down in the patch

Re: [Intel-gfx] [PATCH 09/13] drm/i915: add reset_state for hw_contexts

2013-02-27 Thread Mika Kuoppala
Ian Romanick writes: > On 02/26/2013 03:05 AM, Mika Kuoppala wrote: >> For arb-robustness, every context needs to have it's own >> reset state tracking. Default context will be handled in a identical >> way as the no-context case in further down in the patch set. >> For no-context case, the reset

Re: [Intel-gfx] [PATCH 6/6] drm/i915: More HSW PM clean up.

2013-02-27 Thread Paulo Zanoni
Hi 2013/2/25 Rodrigo Vivi : > This is the last cleaning up patch for HSW, letting render standby > programming sequence like the documented one at HSW PM programing guide. > > Signed-off-by: Rodrigo Vivi > --- > drivers/gpu/drm/i915/intel_pm.c | 9 - > 1 file changed, 9 deletions(-) > >

Re: [Intel-gfx] [PATCH] drm/i915: Increase the RC6p threshold.

2013-02-27 Thread Jesse Barnes
Great, then if it helps Stephane, I think we should merge it. Acked-by-top-post: Jesse Barnes On Wed, 27 Feb 2013 05:50:08 + "Zhang, Ouping" wrote: > Increase the RC6p threshold from 10 to 15, there is no power > difference between the two values. > for example: > (1) I915_WRITE(G

Re: [Intel-gfx] [PATCH 09/13] drm/i915: add reset_state for hw_contexts

2013-02-27 Thread Jesse Barnes
On Tue, 26 Feb 2013 17:47:12 -0800 Ian Romanick wrote: > On 02/26/2013 03:05 AM, Mika Kuoppala wrote: > > For arb-robustness, every context needs to have it's own > > reset state tracking. Default context will be handled in a identical > > way as the no-context case in further down in the patch s

Re: [Intel-gfx] [PATCH] sna: Fix comment on GCC version check

2013-02-27 Thread Chris Wilson
On Tue, Feb 26, 2013 at 03:44:13PM +0100, Sedat Dilek wrote: > On Tue, Feb 26, 2013 at 3:35 PM, Chris Wilson > wrote: > > On Tue, Feb 26, 2013 at 03:27:52PM +0100, Sedat Dilek wrote: > >> The comment was introduced with... > >> > >> commit 528dbf9ebb2688f476ef283be59d0f2232159dcb > >> "sna: Fix b

Re: [Intel-gfx] [PATCH] sna: Fix comment on GCC version check

2013-02-27 Thread Sedat Dilek
On Wed, Feb 27, 2013 at 6:37 PM, Chris Wilson wrote: > On Tue, Feb 26, 2013 at 03:44:13PM +0100, Sedat Dilek wrote: >> On Tue, Feb 26, 2013 at 3:35 PM, Chris Wilson >> wrote: >> > On Tue, Feb 26, 2013 at 03:27:52PM +0100, Sedat Dilek wrote: >> >> The comment was introduced with... >> >> >> >> co

[Intel-gfx] Instrumenting rendercopy, the last bit

2013-02-27 Thread Damien Lespiau
This continues (and finishes for now) the series started: http://lists.freedesktop.org/archives/intel-gfx/2013-February/024954.html To instrument rendercopy (produce an .aub file) for use with the simulator. ___ Intel-gfx mailing list Intel-gfx@lists.fr

[Intel-gfx] [PATCH] tests: Make it easy to tell if rendercopy is doing the right thing

2013-02-27 Thread Damien Lespiau
When dumping an .aub file, gem_render_linear_blit samples a texture to write it in a render target 1:1. When enabling new platforms, it's really handy to be able to see if you are actually rendering to the target which is hard to see currently as all the buffers are initialized with incrementing n

Re: [Intel-gfx] [PATCH] tests: Make it easy to tell if rendercopy is doing the right thing

2013-02-27 Thread Chris Wilson
On Wed, Feb 27, 2013 at 06:28:07PM +, Damien Lespiau wrote: > When dumping an .aub file, gem_render_linear_blit samples a texture to > write it in a render target 1:1. > > When enabling new platforms, it's really handy to be able to see if you > are actually rendering to the target which is ha

Re: [Intel-gfx] [PATCH 2/8] drm/i915: Use cpu_transcoder for HSW_TVIDEO_DIP_* instead of pipe

2013-02-27 Thread Paulo Zanoni
Hi 2013/2/25 Rodrigo Vivi : > While old platforms had 3 transcoders and 3 pipes (1:1), HSW has > 4 transcoders and 3 pipes. > These regs were being used only by HDMI code where pipe is always the same > thing as cpu_transcoder. > This patch allow us to use them for DP, specially for TRANSCODER_EDP

Re: [Intel-gfx] [PATCH 3/8] drm/i915: Added SDP and VSC structures for handling PSR for eDP

2013-02-27 Thread Paulo Zanoni
Hi 2013/2/25 Rodrigo Vivi : > From: Shobhit Kumar > > Signed-off-by: Sateesh Kavuri > > v2: Modified and corrected the structures to be more in line for > kernel coding guidelines and rebased the code on Paulo's DP patchset > > Signed-off-by: Shobhit Kumar > > v3: removing unecessary identation

Re: [Intel-gfx] [PATCH] sna: Fix comment on GCC version check

2013-02-27 Thread Chris Wilson
On Wed, Feb 27, 2013 at 06:59:58PM +0100, Sedat Dilek wrote: > How can I compare a gcc-4.6 and clang-3.2 compiled binaries? > Thinking of miscompiling and/or benchmarking. The real test is whether you can measure a difference in a benchmark like cairo-perf-trace. There are a few traces that are se

Re: [Intel-gfx] [git pull] drm merge for 3.9-rc1

2013-02-27 Thread Sedat Dilek
Hi, I am seeing this also on Linux-Next. /var/log/kern.log:Feb 27 22:52:35 fambox kernel: [ 28.202381] [drm:intel_dp_aux_wait_done] *ERROR* dp aux hw did not signal timeout (has irq: 1)! /var/log/kern.log:Feb 27 22:52:35 fambox kernel: [ 28.210588] [drm:intel_dp_aux_wait_done] *ERROR* dp aux

Re: [Intel-gfx] [git pull] drm merge for 3.9-rc1

2013-02-27 Thread Sedat Dilek
t; (has irq: 1)! > > This seems to be hard reproducible... > Laptop-LCD... Sandybridge Mobile-GT2. > > Is there a way to force the error? > > Possible patch see [1]. > > - Sedat - > > [1] https://patchwork.kernel.org/patch/2192721/ Hmm, I tried to apply the test-patch a

Re: [Intel-gfx] [PATCH] sna: Fix comment on GCC version check

2013-02-27 Thread Sedat Dilek
On Wed, Feb 27, 2013 at 11:27 PM, Chris Wilson wrote: > On Wed, Feb 27, 2013 at 06:59:58PM +0100, Sedat Dilek wrote: >> How can I compare a gcc-4.6 and clang-3.2 compiled binaries? >> Thinking of miscompiling and/or benchmarking. > > The real test is whether you can measure a difference in a bench

Re: [Intel-gfx] [PATCH v.2 03/12] DRM/i915: Convert HPD interrupts to make use of HPD pin assignment in encoders.

2013-02-27 Thread Chris Wilson
On Mon, Feb 25, 2013 at 12:06:50PM -0500, Egbert Eich wrote: > This allows to enable HPD interrupts for individual pins to only receive > hotplug events from lines which are connected and working. > > Signed-off-by: Egbert Eich > --- > drivers/gpu/drm/i915/i915_irq.c | 160 +

Re: [Intel-gfx] [PATCH v.2 06/12] DRM/i915: Add HPD IRQ storm detection.

2013-02-27 Thread Chris Wilson
On Mon, Feb 25, 2013 at 12:06:53PM -0500, Egbert Eich wrote: > Add a hotplug IRQ storm detection (triggered when a hotplug interrupt > fires more than 5 times / sec). > Mask out this specific interrupt and revert to polling on the associated > output. Hmm, not convinced that just marking it as HP

Re: [Intel-gfx] [PATCH v.2 00/12] Detect and deal with Interrupt 'Storms' from noisy Hotplug Lines.

2013-02-27 Thread Chris Wilson
On Mon, Feb 25, 2013 at 12:06:47PM -0500, Egbert Eich wrote: > I've reworked my 'hotplug interrupt storm detection'-patches and > included most of Daniel's suggestions. > I've looked into adding EDID caching but since this requires some > larger scale changes and some changes outside of the Intel

Re: [Intel-gfx] [PATCH 09/13] drm/i915: add reset_state for hw_contexts

2013-02-27 Thread Ian Romanick
On 02/27/2013 01:13 AM, Chris Wilson wrote: On Tue, Feb 26, 2013 at 05:47:12PM -0800, Ian Romanick wrote: On 02/26/2013 03:05 AM, Mika Kuoppala wrote: For arb-robustness, every context needs to have it's own reset state tracking. Default context will be handled in a identical way as the no-cont

[Intel-gfx] [i-g-t][PATH] debugger: Include path for cairo to fix compiler error

2013-02-27 Thread Xiang, Haihao
From: "Xiang, Haihao" CC eudb.o In file included from eudb.c:44:0: ../lib/drmtest.h:34:19: fatal error: cairo.h: No such file or directory compilation terminated. make[3]: *** [eudb.o] Error 1 Signed-off-by: Xiang, Haihao --- debugger/Makefile.am |3 ++- 1 file changed, 2 insertions(