[Xen-ia64-devel] Re: PATCH: ptc.ga for SMP-g

2006-04-10 Thread Alex Williamson
On Fri, 2006-04-07 at 12:37 +0100, Tristan Gingold wrote: > Hi, > > thank you for all the discussion about ptc.ga for SMP-g. > Here is the last patch. > > Tested within SMP-g frame by compiling on domU (4+1+1 cpus) > Tested by boot+halt of dom0+domU. Applied. -- Alex Williamson

Re: [Xen-ia64-devel] Re: [PATCH]: ptc.ga for SMP-g

2006-03-31 Thread Isaku Yamahata
On Fri, Mar 31, 2006 at 09:35:23AM +0100, Tristan Gingold wrote: > Le Vendredi 31 Mars 2006 08:15, Isaku Yamahata a écrit : > [...] > > > The IA64_GRANULE_SIZE (16M/64M) area covering current kernel stack > > > is also mapped by TR. Except for init_task, all kernel stacks are > > > dynamically allo

Re: [Xen-ia64-devel] Re: [PATCH]: ptc.ga for SMP-g

2006-03-31 Thread Tristan Gingold
Le Vendredi 31 Mars 2006 08:15, Isaku Yamahata a écrit : [...] > > The IA64_GRANULE_SIZE (16M/64M) area covering current kernel stack > > is also mapped by TR. Except for init_task, all kernel stacks are > > dynamically allocated from identity mapping area. So if stack of current > > running task h

Re: [Xen-ia64-devel] Re: [PATCH]: ptc.ga for SMP-g

2006-03-30 Thread Tristan Gingold
Le Vendredi 31 Mars 2006 03:23, Isaku Yamahata a écrit : > On Thu, Mar 30, 2006 at 03:58:21PM +0100, Tristan Gingold wrote: > > Le Jeudi 30 Mars 2006 16:17, Isaku Yamahata a écrit : > > > On Thu, Mar 30, 2006 at 04:59:35PM +0800, Xu, Anthony wrote: > > > > There seems no issue, because vcpu_purge_t

Re: [Xen-ia64-devel] Re: [PATCH]: ptc.ga for SMP-g

2006-03-30 Thread Isaku Yamahata
Hi Kevin. Thank you for clarification. On Fri, Mar 31, 2006 at 01:41:51PM +0800, Tian, Kevin wrote: > Let's make some clarification here: > > For IA64 linux, KERNEL_START records start of kernel image which is in > region 5 for 2.6 kernel while in region 7 for 2.4 kernel. PAGE_OFFSET is > sta

RE: [Xen-ia64-devel] Re: [PATCH]: ptc.ga for SMP-g

