2010/3/29 Kristian Høgsberg <k...@bitplanet.net>: > On Fri, Mar 26, 2010 at 7:07 PM, Jesse Barnes <jbar...@virtuousgeek.org> > wrote: >> Drivers may use vblank calls now (e.g. drm_vblank_off) in their unload >> paths, so don't clean up the vblank related structures until after >> driver unload. > > I haven't tested this specific patch on a recent DRM, but I made the > same patch a while ago, and it fixed module unload for me. I sent it > to the list and it fell through the cracks, because "vblank is hard" > or something. > > Reviewed-by: Kristian Høgsberg <k...@bitplanet.net>
I didn't apply it because from what I can see non-kms drivers need to free the vbl stuff on lastclose not unload. I'm nearly sure I said that at the time to. This patch seems to suffer from the same problem from what I can see. And really someone should already have cleaned up the insanity that is vbl struct allocations, vblank might not be hard, but it sure has some ugly. Dave. ------------------------------------------------------------------------------ Download Intel® Parallel Studio Eval Try the new software tools for yourself. Speed compiling, find bugs proactively, and fine-tune applications for parallel performance. See why Intel Parallel Studio got high marks during beta. http://p.sf.net/sfu/intel-sw-dev -- _______________________________________________ Dri-devel mailing list Dri-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/dri-devel
