[Nouveau] [libdrm PATCH 1/4] nouveau: add nouveau_resource_destroy

Christoph Bumiller e0425955 at student.tuwien.ac.at
Fri Jan 22 02:10:58 PST 2010


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
> Nouveau at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/nouveau
>   



More information about the Nouveau mailing list