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-devel

[Xen-devel] RE: [PATCH] X server crashes Xen on xeno-unstable.bk

To: <leendert@xxxxxxxxxxxxxx>, <xen-devel@xxxxxxxxxxxxxxxxxxxxx>
Subject: [Xen-devel] RE: [PATCH] X server crashes Xen on xeno-unstable.bk
From: "Ian Pratt" <m+Ian.Pratt@xxxxxxxxxxxx>
Date: Thu, 6 Jan 2005 20:37:06 -0000
Delivery-date: Thu, 06 Jan 2005 20:38:13 +0000
Envelope-to: xen+James.Bulpin@xxxxxxxxxxxx
List-archive: <http://sourceforge.net/mailarchive/forum.php?forum=xen-devel>
List-help: <mailto:xen-devel-request@lists.sourceforge.net?subject=help>
List-id: List for Xen developers <xen-devel.lists.sourceforge.net>
List-post: <mailto:xen-devel@lists.sourceforge.net>
List-subscribe: <https://lists.sourceforge.net/lists/listinfo/xen-devel>, <mailto:xen-devel-request@lists.sourceforge.net?subject=subscribe>
List-unsubscribe: <https://lists.sourceforge.net/lists/listinfo/xen-devel>, <mailto:xen-devel-request@lists.sourceforge.net?subject=unsubscribe>
Sender: xen-devel-admin@xxxxxxxxxxxxxxxxxxxxx
Thread-index: AcT0LhjM3jZEavTWRrGecm3NKkL1gAAAR6iQ
Thread-topic: [PATCH] X server crashes Xen on xeno-unstable.bk
> Starting an X server on xeno-unstable.bk causes Xen to crash with
> a page fault generated in Xen. It turns out that the fault is
> generated by put_page_from_l1e().
> 
> The culprit is the following dereference:
> 
>      struct domain   *e = page->u.inuse.domain;
> 
> Where the value of page is undefined if the condition
> 
>      if ( !(l1v & _PAGE_PRESENT) || !pfn_is_ram(pfn) )
> 
> does hold. Unfortunately, this condition is checked after the 
> dereference.

Interesting. I'm guessing the reason we haven't seen this is that we
don't build on FC3. The newer version of gcc has obviously spotted the
opportunity to hoist the load.

I'll checkin the fix.

Thanks,
Ian


-------------------------------------------------------
The SF.Net email is sponsored by: Beat the post-holiday blues
Get a FREE limited edition SourceForge.net t-shirt from ThinkGeek.
It's fun and FREE -- well, almost....http://www.thinkgeek.com/sfshirt
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.sourceforge.net/lists/listinfo/xen-devel

<Prev in Thread] Current Thread [Next in Thread>