Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1320539

Re: [PATCH] kexec: unmap reserved pages for each error-return way

From Michael Holzheu <holzheu@linux.vnet.ibm.com>
Newsgroups linux.kernel
Subject Re: [PATCH] kexec: unmap reserved pages for each error-return way
Date 2016-01-28 11:40 +0100
Message-ID <qVS9B-24O-25@gated-at.bofh.it> (permalink)
References <qVwLN-2Tu-29@gated-at.bofh.it> <qVDNi-8kN-47@gated-at.bofh.it>
Organization IBM

Show all headers | View raw


On Wed, 27 Jan 2016 11:15:46 -0800
Andrew Morton <akpm@linux-foundation.org> wrote:

> On Wed, 27 Jan 2016 14:48:31 +0300 Dmitry Safonov <dsafonov@virtuozzo.com> wrote:
> 
> > For allocation of kimage failure or kexec_prepare or load segments
> > errors there is no need to keep crashkernel memory mapped.
> > It will affect only s390 as map/unmap hook defined only for it.
> > As on unmap s390 also changes os_info structure let's check return code
> > and add info only on success.
> > 
> 
> This conflicts (both mechanically and somewhat conceptually) with
> Xunlei Pang's "kexec: Introduce a protection mechanism for the
> crashkernel reserved memory" and "kexec: provide
> arch_kexec_protect(unprotect)_crashkres()".
> 
> http://ozlabs.org/~akpm/mmots/broken-out/kexec-introduce-a-protection-mechanism-for-the-crashkernel-reserved-memory.patch
> http://ozlabs.org/~akpm/mmots/broken-out/kexec-introduce-a-protection-mechanism-for-the-crashkernel-reserved-memory-v4.patch
> 
> and
> 
> http://ozlabs.org/~akpm/mmots/broken-out/kexec-provide-arch_kexec_protectunprotect_crashkres.patch
> http://ozlabs.org/~akpm/mmots/broken-out/kexec-provide-arch_kexec_protectunprotect_crashkres-v4.patch

Hmm, It looks to me that arch_kexec_(un)protect_crashkres() has exactly
the same semantics as crash_(un)map_reserved_pages().

On s390 we don't have the crashkernel memory mapped and therefore need
crash_map_reserved_pages() before loading something into crashkernel
memory.

Perhaps I missed something?
Michael

Back to linux.kernel | Previous | NextNext in thread | Find similar | Unroll thread


Thread

Re: [PATCH] kexec: unmap reserved pages for each error-return way Michael Holzheu <holzheu@linux.vnet.ibm.com> - 2016-01-28 11:40 +0100
  Re: [PATCH] kexec: unmap reserved pages for each error-return way Xunlei Pang <xpang@redhat.com> - 2016-01-28 13:00 +0100
    Re: [PATCH] kexec: unmap reserved pages for each error-return way Michael Holzheu <holzheu@linux.vnet.ibm.com> - 2016-01-28 13:50 +0100
      Re: [PATCH] kexec: unmap reserved pages for each error-return way Xunlei Pang <xpang@redhat.com> - 2016-01-28 14:20 +0100
        Re: [PATCH] kexec: unmap reserved pages for each error-return way Michael Holzheu <holzheu@linux.vnet.ibm.com> - 2016-01-28 15:10 +0100

csiph-web