On Fri, 16 Mar 2018 12:20:25 +0100
Simon Horman <horms@xxxxxxxxxxxx> wrote:
> On Thu, Mar 15, 2018 at 12:13:18PM +0100, Michal Suchánek wrote:
> > On Thu, 15 Mar 2018 11:38:30 +0100
> > Simon Horman <horms@xxxxxxxxxxxx> wrote:
> > >
> > >
> > > if (do_load && (kexec_flags & KEXEC_ON_CRASH) &&
> > > !is_crashkernel_mem_reserved()) {
> > > die("Memory for crashkernel is not reserved\n"
> > > "Please reserve memory by passing"
> > > "\"crashkernel=X@Y\" parameter to kernel\n"
> > > "Then try to loading kdump kernel\n");
> > > }
> >
> > Do you not need memory for kexec -s? This looks broken to start
> > with.
>
> Could you propose a fix? I realise your patchset may not introduce
> this problem. But it seems to me that it makes things slightly worse
> or at the very least perpetuates the notion that the above is correct.
Yes, it makes sense to fix the condition.
>
> > > ...
> > >
> > > if ((result == 0) && do_load_jump_back_helper) {
> >
> > And yes, this should not be allowed with -s
>
> Is the simple fix here for your patch to add an extra condition
> to the if statement above?
This is not supported with -s but nothing prevents setting the flag. So
a test for kexec_load should be added I guess.
Thanks
Michal
_______________________________________________
kexec mailing list
kexec@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/kexec
