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

[Isaku Yamahata <yamahata@xxxxxxxxxxxxx>]

Sorry, I sent out those too early.
Please replace 15168_9296439c5755_fix_off_by_one_p2m_expose_init.patch with 
the attached one.

On Mon, Jun 11, 2007 at 06:13:36PM +0900, Isaku Yamahata wrote:
> 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




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

-- 
yamahata

Attachment: 15168_4785a7fd5dec_fix_off_by_one_p2m_expose_init.patch
Description: Text Data

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