All of lore.kernel.org
 help / color / mirror / Atom feed
From: dwmw2@infradead.org (David Woodhouse)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v34 04/14] arm64: kdump: reserve memory for crash dump kernel
Date: Mon, 03 Apr 2017 09:18:12 +0100	[thread overview]
Message-ID: <1491207492.6020.8.camel@infradead.org> (raw)
In-Reply-To: <20170328065130.16019-2-takahiro.akashi@linaro.org>

On Tue, 2017-03-28 at 15:51 +0900, AKASHI Takahiro wrote:
> 
> +???????if (crash_base == 0) {
> +???????????????/* Current arm64 boot protocol requires 2MB alignment */
> +???????????????crash_base = memblock_find_in_range(0, ARCH_LOW_ADDRESS_LIMIT,
> +???????????????????????????????crash_size, SZ_2M);
> +???????????????if (crash_base == 0) {
> +???????????????????????pr_warn("cannot allocate crashkernel (size:0x%llx)\n",
> +???????????????????????????????crash_size);
> +???????????????????????return;
> +???????????????}
> +???????} else {
> +???????????????/* User specifies base address explicitly. */
> +???????????????if (!memblock_is_region_memory(crash_base, crash_size)) {
> +???????????????????????pr_warn("cannot reserve crashkernel: region is not memory\n");
> +???????????????????????return;
> +???????????????}
> +
> +???????????????if (memblock_is_region_reserved(crash_base, crash_size)) {
> +???????????????????????pr_warn("cannot reserve crashkernel: region overlaps reserved memory\n");
> +???????????????????????return;
> +???????????????}
> +
> +???????????????if (!IS_ALIGNED(crash_base, SZ_2M)) {
> +???????????????????????pr_warn("cannot reserve crashkernel: base address is not 2MB aligned\n");
> +???????????????????????return;
> +???????????????}

You still have typos here.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/x-pkcs7-signature
Size: 4938 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20170403/34889fe5/attachment-0001.bin>

WARNING: multiple messages have this Message-ID (diff)
From: David Woodhouse <dwmw2@infradead.org>
To: AKASHI Takahiro <takahiro.akashi@linaro.org>,
	catalin.marinas@arm.com, will.deacon@arm.com
Cc: mark.rutland@arm.com, panand@redhat.com,
	ard.biesheuvel@linaro.org, geoff@infradead.org,
	kexec@lists.infradead.org, james.morse@arm.com,
	Mark Salter <msalter@redhat.com>,
	bauerman@linux.vnet.ibm.com, sgoel@codeaurora.org,
	dyoung@redhat.com, linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v34 04/14] arm64: kdump: reserve memory for crash dump kernel
Date: Mon, 03 Apr 2017 09:18:12 +0100	[thread overview]
Message-ID: <1491207492.6020.8.camel@infradead.org> (raw)
In-Reply-To: <20170328065130.16019-2-takahiro.akashi@linaro.org>


[-- Attachment #1.1: Type: text/plain, Size: 1730 bytes --]

On Tue, 2017-03-28 at 15:51 +0900, AKASHI Takahiro wrote:
> 
> +       if (crash_base == 0) {
> +               /* Current arm64 boot protocol requires 2MB alignment */
> +               crash_base = memblock_find_in_range(0, ARCH_LOW_ADDRESS_LIMIT,
> +                               crash_size, SZ_2M);
> +               if (crash_base == 0) {
> +                       pr_warn("cannot allocate crashkernel (size:0x%llx)\n",
> +                               crash_size);
> +                       return;
> +               }
> +       } else {
> +               /* User specifies base address explicitly. */
> +               if (!memblock_is_region_memory(crash_base, crash_size)) {
> +                       pr_warn("cannot reserve crashkernel: region is not memory\n");
> +                       return;
> +               }
> +
> +               if (memblock_is_region_reserved(crash_base, crash_size)) {
> +                       pr_warn("cannot reserve crashkernel: region overlaps reserved memory\n");
> +                       return;
> +               }
> +
> +               if (!IS_ALIGNED(crash_base, SZ_2M)) {
> +                       pr_warn("cannot reserve crashkernel: base address is not 2MB aligned\n");
> +                       return;
> +               }

You still have typos here.

[-- Attachment #1.2: smime.p7s --]
[-- Type: application/x-pkcs7-signature, Size: 4938 bytes --]

[-- Attachment #2: Type: text/plain, Size: 143 bytes --]

_______________________________________________
kexec mailing list
kexec@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/kexec

  parent reply	other threads:[~2017-04-03  8:18 UTC|newest]

Thread overview: 95+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-03-28  6:48 [PATCH v34 00/14] arm64: add kdump support AKASHI Takahiro
2017-03-28  6:48 ` AKASHI Takahiro
2017-03-28  6:50 ` [PATCH v34 01/14] memblock: add memblock_clear_nomap() AKASHI Takahiro
2017-03-28  6:50   ` AKASHI Takahiro
2017-03-28  6:50   ` AKASHI Takahiro
2017-03-28  9:47   ` Ard Biesheuvel
2017-03-28  9:47     ` Ard Biesheuvel
2017-03-28  9:47     ` Ard Biesheuvel
2017-03-28  6:50 ` [PATCH v34 02/14] memblock: add memblock_cap_memory_range() AKASHI Takahiro
2017-03-28  6:50   ` AKASHI Takahiro
2017-03-28  6:50   ` AKASHI Takahiro
2017-03-28  9:48   ` Ard Biesheuvel
2017-03-28  9:48     ` Ard Biesheuvel
2017-03-28  9:48     ` Ard Biesheuvel
2017-03-28  6:51 ` [PATCH v34 03/14] arm64: limit memory regions based on DT property, usable-memory-range AKASHI Takahiro
2017-03-28  6:51   ` AKASHI Takahiro
2017-03-28  9:50   ` Ard Biesheuvel
2017-03-28  9:50     ` Ard Biesheuvel
2017-03-28  6:51 ` [PATCH v34 04/14] arm64: kdump: reserve memory for crash dump kernel AKASHI Takahiro
2017-03-28  6:51   ` AKASHI Takahiro
2017-03-28  9:52   ` Ard Biesheuvel
2017-03-28  9:52     ` Ard Biesheuvel
2017-04-03  8:18   ` David Woodhouse [this message]
2017-04-03  8:18     ` David Woodhouse
2017-04-04  5:41     ` AKASHI Takahiro
2017-04-04  5:41       ` AKASHI Takahiro
2017-04-04  6:14       ` David Woodhouse
2017-04-04  6:14         ` David Woodhouse
2017-04-04  7:35         ` AKASHI Takahiro
2017-04-04  7:35           ` AKASHI Takahiro
2017-04-04  7:39           ` Ard Biesheuvel
2017-04-04  7:39             ` Ard Biesheuvel
2017-04-04  7:44           ` David Woodhouse
2017-04-04  7:44             ` David Woodhouse
2017-04-04  9:26             ` Will Deacon
2017-04-04  9:26               ` Will Deacon
2017-04-13 12:15               ` David Woodhouse
2017-04-13 12:15                 ` David Woodhouse
2017-04-13 12:17               ` [PATCH 1/2] arm64: Fix power-of-ten vs. power-of-two prefixes in user-visible messages David Woodhouse
2017-04-13 12:17                 ` David Woodhouse
2017-04-19  9:29                 ` Geert Uytterhoeven
2017-04-19  9:29                   ` Geert Uytterhoeven
2017-04-13 12:18               ` [PATCH 2/2] arm64: Fix power-of-ten vs. power-of-two prefixes in comments etc David Woodhouse
2017-04-13 12:18                 ` David Woodhouse
2017-04-16 23:12                 ` Simon Horman
2017-04-16 23:12                   ` Simon Horman
2017-04-16 23:12                   ` Simon Horman
2017-04-17 11:54                   ` Geert Uytterhoeven
2017-04-17 11:54                     ` Geert Uytterhoeven
2017-04-17 11:54                     ` Geert Uytterhoeven
2017-04-18 14:13                     ` Catalin Marinas
2017-04-18 14:13                       ` Catalin Marinas
2017-04-18 14:13                       ` Catalin Marinas
2017-04-19 14:25                       ` Olof Johansson
2017-04-19 14:25                         ` Olof Johansson
2017-04-19 14:25                         ` Olof Johansson
2017-03-28  6:51 ` [PATCH v34 05/14] arm64: mm: add set_memory_valid() AKASHI Takahiro
2017-03-28  6:51   ` AKASHI Takahiro
2017-03-28  9:54   ` Ard Biesheuvel
2017-03-28  9:54     ` Ard Biesheuvel
2017-03-28  6:51 ` [PATCH v34 06/14] arm64: kdump: protect crash dump kernel memory AKASHI Takahiro
2017-03-28  6:51   ` AKASHI Takahiro
2017-03-28 10:07   ` Ard Biesheuvel
2017-03-28 10:07     ` Ard Biesheuvel
2017-03-28 11:07     ` AKASHI Takahiro
2017-03-28 11:07       ` AKASHI Takahiro
2017-03-28 14:05       ` Ard Biesheuvel
2017-03-28 14:05         ` Ard Biesheuvel
2017-03-30  9:56         ` AKASHI Takahiro
2017-03-30  9:56           ` AKASHI Takahiro
2017-03-30 13:58           ` Ard Biesheuvel
2017-03-30 13:58             ` Ard Biesheuvel
2017-04-03  2:28             ` AKASHI Takahiro
2017-04-03  2:28               ` AKASHI Takahiro
2017-03-28  6:51 ` [PATCH v34 07/14] arm64: hibernate: preserve kdump image around hibernation AKASHI Takahiro
2017-03-28  6:51   ` AKASHI Takahiro
2017-03-28  6:51 ` [PATCH v34 08/14] arm64: kdump: implement machine_crash_shutdown() AKASHI Takahiro
2017-03-28  6:51   ` AKASHI Takahiro
2017-03-28  6:51 ` [PATCH v34 09/14] arm64: kdump: add VMCOREINFO's for user-space tools AKASHI Takahiro
2017-03-28  6:51   ` AKASHI Takahiro
2017-03-28  6:51 ` [PATCH v34 10/14] arm64: kdump: provide /proc/vmcore file AKASHI Takahiro
2017-03-28  6:51   ` AKASHI Takahiro
2017-03-28  6:51 ` [PATCH v34 11/14] arm64: kdump: enable kdump in defconfig AKASHI Takahiro
2017-03-28  6:51   ` AKASHI Takahiro
2017-03-28  6:51 ` [PATCH v34 12/14] Documentation: kdump: describe arm64 port AKASHI Takahiro
2017-03-28  6:51   ` AKASHI Takahiro
     [not found] ` <20170328064831.15894-1-takahiro.akashi-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2017-03-28  6:52   ` [PATCH v34 13/14] Documentation: dt: chosen properties for arm64 kdump AKASHI Takahiro
2017-03-28  6:52     ` AKASHI Takahiro
2017-03-28  6:52     ` AKASHI Takahiro
2017-03-28  6:53   ` [PATCH v34 14/14] efi/libstub/arm*: Set default address and size cells values for an empty dtb AKASHI Takahiro
2017-03-28  6:53     ` AKASHI Takahiro
2017-03-28  6:53     ` AKASHI Takahiro
2017-03-28 10:08     ` Ard Biesheuvel
2017-03-28 10:08       ` Ard Biesheuvel
2017-03-28 10:08       ` Ard Biesheuvel

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1491207492.6020.8.camel@infradead.org \
    --to=dwmw2@infradead.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.