[Intel-gfx] [PATCH 3/3] drm/i915: extract constant offset setting

2011-10-22 Thread Ben Widawsky
Simple refactor. Cc: Keith Packard Reviewed-by: Chris Wilson Signed-off-by: Ben Widawsky --- drivers/gpu/drm/i915/i915_gem_execbuffer.c | 82 1 files changed, 46 insertions(+), 36 deletions(-) diff --git a/drivers/gpu/drm/i915/i915_gem_execbuffer.c b/drivers/gp

[Intel-gfx] [PATCH 2/3] drm/i915: Force sync command ordering (Gen6+)

2011-10-22 Thread Ben Widawsky
The docs say this is required for Gen7, and since the bit was added for Gen6, we are also setting it there pit pf paranoia. Particularly as Chris points out, if PIPE_CONTROL counts as a 3d state packet. This was found through doc inspection by Ken and applies to Gen6+; Cc: Keith Packard Reported

[Intel-gfx] [PATCH 1/3] drm/i915: relative_constants_mode race fix

2011-10-22 Thread Ben Widawsky
After my refactoring, Chris noticed that we had a bug. dev_priv keeps track of the current addressing mode that gets set at execbuffer time. Unfortunately the existing code was doing this before acquiring struct_mutex which leaves a race with another thread also doing an execbuffer. If that wasn't

Re: [Intel-gfx] [PATCH] test/testdisplay:add cursor test

2011-10-22 Thread Eugeni Dodonov
On Oct 22, 2011 10:13 AM, "Hai Lan" wrote: > > Signed-off-by: Hai Lan I like this one, simple and functional. Reviewed-by: Eugeni Dodonov ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-g

Re: [Intel-gfx] CRT not detected via hotplug on resume

2011-10-22 Thread Rainer Dorsch
: http://bokomoko.de/~rd/GMA3500-black-screen/intel_reg_dump-20111022-working- screen.dump and a not working resume http://bokomoko.de/~rd/GMA3500-black-screen/intel_reg_dump-20111022-black- screen.dump Here is the diff of the register dumps, a few registers are different: blackbox:~# diff -u

Re: [Intel-gfx] [PATCH] test/testdisplay:add cursor test

2011-10-22 Thread Lan, Hai
> what does this test do? For the unknowledgeable user what is supposed to > happen, that means, when is the test successful and when not? > I wrote this patch to test whether the cursor can work OK. It needs the tester to watch the screen. If the tester can see the white cursor move from left to

Re: [Intel-gfx] [PATCH] test/testdisplay:add cursor test

2011-10-22 Thread Paul Menzel
Dear Hai, Am Samstag, den 22.10.2011, 22:03 -0400 schrieb Hai Lan: what does this test do? For the unknowledgeable user what is supposed to happen, that means, when is the test successful and when not? > Signed-off-by: Hai Lan > --- > tests/testdisplay.c | 32 +--

[Intel-gfx] [PATCH] test/testdisplay:add cursor test

2011-10-22 Thread Hai Lan
Signed-off-by: Hai Lan --- tests/testdisplay.c | 32 +--- 1 files changed, 25 insertions(+), 7 deletions(-) diff --git a/tests/testdisplay.c b/tests/testdisplay.c index 01f31dc..174d5c7 100644 --- a/tests/testdisplay.c +++ b/tests/testdisplay.c @@ -75,7 +75,7 @@ str