Re: [Intel-gfx] [Mesa-dev] [PATCH 2/5] intel gen4-5: Compute the interpolation status for every variable in one place.

2012-07-16 Thread Paul Berry
On 30 June 2012 11:50, Olivier Galibert wrote: > The program keys are updated accordingly, but the values are not used > yet. > > Signed-off-by: Olivier Galibert > --- > src/mesa/drivers/dri/i965/brw_clip.c| 82 > ++- > src/mesa/drivers/dri/i965/brw_clip.h

Re: [Intel-gfx] [Mesa-dev] [PATCH 1/5] intel gen4/5: fix GL_VERTEX_PROGRAM_TWO_SIDE.

2012-07-16 Thread Paul Berry
On 30 June 2012 11:50, Olivier Galibert wrote: > There was... confusion about which register goes where. With that > patch urb_setup is in line with the vue setup, even when these > annoying backcolor slots are used. And in addition the stray mov into > lalaland is avoided when only one of the

Re: [Intel-gfx] [Mesa-dev] [PATCH 0/5] First batch of gm45 clipping/interpolation fixes

2012-07-16 Thread Paul Berry
On 14 July 2012 02:21, Olivier Galibert wrote: > On Fri, Jul 13, 2012 at 02:45:10PM -0700, Kenneth Graunke wrote: > > Sorry...been really busy, and most of us haven't actually spent much if > > any time in the clipper shaders. I'll try and review it within a week. > > Ok cool, lack of time is so

Re: [Intel-gfx] [PATCH 0/4] [RFC] use HW watchdog timer

2012-07-16 Thread Daniel Vetter
On Mon, Jul 16, 2012 at 11:51:55AM -0700, Ben Widawsky wrote: > This was my pet project for the last few days, but I have to take a > break from working on it for now to do some real work ;-). The patches > compile, and pass a basic test, but that's about it. There is still > quite a bit of work le

Re: [Intel-gfx] Can't start X headless

2012-07-16 Thread Adam Jackson
On Mon, 2012-07-16 at 09:23 +0100, Chris Wilson wrote: > On Mon, 16 Jul 2012 16:18:19 +0800, Chris wrote: > > Hi All, > > > > I am trying desperately to find a working X config that would allow me > > to leverage the intel driver without a monitor being plugged in > > (Headless). We need to do so

Re: [Intel-gfx] [PATCH] drm/i915: Ajdust down threshold in intel_pm.

2012-07-16 Thread Daniel Vetter
On Wed, Jul 04, 2012 at 09:52:11AM +0200, Daniel Vetter wrote: > On Tue, Jul 03, 2012 at 02:16:42PM -0700, Stéphane Marchesin wrote: > > The up and down thresholds are very asymetric, so it is possible > > to have a case where a spike of rendering increases the GPU clock to > > the max (because the

[Intel-gfx] [PATCH 4/4] drm/i915: Display the failing seqno

2012-07-16 Thread Ben Widawsky
We'd really want to do a lot more here, but just as the demo for now. Signed-off-by: Ben Widawsky --- drivers/gpu/drm/i915/i915_irq.c | 14 ++ 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/i915/i915_irq.c b/drivers/gpu/drm/i915/i915_irq.c index a37637

[Intel-gfx] [PATCH 3/4] drm/i915: Add a breadcrumb

2012-07-16 Thread Ben Widawsky
Getting the watchdog interrupt is great, but knowing the exact batch the watchdog fired on is even better. There are other ways to do this without the watchdog, but having the watchdog makes it quite simple. The idea is to simply emit the seqno before the batchbuffer begins running, and if the wat

[Intel-gfx] [PATCH 2/4] drm/i915: Turn on watchdog interrupts

2012-07-16 Thread Ben Widawsky
Now that we are actually using the watch timers, enabling the interrupts just makes sense. Signed-off-by: Ben Widawsky --- drivers/gpu/drm/i915/i915_irq.c | 8 drivers/gpu/drm/i915/i915_reg.h | 3 +++ drivers/gpu/drm/i915/intel_ringbuffer.c | 4 ++-- 3 files changed, 13

[Intel-gfx] [PATCH 1/4] drm/i915: Use HW watchdog for each batch

2012-07-16 Thread Ben Widawsky
The HW watchdog exists for all the rings. It's just a register, but if we writing in the command stream it has the effect we want of generating interrupts if a given batch is taking too long. Unfortunately, our hardware doesn't support interrupts on the blit ring. We still need the software watchd

[Intel-gfx] [PATCH 0/4] [RFC] use HW watchdog timer

2012-07-16 Thread Ben Widawsky
This was my pet project for the last few days, but I have to take a break from working on it for now to do some real work ;-). The patches compile, and pass a basic test, but that's about it. There is still quite a bit of work left to make this useful. The easiest thing would be to tie this into er