2006-03-30 Thread Tian, Kevin
d >Subject: RE: [Xen-ia64-devel] Re: [PATCH]: ptc.ga for SMP-g > >>From: Isaku Yamahata [mailto:[EMAIL PROTECTED] >>Sent: 2006年3月31日 10:53 >>It's right that Linux/ia64 kernel text and statically allocated data >>are in [0xa001, ...]. >>Ho

RE: [Xen-ia64-devel] Re: [PATCH]: ptc.ga for SMP-g

2006-03-30 Thread Xu, Anthony
>From: Isaku Yamahata [mailto:[EMAIL PROTECTED] >Sent: 2006年3月31日 10:53 >It's right that Linux/ia64 kernel text and statically allocated data >are in [0xa001, ...]. >However dynamically allocated pages's virtual address are in >the area of [PAGE_OFFSET, ...]. Dynamically allocated page

Re: [Xen-ia64-devel] Re: [PATCH]: ptc.ga for SMP-g

2006-03-30 Thread Isaku Yamahata
On Fri, Mar 31, 2006 at 10:42:13AM +0800, Xu, Anthony wrote: > >From: Isaku Yamahata >Sent: 2006年3月31日 9:23 > >> PAGE_OFFSET + 64MB is covered by a tr, isn't it ? > > > >[PAGE_OFFSET, PAGE_OFFSET + KERNEL_TR_PAGE_SIZE - 1] is covered. > >Here KERNEL_TR_PAGE_SIZE = 64MB which is defined > >in linux/

RE: [Xen-ia64-devel] Re: [PATCH]: ptc.ga for SMP-g

2006-03-30 Thread Xu, Anthony
>From: Isaku Yamahata >Sent: 2006年3月31日 9:23 >> PAGE_OFFSET + 64MB is covered by a tr, isn't it ? > >[PAGE_OFFSET, PAGE_OFFSET + KERNEL_TR_PAGE_SIZE - 1] is covered. >Here KERNEL_TR_PAGE_SIZE = 64MB which is defined >in linux/asm-ia64/pgtable.h. > >For safety, please consider >PAGE_OFFSET + KERNEL_

RE: [Xen-ia64-devel] Re: [PATCH]: ptc.ga for SMP-g

2006-03-30 Thread Xu, Anthony
>From: Isaku Yamahata>Sent: 2006?3?30? 22:18 >To: Xu, Anthony >Cc: Tristan Gingold; xen-ia64-devel@lists.xensource.com >Subject: Re: [Xen-ia64-devel] Re: [PATCH]: ptc.ga for SMP-g > >Page not present fault is different from tlb miss fault. >So guest OS may handle them d

Re: [Xen-ia64-devel] Re: [PATCH]: ptc.ga for SMP-g

2006-03-30 Thread Isaku Yamahata
On Thu, Mar 30, 2006 at 03:58:21PM +0100, Tristan Gingold wrote: > Le Jeudi 30 Mars 2006 16:17, Isaku Yamahata a écrit : > > On Thu, Mar 30, 2006 at 04:59:35PM +0800, Xu, Anthony wrote: > > > There seems no issue, because vcpu_purge_tr_entry() only sets ->p=0. > > > In some cases, this patch may ca

Re: [Xen-ia64-devel] Re: [PATCH]: ptc.ga for SMP-g

2006-03-30 Thread Tristan Gingold
Le Jeudi 30 Mars 2006 16:17, Isaku Yamahata a écrit : > On Thu, Mar 30, 2006 at 04:59:35PM +0800, Xu, Anthony wrote: > > There seems no issue, because vcpu_purge_tr_entry() only sets ->p=0. > > In some cases, this patch may cause a not-present tlb inserted into > > machine TLB, this is harmless, wh

Re: [Xen-ia64-devel] Re: [PATCH]: ptc.ga for SMP-g

2006-03-30 Thread Isaku Yamahata
On Thu, Mar 30, 2006 at 04:59:35PM +0800, Xu, Anthony wrote: > There seems no issue, because vcpu_purge_tr_entry() only sets ->p=0. > In some cases, this patch may cause a not-present tlb inserted into machine > TLB, this is harmless, when guest accesses this page, page not present fault > happe

RE: [Xen-ia64-devel] Re: [PATCH]: ptc.ga for SMP-g

2006-03-30 Thread Tian, Kevin
>From: Tristan Gingold [mailto:[EMAIL PROTECTED] >Sent: 2006年3月30日 17:25 >For sure, this is almost the safest. But there is still a race-condition: if >vcpu migrates during the IPI, the tlb can be modified by two cpus. So first, you need hint to indicate which domain ptc.g emulation happens on.

Re: [Xen-ia64-devel] Re: [PATCH]: ptc.ga for SMP-g

2006-03-30 Thread Tristan Gingold
Le Jeudi 30 Mars 2006 11:08, Tian, Kevin a écrit : > From: Tristan Gingold [mailto:[EMAIL PROTECTED] > > >Sent: 2006年3月30日 16:56 > > > >> >Can you elaborate ? Do you have a disaster scenario ? > >> >vcpu_purge_tr_entry only clears the p bit. > >> > >> I think it's not safe for one vcpu to operate d

RE: [Xen-ia64-devel] Re: [PATCH]: ptc.ga for SMP-g

2006-03-30 Thread Tian, Kevin
>From: Tristan Gingold [mailto:[EMAIL PROTECTED] >Sent: 2006年3月30日 16:56 > >> >Can you elaborate ? Do you have a disaster scenario ? >> >vcpu_purge_tr_entry only clears the p bit. >> >> I think it's not safe for one vcpu to operate date structure on another >> vcpu. For example, current vcpu is doi

RE: [Xen-ia64-devel] Re: [PATCH]: ptc.ga for SMP-g

2006-03-30 Thread Xu, Anthony
>From: Tristan Gingold >Sent: 2006?3?30? 16:31 >To: Isaku Yamahata >Cc: xen-ia64-devel@lists.xensource.com >Subject: Re: [Xen-ia64-devel] Re: [PATCH]: ptc.ga for SMP-g > >Le Mardi 28 Mars 2006 04:04, Isaku Yamahata a écrit : >> Hi Tristan. >> >> &g

Re: [Xen-ia64-devel] Re: [PATCH]: ptc.ga for SMP-g

2006-03-30 Thread Tristan Gingold
Le Jeudi 30 Mars 2006 10:42, Tian, Kevin a écrit : > From: Tristan Gingold > > >Sent: 2006年3月30日 16:31 > > > >Le Mardi 28 Mars 2006 04:04, Isaku Yamahata a écrit : > >> Hi Tristan. > >> > >> > >> + /* Purge tc entry. > >> + Can we do this directly ? Well, this

RE: [Xen-ia64-devel] Re: [PATCH]: ptc.ga for SMP-g

2006-03-30 Thread Tian, Kevin
>From: Tristan Gingold >Sent: 2006年3月30日 16:31 >Le Mardi 28 Mars 2006 04:04, Isaku Yamahata a écrit : >> Hi Tristan. >> >> >> +/* Purge tc entry. >> + Can we do this directly ? Well, this is just a >> + single atomic write. */ >> +

Re: [Xen-ia64-devel] Re: [PATCH]: ptc.ga for SMP-g

2006-03-30 Thread Tristan Gingold
Le Mardi 28 Mars 2006 04:04, Isaku Yamahata a écrit : > Hi Tristan. > > > + /* Purge tc entry. > +Can we do this directly ? Well, this is just a > +single atomic write. */ > + vcpu_purge_tr_entry(&PSCBX(v,dtlb

Re: [Xen-ia64-devel] Re: [PATCH]: ptc.ga for SMP-g

2006-03-27 Thread Isaku Yamahata
Hi Tristan. + /* Purge tc entry. + Can we do this directly ? Well, this is just a + single atomic write. */ + vcpu_purge_tr_entry(&PSCBX(v,dtlb)); + vcpu_purge_tr_entry(&PSCBX(v,

Re: [Xen-ia64-devel] Re: [PATCH]: ptc.ga for SMP-g

2006-03-27 Thread Alex Williamson
On Mon, 2006-03-27 at 14:15 +0100, Tristan Gingold wrote: > Le Lundi 27 Mars 2006 15:07, Alex Williamson a écrit : > > On Mon, 2006-03-27 at 13:53 +0100, Tristan Gingold wrote: > > > Hi, > > > > > > this is a very simple extension of vcpu_ptc_ga to support SMP-g. > > > The implementation could be o

Re: [Xen-ia64-devel] Re: [PATCH]: ptc.ga for SMP-g

2006-03-27 Thread Tristan Gingold
Le Lundi 27 Mars 2006 15:07, Alex Williamson a écrit : > On Mon, 2006-03-27 at 13:53 +0100, Tristan Gingold wrote: > > Hi, > > > > this is a very simple extension of vcpu_ptc_ga to support SMP-g. > > The implementation could be optimized. > > Hi Tristan, > >There's no patch attached, please res

[Xen-ia64-devel] Re: [PATCH]: ptc.ga for SMP-g

2006-03-27 Thread Alex Williamson
On Mon, 2006-03-27 at 13:53 +0100, Tristan Gingold wrote: > Hi, > > this is a very simple extension of vcpu_ptc_ga to support SMP-g. > The implementation could be optimized. Hi Tristan, There's no patch attached, please resend. Thanks, Alex -- Alex Williamson