On Wed, Jan 18, 2006 at 12:53:54PM +0100, Blaisorblade wrote: > On Wednesday 18 January 2006 02:12, Greg KH wrote: > > > From: Jeff Dike <[EMAIL PROTECTED]>, Paolo 'Blaisorblade' Giarrusso > > > <[EMAIL PROTECTED]> > > > > > > Define a release method for the ubd and network driver so that sysfs > > > doesn't complain when one is removed via: > > > What? No. The kernel is complaining for a reason, don't try to > > out-smart it. > > I'm not trying to ignore the warning.
By providing a release function that does nothing, all you are doing is shutting the kernel up. You are not doing anything to "fix" the real problem at all. > > > host $ uml_mconsole <umid> remove <dev> > > > > Done by Jeff around January for ubd only, later lost, then restored in > > > his tree - however I'm merging it now since there's no reason to leave > > > this here. > > > > We don't need to do any cleanup in the new added method, because when > > > hot-unplug is done by uml_mconsole we already handle cleanup in mconsole > > > infrastructure, i.e. mc_device->remove (net_remove/ubd_remove), which is > > > also the calling method. > > > Huh? You have 2 different release functions for the same object? > > Not sure which ones you refer. net_remove and ubd_remove are for different > devices; mc_device->remove and the sysfs release are in different layers and > for different things. Ok, but you are still adding 2 empty release functions. Please do not do that, if you think you need this, your code logic is wrong. As for your specific bug report, sorry, I really don't know. good luck, greg k-h ------------------------------------------------------- This SF.net email is sponsored by: Splunk Inc. Do you grep through log files for problems? Stop! Download the new AJAX search engine that makes searching your log files as easy as surfing the web. DOWNLOAD SPLUNK! http://sel.as-us.falkag.net/sel?cmd=lnk&kid=103432&bid=230486&dat=121642 _______________________________________________ User-mode-linux-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel
