| |
|
 |
|
 |
|
|
|
| |
|
|
xen-devel
Re: [Xen-devel] [PATCH 00/04] Kexec / Kdump: Release 20061122 (xen-unsta
|
To: |
Magnus Damm <magnus@xxxxxxxxxxxxx> |
|
Subject: |
Re: [Xen-devel] [PATCH 00/04] Kexec / Kdump: Release 20061122 (xen-unstable-12502) |
|
From: |
Ian Campbell <Ian.Campbell@xxxxxxxxxxxxx> |
|
Date: |
Tue, 28 Nov 2006 18:24:04 +0000 |
|
Cc: |
Ian Pratt <m+Ian.Pratt@xxxxxxxxxxxx>, Kazuo Moriwaka <moriwaka@xxxxxxxxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxx, magnus.damm@xxxxxxxxx, Akio Takebe <takebe_akio@xxxxxxxxxxxxxx>, Isaku Yamahata <yamahata@xxxxxxxxxxxxx>, Horms <horms@xxxxxxxxxxxx> |
|
Delivery-date: |
Tue, 28 Nov 2006 10:24:35 -0800 |
|
Envelope-to: |
www-data@xxxxxxxxxxxxxxxxxx |
|
In-reply-to: |
<20061122071050.24010.92547.sendpatchset@localhost> |
|
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/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe> |
|
List-unsubscribe: |
<http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe> |
|
References: |
<20061122071050.24010.92547.sendpatchset@localhost> |
|
Sender: |
xen-devel-bounces@xxxxxxxxxxxxxxxxxxx |
Hi Magnus,
On Wed, 2006-11-22 at 16:10 +0900, Magnus Damm wrote:
> [PATCH 00/04] Kexec / Kdump: Release 20061122 (xen-unstable-12502)
I've been playing a bit more and found a problem.
You define a per CPU variable crash_notes and on crash you loop over
NR_CPUS and clear the notes for CPUS which don't exist. Unfortunately
the percpu regions for CPUs which aren't physically present is returned
to the heap on boot (see percpu_free_unused_areas) -- this means that
you zero out heap pages on crash :-(
You need to use num_{possible,present,online}_cpus() in
machine_crash_kexec() and kexec_get_cpu() instead of NR_CPUS.
Cheers,
Ian.
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
|
|
|
| |
|
Home