On Fri, Jan 6, 2012 at 9:57 AM, Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx> wrote: > On Thu, Jan 05, 2012 at 09:14:10PM -0500, Konrad Rzeszutek Wilk wrote: >> On Fri, Jan 06, 2012 at 07:53:13AM +1000, Ben Skeggs wrote: >> > On Thu, 2012-01-05 at 13:31 -0500, j.glisse@xxxxxxxxx wrote: >> > > From: Jerome Glisse <jglisse@xxxxxxxxxx> >> > > >> > > ttm might call the move notify with null new mem placement, >> > > properly handle this case inside nouveau move notify callback. >> > This has been fixed already in a -next tree I sent to Dave. >> >> I just tried -next with your patch (and two other fixes that I had sent): >> >> drm/ttm/dma: Only call set_pages_array_wb when the page is not in WB pool >> drm/ttm/dma: Fix accounting error when calling ttm_mem_global_free_page and don't try to free freed pages >> >> and Jerome's AGP fix: >> ttm: fix agp since ttm tt rework >> >> and got the crash (but only with NVidia cards) after swapping between Xorg and the VCs. >> Look in drm-next.jpg > > http://darnok.org/vga/drm-next.jpg > >> >> With your patch removed ("drm/nouveau/ttm: fix crash as a result of a recent ttm change") >> and the patch below by Jerome I still get it to crash (see drm-next-with-Jerome-fix-revert-Ben.jpg).. > > http://darnok.org/vga/drm-next-with-Jerome-fix-revert-Ben.jpg > Anything special to trigger it ? I can't trigger it with simple gnome3 session (firefox evince ...) Cheers, Jerome _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel
