Re: [Xen-ia64-devel] save/restore clean up and related bugs

2007-05-08 Thread Isaku Yamahata
On Mon, May 07, 2007 at 06:09:04PM -0600, Alex Williamson wrote: I still see these pretty regularly after a save/restore (running ia64 cset 15029). I'm testing with a 16G 2-way dom0 and a 4-way 4G domU using an lvm disk. I boot up the domU, save, restore, then an 'ls' in the domU usually

Re: [Xen-ia64-devel] [Patch] cleanup warning of UC|WB attributepage

2007-05-08 Thread Akio Takebe
I remade the patch as efi_ucwb() don't check EFI Mermoy Type. If we choice this approch, please apply it. Signed-off-by: Akio Takebe [EMAIL PROTECTED] Best Regards, Akio Takebe cleanup_warning_efi_uc2wb.v3.patch Description: Binary data ___

[Xen-ia64-devel] Xen/IA64 Healthiness Report -Cset#15029

2007-05-08 Thread You, Yongkang
Xen/IA64 Healthiness Report No issue is found in this changeset. Testing Environment: Platform: Tiger4 Processor: Itanium 2 Processor Logic Processors number: 8 (2 processors with Due Core) PAL version: 8.47 Service OS: RHEL4u3 IA64 SMP

Re: [Xen-ia64-devel] [PATCH] make p2m exposure compatible with suspend/resume

2007-05-08 Thread Isaku Yamahata
The previous patch was imcomplete. Please replace it with this one. Sorry for confusion. On Mon, May 07, 2007 at 03:03:50PM +0900, Isaku Yamahata wrote: This patch depends on the patch I sent to xen-devel [PATCH] add two arch hooks xen_pre_suspend() and xen_post_suspend() for suspend/resume.

[Xen-ia64-devel] Re: PATCH: vpsr handling cleanup

2007-05-08 Thread Tristan Gingold
On Mon, May 07, 2007 at 03:35:40PM -0600, Alex Williamson wrote: On Mon, 2007-05-07 at 06:32 +0200, Tristan Gingold wrote: Hi, this is a noop patch: just create a function which really returns the virtual psr and create a new function to set psr for delivering an interrupt. Hi

[Xen-ia64-devel] [PATCH] code clean up using xen machine vector.

2007-05-08 Thread Isaku Yamahata
code clean up using xen machine vector. -- yamahata # HG changeset patch # User [EMAIL PROTECTED] # Date 1178619507 -32400 # Node ID 1c42e2281e1679d7cd662ecbeb5d929967a2d24a # Parent 89b0cc16c053c917adf6219d55356f969b029a38 code clean up using xen machine vector. PATCHNAME:

[Xen-ia64-devel] Re: [Xen-devel] [RFC] [0/4] User-space grants for Console and XenStore

2007-05-08 Thread Isaku Yamahata
On Tue, May 08, 2007 at 03:40:41PM +0100, Derek Murray wrote: On 7 May 2007, at 09:46, Isaku Yamahata wrote: On Wed, May 02, 2007 at 12:15:18PM +0100, Derek Murray wrote: * Will this work on ia64 and PowerPC? With the attached patches, I was able to boot dom0 and create/ destroy domU.

[Xen-ia64-devel] PATCH: cleanup in faults.c

2007-05-08 Thread Tristan Gingold
Hi, a small cleanup pass. Checked by compiling only. Tristan. # HG changeset patch # User Tristan Gingold [EMAIL PROTECTED] # Date 1178629671 -7200 # Node ID 8519e5db6510a066788bb525a17588bad3674203 # Parent d1ce60b8070f640408c702f1fbbef0f6ffda8586 Cleanup: more static added, unused functions

Re: [Xen-ia64-devel] [Patch] cleanup warning of UC|WB attributepage

