Re: [Intel-gfx] [PATCH 13/14] drm/i915: don't set up gem ring functions on gen5 for !kms

2012-04-12 Thread Ben Widawsky
On Wed, 11 Apr 2012 22:12:58 +0200 Daniel Vetter wrote: > We already disallow initialition of gem in this case in the > corresponding ioctl, so don't bother setting up the gem support ring > functions in the legacy dri render ring init. > > Signed-Off-by: Daniel Vetter > --- > drivers/gpu/drm/

Re: [Intel-gfx] [PATCH 14/14] drm/i915: inline enable/disable_irq into ring->get/put_irq

2012-04-12 Thread Ben Widawsky
On Wed, 11 Apr 2012 22:12:59 +0200 Daniel Vetter wrote: > Now that these are properly refactored this additional indirection > doesn't really buy us anything but confusion. Hence inline them. > > This duplicates the ironlake gt enable/disable code snippet, but we've > already separate ilk from g

Re: [Intel-gfx] [PATCH 12/14] drm/i915: consolidate ring->flush a bit

2012-04-12 Thread Ben Widawsky
On Wed, 11 Apr 2012 22:12:57 +0200 Daniel Vetter wrote: > They're indentical, so just kill one. Also give the other a prefix to > distinguish it from the gen6+ functions - this add_request function is > not really generic code. > > Signed-Off-by: Daniel Vetter The subject appears to indicate t

Re: [Intel-gfx] [PATCH] drm/edid: Adding common CVT inferred modes when monitor allows range limited ones trough EDID.

2012-04-12 Thread Rodrigo Vivi
Hi Ajax and Takashi, Thanks for your comments. > The intent here is great, but I don't like the way this is phrased, or > the implementation. To be honest I don't like this implementation as well. I just tried to follow the way it wasa already there. > CVT monitors _must_ accept GTF as well, ED

[Intel-gfx] [PATCH 7/7] drm/edid: Fix some comment typos in the DMT mode list

2012-04-12 Thread Adam Jackson
Signed-off-by: Adam Jackson --- drivers/gpu/drm/drm_edid_modes.h | 10 +- 1 files changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/gpu/drm/drm_edid_modes.h b/drivers/gpu/drm/drm_edid_modes.h index 4be9c1a..ab3a051 100644 --- a/drivers/gpu/drm/drm_edid_modes.h +++ b/drivers

[Intel-gfx] [PATCH 6/7] drm/edid: Add the reduced blanking DMT modes to the DMT list

2012-04-12 Thread Adam Jackson
Copied from the list in xserver. Signed-off-by: Adam Jackson --- drivers/gpu/drm/drm_edid_modes.h | 94 +- 1 files changed, 93 insertions(+), 1 deletions(-) diff --git a/drivers/gpu/drm/drm_edid_modes.h b/drivers/gpu/drm/drm_edid_modes.h index a91ffb1..4be9

[Intel-gfx] [PATCH 5/7] drm/edid: Armor drm_dmt_modes_for_range for reduced blanking modes

2012-04-12 Thread Adam Jackson
Require that the monitor support rb for rb modes to be added. Signed-off-by: Adam Jackson --- drivers/gpu/drm/drm_edid.c |8 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c index 83c51d6..0fc63ca 100644 --- a/dri

[Intel-gfx] [PATCH 4/7] drm/edid: s/drm_gtf_modes_for_range/drm_dmt_modes_for_range/

2012-04-12 Thread Adam Jackson
Slightly more honest naming. Signed-off-by: Adam Jackson --- drivers/gpu/drm/drm_edid.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c index 4f52103..83c51d6 100644 --- a/drivers/gpu/drm/drm_edid.c +++ b/drivers

[Intel-gfx] [PATCH 3/7] drm/edid: Allow drm_mode_find_dmt to hunt for reduced-blanking modes

2012-04-12 Thread Adam Jackson
It won't find any, yet. Fix up callers to match: standard mode codes will look prefer r-b modes for a given size if present, EST3 mode codes will look for exactly the r-b-ness mentioned in the mode code. This might mean fewer modes matched for EST3 mode codes between now and when the DMT mode lis

[Intel-gfx] [PATCH 2/7] drm/edid: Rewrite drm_mode_find_dmt search loop

2012-04-12 Thread Adam Jackson
No functional change, but will make an upcoming change clearer. Signed-off-by: Adam Jackson --- drivers/gpu/drm/drm_edid.c | 19 ++- 1 files changed, 10 insertions(+), 9 deletions(-) diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c index 79a3637..0a23ee5 10

[Intel-gfx] [PATCH 1/7] drm/edid: Document drm_mode_find_dmt

2012-04-12 Thread Adam Jackson
Signed-off-by: Adam Jackson --- drivers/gpu/drm/drm_edid.c | 10 ++ 1 files changed, 10 insertions(+), 0 deletions(-) diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c index 5a18b0d..79a3637 100644 --- a/drivers/gpu/drm/drm_edid.c +++ b/drivers/gpu/drm/drm_edid.c @@

[Intel-gfx] [PATCH 0/7] drm/edid: Add the reduced-blanking modes to the DMT list

2012-04-12 Thread Adam Jackson
These should have been there since day one. This series cleans up the few places that touch the DMT list to handle rb modes correctly, and then imports the modes from xserver's list, along with a few cosmetic changes. - ajax ___ Intel-gfx mailing list

Re: [Intel-gfx] [PATCH] drm/i915: Removed IVB forced enable of sprite dest key

2012-04-12 Thread Daniel Vetter
On Thu, Apr 12, 2012 at 08:39:38PM +, Reese, Armin C wrote: > Prefixed the patch subject with 'drm/i915:'. Nothing else changed. This one is aready merged into -fixes and on track to get merged into 3.4-rc3. -Daniel PS: You're called acreese in both the git commiter and sob line. You can fix

[Intel-gfx] [PATCH] drm/i915: Removed IVB forced enable of sprite dest key

2012-04-12 Thread Reese, Armin C
Prefixed the patch subject with 'drm/i915:'. Nothing else changed. Armin destkey.patch Description: destkey.patch ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx

[Intel-gfx] [PATCH] drm/i915: Mask reserved bits in display/sprite address registers

2012-04-12 Thread Reese, Armin C
The attached patch file was updated to reflect reviewer's comments. The only change I did not make was using PAGE_MASK instead of DISP_BASEADDR_MASK. PAGE_MASK is CPU architecture dependent and I didn't want to tie our GPU addressing arch to that of the CPU. Armin mask1.patch Description: m

Re: [Intel-gfx] [PATCH 02/14] drm/i915: set ring->size in common ring setup code

2012-04-12 Thread Daniel Vetter
On Thu, Apr 12, 2012 at 12:23:47PM -0700, Ben Widawsky wrote: > On Wed, 11 Apr 2012 22:12:47 +0200 > Daniel Vetter wrote: > > > Eventually we want to scale the ring size depending upon available > > gtt space. For now just consolidate this instead of replicating it > > over all ringbuffer templat

Re: [Intel-gfx] Google MapsGL causing X crashes on machines with Sandybridge graphics

2012-04-12 Thread Chris Wilson
On Thu, 12 Apr 2012 19:34:12 + (UTC), Eric Appleman wrote: > It's a known issue, but I've yet to find a comprehensive explanation for > what's > going on. What X crash? The bugs show a GPU hang. X itself should remain stable through such a catastrophe and revert to software rendering (if th

[Intel-gfx] Google MapsGL causing X crashes on machines with Sandybridge graphics

2012-04-12 Thread Eric Appleman
It's a known issue, but I've yet to find a comprehensive explanation for what's going on. [1] https://bugs.freedesktop.org/show_bug.cgi?id=44108 [2] https://bugs.freedesktop.org/show_bug.cgi?id=45806 ___ Intel-gfx mailing list Intel-gfx@lists.freedeskto

Re: [Intel-gfx] [PATCH 02/14] drm/i915: set ring->size in common ring setup code

2012-04-12 Thread Ben Widawsky
On Wed, 11 Apr 2012 22:12:47 +0200 Daniel Vetter wrote: > Eventually we want to scale the ring size depending upon available > gtt space. For now just consolidate this instead of replicating it > over all ringbuffer templates. > > Signed-off-by: Daniel Vetter > --- Definitely in the bikeshed c

Re: [Intel-gfx] [PATCH] drm/edid: Adding common CVT inferred modes when monitor allows range limited ones trough EDID.

2012-04-12 Thread Takashi Iwai
At Wed, 11 Apr 2012 21:59:28 -0300, Rodrigo Vivi wrote: > > There are many bugs open on fd.o regarding missing modes that are supported > on Windows and other closed source drivers. > >From EDID spec we can (might?) infer modes using GTF and CVT when monitor > >allows it trough range limited fla

Re: [Intel-gfx] [PATCH] drm/edid: Adding common CVT inferred modes when monitor allows range limited ones trough EDID.

2012-04-12 Thread Adam Jackson
On Wed, 2012-04-11 at 21:59 -0300, Rodrigo Vivi wrote: > There are many bugs open on fd.o regarding missing modes that are supported > on Windows and other closed source drivers. > From EDID spec we can (might?) infer modes using GTF and CVT when monitor > allows it trough range limited flag... o

Re: [Intel-gfx] bo problem, file-max limit reached within two days

2012-04-12 Thread Knut Petersen
Am 12.04.2012 10:03, schrieb Daniel Vetter: Can you try the drm-intel-next-queued branch from my git repo at http://cgit.freedesktop.org/~danvet/drm-intel/ That contains a patch from Chris Wilson to mitigate mmio offset exhaustion, one possible reason for a -ENOSPC failure. -Daniel It´s a short

[Intel-gfx] Fwd: [pull] drm-intel-next for 3.5

2012-04-12 Thread Daniel Vetter
Forwarding the pull request to Dave to intel-gfx and dri-devel, I've fumbled the cc list on the original mail. -Daniel -- Forwarded message -- From: Daniel Vetter Date: Thu, Apr 12, 2012 at 11:17 Subject: [pull] drm-intel-next for 3.5 To: Dave Airlie Hi Dave First pull request

Re: [Intel-gfx] [PATCH] drm/i915: Release pages en-masse rather than accumulate a pagevec

2012-04-12 Thread Chris Wilson
On Thu, 12 Apr 2012 10:10:11 +0100, Chris Wilson wrote: > As we have a large array of pages to be freed, we can call > release_pages() directly and bypass the accumulation of a pagevec. Ignore this, this patch is a little pre-emptive and depends upon the restructuring for unbound page tracking.

[Intel-gfx] [PATCH] drm/i915: Release pages en-masse rather than accumulate a pagevec

2012-04-12 Thread Chris Wilson
As we have a large array of pages to be freed, we can call release_pages() directly and bypass the accumulation of a pagevec. Signed-off-by: Chris Wilson --- drivers/gpu/drm/i915/i915_gem.c | 21 +++-- 1 file changed, 11 insertions(+), 10 deletions(-) diff --git a/drivers/gpu/

Re: [Intel-gfx] bo problem, file-max limit reached within two days

2012-04-12 Thread Chris Wilson
On Thu, 12 Apr 2012 09:55:11 +0200, Knut Petersen wrote: > Am 02.04.2012 10:44, schrieb Chris Wilson: > > On Mon, 02 Apr 2012 10:36:44 +0200, Knut > > Petersen wrote: > >> After a few unattended hours Xorg was still running, > >> but only a terminal window had survived the night :-( > >> > >> X

Re: [Intel-gfx] bo problem, file-max limit reached within two days

2012-04-12 Thread Daniel Vetter
On Thu, Apr 12, 2012 at 09:55:11AM +0200, Knut Petersen wrote: > Am 02.04.2012 10:44, schrieb Chris Wilson: > >On Mon, 02 Apr 2012 10:36:44 +0200, Knut Petersen > > wrote: > >>After a few unattended hours Xorg was still running, > >>but only a terminal window had survived the night :-( > >> > >>Xo

Re: [Intel-gfx] bo problem, file-max limit reached within two days

2012-04-12 Thread Knut Petersen
Am 02.04.2012 10:44, schrieb Chris Wilson: On Mon, 02 Apr 2012 10:36:44 +0200, Knut Petersen wrote: After a few unattended hours Xorg was still running, but only a terminal window had survived the night :-( Xorg: git, a few days old, kernel: 3.3. Xorg.0.log: Nothing unusual dmesg: [156859.

[Intel-gfx] [pull] drm-intel-fixes for 3.4-rc3

2012-04-12 Thread Daniel Vetter
Hi Dave 3 regression fixes: - disable gmbus again, too broken for 3.4, we'll try again for 3.5 - dp bandwidth computation fix, we've lost the 6bpc dithering flag sometimes, this is a 3.3 regression (maybe even earlier for some configurations). - fix resume regression caused by the gen2/3 fenci

Re: [Intel-gfx] [PATCH] drm/i915: clear fencing tracking state when retiring request

2012-04-12 Thread Daniel Vetter
On Thu, Apr 12, 2012 at 12:38:09AM +0100, Chris Wilson wrote: > On Thu, 12 Apr 2012 01:27:57 +0200, Daniel Vetter > wrote: > > This fixes a regression introduce in > s/introduce/introduced/ > > > commit 7dd4906586274f3945f2aeaaa5a33b451c3b4bba > > Author: Chris Wilson > > Date: Wed Mar 21 10