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

2016-08-03 Thread Daniel Kiper
On Mon, Aug 01, 2016 at 10:25:22AM -0400, Konrad Rzeszutek Wilk wrote: > On Mon, Aug 01, 2016 at 04:15:10PM +0200, Petr Tesarik wrote: > > On Mon, 1 Aug 2016 15:47:58 +0200 [...] > I wonder if Xen should do that - as in 'fix' the bootparams to not have it. > Or perhaps Linux code during bootup

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

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

2016-08-01 Thread Konrad Rzeszutek Wilk
On Mon, Aug 01, 2016 at 04:15:10PM +0200, Petr Tesarik wrote: > 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"

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

2016-08-01 Thread Jan Beulich
>>> On 01.08.16 at 16:15, wrote: > 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"

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 Jan Beulich
(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, wrote: >> > On 13/07/16 13:20, Petr Tesarik wrote: >> >> If a crash kernel is loaded,

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 2/2] Allow kdump with crash_kexec_post_notifiers

2016-08-01 Thread Jan Beulich
>>> 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 crash_kexec_post_notifiers, because >> panic notifiers are run before __crash_kexec()

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

2016-07-13 Thread David Vrabel
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 crash_kexec_post_notifiers, because > panic notifiers are run before __crash_kexec() in that case, and this > Xen hook prevents its being called

[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