[Intel-gfx] [PATCH 01/10] intel: shared header for shader debugging

2011-07-27 Thread Julien Cristau
On Wed, Jul 27, 2011 at 15:08:08 +, Ben Widawsky wrote: > On Thu, Jul 21, 2011 at 11:22:12PM +0200, Julien Cristau wrote: > > On Thu, Jul 21, 2011 at 13:54:41 +, Ben Widawsky wrote: > > > > > On Tue, Jul 19, 2011 at 11:06:17PM +0200, Julien Cristau wrote: > > > > On Wed, Jul 13, 2011 at

[Intel-gfx] [PATCH 01/10] intel: shared header for shader debugging

2011-07-27 Thread Alan Cox
> Ah, I didn't realise this would run as root. Would /run (or /var/run) > be ok then? Alternately, use an abstract domain socket to not have to > care about the underlying filesystem. Abstract sockets are a bit of a Linux specific feature. You also don't want sockets in /home as it may well be

[Intel-gfx] [PATCH 01/10] intel: shared header for shader debugging

2011-07-27 Thread Ben Widawsky
On Wed, Jul 27, 2011 at 05:16:31PM +0200, Julien Cristau wrote: > On Wed, Jul 27, 2011 at 15:08:08 +, Ben Widawsky wrote: > > > On Thu, Jul 21, 2011 at 11:22:12PM +0200, Julien Cristau wrote: > > > On Thu, Jul 21, 2011 at 13:54:41 +, Ben Widawsky wrote: > > > > > > > On Tue, Jul 19, 2011

[Intel-gfx] [PATCH 01/10] intel: shared header for shader debugging

2011-07-27 Thread Ben Widawsky
On Thu, Jul 21, 2011 at 11:22:12PM +0200, Julien Cristau wrote: > On Thu, Jul 21, 2011 at 13:54:41 +, Ben Widawsky wrote: > > > On Tue, Jul 19, 2011 at 11:06:17PM +0200, Julien Cristau wrote: > > > On Wed, Jul 13, 2011 at 13:51:43 -0700, Ben Widawsky wrote: > > > > > > > +#define

[PATCH v2] drm/radeon: Print gart initialization details on all chipsets

2011-07-27 Thread Tormod Volden
From: Tormod Volden The same way it was already done for r300. Also fix typo in gart warning messages. v2: Replace 0x%08X string format by %p Signed-off-by: Tormod Volden --- On Mon, Jul 25, 2011 at 1:03 PM, Dave Airlie wrote: > Causes a new warning per line on

[PATCH] drm/i915: Add quirk to disable SSC on Sony Vaio Y2

2011-07-27 Thread Michel Alexandre Salim
GP SIGNATURE- -- next part -- An embedded and charset-unspecified text was scrubbed... Name: 0001-drm-i915-Add-quirk-to-disable-SSC-on-Sony-Vaio-Y2.patch URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20110727/3db96967/attachment.ksh>

[PATCH] drm/i915: Add quirk to disable SSC on Sony Vaio Y2

2011-07-27 Thread Michel Alexandre Salim
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Using the new quirk added to support disabling SSC on Lenovo U160 (#36656, commit 435793dfb8aec7b2e19f72d5bce8a22fd0b57839), also register the Vaio as a special case and disable SSC for it. This patch fixes #34437 on fdo bugzilla:

[PATCH] drm/radeon/kms: disable CP interrupt when disabling interrupt

2011-07-27 Thread Jerome Glisse
On Wed, Jul 27, 2011 at 11:30 AM, Alex Deucher wrote: > On Tue, Jul 26, 2011 at 7:43 PM, Jerome Glisse wrote: >> On Tue, Jul 26, 2011 at 5:47 PM, Alex Deucher >> wrote: >>> On Tue, Jul 26, 2011 at 5:28 PM, ? wrote: From: Jerome Glisse Some CP interrupt were left enabled when

[patch] drm/radeon: off by one in check_reg() functions

2011-07-27 Thread Dan Carpenter
This off by one range check was copy and pasted a couple places. It's not really harmful, but we should fix it anyway. Signed-off-by: Dan Carpenter diff --git a/drivers/gpu/drm/radeon/r600_cs.c b/drivers/gpu/drm/radeon/r600_cs.c index db8ef19..cf83aa0 100644 ---

[PATCH] drm/radeon/kms: disable CP interrupt when disabling interrupt

2011-07-27 Thread Alex Deucher
On Tue, Jul 26, 2011 at 7:43 PM, Jerome Glisse wrote: > On Tue, Jul 26, 2011 at 5:47 PM, Alex Deucher > wrote: >> On Tue, Jul 26, 2011 at 5:28 PM, ? wrote: >>> From: Jerome Glisse >>> >>> Some CP interrupt were left enabled when disabling interrupt. >>> >> >> Is there a specific issue this

[PATCH] drm/i915: Add quirk to disable SSC on Sony Vaio Y2

2011-07-27 Thread Michel Alexandre Salim
Using the new quirk added to support disabling SSC on Lenovo U160 (#36656, commit 435793dfb8aec7b2e19f72d5bce8a22fd0b57839), also register the Vaio as a special case and disable SSC for it. This patch fixes #34437 on fdo bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=34437 ---

[Intel-gfx] [PATCH] drm/i915: Hold struct_mutex during hotplug processing

2011-07-27 Thread Jesse Barnes
On Wed, 27 Jul 2011 02:21:24 -0700 Keith Packard wrote: > On Tue, 26 Jul 2011 12:12:25 -0700, Jesse Barnes virtuousgeek.org> wrote: > > > I'd like to amend my reviewed by and say the lock shouldn't be held > > around the call to the drm helper function. It queues some work that > > also takes

[git pull] drm pull for -rc1

2011-07-27 Thread Keith Packard
es/dri-devel/attachments/20110727/665556bb/attachment.pgp>

[Intel-gfx] [PATCH] drm/i915: Hold struct_mutex during hotplug processing

2011-07-27 Thread Keith Packard
text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 189 bytes Desc: not available URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20110727/7e8f8dbf/attachment.pgp>

[PATCH] drm/radeon/kms: fix version comment due to merge timing

2011-07-27 Thread alexdeuc...@gmail.com
From: Alex Deucher Compute cs support was actually added in 2.11.0 rather than 2.10.0, but the patch was written prior. Update comment to match. Signed-off-by: Alex Deucher --- drivers/gpu/drm/radeon/radeon_drv.c |4 ++-- 1 files changed, 2 insertions(+), 2

Re: [Intel-gfx] [PATCH] drm/i915: Hold struct_mutex during hotplug processing

2011-07-27 Thread Keith Packard
On Tue, 26 Jul 2011 12:12:25 -0700, Jesse Barnes jbar...@virtuousgeek.org wrote: I'd like to amend my reviewed by and say the lock shouldn't be held around the call to the drm helper function. It queues some work that also takes the mode config lock, which will break. So you can drop it

Re: [git pull] drm pull for -rc1

2011-07-27 Thread Keith Packard
On Tue, 26 Jul 2011 13:02:35 +0100 (IST), Dave Airlie airl...@linux.ie wrote: (and btw I know Keith has a lot of fixes into next merges, its hard to balance the requirement about not merging with the fact his QA team need to test merged trees and I want to pull what the QA team has tested).

[patch] drm/radeon: off by one in check_reg() functions

2011-07-27 Thread Dan Carpenter
This off by one range check was copy and pasted a couple places. It's not really harmful, but we should fix it anyway. Signed-off-by: Dan Carpenter erro...@gmail.com diff --git a/drivers/gpu/drm/radeon/r600_cs.c b/drivers/gpu/drm/radeon/r600_cs.c index db8ef19..cf83aa0 100644 ---

[PATCH v2] drm/radeon: Print gart initialization details on all chipsets

2011-07-27 Thread Tormod Volden
From: Tormod Volden debian.tor...@gmail.com The same way it was already done for r300. Also fix typo in gart warning messages. v2: Replace 0x%08X string format by %p Signed-off-by: Tormod Volden debian.tor...@gmail.com --- On Mon, Jul 25, 2011 at 1:03 PM, Dave Airlie airl...@gmail.com wrote:

Re: [Intel-gfx] [PATCH 01/10] intel: shared header for shader debugging

2011-07-27 Thread Ben Widawsky
On Thu, Jul 21, 2011 at 11:22:12PM +0200, Julien Cristau wrote: On Thu, Jul 21, 2011 at 13:54:41 +, Ben Widawsky wrote: On Tue, Jul 19, 2011 at 11:06:17PM +0200, Julien Cristau wrote: On Wed, Jul 13, 2011 at 13:51:43 -0700, Ben Widawsky wrote: +#define SHADER_DEBUG_SOCKET

Re: [Intel-gfx] [PATCH 01/10] intel: shared header for shader debugging

2011-07-27 Thread Julien Cristau
On Wed, Jul 27, 2011 at 15:08:08 +, Ben Widawsky wrote: On Thu, Jul 21, 2011 at 11:22:12PM +0200, Julien Cristau wrote: On Thu, Jul 21, 2011 at 13:54:41 +, Ben Widawsky wrote: On Tue, Jul 19, 2011 at 11:06:17PM +0200, Julien Cristau wrote: On Wed, Jul 13, 2011 at 13:51:43

Re: [Intel-gfx] [PATCH 01/10] intel: shared header for shader debugging

2011-07-27 Thread Alan Cox
Ah, I didn't realise this would run as root. Would /run (or /var/run) be ok then? Alternately, use an abstract domain socket to not have to care about the underlying filesystem. Abstract sockets are a bit of a Linux specific feature. You also don't want sockets in /home as it may well be NFS

Re: [PATCH] drm/radeon/kms: disable CP interrupt when disabling interrupt

2011-07-27 Thread Alex Deucher
On Tue, Jul 26, 2011 at 7:43 PM, Jerome Glisse j.gli...@gmail.com wrote: On Tue, Jul 26, 2011 at 5:47 PM, Alex Deucher alexdeuc...@gmail.com wrote: On Tue, Jul 26, 2011 at 5:28 PM,  j.gli...@gmail.com wrote: From: Jerome Glisse jgli...@redhat.com Some CP interrupt were left enabled when

Re: [Intel-gfx] [PATCH 01/10] intel: shared header for shader debugging

2011-07-27 Thread Ben Widawsky
On Wed, Jul 27, 2011 at 05:16:31PM +0200, Julien Cristau wrote: On Wed, Jul 27, 2011 at 15:08:08 +, Ben Widawsky wrote: On Thu, Jul 21, 2011 at 11:22:12PM +0200, Julien Cristau wrote: On Thu, Jul 21, 2011 at 13:54:41 +, Ben Widawsky wrote: On Tue, Jul 19, 2011 at 11:06:17PM

Re: [Intel-gfx] [PATCH] drm/i915: Hold struct_mutex during hotplug processing

2011-07-27 Thread Jesse Barnes
On Wed, 27 Jul 2011 02:21:24 -0700 Keith Packard kei...@keithp.com wrote: On Tue, 26 Jul 2011 12:12:25 -0700, Jesse Barnes jbar...@virtuousgeek.org wrote: I'd like to amend my reviewed by and say the lock shouldn't be held around the call to the drm helper function. It queues some work

Re: [PATCH] drm/radeon/kms: disable CP interrupt when disabling interrupt

2011-07-27 Thread Jerome Glisse
On Wed, Jul 27, 2011 at 11:30 AM, Alex Deucher alexdeuc...@gmail.com wrote: On Tue, Jul 26, 2011 at 7:43 PM, Jerome Glisse j.gli...@gmail.com wrote: On Tue, Jul 26, 2011 at 5:47 PM, Alex Deucher alexdeuc...@gmail.com wrote: On Tue, Jul 26, 2011 at 5:28 PM,  j.gli...@gmail.com wrote: From: