[Bug 25741] [R600/KMS] external display flickering

2009-12-29 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=25741





--- Comment #13 from Luca Tettamanti kronos...@gmail.com  2009-12-29 02:48:46 
PST ---
(In reply to comment #12)
 Created an attachment (id=32336)
 -- (http://bugs.freedesktop.org/attachment.cgi?id=32336) [details]
 add missing breaks
 
 This patch should fix it.

The EDID is back, the flickering is still there, but I think I see the problem.
The old code used id as offset into the asSS_Info array; now this is the
PPLL_SS_Info table on my M76:

data_table  ae26  #12  (PPLL_SS_Info):

  Size 000c
  Format Rev.  01
  Param Rev.   00
  Content Rev. 02

  :1e00 0101  2a01 0202   *...

  :  ATOM_COMMON_TABLE_HEADER sHeader:
:  USHORT usStructureSize  = 0x000c
(12)
0002:  UCHAR ucTableFormatRevision = 0x01   (1)
0003:  UCHAR ucTableContentRevision= 0x02   (2)
  0004:  ATOM_SPREAD_SPECTRUM_ASSIGNMENT asSS_Info   [0] :
0004:  USHORT usSpreadSpectrumPercentage   = 0x001e
(30)
0006:  UCHAR ucSpreadSpectrumType  = 0x01   (1)
0007:  UCHAR ucSS_Step = 0x01   (1)
0008:  UCHAR ucSS_Delay= 0x2a  
(42)
0009:  UCHAR ucSS_Id   = 0x01   (1)
000a:  UCHAR ucRecommandedRef_Div  = 0x02   (2)
000b:  UCHAR ucSS_Range= 0x02   (2)
[cut]

Note that the size is 12 bytes, so there's only one
ATOM_SPREAD_SPECTRUM_ASSIGNMENT; ucSS_Id in the LVDS block is 0x1 and the old
code ended up reading past the end of array. For example now it's trying to
set:

[   37.652741]   usSpreadSpectrumPercentage = 0xf8
[   37.652783]   ucSpreadSpectrumType   = 0x1
[   37.652824]   ucSS_Step  = 0x2
[   37.652867]   ucSS_Delay = 0xf
[   37.652910]   ucSS_Range = 0x0
[   37.652953]   ucRecommendedRef_Div   = 0x3c

I guess that atombios validates the data and discards it. The new code picks up
the correct data... and causes flickering. Why is this causing flickering on
the _external_ display? I'm not sure what happens at boot when the outputs are
cloned, but with X running the outputs are uncloned and are using different
CRTCs; the kernel module enables SS on CRTC 1, and xrandr reports that it's
used by LVDS.


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

--
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev 
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 14938] New: Unable to handle kernel paging request in radeon_fence_signaled

2009-12-29 Thread bugzilla-daemon
http://bugzilla.kernel.org/show_bug.cgi?id=14938

   Summary: Unable to handle kernel paging request in
radeon_fence_signaled
   Product: Drivers
   Version: 2.5
Kernel Version: 2.6.33-rc2
  Platform: All
OS/Version: Linux
  Tree: Mainline
Status: NEW
  Severity: normal
  Priority: P1
 Component: Video(DRI - non Intel)
AssignedTo: drivers_video-...@kernel-bugs.osdl.org
ReportedBy: r...@sisk.pl
Blocks: 14885
Regression: Yes


Subject: 2.6.33-rc2: BUG: unable to handle kernel paging request at
13d8, IP=radeon_fence_signaled+0x16/0xc0
Submitter  : Grant Wilson grant.wil...@zen.co.uk
Date   : 2009-12-27 13:40
References : http://marc.info/?l=linux-kernelm=126192123104047w=4
Notify-Also : DRI dri-devel@lists.sourceforge.net

This entry is being used for tracking a regression from 2.6.32.  Please don't
close it until the problem is fixed in the mainline.

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

--
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev 
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 14945] Colors are too bright with KMS on Radeon 9xxx

2009-12-29 Thread bugzilla-daemon
http://bugzilla.kernel.org/show_bug.cgi?id=14945





--- Comment #2 from Samuel Lidén Borell sam...@slbdata.se  2009-12-29 
14:21:43 ---
Created an attachment (id=24346)
 -- (http://bugzilla.kernel.org/attachment.cgi?id=24346)
Output from radeontool regs with KMS disabled and enabled

Here's the output from radeontool regs with KMS disabled and enabled,
respective. As you can see there are a few registers that differ:

-RADEON_DAC_MACRO_CNTL0808
+RADEON_DAC_MACRO_CNTL

-RADEON_TV_DAC_CNTL0740
+RADEON_TV_DAC_CNTL07780142

-RADEON_CRTC_EXT_CNTL8040
+RADEON_CRTC_EXT_CNTL0d008040

Maybe this is somehow related to this bug?

-- 
Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are watching the assignee of the bug.
--
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev 
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 14945] Colors are too bright with KMS on Radeon 9xxx

2009-12-29 Thread bugzilla-daemon
http://bugzilla.kernel.org/show_bug.cgi?id=14945





--- Comment #1 from Samuel Lidén Borell sam...@slbdata.se  2009-12-29 
13:46:05 ---
Created an attachment (id=24345)
 -- (http://bugzilla.kernel.org/attachment.cgi?id=24345)
Output from dmesg

-- 
Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are watching the assignee of the bug.
--
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev 
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 14945] New: Colors are too bright with KMS on Radeon 9xxx

2009-12-29 Thread bugzilla-daemon
http://bugzilla.kernel.org/show_bug.cgi?id=14945

   URL: https://bugzilla.redhat.com/show_bug.cgi?id=505903
   Summary: Colors are too bright with KMS on Radeon 9xxx
   Product: Drivers
   Version: 2.5
Kernel Version: 2.6.32
  Platform: All
OS/Version: Linux
  Tree: Mainline
Status: NEW
  Severity: normal
  Priority: P1
 Component: Video(DRI - non Intel)
AssignedTo: drivers_video-...@kernel-bugs.osdl.org
ReportedBy: sam...@slbdata.se
Regression: No


Created an attachment (id=24344)
 -- (http://bugzilla.kernel.org/attachment.cgi?id=24344)
Test case

The colors on my monitor are too bright when KMS is enabled, both in the
console and in Xorg. I have a Radeon 9500 PRO, but Radeon 9000 is also
affected[1].

I've attached a script to test the behavior in the console. It should display a
gradient with yellow colors, and all colors should be distinguishable (except
possibly the first two depending on the monitor settings). With KMS enabled,
the colors are too bright and all but the last two are indistinguishable.

I'm using Ubuntu 10.04 (alpha version with latest upgrades) with this kernel: 

Linux crashie2 2.6.32-9-generic #13-Ubuntu SMP Thu Dec 17 17:02:51 UTC 2009
i686 GNU/Linux

[1] https://bugzilla.redhat.com/show_bug.cgi?id=505903

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

--
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev 
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


2.6.33-rc2: Reported regressions 2.6.31 - 2.6.32

2009-12-29 Thread Rafael J. Wysocki
[NOTES:
 * I really hope that at least _some_ of the bugs below have been already
   fixed, but I didn't have the time to search through all of the histories.
   _Please_ let me know which of them can be closed.
 * If you close one of the bugs in this list, please do your best to put the
   hash of the fix commit into the Bugzilla entry.]

This message contains a list of some regressions introduced between 2.6.31 and
2.6.32, for which there are no fixes in the mainline I know of.  If any of them
have been fixed already, please let me know.

If you know of any other unresolved regressions introduced between 2.6.31
and 2.6.32, please let me know either and I'll add them to the list.
Also, please let me know if any of the entries below are invalid.

Each entry from the list will be sent additionally in an automatic reply to
this message with CCs to the people involved in reporting and handling the
issue.


Listed regressions statistics:

  Date  Total  Pending  Unresolved
  
  2009-12-29  124   60  57
  2009-11-21   86   29  25
  2009-11-16   84   46  41
  2009-10-26   66   42  37
  2009-10-12   48   31  27
  2009-10-02   22   15   9


Unresolved regressions
--

Bug-Entry   : http://bugzilla.kernel.org/show_bug.cgi?id=14943
Subject : nfs regression?
Submitter   : Nikola Ciprich extmaill...@linuxbox.cz
Date: 2009-12-28 12:10 (2 days old)
References  : http://marc.info/?l=linux-kernelm=126200276223524w=4


Bug-Entry   : http://bugzilla.kernel.org/show_bug.cgi?id=14939
Subject : drm: random hang with i915
Submitter   : Arnd Bergmann a...@arndb.de
Date: 2009-12-07 17:30 (23 days old)
References  : http://marc.info/?l=linux-kernelm=126020704125723w=4
Handled-By  : Jesse Barnes jbar...@virtuousgeek.org


Bug-Entry   : http://bugzilla.kernel.org/show_bug.cgi?id=14933
Subject : OOM killer unexpectedly called
Submitter   : A. Boulan arnaud.bou...@libertysurf.fr
Date: 2009-12-24 23:42 (6 days old)
References  : http://marc.info/?l=linux-kernelm=126169821317492w=4


Bug-Entry   : http://bugzilla.kernel.org/show_bug.cgi?id=14922
Subject : 2.6.32 seemed to have broken nVidia MCP7A sata controller
Submitter   : Mike Cui cui...@gmail.com
Date: 2009-12-19 6:13 (11 days old)
References  : http://marc.info/?l=linux-idem=126120323407742w=4
Handled-By  : Jeff Garzik j...@garzik.org
  Robert Hancock hancock...@gmail.com


Bug-Entry   : http://bugzilla.kernel.org/show_bug.cgi?id=14905
Subject : kernel BUG at kernel/timer.c:951!
Submitter   : Bernard Pidoux bernard.pid...@upmc.fr
Date: 2009-12-19 13:38 (11 days old)
References  : http://marc.info/?l=linux-kernelm=126122997831460w=4
Handled-By  : Jarek Poplawski jark...@gmail.com


Bug-Entry   : http://bugzilla.kernel.org/show_bug.cgi?id=14903
Subject : Kernel panic - not syncing: Attempted to kill init!
Submitter   : Zhiyong Wu zwu.ker...@gmail.com
Date: 2009-12-18 4:08 (12 days old)
References  : http://marc.info/?l=linux-kernelm=126110931124738w=4
Handled-By  : Américo Wang xiyou.wangc...@gmail.com


Bug-Entry   : http://bugzilla.kernel.org/show_bug.cgi?id=14902
Subject : Boot ends not on tty1
Submitter   : Andreas Friedrich andreas.friedr...@ts.fujitsu.com
Date: 2009-12-15 8:05 (15 days old)
References  : http://marc.info/?l=linux-kernelm=126086495304263w=4


Bug-Entry   : http://bugzilla.kernel.org/show_bug.cgi?id=14898
Subject : ksoftirqd problem
Submitter   : Nico segfau...@hotmail.com
Date: 2009-12-13 19:05 (17 days old)
References  : http://marc.info/?l=linux-kernelm=126073114325690w=4


Bug-Entry   : http://bugzilla.kernel.org/show_bug.cgi?id=14897
Subject : i915: Commit 0e442c60 causes flickering
Submitter   : David John david...@xenontk.org
Date: 2009-12-09 17:26 (21 days old)
First-Bad-Commit: 
http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=0e442c60dd39ac6924b11a20497734bd2303744c
References  : http://marc.info/?l=linux-kernelm=126037889600769w=4


Bug-Entry   : http://bugzilla.kernel.org/show_bug.cgi?id=14896
Subject : REGRESSION for RT2561/RT61 in 2.6.32, related to power saving
Submitter   : Alan Stern st...@rowland.harvard.edu
Date: 2009-12-06 22:12 (24 days old)
References  : http://marc.info/?l=linux-kernelm=126013756829691w=4
Handled-By  : Gertjan van Wingerde gwinge...@gmail.com


Bug-Entry   : http://bugzilla.kernel.org/show_bug.cgi?id=14895
Subject : BUG in kernel 2.6.32 when using luks encrypted root and 
RAID0..
Submitter   : r4 m...@centrum.cz
Date: 2009-12-03 18:24 (27 days old)
References 

2.6.33-rc2: Reported regressions from 2.6.32

2009-12-29 Thread Rafael J. Wysocki
[NOTES:
 * There you go, another round of tracking regressions.  It's not too bad at
   the moment, as far as the regressions from 2.6.32 are concerned, but we have
   quite a few -stable regressions, apparently in the DRI area.
 * Regressions from 2.6.31 are still being reported.]

This message contains a list of some regressions from 2.6.32, for which there
are no fixes in the mainline I know of.  If any of them have been fixed already,
please let me know.

If you know of any other unresolved regressions from 2.6.32, please let me know
either and I'll add them to the list.  Also, please let me know if any of the
entries below are invalid.

Each entry from the list will be sent additionally in an automatic reply to
this message with CCs to the people involved in reporting and handling the
issue.


Listed regressions statistics:

  Date  Total  Pending  Unresolved
  
  2009-12-29   36   34  27


Unresolved regressions
--

Bug-Entry   : http://bugzilla.kernel.org/show_bug.cgi?id=14952
Subject : instant reboot on loading 2.6.33-rc on Celeron 900
Submitter   : Meelis Roos mr...@linux.ee
Date: 2009-12-29 13:57 (1 days old)
References  : http://marc.info/?l=linux-kernelm=126209508500393w=4


Bug-Entry   : http://bugzilla.kernel.org/show_bug.cgi?id=14950
Subject : tbench regression with 2.6.33-rc1
Submitter   : Lin Ming ming.m@intel.com
Date: 2009-12-25 11:11 (5 days old)
References  : http://marc.info/?l=linux-kernelm=126174044213172w=4


Bug-Entry   : http://bugzilla.kernel.org/show_bug.cgi?id=14949
Subject : drm_vm.c:drm_mmap: possible circular locking dependency 
detected
Submitter   : Borislav Petkov petko...@googlemail.com
Date: 2009-12-26 9:45 (4 days old)
References  : http://marc.info/?l=linux-kernelm=126182073616279w=4


Bug-Entry   : http://bugzilla.kernel.org/show_bug.cgi?id=14948
Subject : EHCI resume sysfs duplicates
Submitter   : Borislav Petkov petko...@googlemail.com
Date: 2009-12-26 9:36 (4 days old)
References  : http://marc.info/?l=linux-kernelm=126182020615709w=4


Bug-Entry   : http://bugzilla.kernel.org/show_bug.cgi?id=14947
Subject : Blank screen for Intel KMS
Submitter   : Miguel Calleja mcalle...@telecable.es
Date: 2009-12-25 13:10 (5 days old)
First-Bad-Commit: 
http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=fc816655236cd9da162356e96e74c7cfb0834d92
References  : http://marc.info/?l=linux-kernelm=126174693019098w=4


Bug-Entry   : http://bugzilla.kernel.org/show_bug.cgi?id=14946
Subject : All kernels after 2.6.32-git10  show only 1 CPU
Submitter   : Sid Boyce sbo...@blueyonder.co.uk
Date: 2009-12-23 16:55 (7 days old)
References  : http://marc.info/?l=linux-kernelm=126158734326801w=4


Bug-Entry   : http://bugzilla.kernel.org/show_bug.cgi?id=14942
Subject : gkrellm no longer shows all the temperatures on thinkpad x60
Submitter   : Pavel Machek pa...@ucw.cz
Date: 2009-12-27 21:57 (3 days old)
References  : http://marc.info/?l=linux-kernelm=126195107005565w=4
Handled-By  : Henrique de Moraes Holschuh h...@hmh.eng.br


Bug-Entry   : http://bugzilla.kernel.org/show_bug.cgi?id=14940
Subject : cfq-iosched: tiobench regression
Submitter   : Shaohua Li shaohua...@intel.com
Date: 2009-12-24 0:55 (6 days old)
References  : http://marc.info/?l=linux-kernelm=126161612401994w=4


Bug-Entry   : http://bugzilla.kernel.org/show_bug.cgi?id=14938
Subject : Unable to handle kernel paging request in 
radeon_fence_signaled
Submitter   : Grant Wilson grant.wil...@zen.co.uk
Date: 2009-12-27 13:40 (3 days old)
References  : http://marc.info/?l=linux-kernelm=126192123104047w=4


Bug-Entry   : http://bugzilla.kernel.org/show_bug.cgi?id=14937
Subject : WARNING: at kernel/lockdep.c:2830
Submitter   : Grant Wilson grant.wil...@zen.co.uk
Date: 2009-12-27 13:35 (3 days old)
References  : http://marc.info/?l=linux-kernelm=126192220404829w=4


Bug-Entry   : http://bugzilla.kernel.org/show_bug.cgi?id=14936
Subject : kernel BUG at fs/ext4/inode.c:1063 if attempted to use 
non-ext4 partition with ext4
Submitter   : Gene Heskett gene.hesk...@verizon.net
Date: 2009-12-26 14:13 (4 days old)
References  : http://marc.info/?l=linux-kernelm=126183682330809w=4
  http://marc.info/?l=linux-kernelm=126169372613898w=4


Bug-Entry   : http://bugzilla.kernel.org/show_bug.cgi?id=14934
Subject : kernel crash during boot
Submitter   : werner w.landg...@ru.ru
Date: 2009-12-25 5:37 (5 days old)
References  : http://marc.info/?l=linux-kernelm=126171951030608w=4


Bug-Entry   : http://bugzilla.kernel.org/show_bug.cgi?id=14925

[Bug #14949] drm_vm.c:drm_mmap: possible circular locking dependency detected

2009-12-29 Thread Rafael J. Wysocki
This message has been generated automatically as a part of a report
of recent regressions.

The following bug entry is on the current list of known regressions
from 2.6.32.  Please verify if it still should be listed and let me know
(either way).


Bug-Entry   : http://bugzilla.kernel.org/show_bug.cgi?id=14949
Subject : drm_vm.c:drm_mmap: possible circular locking dependency 
detected
Submitter   : Borislav Petkov petko...@googlemail.com
Date: 2009-12-26 9:45 (4 days old)
References  : http://marc.info/?l=linux-kernelm=126182073616279w=4



--
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev 
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: drm/radeon/kms: pm: single frame corruptions on reclocking

2009-12-29 Thread Alex Deucher
2009/12/29 Luca Tettamanti kronos...@gmail.com:
 2009/12/28 Alex Deucher alexdeuc...@gmail.com:
 2009/12/28 Luca Tettamanti kronos...@gmail.com:
 On Mon, Dec 28, 2009 at 01:32:24PM -0500, Alex Deucher wrote:
 2009/12/28 Luca Tettamanti kronos...@gmail.com:
  2009/12/28 Alex Deucher alexdeuc...@gmail.com:
  On Mon, Dec 28, 2009 at 5:53 AM, Luca Tettamanti kronos...@gmail.com 
  wrote:
  Unless there are more subtleties all is needed it to enabled the
  interrupt and catch it in r600_irq_process.
 
  That's pretty much it.  Pre-r6xx asics have a GUI interrupt as well.
  I can look up the details for that if they are not already documented.
 
  I can't find a way to ack the interrupt; I see
  RADEON_GUI_IDLE_INT_TEST_ACK but I think it's for pre-r6xx cards,
  right?

 You don't have to ACK it as the CP generates the interrupt in
 software; similar to the sw interrupts used for fences.

 Ok, good: I've got the stub running on my M76. r100 and rs600 parts are
 untested. Comments?


 Looks pretty good.  I've included the proper defines from the register
 database below and you'll need to ack the gui idle interrupts on
 pre-r600 chips.  Now you just have to do something when you get the
 idle interrupt.
 [...]
 --- linux-2.6.git.orig/drivers/gpu/drm/radeon/r100.c    2009-12-28 
 22:30:59.079748392 +0100
 +++ linux-2.6.git/drivers/gpu/drm/radeon/r100.c 2009-12-28 
 22:41:07.803741755 +0100
 @@ -246,6 +246,9 @@
        if (rdev-irq.sw_int) {
                tmp |= RADEON_SW_INT_ENABLE;
        }
 +       if (rdev-irq.idle_int) {
 +               tmp |= RADEON_GUI_IDLE_INT_ENABLE;
 +       }
        if (rdev-irq.crtc_vblank_int[0]) {
                tmp |= RADEON_CRTC_VBLANK_MASK;
        }
 @@ -278,7 +281,8 @@
        uint32_t irqs = RREG32(RADEON_GEN_INT_STATUS);
        uint32_t irq_mask = RADEON_SW_INT_TEST |
                RADEON_CRTC_VBLANK_STAT | RADEON_CRTC2_VBLANK_STAT |
 -               RADEON_FP_DETECT_STAT | RADEON_FP2_DETECT_STAT;
 +               RADEON_FP_DETECT_STAT | RADEON_FP2_DETECT_STAT |
 +               RADEON_GUI_IDLE_INT_TEST_ACK;

        if (irqs) {
                WREG32(RADEON_GEN_INT_STATUS, irqs);
 @@ -318,6 +322,9 @@
                        queue_hotplug = true;
                        DRM_DEBUG(HPD2\n);
                }
 +               if (status  RADEON_GUI_IDLE_INT_TEST_ACK) {
 +                       DRM_DEBUG(GUI idle\n);
 +               }


 You'll need to ack this on pre-r6xx.

 Clear on write? I've included the relevant bit in irq_mask, so r100
 should be ok.

Right.  missed that. should be ok.


 --- linux-2.6.git.orig/drivers/gpu/drm/radeon/rs600.c   2009-12-28 
 22:32:30.927884090 +0100
 +++ linux-2.6.git/drivers/gpu/drm/radeon/rs600.c        2009-12-28 
 22:46:23.211897501 +0100
 @@ -318,6 +318,9 @@
        if (rdev-irq.sw_int) {
                tmp |= S_40_SW_INT_EN(1);
        }
 +       if (rdev-irq.idle_int) {
 +               tmp |= S_40_GUI_IDLE(1);
 +       }
        if (rdev-irq.crtc_vblank_int[0]) {
                mode_int |= S_006540_D1MODE_VBLANK_INT_MASK(1);
        }
 @@ -411,6 +414,9 @@
                        queue_hotplug = true;
                        DRM_DEBUG(HPD2\n);
                }
 +               if (G_44_GUI_IDLE_STAT(status)) {
 +                       DRM_DEBUG(GUI idle\n);
 +               }

 You'll need to ACK this on pre-r6xx chips.

 Ok, I was mislead by the name of the file ;-)
 rs600 = sort-of-r500?

yes.  rs600 (and rs690/rs740) have avivo display engines, but r4xx 3d
engines.  The irq stuff is the same as r5xx.

Alex

--
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev 
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 14331] Radeon XPRESS 200M: System hang with radeon DRI and Fedora 10 userspace unless DRI=off

2009-12-29 Thread bugzilla-daemon
http://bugzilla.kernel.org/show_bug.cgi?id=14331





--- Comment #13 from Rafael J. Wysocki r...@sisk.pl  2009-12-29 16:19:52 ---
On Tuesday 29 December 2009, Alex Villací­s Lasso wrote:
 El 29/12/09 10:28, Rafael J. Wysocki escribió:
  This message has been generated automatically as a part of a report
  of regressions introduced between 2.6.31 and 2.6.32.
 
  The following bug entry is on the current list of known regressions
  introduced between 2.6.31 and 2.6.32.  Please verify if it still should
  be listed and let me know (either way).
 
 
  Bug-Entry   : http://bugzilla.kernel.org/show_bug.cgi?id=14331
  Subject : Radeon XPRESS 200M: System hang with radeon DRI and 
  Fedora 10 userspace unless DRI=off
  Submitter   : Alex Villacis Lassoavill...@ceibo.fiec.espol.edu.ec
  Date: 2009-10-06 00:29 (85 days old)
 
 
 
 
 I will not be able to test this regression anymore. I recently switched 
 to Fedora 12 on the target machine, and of course the userspace works 
 with the distro kernel.

-- 
Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are watching the assignee of the bug.
--
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev 
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 14331] Radeon XPRESS 200M: System hang with radeon DRI and Fedora 10 userspace unless DRI=off

2009-12-29 Thread bugzilla-daemon
http://bugzilla.kernel.org/show_bug.cgi?id=14331


Rafael J. Wysocki r...@sisk.pl changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||UNREPRODUCIBLE




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

--
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev 
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 14331] Radeon XPRESS 200M: System hang with radeon DRI and Fedora 10 userspace unless DRI=off

2009-12-29 Thread bugzilla-daemon
http://bugzilla.kernel.org/show_bug.cgi?id=14331


Rafael J. Wysocki r...@sisk.pl changed:

   What|Removed |Added

 Status|RESOLVED|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.

--
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev 
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 14945] Colors are too bright with KMS on Radeon 9xxx

2009-12-29 Thread bugzilla-daemon
http://bugzilla.kernel.org/show_bug.cgi?id=14945





--- Comment #3 from Alex Deucher alexdeuc...@gmail.com  2009-12-29 17:08:01 
---
Created an attachment (id=24351)
 -- (http://bugzilla.kernel.org/attachment.cgi?id=24351)
fix primary dac adj values

This patch should fix it.

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

--
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev 
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[PATCH] drm/radeon/kms: add primary dac adj values table

2009-12-29 Thread Alex Deucher
From a71f0302835876038fb608e0d7a570dc3a0f145c Mon Sep 17 00:00:00 2001
From: Alex Deucher alexdeuc...@gmail.com
Date: Tue, 29 Dec 2009 12:09:17 -0500
Subject: [PATCH] drm/radeon/kms: add primary dac adj values table

Look up primary dac adj values from the table if
there is no bios or bios dac table to reference.

The lookup table may need to be adjusted for certain
families.

Should fix kernel bug 14945.

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

diff --git a/drivers/gpu/drm/radeon/radeon_combios.c
b/drivers/gpu/drm/radeon/radeon_combios.c
index 1204234..7018f32 100644
--- a/drivers/gpu/drm/radeon/radeon_combios.c
+++ b/drivers/gpu/drm/radeon/radeon_combios.c
@@ -595,6 +595,34 @@ bool radeon_combios_get_clock_info(struct drm_device *dev)
return false;
 }

+static const uint32_t default_primarydac_adj[CHIP_LAST] = {
+   0x0808, /* r100  */
+   0x0808, /* rv100 */
+   0x0808, /* rs100 */
+   0x0808, /* rv200 */
+   0x0808, /* rs200 */
+   0x0808, /* r200  */
+   0x0808, /* rv250 */
+   0x, /* rs300 */
+   0x0808, /* rv280 */
+   0x0808, /* r300  */
+   0x0808, /* r350  */
+   0x0808, /* rv350 */
+   0x0808, /* rv380 */
+   0x0808, /* r420  */
+   0x0808, /* r423  */
+   0x0808, /* rv410 */
+   0x, /* rs400 */
+   0x, /* rs480 */
+};
+
+static void radeon_legacy_get_primary_dac_info_from_table(struct
radeon_device *rdev,
+ struct 
radeon_encoder_primary_dac *p_dac)
+{
+   p_dac-ps2_pdac_adj = default_primarydac_adj[rdev-family];
+   return;
+}
+
 struct radeon_encoder_primary_dac *radeon_combios_get_primary_dac_info(struct
   
radeon_encoder
   *encoder)
@@ -604,20 +632,20 @@ struct radeon_encoder_primary_dac
*radeon_combios_get_primary_dac_info(struct
uint16_t dac_info;
uint8_t rev, bg, dac;
struct radeon_encoder_primary_dac *p_dac = NULL;
+   int found = 0;

-   if (rdev-bios == NULL)
+   p_dac = kzalloc(sizeof(struct radeon_encoder_primary_dac),
+   GFP_KERNEL);
+
+   if (!p_dac)
return NULL;

+   if (rdev-bios == NULL)
+   goto out;
+
/* check CRT table */
dac_info = combios_get_table_offset(dev, COMBIOS_CRT_INFO_TABLE);
if (dac_info) {
-   p_dac =
-   kzalloc(sizeof(struct radeon_encoder_primary_dac),
-   GFP_KERNEL);
-
-   if (!p_dac)
-   return NULL;
-
rev = RBIOS8(dac_info)  0x3;
if (rev  2) {
bg = RBIOS8(dac_info + 0x2)  0xf;
@@ -628,9 +656,13 @@ struct radeon_encoder_primary_dac
*radeon_combios_get_primary_dac_info(struct
dac = RBIOS8(dac_info + 0x3)  0xf;
p_dac-ps2_pdac_adj = (bg  8) | (dac);
}
-
+   found = 1;
}

+out:
+   if (!found) /* fallback to defaults */
+   radeon_legacy_get_primary_dac_info_from_table(rdev, p_dac);
+
return p_dac;
 }

-- 
1.5.6.3


0001-drm-radeon-kms-add-primary-dac-adj-values-table.patch
Description: application/mbox
--
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev --
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: drm/radeon/kms: pm: single frame corruptions on reclocking

2009-12-29 Thread Luca Tettamanti
Il Mon, Dec 28, 2009 at 05:27:13PM -0500, Alex Deucher ha scritto: 
 2009/12/28 Luca Tettamanti kronos...@gmail.com:
  On Mon, Dec 28, 2009 at 01:32:24PM -0500, Alex Deucher wrote:
  2009/12/28 Luca Tettamanti kronos...@gmail.com:
   2009/12/28 Alex Deucher alexdeuc...@gmail.com:
   On Mon, Dec 28, 2009 at 5:53 AM, Luca Tettamanti kronos...@gmail.com 
   wrote:
   On Sun, Dec 27, 2009 at 1:55 AM, Rafał Miłecki zaj...@gmail.com 
   wrote:
   W dniu 26 grudnia 2009 20:08 użytkownik Alex Deucher
   alexdeuc...@gmail.com napisał:
   It may be that the engine doesn't like to be reclocked while it's
   running.  Perhaps we should use the GUI idle interrupt rather than
   vblanks to reclock the engine.
  
   Could you say something more about GUI idle interrupt, please?
  
   It's mentioned in the documentation of the IH (see r600.c); I guess
   it's enabled by GUI_IDLE_INT_ENABLE.
  
   What is this, do we already have code for that?
  
   Unless there are more subtleties all is needed it to enabled the
   interrupt and catch it in r600_irq_process.
  
   That's pretty much it.  Pre-r6xx asics have a GUI interrupt as well.
   I can look up the details for that if they are not already documented.
  
   I can't find a way to ack the interrupt; I see
   RADEON_GUI_IDLE_INT_TEST_ACK but I think it's for pre-r6xx cards,
   right?
 
  You don't have to ACK it as the CP generates the interrupt in
  software; similar to the sw interrupts used for fences.
 
  Ok, good: I've got the stub running on my M76. r100 and rs600 parts are
  untested. Comments?
 
 
 Looks pretty good.  I've included the proper defines from the register
 database below and you'll need to ack the gui idle interrupts on
 pre-r600 chips.  Now you just have to do something when you get the
 idle interrupt.

I've adapted Rafał's patch to do the reclock when the idle interrupt is
fired (which btw should take care of the special case for nr CRTCs  1).
Unfortunately I still see the black frame when reclocking is performed.
So I tried recloking directly from the IH (yeah, I'm ashamed of
myself...); this got rid of the black frame, but causes corruption of a
horizontal block of the screen (during the reclock, before and after the
screen looks fine). In this second case I've added a spinlock to guard
the access to the CP ring, so nothing touches it while reclocking is
performed; however by the time we process the idle interrupt -
especially considering that multiple events might be queued in the IH
ring - someone else (i.e. one of the other cores) might already have
submitted more work; what do you think?

I'm attaching 3 patches; the first one contains the stub idle IH, the
second one is Rafał's patch adapted for idle interrupt (it's not very
polished), the third one moves reclocking to IH (and as is it's just an
ugly hack).

Luca
-- 
In linea di principio sarei indifferente al natale, se solo il natale
 ricambiasse la cortesia e mi lasciasse in pace. -- Marco d'Itri
--- linux-2.6.git.orig/drivers/gpu/drm/radeon/r600.c	2009-12-28 16:38:38.388825742 +0100
+++ linux-2.6.git/drivers/gpu/drm/radeon/r600.c	2009-12-28 22:03:12.936157804 +0100
@@ -2458,6 +2458,7 @@
 int r600_irq_set(struct radeon_device *rdev)
 {
 	u32 cp_int_cntl = CNTX_BUSY_INT_ENABLE | CNTX_EMPTY_INT_ENABLE;
+	u32 grbm_int_cntl;
 	u32 mode_int = 0;
 	u32 hpd1, hpd2, hpd3, hpd4 = 0, hpd5 = 0, hpd6 = 0;
 
@@ -2465,6 +2466,8 @@
 	if (!rdev-ih.enabled)
 		return 0;
 
+	grbm_int_cntl = RREG32(GRBM_INT_CNTL)  ~GUI_IDLE_INT_ENABLE;
+
 	if (ASIC_IS_DCE3(rdev)) {
 		hpd1 = RREG32(DC_HPD1_INT_CONTROL)  ~DC_HPDx_INT_EN;
 		hpd2 = RREG32(DC_HPD2_INT_CONTROL)  ~DC_HPDx_INT_EN;
@@ -2484,6 +2487,10 @@
 		DRM_DEBUG(r600_irq_set: sw int\n);
 		cp_int_cntl |= RB_INT_ENABLE;
 	}
+	if (rdev-irq.idle_int) {
+		DRM_DEBUG(r600_irq_set: GUI idle int\n);
+		grbm_int_cntl |= GUI_IDLE_INT_ENABLE;
+	}
 	if (rdev-irq.crtc_vblank_int[0]) {
 		DRM_DEBUG(r600_irq_set: vblank 0\n);
 		mode_int |= D1MODE_VBLANK_INT_MASK;
@@ -2518,6 +2525,7 @@
 	}
 
 	WREG32(CP_INT_CNTL, cp_int_cntl);
+	WREG32(GRBM_INT_CNTL, grbm_int_cntl);
 	WREG32(DxMODE_INT_MASK, mode_int);
 	if (ASIC_IS_DCE3(rdev)) {
 		WREG32(DC_HPD1_INT_CONTROL, hpd1);
@@ -2806,6 +2814,9 @@
 		case 181: /* CP EOP event */
 			DRM_DEBUG(IH: CP EOP\n);
 			break;
+		case 233: /* GUI idle event */
+			DRM_DEBUG(IH: GUI idle\n);
+			break;
 		default:
 			DRM_ERROR(Unhandled interrupt: %d %d\n, src_id, src_data);
 			break;
--- linux-2.6.git.orig/drivers/gpu/drm/radeon/radeon.h	2009-12-28 16:38:13.945836481 +0100
+++ linux-2.6.git/drivers/gpu/drm/radeon/radeon.h	2009-12-28 22:02:48.964001788 +0100
@@ -343,6 +343,7 @@
 struct radeon_irq {
 	bool		installed;
 	bool		sw_int;
+	bool		idle_int;
 	/* FIXME: use a define max crtc rather than hardcode it */
 	bool		crtc_vblank_int[2];
 	/* FIXME: use defines for max hpd/dacs */
--- linux-2.6.git.orig/drivers/gpu/drm/radeon/radeon_irq_kms.c	2009-12-28 16:37:10.669823739 +0100
+++ linux-2.6.git/drivers/gpu/drm/radeon/radeon_irq_kms.c	

Re: drm/radeon/kms: pm: single frame corruptions on reclocking

2009-12-29 Thread Xavier Bestel
On Tue, 2009-12-29 at 19:35 +0100, Luca Tettamanti wrote:
 I've adapted Rafał's patch to do the reclock when the idle interrupt is
 fired (which btw should take care of the special case for nr CRTCs  1).
 Unfortunately I still see the black frame when reclocking is performed.
 So I tried recloking directly from the IH (yeah, I'm ashamed of
 myself...); this got rid of the black frame, but causes corruption of a
 horizontal block of the screen (during the reclock, before and after the
 screen looks fine). In this second case I've added a spinlock to guard
 the access to the CP ring, so nothing touches it while reclocking is
 performed; however by the time we process the idle interrupt -
 especially considering that multiple events might be queued in the IH
 ring - someone else (i.e. one of the other cores) might already have
 submitted more work; what do you think?

Maybe a spinlock doesn't cut it. How about some mutex you take when you
need to reclock, to let tasks trying to add to the CP ring just sleep,
and then you unmask the idle interrupt. When you receive the idle
interrupt, you reclock, you free the mutex, and voilà.

Xav




--
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev 
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: 2.6.33-rc2: Reported regressions from 2.6.32

2009-12-29 Thread Rafał Miłecki
2009/12/29 Rafael J. Wysocki r...@sisk.pl:
 Bug-Entry       : http://bugzilla.kernel.org/show_bug.cgi?id=14910
 Subject         : Stable version 2.6.32.2 broke my KMS Radeon setup
 Submitter       : Ruud Linders kerne...@xs4all.nl
 Date            : 2009-12-20 12:02 (10 days old)
 References      : http://marc.info/?l=linux-kernelm=126131105608871w=4

Has patch (I think it was even submitted to Linus).

-- 
Rafał

--
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev 
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 14910] Stable version 2.6.32.2 broke my KMS Radeon setup

2009-12-29 Thread bugzilla-daemon
http://bugzilla.kernel.org/show_bug.cgi?id=14910


Rafael J. Wysocki r...@sisk.pl changed:

   What|Removed |Added

 Blocks|14885   |




--- Comment #2 from Rafael J. Wysocki r...@sisk.pl  2009-12-29 20:43:49 ---
Dropping from the list of recent mainline regressions as a -stable only issue.

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

--
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev 
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: 2.6.33-rc2: Reported regressions from 2.6.32

2009-12-29 Thread Rafael J. Wysocki
On Tuesday 29 December 2009, Rafał Miłecki wrote:
 2009/12/29 Rafael J. Wysocki r...@sisk.pl:
  Bug-Entry   : http://bugzilla.kernel.org/show_bug.cgi?id=14910
  Subject : Stable version 2.6.32.2 broke my KMS Radeon setup
  Submitter   : Ruud Linders kerne...@xs4all.nl
  Date: 2009-12-20 12:02 (10 days old)
  References  : http://marc.info/?l=linux-kernelm=126131105608871w=4
 
 Has patch (I think it was even submitted to Linus).

This is a -stable only issue in fact, so I'm not going to track it from now on.

Please watch the next -stable announcement from Greg and remind him about this
issue if the patch isn't there.

Rafael

--
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev 
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 14956] New: with KMS: kernel lockup on xorg start

2009-12-29 Thread bugzilla-daemon
http://bugzilla.kernel.org/show_bug.cgi?id=14956

   Summary: with KMS: kernel lockup on xorg start
   Product: Drivers
   Version: 2.5
Kernel Version: 2.6.33-rc2
  Platform: All
OS/Version: Linux
  Tree: Mainline
Status: NEW
  Severity: normal
  Priority: P1
 Component: Video(DRI - non Intel)
AssignedTo: drivers_video-...@kernel-bugs.osdl.org
ReportedBy: cstrae...@gmail.com
Regression: No


Created an attachment (id=24358)
 -- (http://bugzilla.kernel.org/attachment.cgi?id=24358)
regs after virgin modprobe radeon modeset=1 before xorg start system freeze

i use the latest git xorg-radeon / libdrm / mesa versions.
this is reproducible behavior on my Ati Mobility Radon 3450 (Vaio laptop):

1)
When booting with radeon.modeset=1 the system freezes while starting up xorg
(nothing is even logged to xorg.0.log)

2)
When booting with radeon.modeset=0 xorg starts and everything works

3)
when booting with radeon.modeset=0 into *text mode*, and then reloading the
radeon kernelmodul with modeset=1 xorg starts without problems

4)
when booting with radeon.modeset=0 into *xorg*, and then stopping xorg and
reloading the radeon kernelmodul with modeset=1 the system freezes even during
modprobe

attached are the radeontool/avivotool regs before xorg start for version 1 and
2.
unfortunately the regs between version 2 and 3 don't differ and since version 4
crashes during modprobe i cant't provide a meaningful dump either

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

--
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev 
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 14956] with KMS: kernel lockup on xorg start

2009-12-29 Thread bugzilla-daemon
http://bugzilla.kernel.org/show_bug.cgi?id=14956





--- Comment #1 from cstrae...@gmail.com  2009-12-29 20:51:34 ---
Created an attachment (id=24359)
 -- (http://bugzilla.kernel.org/attachment.cgi?id=24359)
 radeon regs after virgin modprobe radeon modeset=1 before xorg start system
freeze

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

--
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev 
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 14956] with KMS: kernel lockup on xorg start

2009-12-29 Thread bugzilla-daemon
http://bugzilla.kernel.org/show_bug.cgi?id=14956





--- Comment #2 from cstrae...@gmail.com  2009-12-29 20:54:07 ---
Created an attachment (id=24360)
 -- (http://bugzilla.kernel.org/attachment.cgi?id=24360)
radeon regs after virgin modprobe radeon modeset=0 before working xorg start

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

--
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev 
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 14956] with KMS: kernel lockup on xorg start

2009-12-29 Thread bugzilla-daemon
http://bugzilla.kernel.org/show_bug.cgi?id=14956





--- Comment #3 from cstrae...@gmail.com  2009-12-29 20:55:24 ---
Created an attachment (id=24361)
 -- (http://bugzilla.kernel.org/attachment.cgi?id=24361)
avivo regs after virgin modprobe radeon modeset=0 before working xorg start

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

--
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev 
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 14956] with KMS: kernel lockup on xorg start

2009-12-29 Thread bugzilla-daemon
http://bugzilla.kernel.org/show_bug.cgi?id=14956


cstrae...@gmail.com changed:

   What|Removed |Added

  Attachment #24358|regs after virgin modprobe |avivo regs after virgin
description|radeon modeset=1 before|modprobe radeon modeset=1
   |xorg start system freeze|before xorg start system
   ||freeze




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

--
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev 
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 14945] Colors are too bright with KMS on Radeon 9xxx

2009-12-29 Thread bugzilla-daemon
http://bugzilla.kernel.org/show_bug.cgi?id=14945





--- Comment #4 from Samuel Lidén Borell sam...@slbdata.se  2009-12-29 
21:17:58 ---
No, I still have the same problem with your patch (tried it with both the
Ubuntu DRM modules and the latest modules in Linus' tree).

And the radeontool regs still have the same values.

-- 
Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are watching the assignee of the bug.
--
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev 
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 14945] Colors are too bright with KMS on Radeon 9xxx

2009-12-29 Thread bugzilla-daemon
http://bugzilla.kernel.org/show_bug.cgi?id=14945


Alex Deucher alexdeuc...@gmail.com changed:

   What|Removed |Added

 CC||alexdeuc...@gmail.com




--- Comment #5 from Alex Deucher alexdeuc...@gmail.com  2009-12-29 21:42:42 
---
Can you attach a copy of your video bios?  (as root):
cd /sys/bus/pci/devices/pci bus id
echo 1  rom
cat rom  /tmp/vbios.rom
echo 0  rom

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

--
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev 
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 14956] with KMS: kernel lockup on xorg start

2009-12-29 Thread bugzilla-daemon
http://bugzilla.kernel.org/show_bug.cgi?id=14956


Alex Deucher alexdeuc...@gmail.com changed:

   What|Removed |Added

 CC||alexdeuc...@gmail.com




--- Comment #4 from Alex Deucher alexdeuc...@gmail.com  2009-12-29 21:45:40 
---
Please attach your xorg log and dmesg.

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

--
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev 
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 14945] Colors are too bright with KMS on Radeon 9xxx

2009-12-29 Thread bugzilla-daemon
http://bugzilla.kernel.org/show_bug.cgi?id=14945





--- Comment #6 from Samuel Lidén Borell sam...@slbdata.se  2009-12-29 
21:49:21 ---
Created an attachment (id=24362)
 -- (http://bugzilla.kernel.org/attachment.cgi?id=24362)
Dump of VGA BIOS

-- 
Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are watching the assignee of the bug.
--
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev 
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: 2.6.33-rc2: Reported regressions 2.6.31 - 2.6.32

2009-12-29 Thread Rafael J. Wysocki
On Tuesday 29 December 2009, Jarek Poplawski wrote:
 Rafael J. Wysocki wrote, On 12/29/2009 04:26 PM:
 
 ...
  Bug-Entry   : http://bugzilla.kernel.org/show_bug.cgi?id=14794
  Subject : IP address assigned by DHCP is dropped after ~40 
  seconds
  Submitter   : Márton Németh nm...@freemail.hu
  Date: 2009-12-13 08:59 (17 days old)
  
 
 
 IMHO this bug might be considered as fixed by updating a userspace tool
 (KNetworkManager) - unless somebody wants to seek for the exact reason...

Good, I'll gladly close it. :-)

Rafael

--
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev 
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 14945] Colors are too bright with KMS on Radeon 9xxx

2009-12-29 Thread bugzilla-daemon
http://bugzilla.kernel.org/show_bug.cgi?id=14945


Samuel Lidén Borell sam...@slbdata.se changed:

   What|Removed |Added

  Attachment #24362|Dump of VGA BIOS|Dump of video BIOS
description||




-- 
Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are watching the assignee of the bug.
--
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev 
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 14956] with KMS: kernel lockup on xorg start

2009-12-29 Thread bugzilla-daemon
http://bugzilla.kernel.org/show_bug.cgi?id=14956





--- Comment #5 from cstrae...@gmail.com  2009-12-29 22:08:12 ---
Created an attachment (id=24363)
 -- (http://bugzilla.kernel.org/attachment.cgi?id=24363)
xorg + dmesg working (modeset=0)

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

--
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev 
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 14956] with KMS: kernel lockup on xorg start

2009-12-29 Thread bugzilla-daemon
http://bugzilla.kernel.org/show_bug.cgi?id=14956





--- Comment #6 from cstrae...@gmail.com  2009-12-29 22:10:06 ---
Created an attachment (id=24364)
 -- (http://bugzilla.kernel.org/attachment.cgi?id=24364)
xorg + dmesg crash (modeset=1)

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

--
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev 
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


radeon list corruption bug in 2.6.31.9

2009-12-29 Thread Rik van Riel
Hi David,

here is a call trace of the list corruption I am seeing with
the latest Fedora kernel, 2.6.31.9-174.fc12.x86_64, on my
home system with two radeon video cards:

02:00.0 VGA compatible controller: ATI Technologies Inc RV730 [FirePro 
V5700]
03:00.0 VGA compatible controller: ATI Technologies Inc RV370 5B60 
[Radeon X300 (PCIE)]
03:00.1 Display controller: ATI Technologies Inc RV370 [Radeon X300SE]

Usually these get followed quite quickly by a kernel crash, but
I got a few today that allowed me to make this nice cut'n'paste
for you.

I tried out the 2.6.32.2 based rawhide kernel, but of course
that one did not even boot here, due to the other radeon issue :)

Is there anything I can do to help narrow down the issue, or has
the code been rewritten enough in 2.6.32 and 2.6.33 that I should
just upgrade to .32 once that one works?

[ cut here ]
WARNING: at lib/list_debug.c:30 __list_add+0x68/0x81() (Not tainted)
Hardware name: Precision WorkStation T3500
list_add corruption. prev-next should be next (88032dc32da0), but 
was 8
800ad5aa530. (prev=880292066730).
Modules linked in: fuse tun netconsole configfs autofs4 max6650 coretemp 
sunrpc bridge stp llc ipv6 cpufreq_ondemand acpi_cpufreq freq_table 
dm_multipath kvm_intel kvm uinput snd_hda_codec_analog snd_ens1371 
gameport snd_ac97_codec ppdev snd_hda_intel ac97_bus parport_pc 
snd_usb_audio pl2303 snd_hda_codec snd_seq snd_pcm snd_usb_lib 
snd_rawmidi snd_timer snd_seq_device snd_hwdep usbserial snd parport 
iTCO_wdt tg3 iTCO_vendor_support i2c_i801 dcdbas snd_page_alloc 
serio_raw soundcore wmi radeon ttm drm_kms_helper drm i2c_algo_bit 
i2c_core [last unloaded: microcode]
Pid: 1945, comm: Xorg Not tainted 2.6.31.9-174.fc12.x86_64 #1
Call Trace:
  [81051710] warn_slowpath_common+0x84/0x9c
  [8105177f] warn_slowpath_fmt+0x41/0x43
  [a004df78] ? ttm_buffer_object_init+0x338/0x361 [ttm]
  [81207e23] __list_add+0x68/0x81
  [a007ab3d] radeon_object_create+0x1cb/0x1de [radeon]
  [a007a925] ? radeon_ttm_object_object_destroy+0x0/0x4d [radeon]
  [a008586d] radeon_gem_object_create+0x90/0xfe [radeon]
  [a0086e00] ? radeon_cs_ioctl+0x14a/0x19a [radeon]
  [a00858db] ? radeon_gem_create_ioctl+0x0/0xdf [radeon]
  [a0085935] radeon_gem_create_ioctl+0x5a/0xdf [radeon]
  [a001521f] drm_ioctl+0x237/0x2f4 [drm]
  [81108cc5] vfs_ioctl+0x6f/0x87
  [811091d4] do_vfs_ioctl+0x47b/0x4c1
  [81109270] sys_ioctl+0x56/0x79
  [8141dd20] ? do_device_not_available+0x9/0xb
  [81011cf2] system_call_fastpath+0x16/0x1b
---[ end trace 6286fa6c2ff39104 ]---

--
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev 
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 25733] Latest drm-radeon-testing fails to suspend

2009-12-29 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=25733





--- Comment #1 from Gavin Kinsey ga...@trollgod.org.uk  2009-12-29 16:59:18 
PST ---
It seems this bug only occurs when I use a 64-bit kernel.  The 32-bit kernels
built from the same source suspend fine.  I'm using the same 32-bit userspace
for both.

I just tried the vanilla 2.6.33-rc2 kernel and the bug is in there too.


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

--
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev 
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: drm/radeon/kms: pm: single frame corruptions on reclocking

2009-12-29 Thread Michel Dänzer
On Tue, 2009-12-29 at 19:35 +0100, Luca Tettamanti wrote: 
 Il Mon, Dec 28, 2009 at 05:27:13PM -0500, Alex Deucher ha scritto: 
  2009/12/28 Luca Tettamanti kronos...@gmail.com:
   On Mon, Dec 28, 2009 at 01:32:24PM -0500, Alex Deucher wrote:
   2009/12/28 Luca Tettamanti kronos...@gmail.com:
2009/12/28 Alex Deucher alexdeuc...@gmail.com:
On Mon, Dec 28, 2009 at 5:53 AM, Luca Tettamanti 
kronos...@gmail.com wrote:
On Sun, Dec 27, 2009 at 1:55 AM, Rafał Miłecki zaj...@gmail.com 
wrote:
W dniu 26 grudnia 2009 20:08 użytkownik Alex Deucher
alexdeuc...@gmail.com napisał:
It may be that the engine doesn't like to be reclocked while it's
running.  Perhaps we should use the GUI idle interrupt rather than
vblanks to reclock the engine.
   
Could you say something more about GUI idle interrupt, please?
   
It's mentioned in the documentation of the IH (see r600.c); I guess
it's enabled by GUI_IDLE_INT_ENABLE.
   
What is this, do we already have code for that?
   
Unless there are more subtleties all is needed it to enabled the
interrupt and catch it in r600_irq_process.
   
That's pretty much it.  Pre-r6xx asics have a GUI interrupt as well.
I can look up the details for that if they are not already documented.
   
I can't find a way to ack the interrupt; I see
RADEON_GUI_IDLE_INT_TEST_ACK but I think it's for pre-r6xx cards,
right?
  
   You don't have to ACK it as the CP generates the interrupt in
   software; similar to the sw interrupts used for fences.
  
   Ok, good: I've got the stub running on my M76. r100 and rs600 parts are
   untested. Comments?
  
  
  Looks pretty good.  I've included the proper defines from the register
  database below and you'll need to ack the gui idle interrupts on
  pre-r600 chips.  Now you just have to do something when you get the
  idle interrupt.
 
 I've adapted Rafał's patch to do the reclock when the idle interrupt is
 fired (which btw should take care of the special case for nr CRTCs  1).
 Unfortunately I still see the black frame when reclocking is performed.
 So I tried recloking directly from the IH (yeah, I'm ashamed of
 myself...); this got rid of the black frame, but causes corruption of a
 horizontal block of the screen (during the reclock, before and after the
 screen looks fine).

If you mean the interrupt handler for the idle interrupt, have you tried
doing it in the interrupt handler for the vblank interrupt instead?


-- 
Earthling Michel Dänzer   |http://www.vmware.com
Libre software enthusiast |  Debian, X and DRI developer

--
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev 
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[PATCH] drm/radeon/radeon_cp.c: fix resource leak on error

2009-12-29 Thread Darren Jenkins
If drm_addmap() fails master_priv is leaked.


Coverity CID: 13195

Signed-off-by: Darren Jenkins darrenrjenk...@gmail.com

diff --git drivers/gpu/drm/radeon/radeon_cp.c drivers/gpu/drm/radeon/radeon_cp.c
index 0b2f9c2..06123ba 100644
--- drivers/gpu/drm/radeon/radeon_cp.c
+++ drivers/gpu/drm/radeon/radeon_cp.c
@@ -2145,6 +2145,7 @@ int radeon_master_create(struct drm_device *dev, struct 
drm_master *master)
 master_priv-sarea);
if (ret) {
DRM_ERROR(SAREA setup failed\n);
+   kfree(master_priv);
return ret;
}
master_priv-sarea_priv = master_priv-sarea-handle + sizeof(struct 
drm_sarea);



--
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev 
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[PATCH] drm/radeon/radeon_device.c: move a dereference below a NULL test

2009-12-29 Thread Darren Jenkins
If a NULL value is possible, the dereference should only occur after the
NULL test.

Coverity CID: 13335

Signed-off-by: Darren Jenkins darrenrjenk...@gmail.com

diff --git drivers/gpu/drm/radeon/radeon_device.c 
drivers/gpu/drm/radeon/radeon_device.c
index 7c68480..0c51f8e 100644
--- drivers/gpu/drm/radeon/radeon_device.c
+++ drivers/gpu/drm/radeon/radeon_device.c
@@ -733,16 +733,18 @@ void radeon_device_fini(struct radeon_device *rdev)
  */
 int radeon_suspend_kms(struct drm_device *dev, pm_message_t state)
 {
-   struct radeon_device *rdev = dev-dev_private;
+   struct radeon_device *rdev;
struct drm_crtc *crtc;
int r;
 
-   if (dev == NULL || rdev == NULL) {
+   if (dev == NULL || dev-dev_private == NULL) {
return -ENODEV;
}
if (state.event == PM_EVENT_PRETHAW) {
return 0;
}
+   rdev = dev-dev_private;
+
/* unpin the front buffers */
list_for_each_entry(crtc, dev-mode_config.crtc_list, head) {
struct radeon_framebuffer *rfb = 
to_radeon_framebuffer(crtc-fb);



--
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev 
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[PATCH] drm/radeon: fix a couple of array index errors

2009-12-29 Thread Darren Jenkins
There are a couple of array overruns, and some associated confusion in
the code. 
This is just a wild guess at what the code should actually look like.

Coverity CID: 13305 13306


Signed-off-by: Darren Jenkins darrenrjenk...@gmail.com

diff --git drivers/gpu/drm/radeon/radeon_legacy_tv.c 
drivers/gpu/drm/radeon/radeon_legacy_tv.c
index 3a12bb0..c37535a 100644
--- drivers/gpu/drm/radeon/radeon_legacy_tv.c
+++ drivers/gpu/drm/radeon/radeon_legacy_tv.c
@@ -112,6 +112,8 @@ static const uint16_t vert_timing_NTSC[] = {
0x1817,
0x21d4,
0x0002,
+   0x,
+   0x,
0
 };
 
@@ -623,9 +625,9 @@ void radeon_legacy_tv_mode_set(struct drm_encoder *encoder,
}
flicker_removal = (tmp + 500) / 1000;
 
-   if (flicker_removal  3)
-   flicker_removal = 3;
-   for (i = 0; i  6; ++i) {
+   if (flicker_removal  2)
+   flicker_removal = 2;
+   for (i = 0; i  ARRAY_SIZE(SLOPE_limit); ++i) {
if (flicker_removal == SLOPE_limit[i])
break;
}
diff --git drivers/gpu/drm/radeon/radeon_mode.h 
drivers/gpu/drm/radeon/radeon_mode.h
index 402369d..abee9a9 100644
--- drivers/gpu/drm/radeon/radeon_mode.h
+++ drivers/gpu/drm/radeon/radeon_mode.h
@@ -218,8 +218,8 @@ struct radeon_mode_info {
 
 };
 
-#define MAX_H_CODE_TIMING_LEN 32
-#define MAX_V_CODE_TIMING_LEN 32
+#define MAX_H_CODE_TIMING_LEN 16
+#define MAX_V_CODE_TIMING_LEN 16
 
 /* need to store these as reading
back code tables is excessive */



--
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev 
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[PATCH] gpu/drm/radeon/radeon_irq.c: move a dereference below a NULL test

2009-12-29 Thread Darren Jenkins
If a NULL value is possible, the dereference should only occur after the
NULL test. 


Coverity CID: 13338

Signed-off-by: Darren Jenkins darrenrjenk...@gmail.com 

diff --git drivers/gpu/drm/radeon/radeon_irq.c 
drivers/gpu/drm/radeon/radeon_irq.c
index b79ecc4..2f349a3 100644
--- drivers/gpu/drm/radeon/radeon_irq.c
+++ drivers/gpu/drm/radeon/radeon_irq.c
@@ -289,16 +289,16 @@ int radeon_irq_emit(struct drm_device *dev, void *data, 
struct drm_file *file_pr
drm_radeon_irq_emit_t *emit = data;
int result;
 
-   if ((dev_priv-flags  RADEON_FAMILY_MASK) = CHIP_R600)
-   return -EINVAL;
-
-   LOCK_TEST_WITH_RETURN(dev, file_priv);
-
if (!dev_priv) {
DRM_ERROR(called with no initialization\n);
return -EINVAL;
}
 
+   if ((dev_priv-flags  RADEON_FAMILY_MASK) = CHIP_R600)
+   return -EINVAL;
+
+   LOCK_TEST_WITH_RETURN(dev, file_priv);
+
result = radeon_emit_irq(dev);
 
if (DRM_COPY_TO_USER(emit-irq_seq, result, sizeof(int))) {



--
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev 
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: 2.6.33-rc2: Reported regressions 2.6.31 - 2.6.32

2009-12-29 Thread Jarek Poplawski
On Tue, Dec 29, 2009 at 09:55:13PM +0100, Németh Márton wrote:
 Jarek Poplawski wrote:
  Rafael J. Wysocki wrote, On 12/29/2009 04:26 PM:
  
  ...
  Bug-Entry  : http://bugzilla.kernel.org/show_bug.cgi?id=14794
  Subject: IP address assigned by DHCP is dropped after ~40 
  seconds
  Submitter  : Márton Németh nm...@freemail.hu
  Date   : 2009-12-13 08:59 (17 days old)
 
  
  
  IMHO this bug might be considered as fixed by updating a userspace tool
  (KNetworkManager) - unless somebody wants to seek for the exact reason...
 
 For me (I am the reporter) the problem was solved by upgrading a userspace
 tool, so I think the bug filed against the kernel can be closed marking
 that this was not really a kernel bug.

Yes, let's hope author(s) of KNetworkManager would let us know if they
had to change it because of the kernel error.

Regards,
Jarek P.

--
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev 
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: 2.6.33-rc2: Reported regressions 2.6.31 - 2.6.32

2009-12-29 Thread Németh Márton
Jarek Poplawski wrote:
 Rafael J. Wysocki wrote, On 12/29/2009 04:26 PM:
 
 ...
 Bug-Entry: http://bugzilla.kernel.org/show_bug.cgi?id=14794
 Subject  : IP address assigned by DHCP is dropped after ~40 
 seconds
 Submitter: Márton Németh nm...@freemail.hu
 Date : 2009-12-13 08:59 (17 days old)

 
 
 IMHO this bug might be considered as fixed by updating a userspace tool
 (KNetworkManager) - unless somebody wants to seek for the exact reason...

For me (I am the reporter) the problem was solved by upgrading a userspace
tool, so I think the bug filed against the kernel can be closed marking
that this was not really a kernel bug.

Regards,

Márton Németh



--
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev 
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: 2.6.33-rc2: Reported regressions 2.6.31 - 2.6.32

2009-12-29 Thread Jarek Poplawski
Rafael J. Wysocki wrote, On 12/29/2009 04:26 PM:

...
 Bug-Entry : http://bugzilla.kernel.org/show_bug.cgi?id=14794
 Subject   : IP address assigned by DHCP is dropped after ~40 
 seconds
 Submitter : Márton Németh nm...@freemail.hu
 Date  : 2009-12-13 08:59 (17 days old)
 


IMHO this bug might be considered as fixed by updating a userspace tool
(KNetworkManager) - unless somebody wants to seek for the exact reason...

Jarek P.

--
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev 
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[PATCH] drm/radeon/radeon_fence.c: move a dereference below the NULL test

2009-12-29 Thread Darren Jenkins
If a NULL value is possible, the dereference should only occur after the
NULL test.

Coverity CID: 13334

Signed-off-by: Darren Jenkins darrenrjenk...@gmail.com

diff --git drivers/gpu/drm/radeon/radeon_fence.c 
drivers/gpu/drm/radeon/radeon_fence.c
index 4cdd8b4..f3a8380 100644
--- drivers/gpu/drm/radeon/radeon_fence.c
+++ drivers/gpu/drm/radeon/radeon_fence.c
@@ -140,14 +140,14 @@ int radeon_fence_create(struct radeon_device *rdev, 
struct radeon_fence **fence)
 
 bool radeon_fence_signaled(struct radeon_fence *fence)
 {
-   struct radeon_device *rdev = fence-rdev;
unsigned long irq_flags;
bool signaled = false;
 
-   if (rdev-gpu_lockup) {
+   if (fence == NULL) {
return true;
}
-   if (fence == NULL) {
+
+   if (fence-rdev-gpu_lockup) {
return true;
}
write_lock_irqsave(fence-rdev-fence_drv.lock, irq_flags);



--
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev 
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[PATCH] drm/radeon/r100.c: check for invalid family

2009-12-29 Thread Darren Jenkins
If there is an invalid family the fw_name is NULL and causes an
NULL pointer dereference.
This just adds a check for something unexpected.

Coverity CID: 13251

Signed-off-by: Darren Jenkins darrenrjenk...@gmail.com

diff --git drivers/gpu/drm/radeon/r100.c drivers/gpu/drm/radeon/r100.c
index 7172746..e4b9770 100644
--- drivers/gpu/drm/radeon/r100.c
+++ drivers/gpu/drm/radeon/r100.c
@@ -584,6 +584,8 @@ static int r100_cp_init_microcode(struct radeon_device 
*rdev)
   (rdev-family == CHIP_RV570)) {
DRM_INFO(Loading R500 Microcode\n);
fw_name = FIRMWARE_R520;
+   } else {
+   return -EINVAL;
}
 
err = request_firmware(rdev-me_fw, fw_name, pdev-dev);



--
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev 
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[PATCH] drm/radeon/radeon_connectors.c: add a NULL test before dereference

2009-12-29 Thread Darren Jenkins
The encoder variable can be NULL in this function so I believe it should
be checked before dereference.

Coverity CID: 13253

Signed-off-by: Darren Jenkins darrenrjenk...@gmail.com

diff --git drivers/gpu/drm/radeon/radeon_connectors.c 
drivers/gpu/drm/radeon/radeon_connectors.c
index 2016156..b82ae61 100644
--- drivers/gpu/drm/radeon/radeon_connectors.c
+++ drivers/gpu/drm/radeon/radeon_connectors.c
@@ -615,7 +615,7 @@ static enum drm_connector_status radeon_vga_detect(struct 
drm_connector *connect
ret = connector_status_connected;
}
} else {
-   if (radeon_connector-dac_load_detect) {
+   if (radeon_connector-dac_load_detect  encoder) {
encoder_funcs = encoder-helper_private;
ret = encoder_funcs-detect(encoder, connector);
}



--
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev 
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel