Re: [Xen-ia64-devel][PATCH] add one page attribute to indicate this page is physical IO page

2008-10-17 Thread Isaku Yamahata
On Fri, Oct 17, 2008 at 02:41:59PM +0800, Xu, Anthony wrote: > Isaku Yamahata wrote: > > Thanks for the explanation. > > I took a look at your big patch which you send privately. > > Please correct me if I'm wrong because I haven't reviewed > > it line by line. > > > > It seems like that _PAGE_DIRE

Re: [Xen-ia64-devel][PATCH]a virt_to_maddr fix

2008-10-17 Thread Isaku Yamahata
applied, thanks. On Thu, Oct 16, 2008 at 06:02:28PM +0800, Xu, Anthony wrote: > Putting IA64_MAX_PHYS_BITS to pgtable.h is to avoid compilation error. > > Anthony > > Isaku Yamahata wrote: > > Yes, IA64_MAX_PHYS_BITS makes sense. > > It seems better that the comment is in xenpage.h. > > > > Did

RE: [Xen-ia64-devel][PATCH][VTD]change ioports_permit_access interface

2008-10-17 Thread Xu, Anthony
Isaku Yamahata wrote: > On Thu, Oct 16, 2008 at 07:37:51PM +0800, Xu, Anthony wrote: >> use VTD to assing device, guest port may not be equal to host port. >> Change ioports_permit_access interface >> add deassign_domain_mmio_page interface >> >> Signed-off-by; Anthony Xu < [EMAIL PROTECTED] > > >

RE: [Xen-ia64-devel][PATCH] add one page attribute to indicate this page is physical IO page

2008-10-17 Thread Xu, Anthony
Isaku Yamahata wrote: > On Fri, Oct 17, 2008 at 02:41:59PM +0800, Xu, Anthony wrote: >> Isaku Yamahata wrote: >>> Thanks for the explanation. >>> I took a look at your big patch which you send privately. >>> Please correct me if I'm wrong because I haven't reviewed >>> it line by line. >>> >>> It s

Re: [Xen-ia64-devel][PATCH][VTD]change ioports_permit_access interface

2008-10-17 Thread Isaku Yamahata
On Fri, Oct 17, 2008 at 03:22:18PM +0800, Xu, Anthony wrote: > Isaku Yamahata wrote: > > On Thu, Oct 16, 2008 at 07:37:51PM +0800, Xu, Anthony wrote: > >> use VTD to assing device, guest port may not be equal to host port. > >> Change ioports_permit_access interface > >> add deassign_domain_mmio_pa

RE: [Xen-ia64-devel][PATCH][VTD]change ioports_permit_access interface

2008-10-17 Thread Xu, Anthony
Isaku Yamahata wrote: >> In x86 side, only it is identity mapping, guest can access the port >> directly. >> If it is not identity mapping, guest can not access the port >> directly, at this situation, xen will intercept io port access >> first, and xen access the corresponding physical port. >> >

[PATCH]RE: [Xen-ia64-devel][PATCH][VTD]change ioports_permit_access interface

2008-10-17 Thread Xu, Anthony
use VTD to assing device, guest port may not be equal to host port. Change ioports_permit_access interface Signed-off-by; Anthony Xu < [EMAIL PROTECTED] > Isaku Yamahata wrote: > On Fri, Oct 17, 2008 at 03:22:18PM +0800, Xu, Anthony wrote: >> Isaku Yamahata wrote: >>> On Thu, Oct 16, 2008 at 07

RE: [Xen-ia64-devel][PATCH][VTD]change ioports_permit_access interface

2008-10-17 Thread Xu, Anthony
Hi isaku, If I remembered correctly, last time I encountered issue at line page = mfn_to_page(mfn); Page returns 0. I suspect if page_info for IOPORTs is initialized. Any idea? Thanks, Anthony mm_teardown_pte(struct domain* d, volatile pte_t* pte, unsigned long offset) { pte_t old_pte;

Re: [Xen-ia64-devel][PATCH][VTD]change ioports_permit_access interface

2008-10-17 Thread Isaku Yamahata
On Fri, Oct 17, 2008 at 04:45:09PM +0800, Xu, Anthony wrote: > Hi isaku, > > If I remembered correctly, last time I encountered issue at line page = > mfn_to_page(mfn); > Page returns 0. Which line do you have trouble? Do you mean that mfn_to_page(mfn) returned NULL? or Did you hit BUG_ON(page_g

Re: [PATCH]RE: [Xen-ia64-devel][PATCH][VTD]change ioports_permit_access interface

2008-10-17 Thread Isaku Yamahata
Applied, thanks. On Fri, Oct 17, 2008 at 04:17:08PM +0800, Xu, Anthony wrote: > use VTD to assing device, guest port may not be equal to host port. > Change ioports_permit_access interface > > Signed-off-by; Anthony Xu < [EMAIL PROTECTED] > > > > > Isaku Yamahata wrote: > > On Fri, Oct 17, 200

[Xen-ia64-devel] [PATCH]clean up ioports_{permit, deny}_access()

2008-10-17 Thread Isaku Yamahata
[IA64] clean up ioports_{permit, deny}_access() eliminate code duplication in those functions by introducing helper functions. Signed-off-by: Isaku Yamahata <[EMAIL PROTECTED]> diff -r f27787b9f8d7 xen/arch/ia64/xen/mm.c --- a/xen/arch/ia64/xen/mm.cFri Oct 17 17:40:15 2008 +0900 +++ b/xen/a

RE: [Xen-ia64-devel][PATCH][VTD]change ioports_permit_access interface

2008-10-17 Thread Xu, Anthony
Isaku Yamahata wrote: > On Fri, Oct 17, 2008 at 05:58:45PM +0900, Isaku Yamahata wrote: >> On Fri, Oct 17, 2008 at 04:45:09PM +0800, Xu, Anthony wrote: >>> Hi isaku, >>> >>> If I remembered correctly, last time I encountered issue at line >>> page = mfn_to_page(mfn); Page returns 0. >> >> Which lin

[Xen-ia64-devel] [PATCH] clean up of xenpage.h

2008-10-17 Thread Isaku Yamahata
[IA64] clean up of xenpage.h remove nasty undefs in xenpage.h. Signed-off-by: Isaku Yamahata <[EMAIL PROTECTED]> diff -r 07558e1876e4 xen/include/asm-ia64/linux-xen/asm/page.h --- a/xen/include/asm-ia64/linux-xen/asm/page.h Fri Oct 17 18:16:05 2008 +0900 +++ b/xen/include/asm-ia64/linux-xen/asm/

Re: [Xen-ia64-devel][PATCH][VTD]change ioports_permit_access interface

2008-10-17 Thread Isaku Yamahata
On Fri, Oct 17, 2008 at 05:58:45PM +0900, Isaku Yamahata wrote: > On Fri, Oct 17, 2008 at 04:45:09PM +0800, Xu, Anthony wrote: > > Hi isaku, > > > > If I remembered correctly, last time I encountered issue at line page = > > mfn_to_page(mfn); > > Page returns 0. > > Which line do you have troubl

[Xen-ia64-devel] [IA64] Weekly benchmark results [ww41]

2008-10-17 Thread KUWAMURA Shin'ya
Hi, I report a benchmark result of this week on IPF using ia64/xen-unstable and ia64/linux-2.6.18-xen. All test cases passed. TEST ENVIRONMENT Machine : Tiger4 Kernel : 2.6.18.8-xen Changeset: 18629:9b227eb09263 (ia64/xen-unstable) 70

RE: [Xen-ia64-devel][PATCH][VTD]change ioports_permit_access interface

2008-10-17 Thread Xu, Anthony
Isaku Yamahata wrote: > On Fri, Oct 17, 2008 at 05:58:45PM +0900, Isaku Yamahata wrote: >> On Fri, Oct 17, 2008 at 04:45:09PM +0800, Xu, Anthony wrote: >>> Hi isaku, >>> >>> If I remembered correctly, last time I encountered issue at line >>> page = mfn_to_page(mfn); Page returns 0. >> >> Which lin

RE: [Xen-ia64-devel][PATCH][VTD]change ioports_permit_access interface

2008-10-17 Thread Xu, Anthony
Isaku Yamahata wrote: > On Fri, Oct 17, 2008 at 04:45:09PM +0800, Xu, Anthony wrote: >> Hi isaku, >> >> If I remembered correctly, last time I encountered issue at line >> page = mfn_to_page(mfn); Page returns 0. > > Which line do you have trouble? > Do you mean that mfn_to_page(mfn) returned NULL?

[Xen-ia64-devel] RE: [PATCH 00/33] ia64/xen domU take 12

2008-10-17 Thread Luck, Tony
> This patchset is ia64/xen domU patch take 12. > Tony, please commit those patches. Ok. Committed. -Tony ___ Xen-ia64-devel mailing list Xen-ia64-devel@lists.xensource.com http://lists.xensource.com/xen-ia64-devel