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] xen: make (set|get)_xen_guest_handle available

To: Keir Fraser <keir.fraser@xxxxxxxxxxxxx>, Christoph Egger <Christoph.Egger@xxxxxxx>
Subject: Re: [Xen-devel] [PATCH] xen: make (set|get)_xen_guest_handle available
From: Boris Derzhavets <bderzhavets@xxxxxxxxx>
Date: Thu, 30 Jul 2009 04:16:00 -0700 (PDT)
Cc: "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Thu, 30 Jul 2009 04:16:28 -0700
Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=yahoo.com; s=s1024; t=1248952560; bh=r8h8p+CM+eQPKmqZNCvXsUDG5ox9H2YF8ttchlodgDc=; h=Message-ID:X-YMail-OSG:Received:X-Mailer:Date:From:Subject:To:Cc:In-Reply-To:MIME-Version:Content-Type; b=Z84uTfweFX8RRzrQrZcsqrp7fe01ITYbrm/7PKUGrfqFoAgH+Mn4oNR3YwsB4g2RiQCainReVmA/FOLbkTjf+JawvTlHcXbGwu2OmNByxXiSBWlD9cXIr/pps8xhf1gk9WTwXxyjREyjsBlyl71jvH9ZZDw5+ErVbPvaVSPWHTg=
Domainkey-signature: a=rsa-sha1; q=dns; c=nofws; s=s1024; d=yahoo.com; h=Message-ID:X-YMail-OSG:Received:X-Mailer:Date:From:Subject:To:Cc:In-Reply-To:MIME-Version:Content-Type; b=whR6sRbRW/ZoVVttTvOUBSP2jJpsaqzBn6JfCUQ+gW5W814mzCcAhWyUJcxb5DmQMRx6rXJQ7nUKo6JMGK0fQ9YiV208D0yG+EfzZ4KBoiy5/jJElH+Yo7tebUe7JayqPC4WpyqLGaIVo/6xPhxObNiigiSACQIjDgRvOJHHkik=;
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <200907301234.38307.Christoph.Egger@xxxxxxx>
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
May the recent version of NetBSD run as PV guest ?
Sorry, for stupid question.
Boris.

--- On Thu, 7/30/09, Christoph Egger <Christoph.Egger@xxxxxxx> wrote:

From: Christoph Egger <Christoph.Egger@xxxxxxx>
Subject: Re: [Xen-devel] [PATCH] xen: make (set|get)_xen_guest_handle available
To: "Keir Fraser" <keir.fraser@xxxxxxxxxxxxx>
Cc: "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Date: Thursday, July 30, 2009, 6:34 AM

On Thursday 30 July 2009 12:14:04 Keir Fraser wrote:
> On 30/07/2009 10:12, "Christoph Egger" <Christoph.Egger@xxxxxxx> wrote:
> > Make (set|get)_xen_guest_handle() always available.
> > This avoids code snippets in the guest like this:
> >
> > #if __XEN_ITNERFACE_VERSION__ >= 0x00030201
> > set_xen_guest_handle(hnd, val);
> > #else
> > hnd = val;
> > #endif
>
> Actually I don't see why you'd have code like this. *You*, the guest, gets
> to specify __XEN_INTERFACE_VERSION__ -- that is the point of it -- so
> ifdef'ing based on it in the guest is stupid.
>
> Just specify __XEN_INTERFACE_VERSION__ to be recent enough to have the
> guest handles, and then remove your ifdefs. Simple.

The point is to keep backward compatibility. The patch makes Xen more friendly
with using the old non-structured guest handler if the user wants.

NetBSD offers both way to build & run it with either using the old or new
interface. This allows a smooth migration.

Christoph


--
---to satisfy European Law for business letters:
Advanced Micro Devices GmbH
Karl-Hammerschmidt-Str. 34, 85609 Dornach b. Muenchen
Geschaeftsfuehrer: Thomas M. McCoy, Giuliano Meroni
Sitz: Dornach, Gemeinde Aschheim, Landkreis Muenchen
Registergericht Muenchen, HRB Nr. 43632


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

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