Tilman Sauerbeck [2006-06-11 12:35]: > Tilman Sauerbeck [2006-05-22 19:42]: > > [...] > > > > I found out that the buffer in question was allocated by > > r300BufferData(). Now, the proper call to radeon_mm_free() would have > > been made by r300DeleteBuffer(), but that function was never called. > > > > From looking at the code I think this means that it's an application > > error. > > Now the question is, should Mesa call the "DeleteBuffer" callback for > > all buffers that are still alive when the context is destroyed or should > > r300 be able to cope with it the way it currently is? > > Here's a patch that deletes all VBOs that are still alive when the > context is destroyed. > > Because r300DeleteBuffer() calls radeon_mm_free(), which depends on > ctx->DriverCtx, we now cannot set DriverCtx to NULL before destroying > the Mesa context however. > > Is this a problem? There's lots of driver calls in free_share_state() > that might depend on DriverCtx not being NULL, so I don't think I'm > adding new evil code here.
Can anyone please comment on that patch? Thanks, Tilman -- A: Because it messes up the order in which people normally read text. Q: Why is top-posting such a bad thing? A: Top-posting. Q: What is the most annoying thing on usenet and in e-mail?
pgpSARcDlst60.pgp
Description: PGP signature
Using Tomcat but need to do more? Need to support web services, security? Get stuff done quickly with pre-integrated technology to make your job easier Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
-- _______________________________________________ Dri-devel mailing list Dri-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/dri-devel
