On 21.01.2010 22:33, Marcin Slusarz wrote: > On Fri, Jan 15, 2010 at 12:02:25AM +0100, Maarten Maathuis wrote: > >> Is everyone ok with this api addition (naming ok?)? >> > I guess they are... > > ... Or is there any issue with this patch I'm not aware of? > > Marcin > > In principle I guess we could also make nouveau_resource_free FREE the resource if (!next && !prev && !in_use), but I prefer the extra function.
Also, resource_destroy could iterate through all resources still linked with it and FREE them too, but that would probably not be necessary since all alloc'd ones should already have been released (but would produce a deserved segfault otherwise, if they're still used). If no one speaks in favour of changing anything though, let's push this as is (?) ... Christoph > _______________________________________________ > Nouveau mailing list > [email protected] > http://lists.freedesktop.org/mailman/listinfo/nouveau > _______________________________________________ Nouveau mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/nouveau
