Re: [Regression, post-rc1] Multiple issues after enabling SetVoltage on rs780m

[Dave Airlie <airlied@redhat.com>]

On Mon, 2010-06-07 at 23:37 +0200, Rafael J. Wysocki wrote:
> On Monday 07 June 2010, Alex Deucher wrote:
> > On Mon, Jun 7, 2010 at 6:15 AM, Dave Airlie <airlied@gmail.com> wrote:
> > > On Mon, Jun 7, 2010 at 7:29 PM, Rafael J. Wysocki <rjw@sisk.pl> wrote:
> > >> Hi Alex,
> > >>
> > >> Your commit 9349d5cc920c10845693f906ebd67f394f1d0d04
> > >> (drm/radeon/kms/pm: enable SetVoltage on r7xx/evergreen) has caused my test-bed
> > >> Acer Ferrari One to behave quite unreliably.  The symptoms are:
> > >>
> > >> - the system hangs hard (~ 50% of the time) when starting Xorg
> > >> - the system hangs hard (~ 50% of the time) when stopping Xorg during system
> > >>  reboot
> > >> - the system sometimes hangs hard during suspend to RAM
> > >>
> > >> These problems are not reproducible with the commit above reverted.
> > >>
> > >> Below is the information about the graphics adapter from lspci.
> > >
> > > Reverting that commit on master fixes it?
> > >
> > > that commit touches code paths in rv770 and evergreen that in no way
> > > should affect that chipset which is an rs780, so takes the r600 paths.
> > >
> > > are you sure its not 7ac9aa5a1f1b87adb69bcbec2b89e228f074103a?
> > 
> > It should be that commit if it is indeed the voltage adjust.  That
> > said, I just took a closer look at the voltage adjust on newer IGPs
> > and unfortunately, it doesn't work the same as the discrete cards, so
> > for now we should disable it.  The attached patch should do the trick.
> >  There weren't any problems on my IGP chips, but they don't have a
> > SetVoltage table, so nothing is touching the hw.
> 
> I'm not sure if the adapter is a discrete one.
> 
> Anyway, my testing was done before commit
> 386f40c86d6c8d5b717ef20620af1a750d0dacb4 and I'm unable to reproduce the
> problems with current -git, so they might be a fallout of the bug fixed by that
> commit.
> 

Yeah the sounded a lot more like the vt.c crapfest, since it was when
starting/stopping X.

Dave.