Re: [PATCHES] radeon kms pm patches

2010-03-15 Thread Rafał Miłecki
2010/3/15 Alex Deucher alexdeuc...@gmail.com:
 This weekend I finally got some time to sit down with kms pm, and I
 created a new patch set (against drm-radeon-testing).  For those that
 want to play with the i2c stuff for thermal chips, you'll probably
 grab my recent i2c patches as well.  You can grab my latest pm patches
 here:
 http://people.freedesktop.org/~agd5f/pm2/

You can consider porting
[PATCH 2/2] drm/radeon/kms: prepare for more reclocking operations

You will need to sync with VBLANK few times I believe.

Not sure about GUI syncing, do not know what is this.

-- 
Rafał

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: [PATCHES] radeon kms pm patches

2010-03-15 Thread Rafał Miłecki
2010/3/15 Alex Deucher alexdeuc...@gmail.com:
 This weekend I finally got some time to sit down with kms pm, and I
 created a new patch set (against drm-radeon-testing).  For those that
 want to play with the i2c stuff for thermal chips, you'll probably
 grab my recent i2c patches as well.  You can grab my latest pm patches
 here:
 http://people.freedesktop.org/~agd5f/pm2/

http://people.freedesktop.org/~agd5f/pm2/0009-drm-radeon-kms-r6xx-enable-simd-pm.patch

Didn't you miss there something actually *setting* SIMD(s?)?

-- 
Rafał

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: [PATCHES] radeon kms pm patches

2010-03-15 Thread Alex Deucher
2010/3/15 Rafał Miłecki zaj...@gmail.com:
 2010/3/15 Alex Deucher alexdeuc...@gmail.com:
 This weekend I finally got some time to sit down with kms pm, and I
 created a new patch set (against drm-radeon-testing).  For those that
 want to play with the i2c stuff for thermal chips, you'll probably
 grab my recent i2c patches as well.  You can grab my latest pm patches
 here:
 http://people.freedesktop.org/~agd5f/pm2/

 So far I haven't seen any corruption when changing power modes.

 Why don't you post them inline? It would be possible to comment any
 part of code.

It was a lot of patches.  I can post them with the message next time.


 1) What is GUI?

Graphical User Interface.  In this case it refers to the drawing
engine and related blocks (2D, 3D, CP, etc.).

 2) What is SIMD?

SIMD (Single Instruction Multiple Data) blocks are
the computational blocks of the shader engine.

Alex


 --
 Rafał


--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: [PATCHES] radeon kms pm patches

2010-03-15 Thread Alex Deucher
2010/3/15 Rafał Miłecki zaj...@gmail.com:
 2010/3/15 Alex Deucher alexdeuc...@gmail.com:
 This weekend I finally got some time to sit down with kms pm, and I
 created a new patch set (against drm-radeon-testing).  For those that
 want to play with the i2c stuff for thermal chips, you'll probably
 grab my recent i2c patches as well.  You can grab my latest pm patches
 here:
 http://people.freedesktop.org/~agd5f/pm2/

 http://people.freedesktop.org/~agd5f/pm2/0003-drm-radeon-kms-add-some-comments-about-radeon_asic.patch

 Code you removed was needed for PM debugfs, to do not display memory
 clock on IGP

Oh, right... debugfs support is also currently broken with multiple
cards.  I'll just drop this patch for now.

Alex

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: [PATCHES] radeon kms pm patches

2010-03-15 Thread Alex Deucher
2010/3/15 Rafał Miłecki zaj...@gmail.com:
 2010/3/15 Alex Deucher alexdeuc...@gmail.com:
 This weekend I finally got some time to sit down with kms pm, and I
 created a new patch set (against drm-radeon-testing).  For those that
 want to play with the i2c stuff for thermal chips, you'll probably
 grab my recent i2c patches as well.  You can grab my latest pm patches
 here:
 http://people.freedesktop.org/~agd5f/pm2/

 http://people.freedesktop.org/~agd5f/pm2/0009-drm-radeon-kms-r6xx-enable-simd-pm.patch

 Didn't you miss there something actually *setting* SIMD(s?)?

heh... whoops.  fixed.

Alex

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: [PATCHES] radeon kms pm patches

2010-03-15 Thread Alex Deucher
2010/3/15 Rafał Miłecki zaj...@gmail.com:
 2010/3/15 Alex Deucher alexdeuc...@gmail.com:
 This weekend I finally got some time to sit down with kms pm, and I
 created a new patch set (against drm-radeon-testing).  For those that
 want to play with the i2c stuff for thermal chips, you'll probably
 grab my recent i2c patches as well.  You can grab my latest pm patches
 here:
 http://people.freedesktop.org/~agd5f/pm2/

 You can consider porting
 [PATCH 2/2] drm/radeon/kms: prepare for more reclocking operations

 You will need to sync with VBLANK few times I believe.

 Not sure about GUI syncing, do not know what is this.

I've gone ahead and rebased my patches against Dave's current
drm-radeon-testing branch, so both patches should be in now.

Alex

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: [PATCHES] radeon kms pm patches

2010-03-15 Thread Alex Deucher
2010/3/15 Rafał Miłecki zaj...@gmail.com:
 2010/3/15 Alex Deucher alexdeuc...@gmail.com:
 This weekend I finally got some time to sit down with kms pm, and I
 created a new patch set (against drm-radeon-testing).  For those that
 want to play with the i2c stuff for thermal chips, you'll probably
 grab my recent i2c patches as well.  You can grab my latest pm patches
 here:
 http://people.freedesktop.org/~agd5f/pm2/

 http://people.freedesktop.org/~agd5f/pm2/0006-drm-radeon-kms-wait-for-gpu-idle-before-changing-po.patch

 You made same mistake I did earlier with 0 argument in condition.

 You should use same trick as I did in:
 http://people.freedesktop.org/~agd5f/pm2/0013-drm-radeon-kms-switch-to-condition-waiting-for-recl.patch

Good catch!  fixed.

Alex

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: [PATCH] drm/radeon/kms: gfx init fixes for r6xx/r7xx

2010-03-15 Thread Alex Deucher
Looks like this one might have gotten lost in the shuffle.

Alex

On Fri, Mar 5, 2010 at 2:55 PM, Alex Deucher alexdeuc...@gmail.com wrote:
 From cec90cfdc0f20efcbcd266069a6a8234d230cc0b Mon Sep 17 00:00:00 2001
 From: Alex Deucher alexdeuc...@gmail.com
 Date: Fri, 5 Mar 2010 14:50:37 -0500
 Subject: [PATCH] drm/radeon/kms: gfx init fixes for r6xx/r7xx

 This fixes some issues with the last gfx init patch.

 Signed-off-by: Alex Deucher alexdeuc...@gmail.com
 ---
  drivers/gpu/drm/radeon/r600.c    |    1 +
  drivers/gpu/drm/radeon/r600_cp.c |    3 +++
  drivers/gpu/drm/radeon/rv770.c   |    3 +++
  3 files changed, 7 insertions(+), 0 deletions(-)

 diff --git a/drivers/gpu/drm/radeon/r600.c b/drivers/gpu/drm/radeon/r600.c
 index f9a8335..3980ec9 100644
 --- a/drivers/gpu/drm/radeon/r600.c
 +++ b/drivers/gpu/drm/radeon/r600.c
 @@ -1132,6 +1132,7 @@ void r600_gpu_init(struct radeon_device *rdev)
        /* Setup pipes */
        WREG32(CC_RB_BACKEND_DISABLE, cc_rb_backend_disable);
        WREG32(CC_GC_SHADER_PIPE_CONFIG, cc_gc_shader_pipe_config);
 +       WREG32(GC_USER_SHADER_PIPE_CONFIG, cc_gc_shader_pipe_config);

        tmp = R6XX_MAX_PIPES -
 r600_count_pipe_bits((cc_gc_shader_pipe_config 
 INACTIVE_QD_PIPES_MASK)  8);
        WREG32(VGT_OUT_DEALLOC_CNTL, (tmp * 4)  DEALLOC_DIST_MASK);
 diff --git a/drivers/gpu/drm/radeon/r600_cp.c 
 b/drivers/gpu/drm/radeon/r600_cp.c
 index 40416c0..68e6f43 100644
 --- a/drivers/gpu/drm/radeon/r600_cp.c
 +++ b/drivers/gpu/drm/radeon/r600_cp.c
 @@ -1548,10 +1548,13 @@ static void r700_gfx_init(struct drm_device *dev,

        RADEON_WRITE(R600_CC_RB_BACKEND_DISABLE,      cc_rb_backend_disable);
        RADEON_WRITE(R600_CC_GC_SHADER_PIPE_CONFIG,   
 cc_gc_shader_pipe_config);
 +       RADEON_WRITE(R600_GC_USER_SHADER_PIPE_CONFIG, 
 cc_gc_shader_pipe_config);

        RADEON_WRITE(R700_CC_SYS_RB_BACKEND_DISABLE, cc_rb_backend_disable);
        RADEON_WRITE(R700_CGTS_SYS_TCC_DISABLE, 0);
        RADEON_WRITE(R700_CGTS_TCC_DISABLE, 0);
 +       RADEON_WRITE(R700_CGTS_USER_SYS_TCC_DISABLE, 0);
 +       RADEON_WRITE(R700_CGTS_USER_TCC_DISABLE, 0);

        num_qd_pipes =
                R7XX_MAX_PIPES - 
 r600_count_pipe_bits((cc_gc_shader_pipe_config 
 R600_INACTIVE_QD_PIPES_MASK)  8);
 diff --git a/drivers/gpu/drm/radeon/rv770.c b/drivers/gpu/drm/radeon/rv770.c
 index 37887de..63c181b 100644
 --- a/drivers/gpu/drm/radeon/rv770.c
 +++ b/drivers/gpu/drm/radeon/rv770.c
 @@ -647,10 +647,13 @@ static void rv770_gpu_init(struct radeon_device *rdev)

        WREG32(CC_RB_BACKEND_DISABLE,      cc_rb_backend_disable);
        WREG32(CC_GC_SHADER_PIPE_CONFIG,   cc_gc_shader_pipe_config);
 +       WREG32(GC_USER_SHADER_PIPE_CONFIG, cc_gc_shader_pipe_config);
        WREG32(CC_SYS_RB_BACKEND_DISABLE,  cc_rb_backend_disable);

        WREG32(CGTS_SYS_TCC_DISABLE, 0);
        WREG32(CGTS_TCC_DISABLE, 0);
 +       WREG32(CGTS_USER_SYS_TCC_DISABLE, 0);
 +       WREG32(CGTS_USER_TCC_DISABLE, 0);

        num_qd_pipes =
                R7XX_MAX_PIPES - 
 r600_count_pipe_bits((cc_gc_shader_pipe_config 
 INACTIVE_QD_PIPES_MASK)  8);
 --
 1.5.6.3


--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[PATCH] drm/radeon/kms/pm: fix typo in power table parsing

2010-03-15 Thread Alex Deucher
From b4880d6cb8d7913bf177c463cc8ffce8a2c1785a Mon Sep 17 00:00:00 2001
From: Alex Deucher alexdeuc...@gmail.com
Date: Mon, 15 Mar 2010 01:36:32 -0400
Subject: [PATCH] drm/radeon/kms/pm: fix typo in power table parsing

Signed-off-by: Alex Deucher alexdeuc...@gmail.com
---
 drivers/gpu/drm/radeon/radeon_atombios.c |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/gpu/drm/radeon/radeon_atombios.c
b/drivers/gpu/drm/radeon/radeon_atombios.c
index 815116e..92e6886 100644
--- a/drivers/gpu/drm/radeon/radeon_atombios.c
+++ b/drivers/gpu/drm/radeon/radeon_atombios.c
@@ -1734,7 +1734,7 @@ void radeon_atombios_get_power_modes(struct
radeon_device *rdev)
/* add the i2c bus for thermal/fan chip */
/* no support for internal controller yet */
if (power_info-info_4.sThermalController.ucType  0) {
-   if 
((power_info-info_4.sThermalController.ucType ==
ATOM_PP_THERMALCONTROLLER_RV6xx) 
+   if 
((power_info-info_4.sThermalController.ucType ==
ATOM_PP_THERMALCONTROLLER_RV6xx) ||

(power_info-info_4.sThermalController.ucType ==
ATOM_PP_THERMALCONTROLLER_RV770)) {
DRM_INFO(Internal thermal controller 
%s fan control\n,
 
(power_info-info_4.sThermalController.ucFanParameters 
-- 
1.5.6.3


0001-drm-radeon-kms-pm-fix-typo-in-power-table-parsing.patch
Description: application/mbox
--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 27071] hang in sauerbaten after recent git master upgrade

2010-03-15 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=27071





--- Comment #1 from Pauli suok...@gmail.com  2010-03-15 01:39:57 PST ---
Created an attachment (id=34055)
 -- (http://bugs.freedesktop.org/attachment.cgi?id=34055)
Fix double unmap call when flushing from refill

Does this fix the bug for you?


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

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 27043] [bisected DRI1] drmRadeonCmdBuffer: -22. Kernel failed to parse or rejected command stream. See dmesg for more info.

2010-03-15 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=27043





--- Comment #8 from Fabio Pedretti fabio@libero.it  2010-03-15 01:55:25 
PST ---
(In reply to comment #6)
 (In reply to comment #5)
  Did you actually test glReadPixels() with that?
 
 Err, I mean glCopyPixels and glDrawPixels.

I tested only the game, but shouldn't _mesa_meta_DrawPixels and
_mesa_meta_CopyPixels be already the default BTW?


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

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 26954] [bisected regression] sauerbraten asserts with: Assertion `code-inst_end = 0' failed.

2010-03-15 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=26954





--- Comment #2 from Fabio Pedretti fabio@libero.it  2010-03-15 02:20:35 
PST ---
Created an attachment (id=34056)
 -- (http://bugs.freedesktop.org/attachment.cgi?id=34056)
r300 compiler: workaround for sauerbraten crash

The game always run fine before the recently added assert. The attached patch
uses _mesa_problem instead of the assert. Please apply the patch to the 7.8
branch until the real bug is fixed.


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

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 26954] [bisected regression] sauerbraten asserts with: Assertion `code-inst_end = 0' failed.

2010-03-15 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=26954





--- Comment #3 from Dave Airlie airl...@freedesktop.org  2010-03-15 02:26:44 
PST ---
demos/shadowtex  also has this assert firing.


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

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 27069] OpenGl is slower on dri2

2010-03-15 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=27069





--- Comment #1 from Stefan Dösinger stefandoesin...@gmx.at  2010-03-15 
04:12:49 PST ---
Just for the record, I can confirm this issue here as well. DRI2 barely runs
etracer on my Radeon 9000 mobility, with the dri1 code it runs at 200+ fps.
Similar results on my r500 card.


-- 
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 27083] New: Commit use lcd pll limits when available in drm-radeon-testing breaks my laptop lvds

2010-03-15 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=27083

   Summary: Commit  use lcd pll limits when available in drm-
radeon-testing breaks my laptop lvds
   Product: DRI
   Version: DRI CVS
  Platform: x86-64 (AMD64)
OS/Version: Linux (All)
Status: NEW
  Severity: normal
  Priority: medium
 Component: DRM/Radeon
AssignedTo: dri-devel@lists.sourceforge.net
ReportedBy: scary@gmail.com


As the summary says, commit 86cb2bbfda2cf402aee46779ee90bbb7d915482b
(drm/radeon/kms: use lcd pll limits when available) in drm-radeon-testing
produces unwatchable fuzziness on my laptop screen (toshiba a300, hd3470
mobility). Reverting the commit brings the laptop back to an usable state.
Some system info was already attached at the (old, similar, fixed) bug 
http://bugs.freedesktop.org/show_bug.cgi?id=26358


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

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 26641] RV730 agp xf86-video-ati with kms poor performance

2010-03-15 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=26641





--- Comment #10 from Andy Furniss li...@andyfurniss.entadsl.com  2010-03-15 
05:23:09 PST ---
(In reply to comment #9)
 *** Bug 26816 has been marked as a duplicate of this bug. ***
 

I wasn't too sure about this when it was changed and running todays drt
confirms  for me, that they are not the same issue.

comment #4 has this as a dup of

http://bugzilla.kernel.org/show_bug.cgi?id=15328

which was a regression caused by a specific commit affecting 2D  3D perf over
time, which now seems to be fixed in current drt.

http://bugs.freedesktop.org/show_bug.cgi?id=26816

was not a regression and running todays drt has actually got worse (though
possibly not same xserver) - I can't even play the vid I used in that bug using
x11 now, mplayer shows 125% in vo.


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

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: [PATCHES] radeon drm kms i2c fixes

2010-03-15 Thread Jean Delvare
On Sun, 14 Mar 2010 11:37:02 -0500, Alex Deucher wrote:
 On Wed, Mar 10, 2010 at 7:09 PM, Alex Deucher alexdeuc...@gmail.com wrote:
  Thanks to Jean for helping me with the i2c stuff.  I've attached
  Jean's bit algo patch as my radeon patch depends on it.  I'm not sure
  how we want to get that one upstream (either via drm or i2c).
  Previously, the radeon drm registered i2c buses using the radeon algo
  which would use either the hw i2c engine or bit banging depending on
  the bus in question (some are hw capable, others are not, some chips
  don't have support for their hw engines yet, etc.).  The tricky part
  was that the radeon i2c bit buses require some gpio magic before and
  after a transaction which bit algo didn't previously support.
  Unfortunately, it exposed the internal bit algo bus as well we as the
  radeon algo bus which is bad.  With these patches, if the hw engine is
  supported, we use the radeon algo, if not, we use bit algo directly
  with the pre/post_xfer functions to fix up the gpios.  I've tested on
  several radeons, but more tested would be nice.
 
 Updated patch based on Jean's new version of the pre/post_xfer patch
 (also attached) which he plans to push upstream via the i2c tree.

FYI, the pre/post-xfer patch is upstream now:
http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=0a9c14751377a1407f5e35791e13651d2fc7801c

-- 
Jean Delvare

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: [PATCHES] radeon drm kms i2c fixes

2010-03-15 Thread Jean Delvare
On Thu, 11 Mar 2010 13:35:47 -0500, Alex Deucher wrote:
 On Thu, Mar 11, 2010 at 1:00 PM, Jean Delvare kh...@linux-fr.org wrote:
  I have the following in my machine:
  02:00.0 VGA compatible controller: ATI Technologies Inc RV280 [Radeon 9200] 
  (rev 01)
  02:00.1 Display controller: ATI Technologies Inc RV280 [Radeon 9200] 
  (Secondary) (rev 01)
 
  Can I help with testing? I can follow your instructions.
 
 Sure.  You need a kms-enabled graphics stack.  See this for more:
 http://wiki.x.org/wiki/radeonBuildHowTo
 Then, make sure you have the latest kernel bits and the patches I
 posted.  Note that some users have reported problems with the hw i2c
 engine on some r1xx-r3xx boards.  I suspect a problematic prescale or
 a drive problem.  If you run into issues, please try the patch I
 attached to this bug:
 http://bugs.freedesktop.org/show_bug.cgi?id=26430

Huuu, looks a little too time consuming, I'm afraid. Anything requiring
rebuilding Xorg isn't an option for me. I'm running openSUSE 11.1, it
has Xorg 7.4, I guess it's too old... And openSUSE 11.2 has Xorg 7.4
too, I don't think is is KMS-enabled, so I guess upgrading won't help.
I will probably have to wait for openSUSE 11.3 before I can do any
testing, sorry.

-- 
Jean Delvare

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 26781] RV670 Oops with PM and xrandr.

2010-03-15 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=26781


Andy Furniss li...@andyfurniss.entadsl.com changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED




--- Comment #2 from Andy Furniss li...@andyfurniss.entadsl.com  2010-03-15 
06:58:27 PST ---
(In reply to comment #1)
 Created an attachment (id=33599)
 -- (http://bugs.freedesktop.org/attachment.cgi?id=33599) [details]
 pm oops
 

Seems to be fixed in todays drm-radeon-testing.


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

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 26631] R600 AGP KMS + dynpm,dynclks = stalls

2010-03-15 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=26631


Andy Furniss li...@andyfurniss.entadsl.com changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED




--- Comment #12 from Andy Furniss li...@andyfurniss.entadsl.com  2010-03-15 
07:12:32 PST ---
(In reply to comment #11)
 (In reply to comment #10)
 
  I also notice that even just running gears the clock changes up/down
  accompanied by visual disturbance every couple of seconds, which seems a bit
  pointless, as do a lot of the in game changes.
 
 Another thing I've noticed is that if I have two screens cloned then pm 
 doesn't
 do this. In fact the only time it seems to change clock is to reduce going 
 into
 DPMS and increase to full on coming back out, so I don't get pm during use
 unless I turn off one of the screens with xrandr.
 

Although the above issues still occur with today's drt the stalls that this bug
refers to are now gone - so closing.

Missing vbls/disturbances still happen, but only about 1 time out of 5 now
(85Hz CRT)


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

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 25717] Radeon: system locks up when sending many vertices.

2010-03-15 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=25717





--- Comment #2 from Andy Furniss li...@andyfurniss.entadsl.com  2010-03-15 
07:57:52 PST ---
(In reply to comment #1)

Running todays drt things have changed a bit -

With AGP gart I can now run gltestperf OK, though I suspect this could be by
luck as the glxgears type perf is a bit lower than recent kernels @ 800fps so
maybe the (poor) caching is slowing things down a bit and avoiding it.

With PCIE gart, where glxgears is 1200, I still hit the lockup, but now I am
saved by the new GPU reset code. 

Mar 15 14:29:37 nf7 kernel: radeon :02:00.0: GPU lockup CP stall for more
than 1000msec
Mar 15 14:29:37 nf7 kernel: [ cut here ]
Mar 15 14:29:37 nf7 kernel: WARNING: at
drivers/gpu/drm/radeon/radeon_fence.c:234 radeon_fence_wait+0x262/0x2c0
[radeon]()
Mar 15 14:29:37 nf7 kernel: Hardware name:  
Mar 15 14:29:37 nf7 kernel: GPU lockup (waiting for 0x17D3 last fence id
0x17CB)
Mar 15 14:29:37 nf7 kernel: Modules linked in: radeon ttm drm_kms_helper drm
i2c_algo_bit cfbcopyarea cfbimgblt cfbfillrect fbcon font bitblit softcursor fb
mt352 saa7134_dvb videobuf_dvb dvb_core mt20xx tea5767 tda9887 tda8290 tuner
saa7134 v4l2_common videodev v4l1_compat videobuf_dma_sg videobuf_core tveeprom
i2c_core nvidia_agp ehci_hcd agpgart ohci_hcd usbhid usbcore snd_intel8x0
snd_ac97_codec ac97_bus forcedeth
Mar 15 14:29:37 nf7 kernel: Pid: 2526, comm: gltestperf Tainted: GW 
2.6.33-50657-g5894684 #1
Mar 15 14:29:37 nf7 kernel: Call Trace:
Mar 15 14:29:37 nf7 kernel:  [fa1164a2] ? radeon_fence_wait+0x262/0x2c0
[radeon]
Mar 15 14:29:37 nf7 kernel:  [fa1164a2] ? radeon_fence_wait+0x262/0x2c0
[radeon]
Mar 15 14:29:37 nf7 kernel:  [c102f811] warn_slowpath_common+0x81/0xa0
Mar 15 14:29:37 nf7 kernel:  [fa1164a2] ? radeon_fence_wait+0x262/0x2c0
[radeon]
Mar 15 14:29:37 nf7 kernel:  [c102f87b] warn_slowpath_fmt+0x2b/0x30
Mar 15 14:29:37 nf7 kernel:  [fa1164a2] radeon_fence_wait+0x262/0x2c0
[radeon]
Mar 15 14:29:37 nf7 kernel:  [c104b950] ? autoremove_wake_function+0x0/0x40
Mar 15 14:29:37 nf7 kernel:  [fa117230] ? radeon_sync_obj_wait+0x0/0x20
[radeon]
Mar 15 14:29:37 nf7 kernel:  [fa117241] radeon_sync_obj_wait+0x11/0x20
[radeon]
Mar 15 14:29:37 nf7 kernel:  [f96b6e82] ttm_bo_wait+0xf2/0x1d0 [ttm]
Mar 15 14:29:37 nf7 kernel:  [fa12bb34] radeon_gem_wait_idle_ioctl+0x84/0x120
[radeon]
Mar 15 14:29:37 nf7 kernel:  [f954f719] drm_ioctl+0x259/0x3e0 [drm]
Mar 15 14:29:37 nf7 kernel:  [fa12bab0] ?
radeon_gem_wait_idle_ioctl+0x0/0x120 [radeon]
Mar 15 14:29:37 nf7 kernel:  [c10c05e9] ? do_sync_read+0xb9/0xf0
Mar 15 14:29:37 nf7 kernel:  [c10edd6e] ? __fsnotify_parent+0xe/0x110
Mar 15 14:29:37 nf7 kernel:  [f954f4c0] ? drm_ioctl+0x0/0x3e0 [drm]
Mar 15 14:29:37 nf7 kernel:  [c10ce35d] vfs_ioctl+0x2d/0xa0
Mar 15 14:29:37 nf7 kernel:  [c10ce51a] do_vfs_ioctl+0x6a/0x560
Mar 15 14:29:37 nf7 kernel:  [c10edb0e] ? fsnotify+0xe/0x120
Mar 15 14:29:37 nf7 kernel:  [c10edd6e] ? __fsnotify_parent+0xe/0x110
Mar 15 14:29:37 nf7 kernel:  [c10c0634] ? rw_verify_area+0x14/0xc0
Mar 15 14:29:37 nf7 kernel:  [c10c0e46] ? vfs_read+0x146/0x150
Mar 15 14:29:37 nf7 kernel:  [c10c1b44] ? fget_light+0x14/0xe0
Mar 15 14:29:37 nf7 kernel:  [c10c1b44] ? fget_light+0x14/0xe0
Mar 15 14:29:37 nf7 kernel:  [c10cea4e] sys_ioctl+0x3e/0x60
Mar 15 14:29:37 nf7 kernel:  [c1002c0c] sysenter_do_call+0x12/0x22
Mar 15 14:29:37 nf7 kernel: ---[ end trace eae1ac44941f0643 ]---
Mar 15 14:29:37 nf7 kernel: [drm] Disabling audio support
Mar 15 14:29:37 nf7 kernel: radeon :02:00.0: f72844e0 unpin not necessary
Mar 15 14:29:37 nf7 kernel: radeon :02:00.0: GPU softreset 
Mar 15 14:29:37 nf7 kernel: radeon :02:00.0:  
R_008010_GRBM_STATUS=0xE7731CE0
Mar 15 14:29:37 nf7 kernel: radeon :02:00.0:  
R_008014_GRBM_STATUS2=0x00880103
Mar 15 14:29:37 nf7 kernel: radeon :02:00.0:  
R_000E50_SRBM_STATUS=0x200018C0
Mar 15 14:29:37 nf7 kernel: radeon :02:00.0:  
R_008020_GRBM_SOFT_RESET=0x7FEE
Mar 15 14:29:37 nf7 kernel: radeon :02:00.0:
R_008020_GRBM_SOFT_RESET=0x0001
Mar 15 14:29:37 nf7 kernel: radeon :02:00.0:  
R_008010_GRBM_STATUS=0xA0003030
Mar 15 14:29:37 nf7 kernel: radeon :02:00.0:  
R_008014_GRBM_STATUS2=0x0003
Mar 15 14:29:37 nf7 kernel: radeon :02:00.0:  
R_000E50_SRBM_STATUS=0x200080C0
Mar 15 14:29:37 nf7 kernel: radeon :02:00.0: GPU reset succeed


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

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list

[Bug 27043] [bisected DRI1] drmRadeonCmdBuffer: -22. Kernel failed to parse or rejected command stream. See dmesg for more info.

2010-03-15 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=27043





--- Comment #9 from Alex Deucher ag...@yahoo.com  2010-03-15 08:08:46 PST ---
(In reply to comment #8)
  Err, I mean glCopyPixels and glDrawPixels.
 
 I tested only the game, but shouldn't _mesa_meta_DrawPixels and
 _mesa_meta_CopyPixels be already the default BTW?
 

They weren't enabled until that patch.


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

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 27086] New: radeon pm should have uninitialize function for cleaning pm state

2010-03-15 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=27086

   Summary: radeon pm should have uninitialize function for cleaning
pm state
   Product: DRI
   Version: XOrg CVS
  Platform: Other
OS/Version: All
Status: NEW
  Severity: normal
  Priority: medium
 Component: DRM/Radeon
AssignedTo: dri-devel@lists.sourceforge.net
ReportedBy: suok...@gmail.com


Currently unloading radeon module if dynpm is enabled causes oopses. It also
looks that if only fixing oopses radeon would leave card to power save mode.

When kernel module is unloaded power management should unregister the delayed
work with cancel_delayed_work_sync and set default power state.


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

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 26781] RV670 Oops with PM and xrandr.

2010-03-15 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=26781





--- Comment #3 from Rafał Miłecki zaj...@gmail.com  2010-03-15 08:48:48 PST 
---
Alex's catch:
http://git.kernel.org/?p=linux/kernel/git/airlied/drm-2.6.git;a=commitdiff;h=589468458fa964c2020e23e4d9353be2ef8bd3be


-- 
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 27086] radeon pm should have uninitialize function for cleaning pm state

2010-03-15 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=27086





--- Comment #1 from Rafał Miłecki zaj...@gmail.com  2010-03-15 08:49:31 PST 
---
Could you post fast tip for reloading radeon, please? I heard of some fbcon
unbinding hack.


-- 
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 27086] radeon pm should have uninitialize function for cleaning pm state

2010-03-15 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=27086





--- Comment #2 from Pauli suok...@gmail.com  2010-03-15 08:58:42 PST ---
Created an attachment (id=34074)
 -- (http://bugs.freedesktop.org/attachment.cgi?id=34074)
My script for reloading radeon module

Attached the shell script that I use to reload radeon module.

usage is script [01] [s]{0,1}

where the first parameter is for selecting if modesetting is enabled after
reload. the second paramter is set to s if you want to start gdm immediately
after module reload.


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

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 27086] radeon pm should have uninitialize function for cleaning pm state

2010-03-15 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=27086





--- Comment #3 from Pauli suok...@gmail.com  2010-03-15 09:00:56 PST ---
(In reply to comment #2)
 Created an attachment (id=34074)
 -- (http://bugs.freedesktop.org/attachment.cgi?id=34074) [details]
 My script for reloading radeon module
 
 Attached the shell script that I use to reload radeon module.
 
 usage is script [01] [s]{0,1}
 
 where the first parameter is for selecting if modesetting is enabled after
 reload. the second paramter is set to s if you want to start gdm immediately
 after module reload.
 

warning: There is extra agpmode=4 parameter. I didn't remember that I had added
it there.


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

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: [PATCHES] radeon drm kms i2c fixes

2010-03-15 Thread Johannes Engel
AFAIK recompiling the Xorg driver should be enough in openSUSE 11.2.

Cheers, Johannes

2010/3/15 Jean Delvare kh...@linux-fr.org:
 On Thu, 11 Mar 2010 13:35:47 -0500, Alex Deucher wrote:
 On Thu, Mar 11, 2010 at 1:00 PM, Jean Delvare kh...@linux-fr.org wrote:
  I have the following in my machine:
  02:00.0 VGA compatible controller: ATI Technologies Inc RV280 [Radeon 
  9200] (rev 01)
  02:00.1 Display controller: ATI Technologies Inc RV280 [Radeon 9200] 
  (Secondary) (rev 01)
 
  Can I help with testing? I can follow your instructions.

 Sure.  You need a kms-enabled graphics stack.  See this for more:
 http://wiki.x.org/wiki/radeonBuildHowTo
 Then, make sure you have the latest kernel bits and the patches I
 posted.  Note that some users have reported problems with the hw i2c
 engine on some r1xx-r3xx boards.  I suspect a problematic prescale or
 a drive problem.  If you run into issues, please try the patch I
 attached to this bug:
 http://bugs.freedesktop.org/show_bug.cgi?id=26430

 Huuu, looks a little too time consuming, I'm afraid. Anything requiring
 rebuilding Xorg isn't an option for me. I'm running openSUSE 11.1, it
 has Xorg 7.4, I guess it's too old... And openSUSE 11.2 has Xorg 7.4
 too, I don't think is is KMS-enabled, so I guess upgrading won't help.
 I will probably have to wait for openSUSE 11.3 before I can do any
 testing, sorry.

 --
 Jean Delvare

 --
 Download Intel#174; Parallel Studio Eval
 Try the new software tools for yourself. Speed compiling, find bugs
 proactively, and fine-tune applications for parallel performance.
 See why Intel Parallel Studio got high marks during beta.
 http://p.sf.net/sfu/intel-sw-dev
 --
 ___
 Dri-devel mailing list
 Dri-devel@lists.sourceforge.net
 https://lists.sourceforge.net/lists/listinfo/dri-devel


--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 27069] OpenGl is slower on dri2

2010-03-15 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=27069





--- Comment #2 from Alex Deucher ag...@yahoo.com  2010-03-15 09:52:18 PST ---
this is likely a dup of bug 26599.  The slow down is at least partially caused
by the anti-tearing code used for buffer swaps.


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

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 26641] RV730 agp xf86-video-ati with kms poor performance

2010-03-15 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=26641





--- Comment #11 from Alex Deucher ag...@yahoo.com  2010-03-15 09:58:05 PST ---
Does xf86-video-ati from git master help?  Specifically this commit:
http://cgit.freedesktop.org/xorg/driver/xf86-video-ati/commit/?id=488c9fd8300505cc6c0c2f8f0f00849f27cc5d63


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

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 27083] Commit use lcd pll limits when available in drm-radeon-testing breaks my laptop lvds

2010-03-15 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=27083





--- Comment #1 from Alex Deucher ag...@yahoo.com  2010-03-15 10:08:40 PST ---
Created an attachment (id=34076)
 -- (http://bugs.freedesktop.org/attachment.cgi?id=34076)
remove lvds quirks

Perhaps with the new code the quirk from the other bug isn't necessary.  Does
this patch help?  It removes the LVDS quirks.


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

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 15293] Flash video laggy inside Firefox only with KMS

2010-03-15 Thread bugzilla-daemon
http://bugzilla.kernel.org/show_bug.cgi?id=15293





--- Comment #33 from Rafał Miłecki zaj...@gmail.com  2010-03-15 17:59:36 ---
I've updated kernel and xf86-video-ati and Vimeo works great for me!

Can this be related to
http://cgit.freedesktop.org/xorg/driver/xf86-video-ati/commit/?id=488c9fd8300505cc6c0c2f8f0f00849f27cc5d63
?

-- 
Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are watching the assignee of the bug.
--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 15293] Flash video laggy inside Firefox only with KMS

2010-03-15 Thread bugzilla-daemon
http://bugzilla.kernel.org/show_bug.cgi?id=15293





--- Comment #34 from Alex Deucher alexdeuc...@gmail.com  2010-03-15 18:01:40 
---
(In reply to comment #33)
 I've updated kernel and xf86-video-ati and Vimeo works great for me!
 
 Can this be related to
 http://cgit.freedesktop.org/xorg/driver/xf86-video-ati/commit/?id=488c9fd8300505cc6c0c2f8f0f00849f27cc5d63
 ?

Yes, fixing the domains increased GetImage performance by a factor of ~10 for
me.

-- 
Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are watching the assignee of the bug.

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 15293] Flash video laggy inside Firefox only with KMS

2010-03-15 Thread bugzilla-daemon
http://bugzilla.kernel.org/show_bug.cgi?id=15293





--- Comment #35 from Alex Deucher alexdeuc...@gmail.com  2010-03-15 18:03:22 
---
This bug can probably be closed.

-- 
Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are watching the assignee of the bug.

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 26641] RV730 agp xf86-video-ati with kms poor performance

2010-03-15 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=26641





--- Comment #12 from Andy Furniss li...@andyfurniss.entadsl.com  2010-03-15 
11:15:17 PST ---
(In reply to comment #11)
 Does xf86-video-ati from git master help?  Specifically this commit:
 http://cgit.freedesktop.org/xorg/driver/xf86-video-ati/commit/?id=488c9fd8300505cc6c0c2f8f0f00849f27cc5d63
 

No that doesn't help this -vo x11 issue.

It does boost xv perf so that kms now beats ums, previously kms ate a bit more
cpu, but nothing like as much as this x11 issue ie. not really measurable at SD
res but about 18% more 1280x...@30fps.

I retested with the last drt kernel and the figures are still the same as
posted in my bug - so it's a further drt regression not xserver.


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

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: [Linux-fbdev-devel] drm_fb_helper: Impossible to change video mode

2010-03-15 Thread James Simmons

  Searching the TTM code I couldn't find the handle code so easily. I see
  that the vmwgfx driver provides a example of using ttm.
 
 So handles are purely a userspace interface, in-kernel we don't use handles
 for buffer management, the vmwgfx TTM interface has 
 vmw_user_surface_lookup_handle
 to do the bo lookups.

Yeap, some how userland has to tell which buffer is which. I also noticed 
their is a ttm_base_object_lookup which does the same thing :-) 


--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 26347] powermanagement on rs780 not working

2010-03-15 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=26347


Marc marvi...@gmx.de changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED




--- Comment #26 from Marc marvi...@gmx.de  2010-03-15 11:27:50 PST ---
Thanks Rafał for your efforts!

I tested the patches from Alex today and everything is working so far (beside
some vblank reports).

Hope it will be ready for 2.6.35 - Closing.


-- 
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 26515] Corrupt textures

2010-03-15 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=26515





--- Comment #3 from Maciej Cencora m.cenc...@gmail.com  2010-03-15 11:30:22 
PST ---
Please check if you can reproduce the bug on mesa 7.7 and current mesa from
git.


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

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: [Linux-fbdev-devel] drm_fb_helper: Impossible to change video mode

2010-03-15 Thread James Simmons

  The big issue we have with resizing the buffer is userspace mmaps of the 
  fbdev
  device, and invalidation.
  Previous thread of unresolvedness is here.
  http://www.mail-archive.com/dri-devel@lists.sourceforge.net/msg41878.html
 
 Actually AFAIR (and reading through it again seems to confirm this)
 userspace mappings should be fully handled by the last patch series I
 posted back then[0]. The problem was that the struct fb_ops hooks may be
 called by the kernel from pretty much any context, and neither I nor
 Thomas was sure how to handle the TTM locking given that. Maybe James
 has ideas for this given his better familiarity with fbdev internals.