Re: [Intel-gfx] [PATCH 3/3] drm/i915: Cleanup context switching through do_switch()

2012-07-16 Thread Daniel Vetter
On Sun, Jul 15, 2012 at 12:34:24PM +0100, Chris Wilson wrote: > When bug hunting, I found the interface to do_switch() overly > complicated and I believe festered the earlier bug. This aims to make > the code a little clearer. > > Signed-off-by: Chris Wilson > Cc: Ben Widawsky I've picked up pa

[Intel-gfx] [PATCH 2/2] tests/gem_cacheing: Exercise snoop coherency

2012-07-16 Thread Chris Wilson
This is based on tests/gem_partial_pwrite_pread which aims to detect incoherency between CPU reads and writes to a bo whilst using it as a source and target for GPU writes. Signed-off-by: Chris Wilson --- tests/.gitignore |1 + tests/Makefile.am|1 + tests/gem_cacheing.c | 274 +

[Intel-gfx] [PATCH 1/2] Add bare-metal interface to adjust cacheing (i.e. snoop status) of a bo

2012-07-16 Thread Chris Wilson
Signed-off-by: Chris Wilson --- lib/drmtest.c | 52 lib/drmtest.h |3 +++ 2 files changed, 55 insertions(+) diff --git a/lib/drmtest.c b/lib/drmtest.c index 871c1d0..66f2887 100644 --- a/lib/drmtest.c +++ b/lib/drmtest.c @@ -252,6 +252,5

Re: [Intel-gfx] [PATCH 3/3] drm/i915: Cleanup context switching through do_switch()

2012-07-16 Thread Ben Widawsky
On Sun, 15 Jul 2012 12:34:24 +0100 Chris Wilson wrote: > When bug hunting, I found the interface to do_switch() overly > complicated and I believe festered the earlier bug. This aims to make > the code a little clearer. Would you be willing to split this up into 2 patches? One which reorganizes

Re: [Intel-gfx] [PATCH] drm/i915: Add -EIO to the list of known errors for __wait_seqno

2012-07-16 Thread Daniel Vetter
On Mon, Jul 16, 2012 at 01:05:34PM +0100, Chris Wilson wrote: > This prevents a WARN introduced with > > commit de2b998552c1534e87bfbc51ec5734b02bc89020 > Author: Daniel Vetter > Date: Wed Jul 4 22:52:50 2012 +0200 > > drm/i915: don't return a spurious -EIO from intel_ring_begin >

[Intel-gfx] [PATCH] drm/i915: Add -EIO to the list of known errors for __wait_seqno

2012-07-16 Thread Chris Wilson
This prevents a WARN introduced with commit de2b998552c1534e87bfbc51ec5734b02bc89020 Author: Daniel Vetter Date: Wed Jul 4 22:52:50 2012 +0200 drm/i915: don't return a spurious -EIO from intel_ring_begin Signed-off-by: Chris Wilson --- drivers/gpu/drm/i915/i915_gem.c |1 + 1

Re: [Intel-gfx] [PATCH] drm/i915: set IDICOS to medium uncore resources

2012-07-16 Thread Chris Wilson
On Mon, 18 Jun 2012 10:35:54 +0200, Daniel Vetter wrote: > On Fri, May 04, 2012 at 06:58:59PM -0700, Ben Widawsky wrote: > > I'm seeing about a 5% FPS improvement across various benchmarks on my > > IVB i3. Rumor has it that the higher end parts show even more benefit. > > > > This derives from a

Re: [Intel-gfx] [PATCH 1/3] drm/i915: Flush the context object from the CPU caches upon switching

2012-07-16 Thread Daniel Vetter
On Sun, Jul 15, 2012 at 08:09:36PM +0100, Chris Wilson wrote: > On Sun, 15 Jul 2012 17:16:34 +0200, Daniel Vetter wrote: > > On Sun, Jul 15, 2012 at 12:34:22PM +0100, Chris Wilson wrote: > > > The issue is that we stale data in the CPU caches, when we come to > > > swap-out the object, the CPU may

Re: [Intel-gfx] Can't start X headless

2012-07-16 Thread Chris Wilson
On Mon, 16 Jul 2012 16:18:19 +0800, Chris wrote: > Hi All, > > I am trying desperately to find a working X config that would allow me > to leverage the intel driver without a monitor being plugged in > (Headless). We need to do some opengl rendering which are later saved > on the disk as images a

[Intel-gfx] Can't start X headless

2012-07-16 Thread Chris
Hi All, I am trying desperately to find a working X config that would allow me to leverage the intel driver without a monitor being plugged in (Headless). We need to do some opengl rendering which are later saved on the disk as images and the jobs are executed on servers which by nature are headle