2007-05-08 Thread Alex Williamson
On Tue, 2007-05-08 at 09:58 +0900, Akio Takebe wrote: BTW, I also thought another approch to prevent these case. Because these memory area is accessed as both UC and WB, we need to use new flag. (For example _PAGE_MA_UCE, I'm not sure we can use this flag for UC|WB. or should we make new

Re: [Xen-ia64-devel] [PATCH] make p2m exposure compatible with suspend/resume

2007-05-08 Thread Tristan Gingold
On Tue, May 08, 2007 at 03:41:19PM +0900, Isaku Yamahata wrote: The previous patch was imcomplete. Please replace it with this one. Hi Isaku, you should really define the TR used by the p2m in kregs.h. Using a define is less error-prone than a hard coded number (0x2). (I don't object against

[Xen-ia64-devel] PATCH: define guest_mode (instead of user_mode)

2007-05-08 Thread Tristan Gingold
Hi, subject says all; xen-x86 already defines this macro. Tested only by compilation. Tristan. # HG changeset patch # User Tristan Gingold [EMAIL PROTECTED] # Date 1178630697 -7200 # Node ID 8e5083feaa526653ae9b95b1d4117eb8066bbf35 # Parent 8519e5db6510a066788bb525a17588bad3674203 Defined

[Xen-ia64-devel] psr.sp/is/di/si virtualization

2007-05-08 Thread Tristan Gingold
Hi, these psr bits are not fully virtualized and the current usage is not clear (at least to me): sp (secure performance monitor): I don't know how it is currently used by xenoprofile. Isaku have you any tips on it ? di (disable instruction set transition): should it be hard-coded to 1 or

Re: [Xen-ia64-devel] PATCH: define guest_mode (instead of user_mode)

2007-05-08 Thread Alex Williamson
On Tue, 2007-05-08 at 15:26 +0200, Tristan Gingold wrote: diff -r 8519e5db6510 -r 8e5083feaa52 xen/arch/ia64/vmx/vmx_process.c --- a/xen/arch/ia64/vmx/vmx_process.c Tue May 08 15:07:51 2007 +0200 +++ b/xen/arch/ia64/vmx/vmx_process.c Tue May 08 15:24:57 2007 +0200 @@ -164,7 +164,7 @@

Re: [Xen-ia64-devel] [PATCH] code clean up using xen machine vector.

2007-05-08 Thread Alex Williamson
On Tue, 2007-05-08 at 19:25 +0900, Isaku Yamahata wrote: code clean up using xen machine vector. Nice cleanup, that machine vector is working out already. Thanks, Alex -- Alex Williamson HP Open Source Linux Org.

[Xen-ia64-devel] Re: PATCH: cleanup in faults.c

2007-05-08 Thread Alex Williamson
On Tue, 2007-05-08 at 15:08 +0200, Tristan Gingold wrote: Hi, a small cleanup pass. Checked by compiling only. Applied. Thanks, Alex -- Alex Williamson HP Open Source Linux Org. ___ Xen-ia64-devel

Re: [Xen-ia64-devel] [Patch] cleanup warning of UC|WB attributepage

2007-05-08 Thread Akio Takebe
Hi, Alex On Tue, 2007-05-08 at 09:58 +0900, Akio Takebe wrote: BTW, I also thought another approch to prevent these case. Because these memory area is accessed as both UC and WB, we need to use new flag. (For example _PAGE_MA_UCE, I'm not sure we can use this flag for UC|WB. or should we

[Xen-ia64-devel] Xen/IA64 Healthiness Report -Cset#15044

2007-05-08 Thread You, Yongkang
Xen/IA64 Healthiness Report Nightly testing doesn't find any issue. Testing Environment: Platform: Tiger4 Processor: Itanium 2 Processor Logic Processors number: 8 (2 processors with Due Core and HT) PAL version: 8.47 Service OS: RHEL4u3

Re: [Xen-ia64-devel] psr.sp/is/di/si virtualization

2007-05-08 Thread Isaku Yamahata
On Tue, May 08, 2007 at 03:38:38PM +0200, Tristan Gingold wrote: these psr bits are not fully virtualized and the current usage is not clear (at least to me): sp (secure performance monitor): I don't know how it is currently used by xenoprofile. Isaku have you any tips on it ? The

Re: [Xen-ia64-devel] [PATCH] make p2m exposure compatible with suspend/resume

2007-05-08 Thread Isaku Yamahata
On Tue, May 08, 2007 at 03:32:22PM +0200, Tristan Gingold wrote: you should really define the TR used by the p2m in kregs.h. Using a define is less error-prone than a hard coded number (0x2). (I don't object against this specific patch as the hard-coded number use is already commited in

Re: [Xen-ia64-devel] PATCH: define guest_mode (instead of user_mode)

2007-05-08 Thread Tristan Gingold
On Tue, May 08, 2007 at 10:57:42AM -0600, Alex Williamson wrote: On Tue, 2007-05-08 at 15:26 +0200, Tristan Gingold wrote: diff -r 8519e5db6510 -r 8e5083feaa52 xen/arch/ia64/vmx/vmx_process.c --- a/xen/arch/ia64/vmx/vmx_process.c Tue May 08 15:07:51 2007 +0200 +++

Re: [Xen-ia64-devel] [PATCH] make p2m exposure compatible with suspend/resume

2007-05-08 Thread Tristan Gingold
On Wed, May 09, 2007 at 12:02:44PM +0900, Isaku Yamahata wrote: On Tue, May 08, 2007 at 03:32:22PM +0200, Tristan Gingold wrote: you should really define the TR used by the p2m in kregs.h. Using a define is less error-prone than a hard coded number (0x2). (I don't object against this

[Xen-ia64-devel] PATCH: rewrite vcpu_get_psr

2007-05-08 Thread Tristan Gingold
Hi, a stripped-down version of a previous patch: reimplement vcpu_get_psr. Should be a noop, I don't think performance should be affected. Tristan. # HG changeset patch # User Tristan Gingold [EMAIL PROTECTED] # Date 1178683685 -7200 # Node ID 00b887a409dc883f2bae05da7fe8d80f90d430ad # Parent

[Xen-ia64-devel] PATCH: vcpu_set_psr

2007-05-08 Thread Tristan Gingold
Hi, this patch implements vcpu_set_psr, which is called by vcpu_rfi. I hope performance shouldn't be affected. This implementation also fixes some incorrect assumptions in vcpu_rfi (such as bn=1, ic = 1...). Tristan. # HG changeset patch # User Tristan Gingold [EMAIL PROTECTED] # Date 1178686663