[Xen-ia64-devel] Re: [PATCH] fix the ia64 p2m semantic

[Doi.Tsunehisa@xxxxxxxxxxxxxx]

FYI.

  Original change set is not crashed.

I (Doi.Tsunehisa) said:
> Hi Isaku,
> 
>   Sorry for late response.
> 
> You (yamahata) said:
> > Tsunehisa.
> > I haven't tested arch_memory_op(). Could you verify it?
> 
>   I've checked this patch on the latest change set(cs:14513).
> 
>   It occured hypervisor crash at that xen-platform.ko was inserted.
> And, hypervisor said as follow:
> 
> (XEN) BUG at mm.c:1050
> (XEN) FIXME: implement ia64 dump_execution_state()
> (XEN)
> (XEN) ****************************************
> (XEN) Panic on CPU 1:
> (XEN) BUG at mm.c:1050
> (XEN) ****************************************
> (XEN)
> (XEN) Reboot in five seconds...
> 
>   The code shown by it is...
> 
>    1045
>    1046 static void
>    1047 adjust_page_count_info(struct page_info* page)
>    1048 {
>    1049     struct domain* d = page_get_owner(page);
>    1050     BUG_ON(page->count_info != 1);    <<==== this point.
>    1051     if (d != NULL) {
>    1052         int ret = get_page(page, d);
>    1053         BUG_ON(ret == 0);
>    1054     } else {
>    1055         u64 x, nx, y;
>    1056
>    1057         y = *((u64*)&page->count_info);
>    1058         do {
>    1059             x = y;
>    1060             nx = x + 1;
>    1061
>    1062             BUG_ON((x >> 32) != 0);
>    1063             BUG_ON((nx & PGC_count_mask) != 2);
>    1064             y = cmpxchg((u64*)&page->count_info, x, nx);
>    1065         } while (unlikely(y != x));
>    1066     }
>    1067 }
> 
>   I'll check, which hypercall occures hyper crash.
> 
> Thanks,
> - Tsunehisa Doi
> 

_______________________________________________
Xen-ia64-devel mailing list
Xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-ia64-devel