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] new -xen.c files in sparse tree

To: Keir Fraser <Keir.Fraser@xxxxxxxxxxxx>
Subject: Re: [Xen-devel] new -xen.c files in sparse tree
From: Horms <horms@xxxxxxxxxxxx>
Date: Fri, 9 Feb 2007 10:16:49 +0900
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx, Jan Beulich <jbeulich@xxxxxxxxxx>
Delivery-date: Thu, 08 Feb 2007 17:55:09 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <C1F0F295.91C2%Keir.Fraser@xxxxxxxxxxxx>
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/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <45CB07D8.76E4.0078.0@xxxxxxxxxx> <C1F0F295.91C2%Keir.Fraser@xxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: mutt-ng/devel-r804 (Debian)
On Thu, Feb 08, 2007 at 03:30:29PM +0000, Keir Fraser wrote:
> Empty files can certainly be removed. Where there's any diff at all, it's a
> matter of taste whether we have separate files or ifdef XEN.

IMHO there are two many ways of cooking this egg.

  Patch vs Sparse + ifdef XEN vs separate file

Finding the code can be kind of tricky,
and working out the best way to implement a change seems to be a black art.

Can't this be simplified?

(I'll refrain from saxing axe sparse... oops, I didn't)

-- 
Horms
  H: http://www.vergenet.net/~horms/
  W: http://www.valinux.co.jp/en/


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