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

Re: [Xen-devel] Xen dom0 crash in get_phys_to_machine

To: Gianni Tedesco <gianni.tedesco@xxxxxxxxxx>, Andreas Kinzler <ml-xen-devel@xxxxxx>, sven <ml@xxxxxxxxxxxxx>, Jeremy Fitzhardinge <jeremy@xxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Subject: Re: [Xen-devel] Xen dom0 crash in get_phys_to_machine
From: "Alan J. Wylie" <NDA5OWUy@xxxxxxxxxxx>
Date: Fri, 22 Oct 2010 14:33:05 +0100
Cc:
Delivery-date: Fri, 22 Oct 2010 06:34:01 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <1287752727.12843.4406.camel@xxxxxxxxxxxxxxxxxxxxxx>
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: <19636.5260.149513.257699@xxxxxxxxxxx> <1287752727.12843.4406.camel@xxxxxxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
at 14:05 on Fri 22-Oct-2010 Gianni Tedesco (gianni.tedesco@xxxxxxxxxx) wrote:

> FWIW, when I was checking for any call where pfn > max_pfn - and I got:
> 
>   p2m_top[0][10][104] max_pfn=0
> 
> The p2m seems to have been correctly initialised:
> 
>  xen_build_dynamic_phys_to_machine: topidx=0 mididx=375 max_pfn=192512
> 
> But then it looks like something is trampling max_pfn and possibly other
> important data structures.

I've just been reading through the Documentation/development-process
and discovered "sparse".

Five minutes ago I ran it on mmu.c and got the following interesting
output:

/usr/src/jeremy-git-xen/arch/x86/xen/mmu.c:385:23: warning: symbol 'max_pfn' 
shadows an earlier one
/usr/src/jeremy-git-xen/arch/x86/include/asm/page_64_types.h:58:22: originally 
declared here
/usr/src/jeremy-git-xen/arch/x86/xen/mmu.c:289:47: warning: potentially 
expensive pointer subtraction
/usr/src/jeremy-git-xen/arch/x86/include/asm/xen/page.h:84:9: warning: 
incorrect type in argument 1 (different address spaces)
/usr/src/jeremy-git-xen/arch/x86/include/asm/xen/page.h:84:9:    expected void 
const volatile [noderef] <asn:1>*<noident>
/usr/src/jeremy-git-xen/arch/x86/include/asm/xen/page.h:84:9:    got unsigned 
long *
/usr/src/jeremy-git-xen/arch/x86/include/asm/xen/page.h:84:9: warning: cast 
adds address space to expression (<asn:1>)
/usr/src/jeremy-git-xen/arch/x86/include/asm/xen/page.h:84:9: warning: cast 
adds address space to expression (<asn:1>)
/usr/src/jeremy-git-xen/arch/x86/include/asm/xen/page.h:84:9: warning: cast 
adds address space to expression (<asn:1>)
/usr/src/jeremy-git-xen/arch/x86/include/asm/xen/page.h:84:9: warning: cast 
adds address space to expression (<asn:1>)
/usr/src/jeremy-git-xen/include/linux/mm.h:603:16: warning: potentially 
expensive pointer subtraction
/usr/src/jeremy-git-xen/arch/x86/xen/mmu.c:1269:37: warning: potentially 
expensive pointer subtraction
/usr/src/jeremy-git-xen/include/linux/mm.h:603:16: warning: potentially 
expensive pointer subtraction
/usr/src/jeremy-git-xen/include/linux/mm.h:603:16: warning: potentially 
expensive pointer subtraction
/usr/src/jeremy-git-xen/arch/x86/xen/mmu.c:1410:37: warning: potentially 
expensive pointer subtraction
/usr/src/jeremy-git-xen/include/linux/mm.h:603:16: warning: potentially 
expensive pointer subtraction
/usr/src/jeremy-git-xen/arch/x86/xen/mmu.c:1684:17: error: bad constant 
expression

Is it just a co-incidence that the first two lines refer to the same
symbol that you have just mentioned?

I'm going to try renaming the local symbol and see if things still crash.

The trouble is that the box I've been testing on is supposed to be our
backup file server and is currently doing a rsync of 280GB of files
from a 7 year old windows box. At least I'll be able to leave it
running undisturbed over the weekend.

-- 
Alan J. Wylie                                          http://www.wylie.me.uk/


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