The fb_ops can only be called from fbcon or the fbdev userland interface. 
The fbcon calls should only happen when the VC is in KD_TEXT mode. Now 
with the DRM backend we have the advantage of creating a mapping seperate 
from the console mapping. A fb_open/fb_close could be used to cleaning up 
the userland mmap as well as handle the console pinning. We can supply 
your own fb_mmap hook.

 [0] Though that was really only about making it possible to unpin the
 fbcon BO while it isn't being displayed, resizing it might involve other
 issues I'm not aware of.

Yeap, but both problems are related. Kill two birds with one stone.

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: [PATCH] drm: Allow platform devices to register as DRM devices

2010-03-15 Thread Jordan Crouse
Dave Airlie wrote:
 On Tue, Mar 2, 2010 at 2:00 AM, Jordan Crouse jcro...@codeaurora.org wrote:
 Allow platform devices without PCI resources to be DRM devices.

 Signed-off-by: Jordan Crouse jcro...@codeaurora.org
 
 This patch has a bunch of whitespace damage at least in my inbox and
 also in patchwork

Yes, PBCAK.

 Please also be careful with the places you add copyrights, you moved a
 bunch of code
 from drm_drv.c to drm_pci.c and added a copyright for Code Aurora in
 drm_pci.c? You can
 only assert copyright if you actually wrote the code, otherwise the patch
 contains your authorship details and who contributed the code. I'm
 guessing you might have
 copyright over one file in this that being drm_platform.c, the rest
 I'm less sure about.
 
 I also wonder if we could/should separate the arm drm support out from
 this patch (i.e.
 the __arm__ changes).

That might not be a bad idea, I'll do that.

 Some other misc comments below:
 
  #
  menuconfig DRM
tristate Direct Rendering Manager (XFree86 4.1.0 and higher DRI
 support)
 -   depends on (AGP || AGP=n)  PCI  !EMULATED_CMPXCHG  MMU
 
From what I can see the AGP || AGP==n is still required, you just want
 to drop the PCI
 dependancy??

I guess technically we could also drop the AGP requirement, but since it worked
on my box with AGP=n it seemed to me like a NOP.

 +   depends on !EMULATED_CMPXCHG  MMU

 diff --git a/drivers/gpu/drm/drm_bufs.c b/drivers/gpu/drm/drm_bufs.c
 index 8417cc4..4177f60 100644
 --- a/drivers/gpu/drm/drm_bufs.c
 +++ b/drivers/gpu/drm/drm_bufs.c
 @@ -11,6 +11,7 @@
  *
  * Copyright 1999, 2000 Precision Insight, Inc., Cedar Park, Texas.
  * Copyright 2000 VA Linux Systems, Inc., Sunnyvale, California.
 + * Copyright (c) 2009, Code Aurora Forum.
  * All Rights Reserved.
 
 There have been much bigger changes to these files without copyright
 additions, I'm not sure how big something needs to be but I'm not 100% sure
 this comes close.
 
  resource_size_t drm_get_resource_start(struct drm_device *dev, unsigned int
 resource)
  {
 +   if (drm_core_check_feature(dev, DRIVER_USE_PLATFORM_DEVICE)) {
 +   struct resource *r;
 +   r = platform_get_resource(dev-platformdev, IORESOURCE_MEM,
 +resource);
 +
 +   return r ? r-start : 0;
 +   }
 +
 +#ifdef CONFIG_PCI
return pci_resource_start(dev-pdev, resource);
 +#endif
 +
 +   return 0;
  }
  EXPORT_SYMBOL(drm_get_resource_start);

  resource_size_t drm_get_resource_len(struct drm_device *dev, unsigned int
 resource)
  {
 +   if (drm_core_check_feature(dev, DRIVER_USE_PLATFORM_DEVICE)) {
 +   struct resource *r;
 +   r = platform_get_resource(dev-platformdev, IORESOURCE_MEM,
 +   resource);
 +
 +   return r ? (r-end - r-start) : 0;
 +   }
 +
 +#ifdef CONFIG_PCI
return pci_resource_len(dev-pdev, resource);
 +#endif
 +
 +   return 0;
  }

  EXPORT_SYMBOL(drm_get_resource_len);
 @@ -188,7 +213,7 @@ static int drm_addmap_core(struct drm_device * dev,
 resource_size_t offset,
switch (map-type) {
case _DRM_REGISTERS:
case _DRM_FRAME_BUFFER:
 -#if !defined(__sparc__)  !defined(__alpha__)  !defined(__ia64__) 
 !defined(__powerpc64__)  !defined(__x86_64__)
 +#if !defined(__sparc__)  !defined(__alpha__)  !defined(__ia64__) 
 !defined(__powerpc64__)  !defined(__x86_64__)  !defined(__arm__)
if (map-offset + (map-size-1)  map-offset ||
map-offset  virt_to_phys(high_memory)) {
kfree(map);
 
 diff --git a/drivers/gpu/drm/drm_drv.c b/drivers/gpu/drm/drm_drv.c
 index 766c468..b5171ed 100644
 --- a/drivers/gpu/drm/drm_drv.c
 +++ b/drivers/gpu/drm/drm_drv.c
 @@ -24,6 +24,7 @@
  *
  * Copyright 1999, 2000 Precision Insight, Inc., Cedar Park, Texas.
  * Copyright 2000 VA Linux Systems, Inc., Sunnyvale, California.
 + * Copyright (c) 2009, Code Aurora Forum.
  * All Rights Reserved.
 
 again similiar, but even less code since you mostly remove code.
 
  *
  * Permission is hereby granted, free of charge, to any person obtaining a
 @@ -242,47 +243,20 @@ int drm_lastclose(struct drm_device * dev)
  *
  * Initializes an array of drm_device structures, and attempts to
  * initialize all available devices, using consecutive minors, registering
 the
 - * stubs and initializing the AGP device.
 + * stubs and initializing the device.
  *
  * Expands the \c DRIVER_PREINIT and \c DRIVER_POST_INIT macros before and
  * after the initialization for driver customization.
  */
  int drm_init(struct drm_driver *driver)
  {
 -   struct pci_dev *pdev = NULL;
 -   const struct pci_device_id *pid;
 -   int i;
 -
DRM_DEBUG(\n);
 -
INIT_LIST_HEAD(driver-device_list);

 -   if (driver-driver_features  DRIVER_MODESET)
 -   return pci_register_driver(driver-pci_driver);
 -
 -   /* If not using KMS, fall back to 

Re: [PATCH] drm: Allow platform devices to register as DRM devices

2010-03-15 Thread Ville Syrjälä
On Mon, Mar 01, 2010 at 09:00:09AM -0700, Jordan Crouse wrote:
 
 Allow platform devices without PCI resources to be DRM devices.

I really dislike the fact that drm has bus specific junk all over the
generic code. Some ideas how to clean that up:

Add 'struct device *dev' into drm_device so you don't have to go through
the pdev/platformdev to get it every time. Also use dev_name() instead
of pci_name() and whatever you used for the platform device case.

Add 'int irq' into struct drm_device instead of adding more bus specific
hoops to get at the irq. Not sure I like the generic code mucking about
with the irq directly though but baby steps are easier to handle.

Get rid of the drm_resource_start/len wrappers. AFAICS they're all called
from the low level driver code anyway and the driver knows the bus type
so there's no need for the wrappers.

It would be nice to get the struct pci_driver out of the the drm_driver
structure. Since you now have a new pci specific drm_get_dev() thing
could you also pass the pci_driver as a function parameter instead
of having it live inside the drm_driver?

Also all cases where there's some PCI specific stuff (the busid stuff
mostly) you could just check the drm_device.pdev pointer instead of
having to add another driver flags to identify non-PCI devices. Although
I don't really like having the pdev/platformdev pointers in there at all.

That's sort of my secret drm TODO list but so far didn't have the time
to actually do the coding part.

-- 
Ville Syrjälä
syrj...@sci.fi
http://www.sci.fi/~syrjala/

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 26515] Corrupt textures

2010-03-15 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=26515





--- Comment #4 from Marek Kubica ma...@xivilization.net  2010-03-15 12:59:13 
PST ---
(In reply to comment #3)
 Please check if you can reproduce the bug on mesa 7.7 and current mesa from
 git.

I tried Mesa 7.7 which migrated to Debian, still just as broken as before.
Trying the git version would mean that I have to package the current git
version as debian package to uninstall it cleanly afterwards, which is rather a
lot of work.


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

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[PATCH] drm/radeon/kms: init rdev-num_crtc at asic init

2010-03-15 Thread Alex Deucher
From 2ccd66337a664a7096ff5fba6ad7f9f01295c17b Mon Sep 17 00:00:00 2001
From: Alex Deucher alexdeuc...@gmail.com
Date: Mon, 15 Mar 2010 17:02:12 -0400
Subject: [PATCH] drm/radeon/kms: init rdev-num_crtc at asic init

Replace hardcoded numbers with rdev-num_crtc.

Signed-off-by: Alex Deucher alexdeuc...@gmail.com
---
 drivers/gpu/drm/radeon/radeon_asic.c|   10 ++
 drivers/gpu/drm/radeon/radeon_display.c |9 -
 drivers/gpu/drm/radeon/radeon_irq_kms.c |9 +++--
 3 files changed, 13 insertions(+), 15 deletions(-)

diff --git a/drivers/gpu/drm/radeon/radeon_asic.c
b/drivers/gpu/drm/radeon/radeon_asic.c
index 7e34a00..53ee3f8 100644
--- a/drivers/gpu/drm/radeon/radeon_asic.c
+++ b/drivers/gpu/drm/radeon/radeon_asic.c
@@ -752,6 +752,16 @@ int radeon_asic_init(struct radeon_device *rdev)
rdev-asic-set_memory_clock = NULL;
}

+   /* set the number of crtcs */
+   if (rdev-flags  RADEON_SINGLE_CRTC)
+   rdev-num_crtc = 1;
+   else {
+   if (ASIC_IS_DCE4(rdev))
+   rdev-num_crtc = 6;
+   else
+   rdev-num_crtc = 2;
+   }
+
return 0;
 }

diff --git a/drivers/gpu/drm/radeon/radeon_display.c
b/drivers/gpu/drm/radeon/radeon_display.c
index ff5f099..d65931d 100644
--- a/drivers/gpu/drm/radeon/radeon_display.c
+++ b/drivers/gpu/drm/radeon/radeon_display.c
@@ -1003,15 +1003,6 @@ int radeon_modeset_init(struct radeon_device *rdev)
radeon_combios_check_hardcoded_edid(rdev);
}

-   if (rdev-flags  RADEON_SINGLE_CRTC)
-   rdev-num_crtc = 1;
-   else {
-   if (ASIC_IS_DCE4(rdev))
-   rdev-num_crtc = 6;
-   else
-   rdev-num_crtc = 2;
-   }
-
/* allocate crtcs */
for (i = 0; i  rdev-num_crtc; i++) {
radeon_crtc_init(rdev-ddev, i);
diff --git a/drivers/gpu/drm/radeon/radeon_irq_kms.c
b/drivers/gpu/drm/radeon/radeon_irq_kms.c
index 5c26c0a..45926ea 100644
--- a/drivers/gpu/drm/radeon/radeon_irq_kms.c
+++ b/drivers/gpu/drm/radeon/radeon_irq_kms.c
@@ -96,22 +96,19 @@ void radeon_driver_irq_uninstall_kms(struct drm_device *dev)
/* Disable *all* interrupts */
rdev-irq.sw_int = false;
rdev-irq.gui_idle = false;
-   for (i = 0; i  2; i++) {
+   for (i = 0; i  rdev-num_crtc; i++)
rdev-irq.crtc_vblank_int[i] = false;
+   for (i = 0; i  6; i++)
rdev-irq.hpd[i] = false;
-   }
radeon_irq_set(rdev);
 }

 int radeon_irq_kms_init(struct radeon_device *rdev)
 {
int r = 0;
-   int num_crtc = 2;

-   if (rdev-flags  RADEON_SINGLE_CRTC)
-   num_crtc = 1;
spin_lock_init(rdev-irq.sw_lock);
-   r = drm_vblank_init(rdev-ddev, num_crtc);
+   r = drm_vblank_init(rdev-ddev, rdev-num_crtc);
if (r) {
return r;
}
-- 
1.5.6.3


0001-drm-radeon-kms-init-rdev-num_crtc-at-asic-init.patch
Description: application/mbox
--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 27071] hang in sauerbaten after recent git master upgrade

2010-03-15 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=27071





--- Comment #2 from David Ronis david.ro...@mcgill.ca  2010-03-15 19:00:22 
PST ---
The game has gone back to its previous level of stability; namely, I still
crash, just not 100% of the time.  (I think that this is bug 25597)

 I see the following on the console:

*WARN_ONCE*
File radeon_dma.c function radeonReleaseDmaRegions line 340
Leaking dma buffer object!
***
*** glibc detected *** ./bin_unix/native_client: free(): invalid pointer:
0x084576b4 ***
=== Backtrace: =
/lib/libc.so.6[0xb7307564]
/lib/libc.so.6(cfree+0x90)[0xb730b010]
/usr/lib/dri/r200_dri.so[0xb6484d31]
/usr/lib/dri/r200_dri.so[0xb6482997]
/usr/lib/dri/r200_dri.so[0xb647ea9e]
/usr/lib/dri/r200_dri.so[0xb644a5d1]
/usr/lib/dri/r200_dri.so[0xb6441cc8]
/usr/lib/libGL.so.1[0xb75700b9]
/usr/lib/libGL.so.1[0xb7537c3a]
/usr/lib/libGL.so.1(glXDestroyContext+0x18)[0xb7537d9e]
/usr/lib/libSDL-1.2.so.0[0xb76cb1b4]

Here's a backtrace I was able to capture after the last crash (with your patch
applied).

#5  0xb6484d31 in radeonReleaseDmaRegions (rmesa=0x8457698) at radeon_dma.c:343
dma_bo = (struct radeon_dma_bo *) 0x84576b4
temp = (struct radeon_dma_bo *) 0x96d8cf8
expire_at = 64069
time = 63969
__FUNCTION__ = radeonReleaseDmaRegions
#6  0xb6482997 in rcommonFlushCmdBuf (rmesa=0x8457698, 
caller=0xb66bb428 radeonDestroyContext) at radeon_common.c:1224
ret = -1236282016
#7  0xb647ea9e in radeonDestroyContext (driContextPriv=0x842eac0)
at radeon_common_context.c:308
ctx = (GLcontext *) 0x0
radeon = (radeonContextPtr) 0x8457698
current = (radeonContextPtr) 0x0
__PRETTY_FUNCTION__ = radeonDestroyContext
__FUNCTION__ = radeonDestroyContext
#8  0xb644a5d1 in r200DestroyContext (driContextPriv=0x842eac0)
at r200_context.c:511
i = 6
rmesa = (r200ContextPtr) 0x8457698
#9  0xb6441cc8 in driDestroyContext (pcp=0x842eac0) at ../common/dri_util.c:551
No locals.
#10 0xb75700b9 in driDestroyContext (context=0x84554d0, psc=0x8454aa0, 
dpy=0x84116d8) at dri_glx.c:482
pcp = (__GLXDRIcontextPrivate *) 0x84554d0
#11 0xb7537c3a in DestroyContext (dpy=0x84116d8, gc=0x842cc10) at glxcmds.c:548
req = (xGLXDestroyContextReq *) 0xb73e3ff4
xid = 67108866
opcode = 151 '\227'
imported = 0 '\0'
#12 0xb7537d9e in glXDestroyContext (dpy=0x84116d8, gc=0x842cc10)
at glxcmds.c:584
No locals.
#13 0xb76cb1b4 in X11_GL_Shutdown () from /usr/lib/libSDL-1.2.so.0
No locals.
#14 0x0842cc10 in ?? ()
No symbol table info available.
#15 0x in ?? ()
No symbol table info available.


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

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: [PATCH] drm: Allow platform devices to register as DRM devices

2010-03-15 Thread Dave Airlie

 I guess technically we could also drop the AGP requirement, but since it
 worked
 on my box with AGP=n it seemed to me like a NOP.

Its not a NOP, otherwise we'd remove it, AGP || AGP=n means if
AGP is enabled DRM must be enabled similiarly, it stops AGP=m + DRM=y
basically.

  EXPORT_SYMBOL(drm_init);
 diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
 index ab6c973..48a14a0 100644
 --- a/drivers/gpu/drm/drm_edid.c
 +++ b/drivers/gpu/drm/drm_edid.c
 @@ -1220,6 +1220,9 @@ struct edid *drm_get_edid(struct drm_connector
 *connector,
       int ret;
       struct edid *edid;

 +       if (drm_core_check_feature(connector-dev,
 DRIVER_USE_PLATFORM_DEVICE))
 +               return NULL;
 +


 This makes no sense, having the ability to probe EDID or not is most
 definitely not a platform vs PCI problem.

 Yeah, that was my poor man's Don't probe EDID hack.  I'm not sure if there
 is a smart way to implicitly check to see if EDID should be probed, but I'm
 worried about abusing the features flag too badly, though if a
 DRIVER_USE_EDID
 is needed then we shouldn't be shy about using it.

The generic code never calls this only the driver, so there should be no
need for flags at all.

 Not 100% sure about this, but if you only intend on KMS and don't need to
 inform userspace of irq support this should be okay.

 Again, a don't do this hack.  I'll look at this more carefully and see if
 there
 is a good way to evaluate this based on the hooks that the platform has
 defined.

Its mostly used in UMS to inform userspace for some strange reason
its pretty legacy at this point, new driver should probably not hit it.

 @@ -60,7 +60,7 @@ static pgprot_t drm_io_prot(uint32_t map_type, struct


 This also doesn't address noueau/vmwgfx entry points.

 Yep, thats my bad.  I'll refresh and push better patches without whitespace
 stupidity.

Thanks,
Dave.

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel