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] [PATCH 14/14] memmap: allow huge size efi memory ma

To: Alex Williamson <alex.williamson@xxxxxx>
Subject: Re: [Xen-ia64-devel] [PATCH 14/14] memmap: allow huge size efi memory map of real machine
From: Isaku Yamahata <yamahata@xxxxxxxxxxxxx>
Date: Wed, 23 May 2007 21:14:44 +0900
Cc: xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Wed, 23 May 2007 05:12:49 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <1179883948.6134.182.camel@bling>
List-help: <mailto:xen-ia64-devel-request@lists.xensource.com?subject=help>
List-id: Discussion of the ia64 port of Xen <xen-ia64-devel.lists.xensource.com>
List-post: <mailto:xen-ia64-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-ia64-devel>, <mailto:xen-ia64-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-ia64-devel>, <mailto:xen-ia64-devel-request@lists.xensource.com?subject=unsubscribe>
References: <20070517104357.GA14091%yamahata@xxxxxxxxxxxxx> <1179883948.6134.182.camel@bling>
Sender: xen-ia64-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mutt/1.4.2.1i
On Tue, May 22, 2007 at 07:32:28PM -0600, Alex Williamson wrote:

> Even with that fix, my system hangs at the xfree(fw_tables) call near
> the end of this chunk.  If I skip that xfree(), we hang at the next call
> to xfree.  Looks like we corrupted memory somewhere, but I haven't found
> it yet.

Thanks for debugging and sorry for those bugs.
Although I reviewed the patch, I've not found memory corruption yet.
I attached my current clean up patch for those who want to debug it.


> In this case NUM_MEM_DESCS should be very small since it's only adding
> the descriptors we add for xen and the rest is dynamically sized based
> on the number of entries in the MDT provided by firmware.  Thanks,

I'll postpone reducing NUM_MEM_DESCS until memory corruption fix.

-- 
yamahata

Attachment: changes.patch
Description: Text Data

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