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-merge

Re: [Xen-merge] [PATCH] merge i386's agp.h

To: Jan Beulich <JBeulich@xxxxxxxxxx>
Subject: Re: [Xen-merge] [PATCH] merge i386's agp.h
From: Vincent Hanquez <vincent.hanquez@xxxxxxxxxxxx>
Date: Tue, 20 Dec 2005 19:01:37 +0100
Cc: xen-merge@xxxxxxxxxxxxxxxxxxx
Delivery-date: Tue, 20 Dec 2005 18:04:12 +0000
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <43A81FF4.76F0.0078.0@xxxxxxxxxx>
List-help: <mailto:xen-merge-request@lists.xensource.com?subject=help>
List-id: xen-merge <xen-merge.lists.xensource.com>
List-post: <mailto:xen-merge@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-merge>, <mailto:xen-merge-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-merge>, <mailto:xen-merge-request@lists.xensource.com?subject=unsubscribe>
References: <43A81FF4.76F0.0078.0@xxxxxxxxxx>
Sender: xen-merge-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mutt/1.5.9i
On Tue, Dec 20, 2005 at 03:15:00PM +0100, Jan Beulich wrote:
> Subject says it all.

I'm not sure that is the right approch. Adding ifdef all over [1]
has never been well accepted by kernel people ...

But it look backwards from Chris' approch to split files into mach and
generic (especially floppy.h)
(which is, imho not nicer on a certain POV.)

Maybe a mix of the two would be the `cleanest' [2] solution.

[1] there's not that much ifdef so "all over" is a bit strong here ...
[2] I'm afraid there's no such thing here ..
-- 
Vincent Hanquez

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