On Tue, Aug 08, 2006 at 11:52:41AM -0700, Yu, Fenghua wrote: > > Seems the following part is missed in your patch: > > --- arch/ia64/sn/kernel/xpc_main.c 2006-02-13 12:35:03.000000000 > -0800 > +++ arch/ia64/sn/kernel/xpc_main.c.new 2006-08-08 11:47:03.000000000 > -0700 > @@ -1374,6 +1374,7 @@ xpc_init(void) > complete(&xpc_discovery_exited); > > xpc_do_exit(xpcUnloading); > + kfree(xpc_remote_copy_buffer_base); > return -EBUSY; > } Actually, the very last line of xpc_do_exit() is the following: kfree(xpc_remote_copy_buffer_base); } I realize it's perhaps not the most straight-forward arrangement, but it does have something to do with the various avenues through which our exit processing can get invoked (not just by rmmod's call to xpc_exit()). I really do appreciate your feedback. Thanks. Dean - : send the line "unsubscribe linux-ia64" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
![]() |