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/
Home Products Support Community News


Re: [Xen-devel] regression with c/s 21223

To: Jim Fehlig <jfehlig@xxxxxxxxxx>
Subject: Re: [Xen-devel] regression with c/s 21223
From: Keir Fraser <keir.fraser@xxxxxxxxxxxxx>
Date: Fri, 7 May 2010 20:41:03 +0100
Cc: Xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxx>, Ryan O'Connor <rjo@xxxxxxxxx>
Delivery-date: Fri, 07 May 2010 12:41:48 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <4BE46A75.8010207@xxxxxxxxxx>
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>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AcruG9k2Lyl+/clKRNqxHgIIm71sSgAAWDro
Thread-topic: [Xen-devel] regression with c/s 21223
User-agent: Microsoft-Entourage/
On 07/05/2010 20:31, "Jim Fehlig" <jfehlig@xxxxxxxxxx> wrote:

>>  I suppose it really just punts
>> the tap2 issues, unless we also get rid of the tap2-falls-back-to-tap1
>> logic?
> It reverts 2 hunks of c/s 19874, which implicitly converts the device to
> tap2 in a xend client app!  If the same configuration is provided to
> xend through libvirt, this implicit conversion does not occur.  I'd
> suspect this is true for direct users of XenAPI as well.
> But yes, I agree that if an explicit differentiation between tap and
> tap2 exists, then the tap2-fall-back-to-tap1 logic should be removed.
> It would be nice to get input from others, particularly authors of
> blktap2 integration patches :-).

Well, in the absence of that I'm happy to take the patch, and I guess
backport it for 4.0.1.

 -- Keir

Xen-devel mailing list