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-ia64-devel] [patch 1/5] sn2+machvec support

To: Alex Williamson <alex.williamson@xxxxxx>
Subject: Re: [Xen-ia64-devel] [patch 1/5] sn2+machvec support
From: Jes Sorensen <jes@xxxxxxx>
Date: Thu, 14 Dec 2006 10:53:07 +0100
Cc: xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Thu, 14 Dec 2006 01:53:07 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <1166055164.5653.36.camel@lappy>
List-help: <mailto:xen-ia64-devel-request@lists.xensource.com?subject=help>
List-id: Discussion of the ia64 port of Xen <xen-ia64-devel.lists.xensource.com>
List-post: <mailto:xen-ia64-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-ia64-devel>, <mailto:xen-ia64-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-ia64-devel>, <mailto:xen-ia64-devel-request@lists.xensource.com?subject=unsubscribe>
References: <45803465.1010703@xxxxxxx> <1166055164.5653.36.camel@lappy>
Sender: xen-ia64-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Thunderbird (X11/20060527)
Alex Williamson wrote:
> Hi Jes,
>    Thanks for the cleanup.  I've added most of these to the mercurial
> tree here:
> http://free.linux.hp.com/~awilliam/xen-ia64-machvec/
> I found a couple more README.origin files that needed updating and
> reworked parts of patch 4 (adding lots more #ifdef XEN where code was
> simply added or removed before).  Overall, it looks pretty good.  The
> things not in this tree are the updates to sn_sal.h and list.h.
>    The sn_sal.h in the patch looked like a mix of various upstream
> versions of the file.  I re-based sn_sal.h in the tree above to 2.6.19,
> please re-implement the Xen fixes against that version.

Thats why I didn't add sn_sal.h to the README.origin since it was unclear

>    The Xen list.h doesn't seem to cleanly match any upstream Linux base,
> which makes me suspicious that it's had functionality intentionally
> pruned out of it.  Did you figure out what it's currently based on?  I'm
> wondering if we should try to only include the subset of functionality
> we need rather than move the whole file forward.  It's probably worth a
> discussion on xen-devel to make sure we don't get any surprises when we
> try to merge it upstream.  Thanks,
>       Alex

Xen-ia64-devel mailing list