From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bastian Blank Subject: Re: [Pkg-xen-devel] xen 4.1 blktap2 support Date: Fri, 29 Apr 2011 13:08:08 +0200 Message-ID: <20110429110808.GA14859@wavehammer.waldi.eu.org> References: <4DB82B38.6090207@gmail.com> <4DB89A39.6070203@gmail.com> <4DB91947.2070203@gmail.com> <1303977639.25988.1447.camel@localhost.localdomain> <20110428110658.GA26816@wavehammer.waldi.eu.org> <1303990128.25988.1570.camel@localhost.localdomain> <20110428122719.GA27897@wavehammer.waldi.eu.org> <1304000458.25988.1731.camel@localhost.localdomain> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Return-path: Content-Disposition: inline In-Reply-To: <1304000458.25988.1731.camel@localhost.localdomain> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: Ian Campbell Cc: pkg-xen-devel@lists.alioth.debian.org, xen-devel List-Id: xen-devel@lists.xenproject.org On Thu, Apr 28, 2011 at 03:20:58PM +0100, Ian Campbell wrote: > On Thu, 2011-04-28 at 14:27 +0200, Bastian Blank wrote: > > - Silent fail if qemu-dm[1] is missing or failing. It lacks error > > checking. > Yes, this could certainly be improved. A hang without any feedback is the worst user-experience possible. > > - Probably missing close-on-exit flags for several file handlers. > You mean close-on-exec? Yes. > the libxl interfaces for exec'ing takes care of closing file handles and It tries to, but it can't ever succeed (there is no practical upper limit for the fd value). Usually this is used to _hide_ other errors. > Do you know of specific instances where the CLOEXEC flag is needed but > missing? Not right now. But because of the hiding properties it is not possible to detect without changes anyway. > I don't think any of the above qualifies xl as so broken we shouldn't > even suggest people try it, as you started out by saying... Failing without feedback and only running in an environment not security supportable is broken. Bastian -- Without followers, evil cannot spread. -- Spock, "And The Children Shall Lead", stardate 5029.5