Re: [PATCH] drivers/radeon/kms: Workaround RV410/R420 CP errata.
On Fri, Dec 18, 2009 at 01:11:45AM -0800, Corbin Simpson wrote: > >From 7c1de3201bd4e965da7c1d542c46d8b2725bf42d Mon Sep 17 00:00:00 2001 > From: Corbin Simpson > Date: Fri, 18 Dec 2009 01:00:57 -0800 > Subject: [PATCH] drivers/radeon/kms: Workaround RV410/R420 CP errata. > > Long story short, this fixes sporadic hardlocks with my rv410 during > times of intense 2D acceleration (Flash on Fx3). > > Signed-off-by: Corbin Simpson It seems you are using space and not tab. Cheers, Jerome -- 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: [PATCH] drivers/radeon/kms: Workaround RV410/R420 CP errata.
2009/12/18 Corbin Simpson : > >From 7c1de3201bd4e965da7c1d542c46d8b2725bf42d Mon Sep 17 00:00:00 2001 > From: Corbin Simpson > Date: Fri, 18 Dec 2009 01:00:57 -0800 > Subject: [PATCH] drivers/radeon/kms: Workaround RV410/R420 CP errata. > > Long story short, this fixes sporadic hardlocks with my rv410 during > times of intense 2D acceleration (Flash on Fx3). Can this be issue on newer chipsets as well? I get hard lockup on RV620 from time to time, very easy reproducible with just starting JDownloader (java application). It seems to do some heavy 2D and locks up my machine in something between 1 second and few minutes. This does not happen when I use UMS (non-KMS) or when I start JDownloader on KDE's virtual desktop #2 and switch to desktop #1. -- 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
Re: [PATCH] drivers/radeon/kms: Workaround RV410/R420 CP errata.
On Fri, 2009-12-18 at 11:54 +0100, Rafał Miłecki wrote: > 2009/12/18 Corbin Simpson : > > >From 7c1de3201bd4e965da7c1d542c46d8b2725bf42d Mon Sep 17 00:00:00 2001 > > From: Corbin Simpson > > Date: Fri, 18 Dec 2009 01:00:57 -0800 > > Subject: [PATCH] drivers/radeon/kms: Workaround RV410/R420 CP errata. > > > > Long story short, this fixes sporadic hardlocks with my rv410 during > > times of intense 2D acceleration (Flash on Fx3). > > Can this be issue on newer chipsets as well? I get hard lockup on > RV620 from time to time, very easy reproducible with just starting > JDownloader (java application). It seems to do some heavy 2D and locks > up my machine in something between 1 second and few minutes. > > This does not happen when I use UMS (non-KMS) or when I start > JDownloader on KDE's virtual desktop #2 and switch to desktop #1. It can't be the same problem: it's specific to the 2D engine, which doesn't exist anymore as of R6xx, and it would also happen with UMS. -- 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
Re: [PATCH] drivers/radeon/kms: Workaround RV410/R420 CP errata.
On Fri, 2009-12-18 at 01:11 -0800, Corbin Simpson wrote: [...] > @@ -194,6 +225,9 @@ static int r420_startup(struct radeon_device *rdev) > dev_err(rdev->dev, "failled initializing CP (%d).\n", r); > return r; > } > +/* Handle CP errata. */ > +r420_cp_errata_init(rdev); > + > r = r100_wb_init(rdev); > if (r) { On a purely cosmetic POV, it looks like you indent with spaces in a file previously indented with tabs. 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: [PATCH] drivers/radeon/kms: Workaround RV410/R420 CP errata.
On Fri, Dec 18, 2009 at 01:11:45AM -0800, Corbin Simpson wrote: > >From 7c1de3201bd4e965da7c1d542c46d8b2725bf42d Mon Sep 17 00:00:00 2001 > From: Corbin Simpson > Date: Fri, 18 Dec 2009 01:00:57 -0800 > Subject: [PATCH] drivers/radeon/kms: Workaround RV410/R420 CP errata. > > Long story short, this fixes sporadic hardlocks with my rv410 during > times of intense 2D acceleration (Flash on Fx3). > > Signed-off-by: Corbin Simpson > --- Beside the tab issue you should also avoid to reallocate scratch register if it was already allocated. Maybe init the scratch to -1 and test for that. Otherwise after few suspend/resume you won't have scratch reg left. Cheers, Jerome -- 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