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: [RFC] tmem ABI change... backwards compatibility unneces

To: Jan Beulich <JBeulich@xxxxxxxxxx>
Subject: [Xen-devel] RE: [RFC] tmem ABI change... backwards compatibility unnecessary?
From: Dan Magenheimer <dan.magenheimer@xxxxxxxxxx>
Date: Thu, 2 Sep 2010 16:19:52 -0700 (PDT)
Cc: JeremyFitzhardinge <jeremy@xxxxxxxx>, "Xen-Devel \(xen-devel@xxxxxxxxxxxxxxxxxxx\)" <xen-devel@xxxxxxxxxxxxxxxxxxx>, tmem-devel@xxxxxxxxxxxxxx, Kurt Hackel <kurt.hackel@xxxxxxxxxx>, Vasiliy G Tolstov <v.tolstov@xxxxxxxxx>, Keir Fraser <keir.fraser@xxxxxxxxxxxxx>, stephen.spector@xxxxxxxxxx, Chris Mason <chris.mason@xxxxxxxxxx>
Delivery-date: Thu, 02 Sep 2010 16:22:14 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <4C7E87970200007800013C2C@xxxxxxxxxxxxxxxxxx>
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>
References: <1e601c02-1f50-4396-b4d1-e1e21ebf3dc8@default 4C7E87970200007800013C2C@xxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
> From: Jan Beulich [mailto:JBeulich@xxxxxxxxxx]
> Sent: Wednesday, September 01, 2010 9:04 AM
> To: Dan Magenheimer
> Cc: stephen.spector@xxxxxxxxxx; Keir Fraser; JeremyFitzhardinge; Xen-
> Devel (xen-devel@xxxxxxxxxxxxxxxxxxx); Chris Mason; Kurt Hackel; tmem-
> devel@xxxxxxxxxxxxxx; Vasiliy G Tolstov
> Subject: Re: [RFC] tmem ABI change... backwards compatibility
> unnecessary?
> 
> >>> On 01.09.10 at 16:36, Dan Magenheimer <dan.magenheimer@xxxxxxxxxx>
> wrote:
> > I *think* it is still the case that tmem is experimental
> > and is not used anywhere yet in production.  If I am
> > wrong, PLEASE LET ME KNOW ASAP.
> 
> Well, if you call us shipping it (default disabled) in a couple of
> releases "not used in production"...
> 
> > I am inclined to update the Xen tmem implementation
> > to only support v1 and gracefully fail v0.
> 
> If "graceful" really means what it says, this would appear to be
> acceptable irrespective of my note above.

OK, I will submit a patch tomorrow with the following characteristics:

v0 (current) hypervisor + v0 guest: succeeds
v1 (patched) hypervisor + v1 guest: succeeds
v0 (current) hypervisor + v1 guest: fails
v1 (patched) hypervisor + v0 guest: fails

where fails is an xm dmesg message that says "unsupported
spec version" when the guest attempts to create a pool.
And pool creation failure ensures that all further tmem
operations also fail (indeed never even result in a
hypercall for most tmem-enabled kernels).

Thank goodness ABI versioning was built into tmem from
the beginning!

Dan

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