Re: [Xen-ia64-devel] PV-on-HVM for IPF (take 3)

2006-09-12 Thread Isaku Yamahata
Hi. Your patch isn't SMP-safe. domVTi code assumes that the p2m table is build at the domain creation and read-only after that. However your patch breaks the assumption and opened the door to SMP world. Especially you must resolve the race between tlb insert and tlb purge. Do you have any plan

Re: [Xen-ia64-devel] PV-on-HVM for IPF (take 3)

2006-09-12 Thread Doi . Tsunehisa
Hi. You (yamahata) said: Your patch isn't SMP-safe. domVTi code assumes that the p2m table is build at the domain creation and read-only after that. However your patch breaks the assumption and opened the door to SMP world. Especially you must resolve the race between tlb insert and tlb

Re: [Xen-ia64-devel] PV-on-HVM for IPF (take 3)

2006-09-03 Thread Doi . Tsunehisa
Hi Alex, You (alex.williamson) said: I'll post new expand-memory-op.patch2. I applied this, but I did have to fix mm.h to make it build. Thanks, Thank you, and sorry. I forgot to append mm.h patch with expand-memory-op.patch2. -- ÅÚÈî¤Ç¤·¤¿¡¥¡¥¡¥

Re: [Xen-ia64-devel] PV-on-HVM for IPF (take 3)

2006-09-01 Thread Alex Williamson
On Wed, 2006-08-30 at 14:16 +0900, DOI Tsunehisa wrote: You should add least add a comment because it is unusable to see VMX_DOMAIN() within paravirtualization area. OK. I'll append comment to descibe more detail reason. I'll post new expand-memory-op.patch2. I applied this, but

[Xen-ia64-devel] PV-on-HVM for IPF (take 3)

2006-08-29 Thread DOI Tsunehisa
Hi all, We have been porting PV-on-HVM feature for ia64 platform. I will post patches for PV-on-HVM on ia64 platform. These patches modify common code for PV-on-HVM on IPF. We ported PV-on-HVM for IPF under this consideration: * Expand memory_op hypercall + Introduce

Re: [Xen-ia64-devel] PV-on-HVM for IPF (take 3)

2006-08-29 Thread Tristan Gingold
Le Mardi 29 Août 2006 13:50, DOI Tsunehisa a écrit : Hi all, We have been porting PV-on-HVM feature for ia64 platform. Hi, I have a question about this modification: void vcpu_flush_vtlb_all(struct vcpu *v) { - /* First VCPU tlb. */ - vcpu_purge_tr_entry(PSCBX(v,dtlb)); -

Re: [Xen-ia64-devel] PV-on-HVM for IPF (take 3)

2006-08-29 Thread Doi . Tsunehisa
Hi Tristan, Thank you for your comment. You (Tristan.Gingold) said: I have a question about this modification: void vcpu_flush_vtlb_all(struct vcpu *v) { - /* First VCPU tlb. */ - vcpu_purge_tr_entry(PSCBX(v,dtlb)); - vcpu_purge_tr_entry(PSCBX(v,itlb)); - - /*

Re: [Xen-ia64-devel] PV-on-HVM for IPF (take 3)

2006-08-29 Thread DOI Tsunehisa
You should add least add a comment because it is unusable to see VMX_DOMAIN() within paravirtualization area. OK. I'll append comment to descibe more detail reason. I'll post new expand-memory-op.patch2. Thanks, - Tsunehisa Doi Expand memory_op for PV-on-HVM on IPF Signed-off-by: