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

Re: [xen-devel][PATCH] PV driver compatibility

To: Ian Campbell <Ian.Campbell@xxxxxxxxxxxxx>
Subject: Re: [xen-devel][PATCH] PV driver compatibility
From: Keir Fraser <keir.fraser@xxxxxxxxxxxxx>
Date: Thu, 21 Jan 2010 21:46:56 +0000
Cc: ksrinivasan <ksrinivasan@xxxxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Thu, 21 Jan 2010 13:47:24 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <1264106300.21707.57.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>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: Acqa2a3ckXfYIqUnSDSLeX61b0LsKQACZJqZ
Thread-topic: [xen-devel][PATCH] PV driver compatibility
User-agent: Microsoft-Entourage/12.23.0.091001
On 21/01/2010 20:38, "Ian Campbell" <Ian.Campbell@xxxxxxxxxxxxx> wrote:

>> You introduced the magic port value check, in xen-unstable:19964.
> 
> I'm guilty of pretty poor changelogging there, aren't I, I've no idea
> how the unmodified drivers part of the change relates to the comment :-(

Yes, I wonder if it was even meant to be checked in. Or if it's an
accidental merge of two patches both of which you intended to go in.

I'm not sure unconditional unplug, as implemented, is a good idea. I can
imagine people with setups in which emulated devices coexist with pv
devices. Such a setup breaks if emulated devices all get unplugged when pv
drivers load. Presumably in the environment this patch came from (Citrix
XenServer) such a situation is disallowed, but I'm not so sure about
proscribing it more generally.

Some people do want unplug though, so I think making it a non-default module
option is a good idea.

 -- Keir



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