From mboxrd@z Thu Jan 1 00:00:00 1970 From: "James (song wei)" Subject: Re: earlier remove the backend of tapdisk device in xenstore to release the resource allocated in backend driver lies in dom0'kernel Date: Thu, 22 Apr 2010 18:48:56 -0700 (PDT) Message-ID: <28336546.post@talk.nabble.com> References: <28325456.post@talk.nabble.com> <4BD04B87020000780003B5F4@vpn.id2.novell.com> <4BD083E1.10509@novell.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <4BD083E1.10509@novell.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: xen-devel@lists.xensource.com List-Id: xen-devel@lists.xenproject.org Jim Fehlig wrote: > >>Agreed. qemu-dm should clean up these resources on shutdown. AFAICT, >>it currently relies on receiving CTRMSG_CLOSE from blktapctrl (see >>ioemu-dir/hw/xen_blktap.c), which it may never receive before exiting. > > No, I don't think so. The root cause in this issue is qemu-dm exit too > earlier. > Qemu-dm has been destroyed before blktapctrl send "CTRMSG_CLOSE", > moreover, removing the backend node in xenstore trigers the callback in > blktapctrl to send "CTRMSG_CLOSE". > It's xend to decide who ought to be occured earlier between sending SIGHUP > to qemu-dm and removing the backend node in xenstore. Obviously, xend is > the role of manager. So I think fix this issue in xend is appropriate. > In addition, If we really want to fix it in qemu, we have to intercept > the exit signal sent by kill() in xend. I don't think intercept this > singal in qemu for this point is proper. > > -Jame (Song Wei) > > > _______________________________________________ > Xen-devel mailing list > Xen-devel@lists.xensource.com > http://lists.xensource.com/xen-devel > > -- View this message in context: http://old.nabble.com/earlier-remove-the-backend-of-tapdisk-device-in-xenstore-to-release-the-resource-allocated-in-backend-driver-lies-in-dom0%27kernel-tp28325456p28336546.html Sent from the Xen - Dev mailing list archive at Nabble.com.