[Bug 22344] [regression OGLC] 7 cases failed

2009-06-18 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=22344


Shuang He shuang...@intel.com changed:

   What|Removed |Added

 AssignedTo|dri-|i...@freedesktop.org
   |de...@lists.sourceforge.net |




-- 
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.

--
Crystal Reports - New Free Runtime and 30 Day Trial
Check out the new simplified licensing option that enables unlimited
royalty-free distribution of the report engine for externally facing 
server and web deployment.
http://p.sf.net/sfu/businessobjects
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[PATCH] drm/radeon: this VRAM vs aperture test is wrong, just remove it.

2009-06-18 Thread Dave Airlie
From: Dave Airlie airl...@itt42.(none)

Its quite valid to have VRAM  aperture size.

Signed-off-by: Dave Airlie airl...@redhat.com
---
 drivers/gpu/drm/radeon/r100.c |6 --
 1 files changed, 0 insertions(+), 6 deletions(-)

diff --git a/drivers/gpu/drm/radeon/r100.c b/drivers/gpu/drm/radeon/r100.c
index 5225f5b..8f41f70 100644
--- a/drivers/gpu/drm/radeon/r100.c
+++ b/drivers/gpu/drm/radeon/r100.c
@@ -1267,12 +1267,6 @@ void r100_vram_info(struct radeon_device *rdev)
 
rdev-mc.aper_base = drm_get_resource_start(rdev-ddev, 0);
rdev-mc.aper_size = drm_get_resource_len(rdev-ddev, 0);
-   if (rdev-mc.aper_size  rdev-mc.vram_size) {
-   /* Why does some hw doesn't have CONFIG_MEMSIZE properly
-* setup ? */
-   rdev-mc.vram_size = rdev-mc.aper_size;
-   WREG32(RADEON_CONFIG_MEMSIZE, rdev-mc.vram_size);
-   }
 }
 
 
-- 
1.6.2.2


--
Crystal Reports - New Free Runtime and 30 Day Trial
Check out the new simplified licensing option that enables unlimited
royalty-free distribution of the report engine for externally facing 
server and web deployment.
http://p.sf.net/sfu/businessobjects
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 22347] New: [KMS] mesa demo cubemap broken for rv280

2009-06-18 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=22347

   Summary: [KMS] mesa demo cubemap broken for rv280
   Product: Mesa
   Version: CVS
  Platform: x86 (IA32)
OS/Version: Linux (All)
Status: NEW
  Severity: normal
  Priority: medium
 Component: Drivers/DRI/Radeon
AssignedTo: dri-devel@lists.sourceforge.net
ReportedBy: rand...@mail.ru


Created an attachment (id=26918)
 -- (http://bugs.freedesktop.org/attachment.cgi?id=26918)
glxinfo log

Using kernel 2.6.30-git up to 

commit 3fe0344faf7fdcb158bd5c1a9aec960a8d70c8e8
Merge: c30938d... 2030117...
Author: Linus Torvalds torva...@linux-foundation.org
Date:   Wed Jun 17 10:42:21 2009 -0700

Mesa master commit 3addc4e3078b99db07598ac0b2473f7612b83a09
Author: Eric Anholt e...@anholt.net
Date:   Wed Jun 17 20:46:12 2009 -0700

i965: Add decode for the G4X x,y offset in surface state.

libdrm commit 2fa2db138ba989bfa1a8cd9ab66d83fb7369249e
Author: Dave Airlie airl...@linux.ie
Date:   Wed Jun 17 17:47:42 2009 +1000

libdrm/radeon: add initial libdrm_radeon

xf86-video-ati from ~/glisse/xf86-video-ati radeon-gem-cs3 branch commit
7a514a750fc6bca34e9549eb28e344576c0de4f1
Author: Jerome Glisse jgli...@redhat.com
Date:   Thu Jun 4 22:33:39 2009 +0200

radeon: scissor rendering to pixmap widht and height

(will try radeon-kms branch in few minutes)

i got broken rendring in mesa/demos/cubemap (default window).

My hardware - 

01:00.0 VGA compatible controller: ATI Technologies Inc RV280 [Radeon 9200 SE]
(rev 01) (prog-if 00 [VGA controller])
Subsystem: Hightech Information System Ltd. Excalibur 9200SE VIVO 128M
Flags: bus master, 66MHz, medium devsel, latency 64, IRQ 16
Memory at c800 (32-bit, prefetchable) [size=128M]
I/O ports at b800 [size=256]
Memory at d7ef (32-bit, non-prefetchable) [size=64K]
Expansion ROM at d7ec [disabled] [size=128K]
Capabilities: [58] AGP version 2.0
Capabilities: [50] Power Management version 2
Kernel driver in use: radeon
Kernel modules: radeon, radeonfb

radeon was loaded with agpmode=-1 modeset=1 parameters.


-- 
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.

--
Crystal Reports - New Free Runtime and 30 Day Trial
Check out the new simplified licensing option that enables unlimited
royalty-free distribution of the report engine for externally facing 
server and web deployment.
http://p.sf.net/sfu/businessobjects
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 22347] [KMS] mesa demo cubemap broken for rv280

2009-06-18 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=22347


Andrew Randrianasulu rand...@mail.ru changed:

   What|Removed |Added

  Attachment #26918|0   |1
is obsolete||




--- Comment #1 from Andrew Randrianasulu rand...@mail.ru  2009-06-18 05:19:57 
PST ---
Created an attachment (id=26919)
 -- (http://bugs.freedesktop.org/attachment.cgi?id=26919)
glxinfo -l

Strange, GL_MAX_CUBE_MAP_TEXTURE_SIZE_ARB = 4096 (on rv280?)


-- 
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.

--
Crystal Reports - New Free Runtime and 30 Day Trial
Check out the new simplified licensing option that enables unlimited
royalty-free distribution of the report engine for externally facing 
server and web deployment.
http://p.sf.net/sfu/businessobjects
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[PATCH] radeon: command stream checker for r3xx-r5xx hardware

2009-06-18 Thread Jerome Glisse
This finish 3D state cs checking for r3xx-r5xx, you will
need lastest DDX and mesa as some previously allowed register
are now forbidden.

Jerome
From 246cb55f8345bdf3ef071b587b311ab1088280ce Mon Sep 17 00:00:00 2001
From: Jerome Glisse gli...@freedesktop.org
Date: Wed, 17 Jun 2009 13:28:30 +0200
Subject: [PATCH] radeon: command stream checker for r3xx-r5xx hardware

For security purpose we want to make sure the userspace process doesn't
access memory beyond buffer it owns. To achieve this we need to check
states the userspace program. For color buffer and zbuffer we check that
the clipping register will discard access beyond buffers set as color
or zbuffer. For vertex buffer we check that no vertex fetch will happen
beyond buffer end. For texture we check various texture states (number
of mipmap level, texture size, texture depth, ...) to compute the amount
of memory the texture fetcher might access.

The command stream checking impact the performances so far quick benchmark
shows an average of 3% decrease in fps of various applications. It can
be optimized a bit more by caching result of checking and thus avoid a
full recheck if no states changed since last check.

Note that this patch is still incomplete on checking side as it doesn't
check 2d rendering states.

Signed-off-by: Jerome Glisse jgli...@redhat.com
---
 drivers/gpu/drm/radeon/r100.c  |   76 +++--
 drivers/gpu/drm/radeon/r300.c  |  478 +--
 drivers/gpu/drm/radeon/r300.h  |   36 +++
 drivers/gpu/drm/radeon/radeon.h|9 +-
 drivers/gpu/drm/radeon/radeon_asic.h   |   15 +-
 drivers/gpu/drm/radeon/radeon_device.c |4 +
 drivers/gpu/drm/radeon/rv515.c |   58 
 7 files changed, 611 insertions(+), 65 deletions(-)
 create mode 100644 drivers/gpu/drm/radeon/r300.h

diff --git a/drivers/gpu/drm/radeon/r100.c b/drivers/gpu/drm/radeon/r100.c
index 5225f5b..9c762dc 100644
--- a/drivers/gpu/drm/radeon/r100.c
+++ b/drivers/gpu/drm/radeon/r100.c
@@ -644,7 +644,7 @@ int r100_cp_reset(struct radeon_device *rdev)
  */
 int r100_cs_parse_packet0(struct radeon_cs_parser *p,
 			  struct radeon_cs_packet *pkt,
-			  unsigned *auth, unsigned n,
+			  const unsigned *auth, unsigned n,
 			  radeon_packet0_check_t check)
 {
 	unsigned reg;
@@ -654,6 +654,10 @@ int r100_cs_parse_packet0(struct radeon_cs_parser *p,
 
 	idx = pkt-idx + 1;
 	reg = pkt-reg;
+	/* Check that register fall into register range
+	 * determined by the number of entry (n) in the
+	 * safe register bitmap.
+	 */
 	if (pkt-one_reg_wr) {
 		if ((reg  7)  n) {
 			return -EINVAL;
@@ -683,24 +687,6 @@ int r100_cs_parse_packet0(struct radeon_cs_parser *p,
 	return 0;
 }
 
-int r100_cs_parse_packet3(struct radeon_cs_parser *p,
-			  struct radeon_cs_packet *pkt,
-			  unsigned *auth, unsigned n,
-			  radeon_packet3_check_t check)
-{
-	unsigned i, m;
-
-	if ((pkt-opcode  5)  n) {
-		return -EINVAL;
-	}
-	i = pkt-opcode  5;
-	m = 1  (pkt-opcode  31);
-	if (auth[i]  m) {
-		return check(p, pkt);
-	}
-	return 0;
-}
-
 void r100_cs_dump_packet(struct radeon_cs_parser *p,
 			 struct radeon_cs_packet *pkt)
 {
@@ -901,6 +887,25 @@ static int r100_packet0_check(struct radeon_cs_parser *p,
 	return 0;
 }
 
+int r100_cs_track_check_pkt3_indx_buffer(struct radeon_cs_parser *p,
+	 struct radeon_cs_packet *pkt,
+	 struct radeon_object *robj)
+{
+	struct radeon_cs_chunk *ib_chunk;
+	unsigned idx;
+
+	ib_chunk = p-chunks[p-chunk_ib_idx];
+	idx = pkt-idx + 1;
+	if ((ib_chunk-kdata[idx+2] + 1)  radeon_object_size(robj)) {
+		DRM_ERROR([drm] Buffer too small for PACKET3 INDX_BUFFER 
+			  (need %u have %lu) !\n,
+			  ib_chunk-kdata[idx+2] + 1,
+			  radeon_object_size(robj));
+		return -EINVAL;
+	}
+	return 0;
+}
+
 static int r100_packet3_check(struct radeon_cs_parser *p,
 			  struct radeon_cs_packet *pkt)
 {
@@ -954,6 +959,10 @@ static int r100_packet3_check(struct radeon_cs_parser *p,
 			return r;
 		}
 		ib[idx+1] = ib_chunk-kdata[idx+1] + ((u32)reloc-lobj.gpu_offset);
+		r = r100_cs_track_check_pkt3_indx_buffer(p, pkt, reloc-robj);
+		if (r) {
+			return r;
+		}
 		break;
 	case 0x23:
 		/* FIXME: cleanup */
@@ -999,18 +1008,18 @@ int r100_cs_parse(struct radeon_cs_parser *p)
 		}
 		p-idx += pkt.count + 2;
 		switch (pkt.type) {
-		case PACKET_TYPE0:
-			r = r100_packet0_check(p, pkt);
-			break;
-		case PACKET_TYPE2:
-			break;
-		case PACKET_TYPE3:
-			r = r100_packet3_check(p, pkt);
-			break;
-		default:
-			DRM_ERROR(Unknown packet type %d !\n,
-	pkt.type);
-			return -EINVAL;
+			case PACKET_TYPE0:
+r = r100_packet0_check(p, pkt);
+break;
+			case PACKET_TYPE2:
+break;
+			case PACKET_TYPE3:
+r = r100_packet3_check(p, pkt);
+break;
+			default:
+DRM_ERROR(Unknown packet type %d !\n,
+	  pkt.type);
+return -EINVAL;
 		}
 		if (r) {
 			return r;
@@ -1352,6 +1361,11 @@ void r100_mm_wreg(struct radeon_device *rdev, uint32_t reg, uint32_t v)
 	}
 }
 
+int r100_init(struct radeon_device 

[Bug 22347] [KMS] mesa demo cubemap broken for rv280

2009-06-18 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=22347





--- Comment #3 from Andrew Randrianasulu rand...@mail.ru  2009-06-18 09:13:44 
PST ---
Created an attachment (id=26927)
 -- (http://bugs.freedesktop.org/attachment.cgi?id=26927)
Screenshot

Corruption may look different at each run.


-- 
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.

--
Crystal Reports - New Free Runtime and 30 Day Trial
Check out the new simplified licensing option that enables unlimited
royalty-free distribution of the report engine for externally facing 
server and web deployment.
http://p.sf.net/sfu/businessobjects
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: kerneloops.org report for the week of June 14 2009

2009-06-18 Thread Bob Copeland
On Sun, Jun 14, 2009 at 8:33 PM, Arjan van de Venar...@infradead.org wrote:
 Rank 9: minstrel_get_rate (warning)
        Reported 64 times (384 total reports)
        Issue with the ath5k driver
        This warning was last seen in version 2.6.30-rc3, and first seen in 
 2.6.27.12.
        More info: 
 http://www.kerneloops.org/searchweek.php?search=minstrel_get_rate

I believe this is fixed by http://lkml.org/lkml/2009/6/5/269, now
in pre-2.6.31.  Hmm, odd though, we should be seeing reports from
other wireless drivers since it's a general memory corruption
unrelated to ath5k (but not all use the mac80211 rate controllers).

(2.6.27.12 must be a backport, minstrel didn't exist until .29)

-- 
Bob Copeland %% www.bobcopeland.com

--
Crystal Reports - New Free Runtime and 30 Day Trial
Check out the new simplified licensing option that enables unlimited
royalty-free distribution of the report engine for externally facing 
server and web deployment.
http://p.sf.net/sfu/businessobjects
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[GIT PULL] drm-malloc branch for merge

2009-06-18 Thread Eric Anholt
The following changes since commit 1d89b30cc9be41af87881682ec82e2c107849dbe:
  Matthew Wilcox (1):
ia64: Fix resource assignment for root busses

are available in the git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/anholt/drm-intel 
..BRANCH.NOT.VERIFIED..

I'm doing this slightly more formally as I've got a big change of my own
that affects the DRM in general: Removing the unused malloc debugging.
I've threatened it before, and shown the previous iterations of the
patch around, but here it finally is for this merge window.

Ben Gamari (2):
  drm/i915: Warn when inteldrmfb fails to restore its framebuffer config
  drm/i915: A few debugfs formatting fixes

Chris Wilson (3):
  drm/i915: detach/attach get/put pages symmetry
  drm/i915: Install fence register for tiled scanout on i915
  drm/i915: Clear fence register on tiling stride change.

Eric Anholt (1):
  drm: Remove memory debugging infrastructure.

 drivers/gpu/drm/drm_agpsupport.c|   12 +-
 drivers/gpu/drm/drm_auth.c  |4 +-
 drivers/gpu/drm/drm_bufs.c  |  140 ++-
 drivers/gpu/drm/drm_context.c   |4 +-
 drivers/gpu/drm/drm_debugfs.c   |9 +-
 drivers/gpu/drm/drm_dma.c   |   31 +---
 drivers/gpu/drm/drm_drawable.c  |   25 +--
 drivers/gpu/drm/drm_drv.c   |   18 +--
 drivers/gpu/drm/drm_fops.c  |8 +-
 drivers/gpu/drm/drm_gem.c   |8 +-
 drivers/gpu/drm/drm_hashtab.c   |6 +-
 drivers/gpu/drm/drm_ioctl.c |   14 +-
 drivers/gpu/drm/drm_irq.c   |   44 ++---
 drivers/gpu/drm/drm_memory.c|   25 ---
 drivers/gpu/drm/drm_pci.c   |   53 +-
 drivers/gpu/drm/drm_proc.c  |8 +-
 drivers/gpu/drm/drm_scatter.c   |   33 ++--
 drivers/gpu/drm/drm_sman.c  |   29 ++--
 drivers/gpu/drm/drm_stub.c  |   19 +-
 drivers/gpu/drm/drm_vm.c|8 +-
 drivers/gpu/drm/i810/i810_dma.c |6 +-
 drivers/gpu/drm/i830/i830_dma.c |6 +-
 drivers/gpu/drm/i915/i915_dma.c |   40 ++---
 drivers/gpu/drm/i915/i915_drv.h |2 +
 drivers/gpu/drm/i915/i915_gem.c |   86 ++---
 drivers/gpu/drm/i915/i915_gem_debugfs.c |4 +-
 drivers/gpu/drm/i915/i915_gem_tiling.c  |   67 +--
 drivers/gpu/drm/i915/i915_mem.c |   24 ++--
 drivers/gpu/drm/i915/intel_bios.c   |6 +-
 drivers/gpu/drm/i915/intel_display.c|   20 ++-
 drivers/gpu/drm/i915/intel_fb.c |6 +-
 drivers/gpu/drm/i915/intel_tv.c |   11 +-
 drivers/gpu/drm/mga/mga_dma.c   |   14 +-
 drivers/gpu/drm/r128/r128_cce.c |   12 +-
 drivers/gpu/drm/r128/r128_state.c   |   84 
 drivers/gpu/drm/radeon/radeon_cp.c  |9 +-
 drivers/gpu/drm/radeon/radeon_i2c.c |6 +-
 drivers/gpu/drm/radeon/radeon_kms.c |4 +-
 drivers/gpu/drm/radeon/radeon_mem.c |   24 ++--
 drivers/gpu/drm/radeon/radeon_state.c   |   16 +-
 drivers/gpu/drm/savage/savage_bci.c |   21 +--
 drivers/gpu/drm/savage/savage_state.c   |   17 +-
 drivers/gpu/drm/sis/sis_drv.c   |6 +-
 drivers/gpu/drm/via/via_map.c   |8 +-
 include/drm/drmP.h  |   52 -
 include/drm/drm_memory_debug.h  |  309 ---
 46 files changed, 444 insertions(+), 914 deletions(-)
 delete mode 100644 include/drm/drm_memory_debug.h

-- 
Eric Anholt
e...@anholt.net eric.anh...@intel.com




signature.asc
Description: This is a digitally signed message part
--
Crystal Reports - New Free Runtime and 30 Day Trial
Check out the new simplified licensing option that enables unlimited
royalty-free distribution of the report engine for externally facing 
server and web deployment.
http://p.sf.net/sfu/businessobjects--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: kerneloops.org report for the week of June 14 2009

2009-06-18 Thread Arjan van de Ven
On Thu, 18 Jun 2009 13:08:39 -0400
Bob Copeland m...@bobcopeland.com wrote:

 On Sun, Jun 14, 2009 at 8:33 PM, Arjan van de
 Venar...@infradead.org wrote:
  Rank 9: minstrel_get_rate (warning)
         Reported 64 times (384 total reports)
         Issue with the ath5k driver
         This warning was last seen in version 2.6.30-rc3, and first
  seen in 2.6.27.12. More info:
  http://www.kerneloops.org/searchweek.php?search=minstrel_get_rate
 
 I believe this is fixed by http://lkml.org/lkml/2009/6/5/269, now
 in pre-2.6.31.  Hmm, odd though, we should be seeing reports from
 other wireless drivers since it's a general memory corruption
 unrelated to ath5k (but not all use the mac80211 rate controllers).
 

well.. out of the 534 reports or so that I have by now... 525 have
ath5k loaded.

it's not 100% but it'd pretty close.


-- 
Arjan van de VenIntel Open Source Technology Centre
For development, discussion and tips for power savings, 
visit http://www.lesswatts.org

--
Crystal Reports - New Free Runtime and 30 Day Trial
Check out the new simplified licensing option that enables unlimited
royalty-free distribution of the report engine for externally facing 
server and web deployment.
http://p.sf.net/sfu/businessobjects
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel