WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-ia64-devel

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

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