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/

Home

xen-merge

[Top] [All Lists]

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

from [Jan Beulich]

[Permanent Link][Original]

To:	"Vincent Hanquez" <vincent.hanquez@xxxxxxxxxxxx>
Subject:	Re: [Xen-merge] [PATCH] merge i386's agp.h
From:	"Jan Beulich" <JBeulich@xxxxxxxxxx>
Date:	Wed, 21 Dec 2005 08:31:24 +0100
Cc:	xen-merge@xxxxxxxxxxxxxxxxxxx
Delivery-date:	Wed, 21 Dec 2005 07:33:19 +0000
Envelope-to:	www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to:	<20051220180137.GA13192@xxxxxxxxx>
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> <20051220180137.GA13192@xxxxxxxxx>
Sender:	xen-merge-bounces@xxxxxxxxxxxxxxxxxxx

>>> Vincent Hanquez <vincent.hanquez@xxxxxxxxxxxx> 20.12.05 19:01 >>>
>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 ...

I know.

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

That is what I'm aiming towards. If more splitting is needed, this can
always be done in a second round. But we should get started somehow...

>[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 ..

Jan

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

[More with this subject...]

<Prev in Thread]	Current Thread	[Next in Thread>
[Xen-merge] [PATCH] merge i386's highmem.h, (continued) [Xen-merge] [PATCH] merge i386's highmem.h, Jan Beulich [Xen-merge] [PATCH] merge i386's hw_irq.h, Jan Beulich [Xen-merge] [PATCH] move i386's hypercall.h, Jan Beulich [Xen-merge] [PATCH] move i386's hypervisor.h, Jan Beulich [Xen-merge] [PATCH] move i386's swiotlb.h, Jan Beulich Re: [Xen-merge] [PATCH] move i386's swiotlb.h, Vincent Hanquez Re: [Xen-merge] [PATCH] move i386's swiotlb.h, Jan Beulich Re: [Xen-merge] [PATCH] move i386's swiotlb.h, Vincent Hanquez Re: [Xen-merge] [PATCH] move i386's swiotlb.h, Muli Ben-Yehuda Re: [Xen-merge] [PATCH] merge i386's agp.h, Vincent Hanquez Re: [Xen-merge] [PATCH] merge i386's agp.h, Jan Beulich <= [Xen-merge] [PATCH] merge i386's kmap_types.h, Jan Beulich [Xen-merge] [PATCH] merge i386's mmu.h, Jan Beulich [Xen-merge] [PATCH] merge i386's mmu_context.h, Jan Beulich [Xen-merge] [PATCH] merge i386's pci.h, Jan Beulich [Xen-merge] [PATCH] merge i386's io.h, Jan Beulich [Xen-merge] [PATCH] merge i386's page.h, Jan Beulich *[Xen-merge] [PATCH] merge i386's pgtable.h*, Jan Beulich* [Xen-merge] [PATCH] merge i386's segment.h, Jan Beulich [Xen-merge] [PATCH] merge i386's processor.h, Jan Beulich [Xen-merge] [PATCH] merge i386's system.h, Jan Beulich

Previous by Date:	RE: [Xen-merge] disallow CONFIG_PCI_MSI, Nakajima, Jun
Next by Date:	[Xen-merge] [PATCH] fix boot image generation, Jan Beulich
Previous by Thread:	Re: [Xen-merge] [PATCH] merge i386's agp.h, Vincent Hanquez
Next by Thread:	[Xen-merge] [PATCH] merge i386's kmap_types.h, Jan Beulich
Indexes:	[Date] [Thread] [Top] [All Lists]

This site is hosted by Citrix