Re: [Xen-ia64-devel] [Q] about assign_domain_page_replace

[Isaku Yamahata <yamahata@xxxxxxxxxxxxx>]

On Mon, Jun 11, 2007 at 03:55:30PM +0900, Akio Takebe wrote:
Content-Description: Mail message body

> >On Fri, Jun 08, 2007 at 11:52:30PM +0900, Isaku Yamahata wrote:
> >> Thank you very much for information. It greatly helped.
> >> Could you try the attached patch with the previsou patch?
> >> I hope it fixes the foreign domain page mapping case.
> >> However there remains the grant table mapping issue.
> >
> >When backend maps ring page of frontend domain,
> >the debug message is printed out.
> >So only a few messages is okay.
> >When you tested, did you see only a few messages whose backtrace
> >includes grant table or many messages?
> I retried 15165 + test.patch (including your patch).
> I had the only following message while a booting of domU.
> But if I do shutdown on the domU, then Hypervisor doesn't panic.
> The following backtraces include __do_dom0vp_add_physmap(),
> Is that OK? 

Unfortunately no. I reviewed the linux side code and found an off-by-one bug.
Could you try again with the three attached patches (+ debug message patch).
(Note that I sent out two of three attached patch. I attached those just
for convinience.)
Thank you very much for your patience. You found three bugs!
I hope that you won't hit more bugs.

thanks,
-- 
yamahata

Attachment: 15168_9296439c5755_fix_off_by_one_p2m_expose_init.patch
Description: Text Data

Attachment: 15167_7305e027b17d_fix_one_more_zero_p2m_page.patch
Description: Text Data

Attachment: 15166_afbcf495550d_fix_p2m_exposure.patch
Description: Text Data

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