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

2017-04-15 Thread Petr Tesarik
On Sat, 15 Apr 2017 00:26:05 +0200 Daniel Kiper wrote: > On Fri, Apr 14, 2017 at 06:53:36PM +0200, Petr Tesarik wrote: >[...] > > shifted towards libkdumpfile (https://github.com/ptesarik/libkdumpfile), > > and this library can open PV guest dump files without any issues. &g

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

2017-04-14 Thread Petr Tesarik
On Tue, 11 Apr 2017 19:20:08 +0200 Daniel Kiper wrote: > On Tue, Apr 11, 2017 at 04:59:16PM +0200, Petr Tesarik wrote: >[...] > > Tested-by: Petr Tesarik > > > > I copied the complete /proc/vmcore to a directory on disk. Exactly > > as expected, crash works both

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

2017-04-11 Thread Petr Tesarik
> > 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 > > > > So can I add your R-b: now? > > My R-b is still valid. Though, let'

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

2017-04-11 Thread Petr Tesarik
On Mon, 10 Apr 2017 22:49:33 +0200 Daniel Kiper wrote: > On Fri, Apr 07, 2017 at 11:16:22AM +0200, Petr Tesarik wrote: > > On Wed, 5 Apr 2017 13:13:00 +0200 > > Petr Tesarik wrote: > > > > > On Tue, 4 Apr 2017 12:42:53 -0700 (PDT) > > > Daniel Kiper wrot

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

2017-04-07 Thread Petr Tesarik
On Wed, 5 Apr 2017 13:13:00 +0200 Petr Tesarik wrote: > On Tue, 4 Apr 2017 12:42:53 -0700 (PDT) > Daniel Kiper wrote: > >[...] > > So, if Petr did relevant tests that is nice. However, then, IMO, this > > patch begs Petr Tested-by. > > Actually, I tested wit

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

2017-04-05 Thread Petr Tesarik
On Tue, 4 Apr 2017 12:42:53 -0700 (PDT) Daniel Kiper wrote: > > 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 t

Re: [Xen-devel] [PATCH 2/2] Allow kdump with crash_kexec_post_notifiers

2016-08-01 Thread Petr Tesarik
On Mon, 1 Aug 2016 16:22:18 +0200 "Jan Beulich" wrote: > >>> On 01.08.16 at 16:15, wrote: >[...] > > But let me repeat: if PV kexec works, then it has always had priority > > over the hypercall. If it doesn't work, then it has always been broken. > > For the latter case, I agree that the kernel

Re: [Xen-devel] [PATCH 2/2] Allow kdump with crash_kexec_post_notifiers

2016-08-01 Thread Petr Tesarik
On Mon, 1 Aug 2016 15:47:58 +0200 "Jan Beulich" wrote: > (re-adding xen-devel) > > >>> On 01.08.16 at 15:02, wrote: > > On Mon, 1 Aug 2016 13:55:01 +0200 > > "Jan Beulich" wrote: > > > >> >>> On 13.07.16 at 14:53

Re: [Xen-devel] [PATCH 2/2] Allow kdump with crash_kexec_post_notifiers

2016-08-01 Thread Petr Tesarik
On Mon, 1 Aug 2016 13:55:01 +0200 "Jan Beulich" wrote: > >>> On 13.07.16 at 14:53, wrote: > > On 13/07/16 13:20, Petr Tesarik wrote: > >> If a crash kernel is loaded, do not crash the running domain. This is > >> needed if the kernel is loaded with

Re: [Xen-devel] [PATCH 1/2] Add a kexec_crash_loaded() function

2016-07-13 Thread Petr Tesarik
On Wed, 13 Jul 2016 05:52:33 -0700 Josh Triplett wrote: > On Wed, Jul 13, 2016 at 02:19:55PM +0200, Petr Tesarik wrote: > > --- a/kernel/kexec_core.c > > +++ b/kernel/kexec_core.c > > @@ -95,6 +95,12 @@ int kexec_should_crash(struct task_struct *p) > > retu

Re: [Xen-devel] [PATCH 0/2] Allow crash dumps with crash_kexec_post_notifiers

2016-07-13 Thread Petr Tesarik
On Wed, 13 Jul 2016 14:19:50 +0200 Petr Tesarik wrote: > Hello all, > > this patch series makes it possible to save a kernel crash dump when the > kernel command line includes "crash_kexec_post_notifiers". Oh ... I forgot to add: This only applies to running Linux under X

[Xen-devel] [PATCH 1/2] Add a kexec_crash_loaded() function

2016-07-13 Thread Petr Tesarik
dules separately to enable the use of PV drivers with unmodified bare-metal kernels. Signed-off-by: Petr Tesarik --- include/linux/kexec.h |2 ++ kernel/kexec_core.c |6 ++ kernel/ksysfs.c |2 +- 3 files changed, 9 insertions(+), 1 deletion(-) diff --git a/include/

[Xen-devel] [PATCH 0/2] Allow crash dumps with crash_kexec_post_notifiers

2016-07-13 Thread Petr Tesarik
rs are run or not: If you load a crash kernel with kexec, it will be used, otherwise the hypervisor facility is used (using a hypercall). Feedback welcome! Petr T --- Petr Tesarik (2): Add a kexec_crash_loaded() function Allow kdump with crash_kexec_post_notifiers arch/x86/xen/e

[Xen-devel] [PATCH 2/2] Allow kdump with crash_kexec_post_notifiers

2016-07-13 Thread Petr Tesarik
If a crash kernel is loaded, do not crash the running domain. This is needed if the kernel is loaded with crash_kexec_post_notifiers, because panic notifiers are run before __crash_kexec() in that case, and this Xen hook prevents its being called later. Signed-off-by: Petr Tesarik --- arch/x86

Re: [Xen-devel] crash tool - problem with new Xen linear virtual mapped sparse p2m list

2015-11-24 Thread Petr Tesarik
V Tue, 24 Nov 2015 10:35:03 + Andrew Cooper napsáno: > On 24/11/15 10:17, Petr Tesarik wrote: > > On Tue, 24 Nov 2015 10:09:01 + > > David Vrabel wrote: > > > >> On 24/11/15 09:55, Malcolm Crossley wrote: > >>> On 24/11/15 08:59, Jan Beulich wro

Re: [Xen-devel] crash tool - problem with new Xen linear virtual mapped sparse p2m list

2015-11-24 Thread Petr Tesarik
On Tue, 24 Nov 2015 10:09:01 + David Vrabel wrote: > On 24/11/15 09:55, Malcolm Crossley wrote: > > On 24/11/15 08:59, Jan Beulich wrote: > > On 24.11.15 at 07:55, wrote: > >>> What about: > >>> > >>> 4) Instead of relying on the kernel maintained p2m list for m2p > >>>conversion use

Re: [Xen-devel] More on kexec/purgatory handover

2015-05-13 Thread Petr Tesarik
On Wed, 13 May 2015 11:01:24 +0100 "Jan Beulich" wrote: > >>> On 13.05.15 at 11:53, wrote: > > On 13/05/15 09:12, Jan Beulich wrote: > > On 13.05.15 at 09:35, wrote: > >>> Fundamentally if you are transfering control in long mode you have to > >>> set up some page table. I giant identity m

Re: [Xen-devel] More on kexec/purgatory handover

2015-05-13 Thread Petr Tesarik
On Wed, 13 May 2015 09:12:47 +0100 "Jan Beulich" wrote: > >>> On 13.05.15 at 09:35, wrote: > > Fundamentally if you are transfering control in long mode you have to > > set up some page table. I giant identity mapped page table that can use > > 1G or 2M pages takes up very little memory, and ca

[Xen-devel] More on kexec/purgatory handover

2015-05-09 Thread Petr Tesarik
kernel. If Xen plays the role of the old kernel, it must also set up paging. The question is how. Let's start a discussion on the kexec mailing list (in Cc) to clarify what should be done by the old kernel and what should be done by the purgatory code. Petr Tesarik __

Re: [Xen-devel] [PATCH] kexec-tools: Read always one vmcoreinfo file

2014-11-13 Thread Petr Tesarik
On Thu, 13 Nov 2014 18:22:17 +0100 Daniel Kiper wrote: > Hi Petr, > > On Thu, Nov 13, 2014 at 04:51:48PM +0100, Petr Tesarik wrote: > > Hi all, > > > > this thread got somehow forgotten because of vacations... > > Anyway, read below. > > > > On Tue

Re: [Xen-devel] [PATCH] kexec-tools: Read always one vmcoreinfo file

2014-11-13 Thread Petr Tesarik
Hi all, this thread got somehow forgotten because of vacations... Anyway, read below. On Tue, 24 Jul 2012 15:54:10 +0200 Daniel Kiper wrote: > On Tue, Jul 24, 2012 at 10:18:34AM +0200, Petr Tesarik wrote: > > Dne Po 23. ??ervence 2012 22:10:59 Daniel Kiper napsal(a): >