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] Partial fix for compat build non-portability

To: John Levon <levon@xxxxxxxxxxxxxxxxx>, Jan Beulich <jbeulich@xxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] Partial fix for compat build non-portability
From: Keir Fraser <keir@xxxxxxxxxxxxx>
Date: Wed, 10 Jan 2007 18:27:30 +0000
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Wed, 10 Jan 2007 10:27:15 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <20070110175045.GA31773@xxxxxxxxxxxxxxxxxxxxxxx>
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>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: Acc05PyxOx3uMqDYEduNDgAX8io7RQ==
Thread-topic: [Xen-devel] [PATCH] Partial fix for compat build non-portability
User-agent: Microsoft-Entourage/11.2.5.060620
On 10/1/07 17:50, "John Levon" <levon@xxxxxxxxxxxxxxxxx> wrote:

> On Wed, Jan 10, 2007 at 05:43:04PM +0000, Jan Beulich wrote:
> 
>> I certainly dislike this hunk - I intentionally didn't do it this way
> 
> globs in makefiles are a horrible idea.

Well, I would agree when it comes to filtered globs at least. If the file
list has particular constraints I think a white list is clearer than a
pattern-based black list.

While we're on the subject of the compat code, I wonder whether some of it
might not better belong under arch/x86 (and even arch/x86/x86_64). The
concept of what 'compat' means in an architecture-neutral sense isn't well
defined, I believe. For example, the particular structs and headers that
need special treatment will surely be different between platforms that
choose to use CONFIG_COMPAT (if indeed any other than pae-on-x86-64 ever
materialises, which is itself quite a doubtful assumption). Something to
think about at least...

 -- Keir


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