[PATCH] drm/nouveau: fix ttm move notify callback

[Konrad Rzeszutek Wilk]

On Tue, Jan 10, 2012 at 01:46:05PM +1000, Ben Skeggs wrote:
> On Fri, 2012-01-06 at 16:00 -0500, Jerome Glisse wrote:
> > On Fri, Jan 06, 2012 at 02:52:49PM -0500, Konrad Rzeszutek Wilk wrote:
> > > > Still having difficulty to reproduce can you reproduce with the attached
> > > > printk debuging patch and provide the log (only few printk preceding the
> > > > oops or segfault are interesting).
> > > 
> > > http://darnok.org/vga/move_notify-v212.log
> > > 
> > 
> > Looks like nouveau doesn't like move notify being call on driver
> > shutdown or when somethings om nv50 is down. Ben i think you will
> > be better at finding a fix for that than me.
> I'm also not able to reproduce this issue on a NV98 (so, i'd expect
> every nv50+ chipset to behave the same) chipset with the current code in
> Dave's drm-core-next tree..

I was using 3.2 and then merged drm-core-next tree on top of that.

> 
> Am I missing something?

I am using a stock Fedora 16 with X Server 1.11.2. Machine has 8GB,
and one DVI monitor and is an AMD box. The kernel was compiled
using the default Fedora Core .config and for any new options I just
hit enter.

Don't have the experimental libGL code, so using:
OpenGL version string: 2.1 Mesa 7.11.2

for the rendering. And the test setup is fairly easy - launch etracer,
switch to a FB VC (Ctrl-Alt-F2), login, find the etracer pid and run
perf --record --pid X and then switch back. Finish playing the game, exit
it and then switch to the FB VC to turn it off, and it happens.

Sometimes it happens when I just finished the game.

I also can reproduce this with an AGP card (GeForce 4 Ti4200?) on an
Intel Prescott box (2GB of memory) - also with stock Fedora 16.
Thought the crash is different:

http://darnok.org/vga/agp_nouveau_crash.jpg

Hmm, I can hook up a serial console to that box to get a better output - but
perhaps before I do that should is there a debug patch I should compile in?