Re: [PATCH v2] xen, kdump: handle pv domain in paddr_vmcoreinfo_note()

From: Juergen Gross <jgross@suse.com>
To: Daniel Kiper <dkiper@net-space.pl>
Cc: ptesarik@suse.com, andrew.cooper3@citrix.com,
	daniel.kiper@oracle.com, linux-kernel@vger.kernel.org,
	xen-devel@lists.xenproject.org, boris.ostrovsky@oracle.com,
	eric.devolder@oracle.com
Subject: Re: [PATCH v2] xen, kdump: handle pv domain in paddr_vmcoreinfo_note()
Date: Tue, 4 Apr 2017 13:55:23 +0200	[thread overview]
Message-ID: <93654826-75f5-1530-68b5-46cd719722bb__28156.5376827077$1491306996$gmane$org@suse.com> (raw)
In-Reply-To: <20170403124257.GA3439@router-fw-old.local.net-space.pl>

On 03/04/17 14:42, Daniel Kiper wrote:
> On Fri, Mar 31, 2017 at 12:14:38PM +0200, Juergen Gross wrote:
>> For kdump to work correctly it needs the physical address of
>> vmcoreinfo_note. When running as dom0 this means the virtual address
>> has to be translated to the related machine address.
>>
>> paddr_vmcoreinfo_note() is meant to do the translation via
>> __pa_symbol() only, but being attributed "weak" it can be replaced
>> easily in Xen case.
>>
>> Signed-off-by: Juergen Gross <jgross@suse.com>
> 
> Have you tested this patch with latest crash tool? Do dom0 and Xen
> hypervisor analysis work without any issue (at least basic commands
> like dmesg, bt, ps, etc.)? If yes for both you can add:
> 
> Reviewed-by: Daniel Kiper <daniel.kiper@oracle.com>

This patch isn't for dump analysis, but for dump creation. Petr has
verified that the dump is in the expected format. Please ask Petr
for further details, e.g. user side modifications being necessary.

Juergen

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel