[Bug 22396] New: [KMS i915] ut2004 will crash the system

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

   Summary: [KMS i915] ut2004 will crash the system
   Product: Mesa
   Version: unspecified
  Platform: Other
OS/Version: Linux (All)
Status: NEW
  Severity: normal
  Priority: medium
 Component: Drivers/DRI/i915
AssignedTo: dri-devel@lists.sourceforge.net
ReportedBy: jian.j.z...@intel.com


Created an attachment (id=26989)
 -- (http://bugs.freedesktop.org/attachment.cgi?id=26989)
xorg.conf

System Environment:
--
Host:   945GM
Arch:   i386
Libdrm: (master)2fa2db138ba989bfa1a8cd9ab66d83fb7369249e
Mesa:   (master)df70d3049a396af3601d2a1747770635a74120bb
Xserver:(master)ae20e748cd3a656173e1f50109bfd4af0712bb87
Xf86_video_intel:(master)0b7522372c82cd3b485c7e81867ccdbb710451e8

Bug detailed description:
--
start X and run ut2004-demo, then it will crash the system leaving there a
blurred screen. It failed both with KMS and UMS. On G45, it didn't crash the
system, but sometimes has a black screen. And its log is unreadable. 
[r...@x-945gm ~]# ut2004
WARNING: ALC_EXT_capture is subject to change!
libGL: OpenDriver: trying /opt/X11R7/lib/dri/i915_dri.so
Xlib:  extension XiG-SUNDRY-NONSTANDARD missing on display :0.0.


Reproduce steps:

1.xinit 
2. run ut2004(or its demo, benchmark.sh)


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

--
Are you an open source citizen? Join us for the Open Source Bridge conference!
Portland, OR, June 17-19. Two days of sessions, one day of unconference: $250.
Need another reason to go? 24-hour hacker lounge. Register today!
http://ad.doubleclick.net/clk;215844324;13503038;v?http://opensourcebridge.org
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 22396] [KMS i915] ut2004 will crash the system

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





--- Comment #1 from zhao jian jian.j.z...@intel.com  2009-06-21 02:27:03 PST 
---
Created an attachment (id=26990)
 -- (http://bugs.freedesktop.org/attachment.cgi?id=26990)
xorg.0.log


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

--
Are you an open source citizen? Join us for the Open Source Bridge conference!
Portland, OR, June 17-19. Two days of sessions, one day of unconference: $250.
Need another reason to go? 24-hour hacker lounge. Register today!
http://ad.doubleclick.net/clk;215844324;13503038;v?http://opensourcebridge.org
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 22396] [KMS i915] ut2004 will crash the system

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





--- Comment #2 from zhao jian jian.j.z...@intel.com  2009-06-21 02:33:33 PST 
---
And it works well with our RC2 image: 
Libdrm: (master)2fa2db138ba989bfa1a8cd9ab66d83fb7369249e
Mesa:   (mesa_7_5_branch)8f382fd3f396e182255fe084bc32648b98ca1d94
Xserver: (server-1.6-branch)6be19e8f43086fb4b7fb30a47b89b5f3eed798ef
Xf86_video_intel: (master)b5cd2130f97591f4a387db1b98c940c30bc6404c


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

--
Are you an open source citizen? Join us for the Open Source Bridge conference!
Portland, OR, June 17-19. Two days of sessions, one day of unconference: $250.
Need another reason to go? 24-hour hacker lounge. Register today!
http://ad.doubleclick.net/clk;215844324;13503038;v?http://opensourcebridge.org
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 20607] FlightGear asserts and aborts in r300_mipmap_tree.c:calculate_miptree_layout

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





--- Comment #15 from Mukund Sivaraman m...@banu.com  2009-06-21 04:04:34 PST 
---
(In reply to comment #7)
 (In reply to comment #6)
  (In reply to comment #4)
   *WARN_ONCE*
   File r300_state.c function r500SetupRSUnit line 1907
   Don't know how to satisfy InputsRead=0x0008
  
  This looks like bug 17929, which was fixed before Mesa 7.3, but maybe the 
  fix
  needs to be adapted for R5xx cards as well...
 
 Nope, this bug and other fog related has been fixed by
 d8b8fb68954e6eebd0b38708c25a5bec4cf1a26c and
 7ad7abc4cd5c94b83feea4553ffb5a69d8ad4757. These commits aren't in 7.3 neither
 in 7.4.
 

Fedora 11 shipped with Mesa RPM versions 7.5-0.14.fc11.x86_64, but it's unclear
what release they picked as Mesa 7.5 doesn't seem to be released (?). Maybe
it's one of the RC releases.

I am still able to reproduce the exact same bug as in the description with this
RPM version on the same hardware.

Let's keep this bug in the resolved state while I get a chance to verify if it
has been fixed. Please do not mark this bug as closed until then.


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

--
Are you an open source citizen? Join us for the Open Source Bridge conference!
Portland, OR, June 17-19. Two days of sessions, one day of unconference: $250.
Need another reason to go? 24-hour hacker lounge. Register today!
http://ad.doubleclick.net/clk;215844324;13503038;v?http://opensourcebridge.org
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 20607] FlightGear asserts and aborts in r300_mipmap_tree.c:calculate_miptree_layout

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





--- Comment #16 from Mukund Sivaraman m...@banu.com  2009-06-21 04:13:08 PST 
---
The source file of the assertion is different, but it's in the same function,
so maybe the file was renamed:

[m...@naina ~]$ fgfs
  Model Author:  Unknown
  Creation Date: 2002-01-01
  Version:   $Id: c172p.xml,v 1.20 2008/09/01 15:14:33 torsten Exp $
  Description:   Cessna C-172
FGMultiplayMgr - No receiver port, Multiplayermode disabled
KI266 dme indicator #0 initialized
Initializing Nasal Electrical System
fgfs: radeon_mipmap_tree.c:142: calculate_miptree_layout: Assertion `numLevels
= 12' failed.
Aborted
[m...@naina ~]$ 


New package versions:

FlightGear-1.9.1-4.fc11.x86_64
xorg-x11-drv-ati-6.12.2-14.fc11.x86_64
mesa-libGL-7.5-0.14.fc11.x86_64
libdrm-2.4.6-7.fc11.x86_64


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

--
Are you an open source citizen? Join us for the Open Source Bridge conference!
Portland, OR, June 17-19. Two days of sessions, one day of unconference: $250.
Need another reason to go? 24-hour hacker lounge. Register today!
http://ad.doubleclick.net/clk;215844324;13503038;v?http://opensourcebridge.org
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 9253] rendering problems on radeon 9600 XT + compiz

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


Pauli suok...@gmail.com changed:

   What|Removed |Added

 CC||dragon_...@email.it




--- Comment #3 from Pauli suok...@gmail.com  2009-06-21 04:29:15 PST ---
*** Bug 22395 has been marked as a duplicate of this bug. ***


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

--
Are you an open source citizen? Join us for the Open Source Bridge conference!
Portland, OR, June 17-19. Two days of sessions, one day of unconference: $250.
Need another reason to go? 24-hour hacker lounge. Register today!
http://ad.doubleclick.net/clk;215844324;13503038;v?http://opensourcebridge.org
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 22398] New: Googleearth crashes and Mplayer crash

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

   Summary: Googleearth crashes and Mplayer crash
   Product: Mesa
   Version: CVS
  Platform: Other
OS/Version: All
Status: NEW
  Severity: normal
  Priority: medium
 Component: Drivers/DRI/r300
AssignedTo: dri-devel@lists.sourceforge.net
ReportedBy: ashl1fut...@gmail.com


I have Gentoo Linux amd64 (x86_64).
05:00.0 VGA compatible controller: ATI Technologies Inc RV380 0x3e50 [Radeon
X600]

media-libs/mesa-7.5_rc3
x11-apps/mesa-progs-7.4.1

DRI is enabled in kernel 2.6.29-r5

Google crash with this:
libGL warning: 3D driver claims to not support visual 0x68   
libGL warning: 3D driver claims to not support visual 0x69   
libGL warning: 3D driver claims to not support visual 0x57   
libGL warning: 3D driver claims to not support visual 0x6a   
libGL warning: 3D driver claims to not support visual 0x6b   
libGL warning: 3D driver claims to not support visual 0x6c   
libGL warning: 3D driver claims to not support visual 0x21   
libGL warning: 3D driver claims to not support visual 0x6d   
libGL warning: 3D driver claims to not support visual 0x6e   
libGL warning: 3D driver claims to not support visual 0x6f   
libGL warning: 3D driver claims to not support visual 0x70   
libGL warning: 3D driver claims to not support visual 0x71   
libGL warning: 3D driver claims to not support visual 0x72   
libGL warning: 3D driver claims to not support visual 0x73   
libGL warning: 3D driver claims to not support visual 0x22   
libGL warning: 3D driver claims to not support visual 0x74   
*WARN_ONCE*  
File r300_render.c function r300Fallback line 428
Software fallback:ctx-Line.SmoothFlag   
***  
Try R300_SPAN_DISABLE_LOCKING env var if this hangs. 
*WARN_ONCE*  
File r300_state.c function r300Enable line 512   
TODO - double side stencil ! 
***  
*WARN_ONCE*  
File r300_state.c function r300_setup_rs_unit line 1391  
fragprog wants coords for tex0, vp doesn't provide them! 
***
drmRadeonCmdBuffer: -22 (exiting)

Mplayer randomly crashes with this:
drmRadeonCmdBuffer: -12 or similar
I cannot recognize it.

It seems related to:
https://bugs.freedesktop.org/show_bug.cgi?id=7445
https://bugs.freedesktop.org/show_bug.cgi?id=10753


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

--
Are you an open source citizen? Join us for the Open Source Bridge conference!
Portland, OR, June 17-19. Two days of sessions, one day of unconference: $250.
Need another reason to go? 24-hour hacker lounge. Register today!
http://ad.doubleclick.net/clk;215844324;13503038;v?http://opensourcebridge.org
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Bulding DRM on MIPS

2009-06-21 Thread shachar . kaufman

HI all,

I'm developing a driver for my embedded graphics device on a MIPS platform.

I've really just started this effort when I encountered use of:
#include asm/agp.h
(in drmP.h, without any conditioning on AGP support in the OS)

This lead to think - what is the proper configuration I need to use in  
order to build DRM for MIPS systems?
--
Are you an open source citizen? Join us for the Open Source Bridge conference!
Portland, OR, June 17-19. Two days of sessions, one day of unconference: $250.
Need another reason to go? 24-hour hacker lounge. Register today!
http://ad.doubleclick.net/clk;215844324;13503038;v?http://opensourcebridge.org--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: [git pull] drm: previous pull req + 1.

2009-06-21 Thread Maxim Levitsky
On Sat, 2009-06-20 at 17:42 -0700, Linus Torvalds wrote:
 
 On Sun, 21 Jun 2009, Maxim Levitsky wrote:
  
  Something from this tree breaks my i965.
  Using -git just before this was pulled,
  
   a552f0af753eb4b5bbbe9eff205fe874b04c4583 works, but using latest git
  
  makes google earth stall, it doesn't update its main window. It appears
  that openining and closing its menu, allows it to progress frame after
  frame. No crashes hangs however.
 
 Can you bisect? There's not a tons of commit there, so it shouldn't be 
 more than a couple of recompiles/reboots, and you'd be able to pinpoint 
 the exact commit that breaks. That will help people figure it out, or at 
 worst just pinpoint what we need to revert.
 
   Linus


Here the result:


 52dc7d32b88156248167864f77a9026abe27b432 is first bad commit
 commit 52dc7d32b88156248167864f77a9026abe27b432
 Author: Chris Wilson ch...@chris-wilson.co.uk
 Date:   Sat Jun 6 09:46:01 2009 +0100
 
 drm/i915: Clear fence register on tiling stride change.
 
 The fence register value also depends upon the stride of the object, so we
 need to clear the fence if that is changed as well.
 
 Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk
 [anholt: Added 8xx and 965 paths, and renamed the confusing
 i915_gem_object_tiling_ok function to i915_gem_object_fence_offset_ok]
 Signed-off-by: Eric Anholt e...@anholt.net
 



However I can't reproduce the situation I have earlier, maybe I have changed 
some settings, don't know.
Now, the bad behavior (and I reproduced it many times, is that GE shows 
incorrect textures 
(like they are divided in tiny interlaced rows, one row ok, other contain image 
from other part of world), only few textures are such
it seems logical that this is related to tiling.

Also, if I maximize it, it hangs. This seems to be a separate bug introduced by 
these series.

commit 43813f399c72aa22e01a680559c1cb5274bf2140 both textures and maximize 
broken
commit 52dc7d32b88156248167864f77a9026abe27b432, shows this incorect textures, 
but doesn't hang the system on maximize
commit 8c4b8c3f34de4e2da20df042bba173fe557f8b45 works just fine



Unfortunaly I have no time now to do another bisect now, I try to do such as 
soon as possible.

Thanks,
Maxim Levitsky



--
Are you an open source citizen? Join us for the Open Source Bridge conference!
Portland, OR, June 17-19. Two days of sessions, one day of unconference: $250.
Need another reason to go? 24-hour hacker lounge. Register today!
http://ad.doubleclick.net/clk;215844324;13503038;v?http://opensourcebridge.org
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: [git pull] drm: previous pull req + 1.

2009-06-21 Thread Linus Torvalds


On Sun, 21 Jun 2009, Dave Airlie wrote:
  
  I tried this tree (specifically, a merge of Linus' fb20871 this tree) on
  Fedora 11 with modesetting enabled on an integrated Radeon 2100, and 
  plymouthd
  crashes immediately with a corrupt page table.  Photo attached.  After the
  crash, bootup stops, although ctrl-alt-del works.
 
 You need a different userspace, -ati from koji in F11 should do it.

Dave - no amount of userspace differences make a corrupted page table 
acceptable. 

This needs to be fixed. No excuses. Kernel crashes are never an issue of 
you used the wrong user space.

Linus

--
Are you an open source citizen? Join us for the Open Source Bridge conference!
Portland, OR, June 17-19. Two days of sessions, one day of unconference: $250.
Need another reason to go? 24-hour hacker lounge. Register today!
http://ad.doubleclick.net/clk;215844324;13503038;v?http://opensourcebridge.org
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: [git pull] drm: previous pull req + 1.

2009-06-21 Thread Linus Torvalds


On Sun, 21 Jun 2009, Linus Torvalds wrote:
 
 Dave - no amount of userspace differences make a corrupted page table 
 acceptable. 
 
 This needs to be fixed. No excuses. Kernel crashes are never an issue of 
 you used the wrong user space.

So corrupted page table means that one of the reserved bits was set, and 
we get a page fault with the PF_RSVD bit on in the error code.

Looking at the debug output, it says

PGD12148a067
PUD12148b067
PMD121496067
PTE c90011780237

where the top-level entries look fine, but the PTE is total crap. It looks 
like it has filled in the page frame number with a virtual address rather 
than with an actual page

The PTE _should_ look like this:

 - bit 63: NX
 - bits 62-52: zero (available to sw, but I don't think we use them)
 - bits 51-47: zero (reserved)
 - bits 46-12: page frame
 - bits 11-0: protection and PAT bits etc (bits 8-7 are also reserved)

and that PTE clearly does not match.

Strictly speaking, that 47-bit physical address is purely theoretical. I 
think existing CPU's are limited to 40 bits or so, so there are even more 
reserved bits. 

Anyway, here's a totally UNTESTED patch that hopefully gives a warning on 
where exactly we set the invalid bits. Andy, mind trying it out? You 
should get the warnign much earlier, and it should have a much more useful 
back-trace.

(But this is _untested_, so maybe I screwed up and it doesn't compile or 
work. The BAD_PTE_BITS mask could also be improved upon, but that mask 
should be good enough - it doesn't include _all_ the bits it could, 
but it certainly has enough bits set to trigger that obviously bad case).

Linus

---
 arch/x86/include/asm/pgtable_64.h |3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/arch/x86/include/asm/pgtable_64.h 
b/arch/x86/include/asm/pgtable_64.h
index c57a301..b95828e 100644
--- a/arch/x86/include/asm/pgtable_64.h
+++ b/arch/x86/include/asm/pgtable_64.h
@@ -49,8 +49,11 @@ static inline void native_pte_clear(struct mm_struct *mm, 
unsigned long addr,
*ptep = native_make_pte(0);
 }
 
+#define BAD_PTE_BITS (_PAGE_NX - (1ul  __PHYSICAL_MASK_SHIFT))
+
 static inline void native_set_pte(pte_t *ptep, pte_t pte)
 {
+   WARN_ON_ONCE(pte.pte  BAD_PTE_BITS);
*ptep = pte;
 }
 

--
Are you an open source citizen? Join us for the Open Source Bridge conference!
Portland, OR, June 17-19. Two days of sessions, one day of unconference: $250.
Need another reason to go? 24-hour hacker lounge. Register today!
http://ad.doubleclick.net/clk;215844324;13503038;v?http://opensourcebridge.org
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


i915: readpix demo regression

2009-06-21 Thread Eric Anholt
I was checking on some driver regressions this weekend, and thought that
readpixels failure on i915 might have been my fault, but bisect claims
it's:

dd26899ca39111e0866afed9df94bfb1618dd363 is first bad commit
commit dd26899ca39111e0866afed9df94bfb1618dd363
Author: Michel Dänzer daen...@vmware.com
Date:   Fri Jun 19 23:55:55 2009 +0200

intel: Fixups for 'mesa: create/destroy buffer objects via driver 
functions'.

Initialize all driver function hooks before calling 
_mesa_initialize_context(),
and handle all buffer objects in intel_buffer_object().

Fixes assertion failure when running glxinfo.

:04 04 d2d5b6bc870422b08b9fb5a6d196fb390815cbdf 
144a2d92751f48321a44430bfe8c4768eec59a97 M  src

The effect is that the Read/DrawPixels area of the readpix demo is
white, and conformance tests all fail.

-- 

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




signature.asc
Description: This is a digitally signed message part
--
Are you an open source citizen? Join us for the Open Source Bridge conference!
Portland, OR, June 17-19. Two days of sessions, one day of unconference: $250.
Need another reason to go? 24-hour hacker lounge. Register today!
http://ad.doubleclick.net/clk;215844324;13503038;v?http://opensourcebridge.org--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: [git pull] drm: previous pull req + 1.

2009-06-21 Thread Linus Torvalds


On Sun, 21 Jun 2009, Andrew Lutomirski wrote:
 
  Anyway, here's a totally UNTESTED patch that hopefully gives a warning on
  where exactly we set the invalid bits. Andy, mind trying it out? You
  should get the warnign much earlier, and it should have a much more useful
  back-trace.
 
 Your patch worked.  Photo attached.

Ok.

So it's fb_mmap() that uses an invalid page frame number when it does the 
io_remap_pfn_range() thing. 

And the way it gets that page frame number is basically

 - Offset (in bytes) from start of mapping:

off = vma-vm_pgoff  PAGE_SHIFT;
..

 - frame buffer start address:

/* frame buffer memory */
start = info-fix.smem_start;
len = PAGE_ALIGN((start  ~PAGE_MASK) + info-fix.smem_len);
..
off += start;

 - do the remap:

io_remap_pfn_range(vma, vma-vm_start, off  PAGE_SHIFT, ..

and there has been no changes to this logic in drivers/video/fbmem.c 
lately.

What *has* changed is that we have a newradeon driver, and it looks like 
that new radeon driver is crap, and does this:

info-fix.smem_start = (unsigned long)fbptr;

which is totally screwed up. It assigns a _virtual_ address to that 
smem_start thing, even though it should be a physical one. 

I don't know the radeon driver, so I don't know where to find the physical 
address.  It's also possible that there is no good single physical 
address, and the radeon driver should implement a fb_mmap function.

Does this patch make the warning and the oops at least go away? Obviously 
it won't result in a working frame buffer, but that's a separate issue

Linus

---
 drivers/gpu/drm/radeon/radeon_fb.c |9 +
 1 files changed, 9 insertions(+), 0 deletions(-)

diff --git a/drivers/gpu/drm/radeon/radeon_fb.c 
b/drivers/gpu/drm/radeon/radeon_fb.c
index fa86d39..4aa151e 100644
--- a/drivers/gpu/drm/radeon/radeon_fb.c
+++ b/drivers/gpu/drm/radeon/radeon_fb.c
@@ -380,6 +380,14 @@ int radeonfb_blank(int blank, struct fb_info *info)
return 0;
 }
 
+/*
+ * Not yet implemented. The fix.smem_start is crap.
+ */
+static int radeonfb_mmap(struct fb_info *info, struct vm_area_struct *vma)
+{
+   return -EINVAL;
+}
+
 static struct fb_ops radeonfb_ops = {
.owner = THIS_MODULE,
.fb_check_var = radeonfb_check_var,
@@ -390,6 +398,7 @@ static struct fb_ops radeonfb_ops = {
.fb_imageblit = cfb_imageblit,
.fb_pan_display = radeonfb_pan_display,
.fb_blank = radeonfb_blank,
+   .fb_mmap = radeonfb_mmap,
 };
 
 /**

--
Are you an open source citizen? Join us for the Open Source Bridge conference!
Portland, OR, June 17-19. Two days of sessions, one day of unconference: $250.
Need another reason to go? 24-hour hacker lounge. Register today!
http://ad.doubleclick.net/clk;215844324;13503038;v?http://opensourcebridge.org
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: [git pull] drm: previous pull req + 1.

2009-06-21 Thread Chris Wilson
On Sun, 2009-06-21 at 17:47 +0300, Maxim Levitsky wrote:
  52dc7d32b88156248167864f77a9026abe27b432 is first bad commit
  commit 52dc7d32b88156248167864f77a9026abe27b432
  Author: Chris Wilson ch...@chris-wilson.co.uk
  Date:   Sat Jun 6 09:46:01 2009 +0100

The error here seems to be my presumption that only the i915 was using
fences for GPU access. (In hindsight, it seems obvious that we do not
know why the fence was allocated for the object and so if it has
outstanding rendering, we must assume that it is using a fence for a
rendering op.)

To confirm, please can you try:

diff --git a/drivers/gpu/drm/i915/i915_gem.c
b/drivers/gpu/drm/i915/i915_gem.c
index fd2b8bd..0735518 100644
--- a/drivers/gpu/drm/i915/i915_gem.c
+++ b/drivers/gpu/drm/i915/i915_gem.c
@@ -2347,7 +2347,7 @@ i915_gem_object_put_fence_reg(struct
drm_gem_object *obj)
 * therefore we must wait for any outstanding access to complete
 * before clearing the fence.
 */
-   if (!IS_I965G(dev)) {
+   if (1) {
int ret;
 
i915_gem_object_flush_gpu_write_domain(obj);

(I'm travelling tomorrow, so if this does turn out to be the fault, I'd
appreciate it if someone wrote it up as a complete patch.)

 However I can't reproduce the situation I have earlier, maybe I have changed 
 some settings, don't know.
 Now, the bad behavior (and I reproduced it many times, is that GE shows 
 incorrect textures 
 (like they are divided in tiny interlaced rows, one row ok, other contain 
 image from other part of world), only few textures are such
 it seems logical that this is related to tiling.

What you describe is exactly the artefact you would see when you access
a tiled texture using linear addressing. Pretty conclusive evidence that
we do need to flush outstanding GPU access.

 Also, if I maximize it, it hangs. This seems to be a separate bug introduced 
 by these series.

It does seem like a separate bug. Hopefully, we can get a better handle
on it after we fix this obnoxious tiling issue.
-ickle


--
Are you an open source citizen? Join us for the Open Source Bridge conference!
Portland, OR, June 17-19. Two days of sessions, one day of unconference: $250.
Need another reason to go? 24-hour hacker lounge. Register today!
http://ad.doubleclick.net/clk;215844324;13503038;v?http://opensourcebridge.org
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: [git pull] drm: previous pull req + 1.

2009-06-21 Thread Dave Airlie
 
 
 On Sun, 21 Jun 2009, Andrew Lutomirski wrote:
  
   Anyway, here's a totally UNTESTED patch that hopefully gives a warning on
   where exactly we set the invalid bits. Andy, mind trying it out? You
   should get the warnign much earlier, and it should have a much more useful
   back-trace.
  
  Your patch worked.  Photo attached.
 
 Ok.
 
 So it's fb_mmap() that uses an invalid page frame number when it does the 
 io_remap_pfn_range() thing. 
 
 And the way it gets that page frame number is basically
 
  - Offset (in bytes) from start of mapping:
 
   off = vma-vm_pgoff  PAGE_SHIFT;
   ..
 
  - frame buffer start address:
 
 /* frame buffer memory */
 start = info-fix.smem_start;
 len = PAGE_ALIGN((start  ~PAGE_MASK) + info-fix.smem_len);
   ..
   off += start;
 
  - do the remap:
 
   io_remap_pfn_range(vma, vma-vm_start, off  PAGE_SHIFT, ..
 
 and there has been no changes to this logic in drivers/video/fbmem.c 
 lately.
 
 What *has* changed is that we have a newradeon driver, and it looks like 
 that new radeon driver is crap, and does this:
 
   info-fix.smem_start = (unsigned long)fbptr;
 
 which is totally screwed up. It assigns a _virtual_ address to that 
 smem_start thing, even though it should be a physical one. 
 
 I don't know the radeon driver, so I don't know where to find the physical 
 address.  It's also possible that there is no good single physical 
 address, and the radeon driver should implement a fb_mmap function.
 
 Does this patch make the warning and the oops at least go away? Obviously 
 it won't result in a working frame buffer, but that's a separate issue
 
   Linus

I noticed the same bogus line myself last night, I'll get Jerome to look 
at it since its his code, he was trying to be smart about how the 
radeon fbdev emulation should work, but fbdev isn't smart enough to do 
what he wants, so I've asked him to go back to the dumb pin the fbcon in 
VRAM until we can fix fbdev to do some sort of prepare/commit type hooks 
around blocks of reads/writes.

With the safe method we end up with an 8MB pinned fbcon on 32MB in some 
scenarios, which is still totally unacceptable from a user pov.

Btw this driver is under staging for a good reason, nobody claimed it was 
perfect, we just said we felt it was a good baseline to build the final 
driver on top off. Maybe I can add 
CONFIG_DONT_CC_LINUS_ON_STAGING_DRIVERS.

Dave.

  
 ---
  drivers/gpu/drm/radeon/radeon_fb.c |9 +
  1 files changed, 9 insertions(+), 0 deletions(-)
 
 diff --git a/drivers/gpu/drm/radeon/radeon_fb.c 
 b/drivers/gpu/drm/radeon/radeon_fb.c
 index fa86d39..4aa151e 100644
 --- a/drivers/gpu/drm/radeon/radeon_fb.c
 +++ b/drivers/gpu/drm/radeon/radeon_fb.c
 @@ -380,6 +380,14 @@ int radeonfb_blank(int blank, struct fb_info *info)
   return 0;
  }
  
 +/*
 + * Not yet implemented. The fix.smem_start is crap.
 + */
 +static int radeonfb_mmap(struct fb_info *info, struct vm_area_struct *vma)
 +{
 + return -EINVAL;
 +}
 +
  static struct fb_ops radeonfb_ops = {
   .owner = THIS_MODULE,
   .fb_check_var = radeonfb_check_var,
 @@ -390,6 +398,7 @@ static struct fb_ops radeonfb_ops = {
   .fb_imageblit = cfb_imageblit,
   .fb_pan_display = radeonfb_pan_display,
   .fb_blank = radeonfb_blank,
 + .fb_mmap = radeonfb_mmap,
  };
  
  /**
 
 --
 Are you an open source citizen? Join us for the Open Source Bridge conference!
 Portland, OR, June 17-19. Two days of sessions, one day of unconference: $250.
 Need another reason to go? 24-hour hacker lounge. Register today!
 http://ad.doubleclick.net/clk;215844324;13503038;v?http://opensourcebridge.org
 --
 ___
 Dri-devel mailing list
 Dri-devel@lists.sourceforge.net
 https://lists.sourceforge.net/lists/listinfo/dri-devel
 
 

--
Are you an open source citizen? Join us for the Open Source Bridge conference!
Portland, OR, June 17-19. Two days of sessions, one day of unconference: $250.
Need another reason to go? 24-hour hacker lounge. Register today!
http://ad.doubleclick.net/clk;215844324;13503038;v?http://opensourcebridge.org
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: [git pull] drm: previous pull req + 1.

2009-06-21 Thread Dave Airlie
 
 
 On Sun, 21 Jun 2009, Dave Airlie wrote:
   
   I tried this tree (specifically, a merge of Linus' fb20871 this tree) on
   Fedora 11 with modesetting enabled on an integrated Radeon 2100, and 
   plymouthd
   crashes immediately with a corrupt page table.  Photo attached.  After the
   crash, bootup stops, although ctrl-alt-del works.
  
  You need a different userspace, -ati from koji in F11 should do it.
 
 Dave - no amount of userspace differences make a corrupted page table 
 acceptable. 

I was original responding to a line I saw in his dmesg (which was about an 
ioctl not being available, I only got to parsing the crash later.

 
 This needs to be fixed. No excuses. Kernel crashes are never an issue of 
 you used the wrong user space.

Agreed, and we block old userspaces running on KMS kernels (at least 
trying to use DRI) and I thought that was what he was seeing.

Dave.

 
   Linus
 
 --
 Are you an open source citizen? Join us for the Open Source Bridge conference!
 Portland, OR, June 17-19. Two days of sessions, one day of unconference: $250.
 Need another reason to go? 24-hour hacker lounge. Register today!
 http://ad.doubleclick.net/clk;215844324;13503038;v?http://opensourcebridge.org
 --
 ___
 Dri-devel mailing list
 Dri-devel@lists.sourceforge.net
 https://lists.sourceforge.net/lists/listinfo/dri-devel
 
 

--
Are you an open source citizen? Join us for the Open Source Bridge conference!
Portland, OR, June 17-19. Two days of sessions, one day of unconference: $250.
Need another reason to go? 24-hour hacker lounge. Register today!
http://ad.doubleclick.net/clk;215844324;13503038;v?http://opensourcebridge.org
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 22405] New: [regression OGLC] most of cases failed due to commit dd26899ca39111e0866afed9df94bfb1618dd363

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

   Summary: [regression OGLC] most of cases failed due to commit
dd26899ca39111e0866afed9df94bfb1618dd363
   Product: Mesa
   Version: CVS
  Platform: x86 (IA32)
OS/Version: Linux (All)
Status: NEW
  Severity: blocker
  Priority: highest
 Component: Drivers/DRI/i915
AssignedTo: dri-devel@lists.sourceforge.net
ReportedBy: shuang...@intel.com


System Environment:
--
Arch:   i386
Platform:   945GM
Mesa(master)df70d3049a396af3601d2a1747770635a74120bb
Xserver (master)ae20e748cd3a656173e1f50109bfd4af0712bb87
Xf86_video_intel(master)534e73ad4f234a04755917f2bf17ba821c27eb52
Libdrm  (master)2fa2db138ba989bfa1a8cd9ab66d83fb7369249e 

Bug detailed description:
-
most OGLC case (like mustpass.c, xformmix.c, vorder.c, ...) failed now. This is
caused by following commit:

commit dd26899ca39111e0866afed9df94bfb1618dd363
Author: Michel Dänzer daen...@vmware.com
Date:   Fri Jun 19 23:55:55 2009 +0200

intel: Fixups for 'mesa: create/destroy buffer objects via driver
functions'.

Initialize all driver function hooks before calling
_mesa_initialize_context(),
and handle all buffer objects in intel_buffer_object().

Fixes assertion failure when running glxinfo.



Reproduce steps:

1.xinit
2. run olgc/mustpass.c ...


-- 
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
--
Are you an open source citizen? Join us for the Open Source Bridge conference!
Portland, OR, June 17-19. Two days of sessions, one day of unconference: $250.
Need another reason to go? 24-hour hacker lounge. Register today!
http://ad.doubleclick.net/clk;215844324;13503038;v?http://opensourcebridge.org
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel