On Tue, Dec 13, 2011 at 11:40:31AM -0500, Konrad Rzeszutek Wilk wrote: > On Tue, Dec 13, 2011 at 05:23:30PM +0100, Thomas Hellstrom wrote: > > On 12/13/2011 05:07 PM, Jerome Glisse wrote: > > >On Mon, Dec 12, 2011 at 03:09:26PM -0500, Konrad Rzeszutek Wilk wrote: > > >>Jerome pointed me to some accounting error in the DMA API debugging code and > > >>while I can't figure it out yet, I did notice some extreme slowness - which > > >>is due to the nouveau driver calling the unpopulate (now that unbind + > > >>unpopulate are squashed) quite a lot (this is using Gnome Shell - I think GNOME2 > > >>did not have those issues but I can't recall). > > >> > > >>Anyhow these patches fix the 50% perf regression I saw and also some minor bugs > > >>that I noticed. > > >> > > >Gonna review those today and test them. > > > > > >Cheers, > > >Jerome > > Hi! > > > > I'm not whether any drivers are still using the AGP backend? > > Uh, probably they do if the cards are AGP? > The problem I encountered was with an PCIe Nvidia card: > > 01:00.0 VGA compatible controller: nVidia Corporation G84 [GeForce 8600 GT] (rev a1 > > > Calling unpopulate / (previous clear) each time unbind is done > > should be quite > > inefficient with that one, as AGP sets up its own data structures > > and copies page tables > > on each populate. That should really be avoided unless there is a > > good reason to have it. > > nouveau_bo_rd32 and nv50_crtc_cursor_set showed up as the callers that > were causing the unpopulate calls. It did happen _a lot_ when I moved the > cursor madly. Looked at that and i am pretty sure that those function can't trigger unpopulate event (but i might have missed a path). I tested before and after the ttm change and those function are as time consuming as before. I think nouveau is doing something stupid in userspace when it comes to cursor (might be the ddx or just gnome3 or X server haven't tracked that down). Anyway, bottom line is i believe we don't call unpopulate more than before the ttm changes. It seems to be the same from code inspection, previously each time we called clear we also freed the pages so clear is equivalent to unpopulate. In theory we don't call unpopulate on each unbind but given the current user we do (only path were unbind is not followed by unpopulate is ttm_bo_move_ttm same as before the patch so no change there). So, Konrad patch to avoid calling set_pages_array_wb when the page is cached is a proper fix, this is what happened before, and this is what the ttm change broke. I believe there is no performance regression with this ttm change on radeon or nouveau (tested only with gnome3+nexuiz on nv50 and r600,r700,evergreen). Cheers, Jerome _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel
