WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

[Xen-devel] Re: [Pkg-xen-devel] xen 4.1 blktap2 support

To: Ian Campbell <ijc@xxxxxxxxxxxxxx>
Subject: [Xen-devel] Re: [Pkg-xen-devel] xen 4.1 blktap2 support
From: Bastian Blank <waldi@xxxxxxxxxx>
Date: Fri, 29 Apr 2011 13:08:08 +0200
Cc: pkg-xen-devel@xxxxxxxxxxxxxxxxxxxxxxx, xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Fri, 29 Apr 2011 04:09:16 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <1304000458.25988.1731.camel@xxxxxxxxxxxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Mail-followup-to: Ian Campbell <ijc@xxxxxxxxxxxxxx>, xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxx>, pkg-xen-devel@xxxxxxxxxxxxxxxxxxxxxxx
References: <4DB82B38.6090207@xxxxxxxxx> <BAY148-w56B2B88110870F89DE84D4EF980@xxxxxxx> <4DB89A39.6070203@xxxxxxxxx> <BAY148-w282F83283CEBAD8F438117EF9B0@xxxxxxx> <4DB91947.2070203@xxxxxxxxx> <1303977639.25988.1447.camel@xxxxxxxxxxxxxxxxxxxxx> <20110428110658.GA26816@xxxxxxxxxxxxxxxxxxxxxxx> <1303990128.25988.1570.camel@xxxxxxxxxxxxxxxxxxxxx> <20110428122719.GA27897@xxxxxxxxxxxxxxxxxxxxxxx> <1304000458.25988.1731.camel@xxxxxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mutt/1.5.20 (2009-06-14)
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

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel