Mailing List Archive

[Xen-merge] [PATCH] move i386's swiotlb.h
Subject says it all.
Re: [Xen-merge] [PATCH] move i386's swiotlb.h [ In reply to ]
On Fri, Dec 23, 2005 at 01:27:22PM +0100, Jan Beulich wrote:
> Subject says it all.

> Index: 2005-12-16/include/asm-i386/mach-xen/asm/swiotlb.h
> ===================================================================
> --- 2005-12-16.orig/include/asm-i386/mach-xen/asm/swiotlb.h 2005-11-21 16:42:07.000000000 +0100
> +++ /dev/null 1970-01-01 00:00:00.000000000 +0000

isn't the swiotlb was suppose to move to lib/ and be available to almost
all arch ? IIRC there was talk to do that on the LKML (well at least
powerpc is trying to add it).

Maybe we could just reorganize directly to this purpose and get rid of
swiotlb in our tree ?

--
Vincent Hanquez

_______________________________________________
Xen-merge mailing list
Xen-merge@lists.xensource.com
http://lists.xensource.com/xen-merge
Re: [Xen-merge] [PATCH] move i386's swiotlb.h [ In reply to ]
I don't know; if so, it'd have been nice to tell me when I submitted to
original merge proposal. And anyway, this can always be done as a second
step. My main goal right now is to get rid of the
include/asm-*/mach-xen/asm directories. Jan

>>> Vincent Hanquez <vincent.hanquez@cl.cam.ac.uk> 23.12.05 13:34:51
>>>
On Fri, Dec 23, 2005 at 01:27:22PM +0100, Jan Beulich wrote:
> Subject says it all.

> Index: 2005-12-16/include/asm-i386/mach-xen/asm/swiotlb.h
> ===================================================================
> ---
2005-12-16.orig/include/asm-i386/mach-xen/asm/swiotlb.h 2005-11-21
16:42:07.000000000 +0100
> +++ /dev/null 1970-01-01 00:00:00.000000000 +0000

isn't the swiotlb was suppose to move to lib/ and be available to
almost
all arch ? IIRC there was talk to do that on the LKML (well at least
powerpc is trying to add it).

Maybe we could just reorganize directly to this purpose and get rid of
swiotlb in our tree ?

--
Vincent Hanquez

_______________________________________________
Xen-merge mailing list
Xen-merge@lists.xensource.com
http://lists.xensource.com/xen-merge
Re: [Xen-merge] [PATCH] move i386's swiotlb.h [ In reply to ]
On Fri, Dec 23, 2005 at 01:43:35PM +0100, Jan Beulich wrote:
> I don't know; if so, it'd have been nice to tell me when I submitted to
> original merge proposal.

This is actually quite new, at least for the powerpc part.
(it's related to the BCM48xxx that is totally buggy with memory > 1GB.)

I think for x86_64, Andi did lots of works to merge ia64 and x86_64 file.
I'll have a look at the fact ..

> And anyway, this can always be done as a second
> step. My main goal right now is to get rid of the
> include/asm-*/mach-xen/asm directories.

sure.

--
Vincent Hanquez

_______________________________________________
Xen-merge mailing list
Xen-merge@lists.xensource.com
http://lists.xensource.com/xen-merge
RE: [Xen-merge] [PATCH] move i386's swiotlb.h [ In reply to ]
> isn't the swiotlb was suppose to move to lib/ and be
> available to almost all arch ? IIRC there was talk to do that
> on the LKML (well at least powerpc is trying to add it).
>
> Maybe we could just reorganize directly to this purpose and
> get rid of swiotlb in our tree ?

Our swiotlb is quite different from the original thesedays, so we'll
need our own for the foreseeable future.

Once we get the tree into a state we all agree on and start carving
things off in to patches I think we'll should prioritize the relatively
small subset that's required to make unprivileged linux guests work (we
can even leave some of the optimizations for later). Most of the patches
in the tree are required for building privilged guests capable of
running drivers, and we can leave them until last.

Ian

_______________________________________________
Xen-merge mailing list
Xen-merge@lists.xensource.com
http://lists.xensource.com/xen-merge
Re: [Xen-merge] [PATCH] move i386's swiotlb.h [ In reply to ]
On Fri, Dec 23, 2005 at 01:34:51PM +0100, Vincent Hanquez wrote:
> On Fri, Dec 23, 2005 at 01:27:22PM +0100, Jan Beulich wrote:
> > Subject says it all.
>
> > Index: 2005-12-16/include/asm-i386/mach-xen/asm/swiotlb.h
> > ===================================================================
> > --- 2005-12-16.orig/include/asm-i386/mach-xen/asm/swiotlb.h 2005-11-21 16:42:07.000000000 +0100
> > +++ /dev/null 1970-01-01 00:00:00.000000000 +0000
>
> isn't the swiotlb was suppose to move to lib/ and be available to almost
> all arch ? IIRC there was talk to do that on the LKML (well at least
> powerpc is trying to add it).

There are several relevant things in progress here:

- in 2.6.15, swiotlb.c is available under lib/ for both x86-64 and
IA64. Unfortunately Xen's swiotlb.c is quite different.

- in 2.6.16 (I hope), swiotlb.h will be moved to asm-generic. This is
a patch that's in -mm at the moment. There's no reason Xen couldn't
use this header file.

- there's a patch in Andi Kleen's queue to make multiple IOMMU's
usable for x86-64 relatively cleanly (the dma-mapping-ops patch). This
will make it slighly cleaner to support Xen's copy of swiotlb.c.

Cheers,
Muli
--
Muli Ben-Yehuda
http://www.mulix.org | http://mulix.livejournal.com/


_______________________________________________
Xen-merge mailing list
Xen-merge@lists.xensource.com
http://lists.xensource.com/xen-merge
RE: [Xen-merge] [PATCH] move i386's swiotlb.h [ In reply to ]
> There are several relevant things in progress here:
>
> - in 2.6.15, swiotlb.c is available under lib/ for both x86-64 and
> IA64. Unfortunately Xen's swiotlb.c is quite different.

As a relevant (to the CONFIG_xxx_XEN discussion) aside,
Xenlinux for ia64 uses Linux's normal swiotlb.c.

Dan

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