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

[Doi.Tsunehisa@xxxxxxxxxxxxxx]

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