 Home |
Products |
Support |
Community |
News |
xen-devel
Re: [Xen-devel] bogus gfn - mfn - gfn - mfn checks in guest_physmap_add_
| To: | Tim Deegan <Tim.Deegan@xxxxxxxxxx> |
|---|---|
| Subject: | Re: [Xen-devel] bogus gfn - mfn - gfn - mfn checks in guest_physmap_add_entry |
| From: | Olaf Hering <olaf@xxxxxxxxx> |
| Date: | Thu, 25 Nov 2010 21:56:49 +0100 |
| Cc: | Patrick Colp <pjcolp@xxxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx> |
| Delivery-date: | Thu, 25 Nov 2010 12:58:00 -0800 |
| Dkim-signature: | v=1; a=rsa-sha1; c=relaxed/relaxed; t=1290718615; l=617; s=domk; d=aepfle.de; h=In-Reply-To:Content-Type:MIME-Version:References:Subject:Cc:To:From: Date:X-RZG-CLASS-ID:X-RZG-AUTH; bh=5ijp9U5/+GUYPtv2ME67CFqmQ1o=; b=ceT7aV//Qsa4LxArdDsR/TQAAOk9rldMeRQGEfuPpqDpVBlcAmJ0Ei2xXrh/3TgbrZ1 pFruoKW8i2vHvHsCs5kZCOXF78BoXOUudSeqVqJvuuQEs3XoVql3ez4QcHClyskM7MqJc PNkuxSlUvt4/ZmIHKb53oPG93Kp6BUMIuu0= |
| Envelope-to: | www-data@xxxxxxxxxxxxxxxxxxx |
| In-reply-to: | <20101125153216.GJ19638@xxxxxxxxxxxxxxxxxxxxxxx> |
| List-help: | <mailto:xen-devel-request@lists.xensource.com?subject=help> |
| List-id: | Xen developer discussion <xen-devel.lists.xensource.com> |
| List-post: | <mailto:xen-devel@lists.xensource.com> |
| List-subscribe: | <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe> |
| List-unsubscribe: | <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe> |
| References: | <20101123210158.GA9425@xxxxxxxxx> <20101124102202.GF19638@xxxxxxxxxxxxxxxxxxxxxxx> <20101124144138.GA25619@xxxxxxxxx> <20101124145326.GH19638@xxxxxxxxxxxxxxxxxxxxxxx> <20101125150310.GA12431@xxxxxxxxx> <20101125153216.GJ19638@xxxxxxxxxxxxxxxxxxxxxxx> |
| Sender: | xen-devel-bounces@xxxxxxxxxxxxxxxxxxx |
| User-agent: | Mutt/1.5.20 (2009-06-14) |
On Thu, Nov 25, Tim Deegan wrote: > Good catch. It might be better to hook page_set_owner(), which ought to > imply that the old m2p info is stale. I had a quick look at all its > callers and I _think_ it would be OK, but I haven't tested that at > all. :) I tried page_set_owner(). Blindly adding set_gpfn_from_mfn there leads to compile errors, thanks to all the redefinitions of mfn_t related macros. Also, the page_set_owner() in alloc_heap_pages would cause Xen to crash early. It appears that maintaing machine_to_phys_mapping[] in free_domheap_pages() is enough to avoid the ASSERTS. Olaf _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxxxxxxxx http://lists.xensource.com/xen-devel |
| Previous by Date: | Re: [Xen-devel] bogus gfn - mfn - gfn - mfn checks in guest_physmap_add_entry, Olaf Hering |
|---|---|
| Next by Date: | Re: [Xen-devel] Re: how to avoid lost trace records?, Olaf Hering |
| Previous by Thread: | Re: [Xen-devel] bogus gfn - mfn - gfn - mfn checks in guest_physmap_add_entry, Tim Deegan |
| Next by Thread: | Re: [Xen-devel] bogus gfn - mfn - gfn - mfn checks in guest_physmap_add_entry, Keir Fraser |
| Indexes: | [Date] [Thread] [Top] [All Lists] |
