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

[Doi.Tsunehisa@xxxxxxxxxxxxxx]

You (yamahata) said:
>>   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:
> 
> Thank you for testing.
> Do you have stack trace at the crash?

  I don't have the stack trace in the console log.

> page->count_info == PGC_allocated | 1?

  You are right!! The page->count_info value is 0x80000001 at the crash.

> If so, the BUG_ON() is bogus. Probably it should be something like
> BUG_ON(page->count_info != 1 && page->count_info != (PGC_allocated | 1)).

  BUG_ON((page->count_info & PGC_count_mask) != 1) is better, I think.

Thanks,
- Tsunehisa Doi

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