From: Kevin Wolf <kwolf@suse.de>
To: Ian Jackson <Ian.Jackson@eu.citrix.com>
Cc: xen-devel@lists.xensource.com
Subject: Re: [RFC][PATCH] Use ioemu block drivers through blktap
Date: Mon, 31 Mar 2008 15:37:00 +0200 [thread overview]
Message-ID: <47F0E8FC.6020405@suse.de> (raw)
In-Reply-To: <18416.58810.250091.262660@mariner.uk.xensource.com>
Ian Jackson schrieb:
> Yes, but to avoid gradually accumulating an ever increasing amount of
> cruft, that cleanup mechanism ought to be capable of cleaning up even
> `lost' garbage of one kind or another. And once you've got such a
> garbage collecting cleanup arrangement, you don't need any separate
> atexit-like arrangement which cleans up specifically after the current
> run.
Well then, feel free to implement it. ;-)
> It is best just to run the garbage collector at a suitable point. Do
> you have a garbage collector which deletes pipes from stale domains ?
Not directly, no. When a domain gets the same domain ID, the pipe is
reused and then freed when the domain is destroyed. But you don't want
to reboot your Dom0 too often, so you won't get the same domain ID too
early...
> OTOH if you write special-case code like this then you have the costs
> of writing extra code. Code costs effort to maintain and typically
> has a nonzero bug rate, causing actual problems when it goes wrong.
If you think we were better off just dropping this code, for my part we
can do so. I just thought I'd port also the shutdown handlers from
tapdisk while I'm at it.
Kevin
prev parent reply other threads:[~2008-03-31 13:37 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-03-10 17:03 [RFC][PATCH] Use ioemu block drivers through blktap Kevin Wolf
2008-03-13 12:25 ` Kevin Wolf
2008-03-13 14:21 ` Konrad Rzeszutek
2008-03-14 9:37 ` Kevin Wolf
2008-03-17 14:12 ` Konrad Rzeszutek
2008-03-28 15:22 ` Ian Jackson
2008-03-31 9:10 ` Kevin Wolf
2008-03-31 9:36 ` Ian Jackson
2008-03-31 11:11 ` Kevin Wolf
2008-03-31 13:23 ` Ian Jackson
2008-03-31 13:31 ` Keir Fraser
2008-03-31 13:37 ` Ian Jackson
2008-03-31 13:37 ` Kevin Wolf [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=47F0E8FC.6020405@suse.de \
--to=kwolf@suse.de \
--cc=Ian.Jackson@eu.citrix.com \
--cc=xen-devel@lists.xensource.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.