All of lore.kernel.org
 help / color / mirror / Atom feed
From: AKASHI Takahiro <takahiro.akashi@linaro.org>
To: Bhupesh Sharma <bhsharma@redhat.com>
Cc: Mark Rutland <mark.rutland@arm.com>,
	"linux-efi@vger.kernel.org" <linux-efi@vger.kernel.org>,
	Ard Biesheuvel <ard.biesheuvel@linaro.org>,
	Matt Fleming <matt@codeblueprint.co.uk>,
	kexec@lists.infradead.org, James Morse <james.morse@arm.com>,
	Bhupesh SHARMA <bhupesh.linux@gmail.com>,
	Dave Young <dyoung@redhat.com>,
	"linux-arm-kernel@lists.infradead.org"
	<linux-arm-kernel@lists.infradead.org>
Subject: Re: arm64 crashkernel fails to boot on acpi-only machines due to ACPI regions being no longer mapped as NOMAP
Date: Tue, 9 Jan 2018 13:42:53 +0900	[thread overview]
Message-ID: <20180109030717.GA18820@linaro.org> (raw)
In-Reply-To: <CACi5LpNeSNHoUcM9xOq0bjN_okaEUDbaz1qyuqAct7BSNLQqKQ@mail.gmail.com>

Bhupesh,

On Tue, Jan 09, 2018 at 01:30:07AM +0530, Bhupesh Sharma wrote:
> Hello Akashi,
> 
> On Tue, Dec 26, 2017 at 8:26 AM, Bhupesh Sharma <bhsharma@redhat.com> wrote:
> > On Tue, Dec 26, 2017 at 7:58 AM, AKASHI Takahiro
> > <takahiro.akashi@linaro.org> wrote:
> >> On Tue, Dec 26, 2017 at 09:35:17AM +0800, Dave Young wrote:
> >>> [snip]
> >>> > > > Well, we may be able to change pr_warn() to pr_warn_once() here, but
> >>> > > > I hope that adding "numa=off" to kernel command line should also work.
> >>> > >
> >>> > > Hmm, adding "numa=off" to crashkernel bootargs works, and TBH it was
> >>> > > my initial thought process as well, but I am not sure if this will
> >>> > > cause any regressions on aarch64 systems which use crashdump feature.
> >>> >
> >>> > It should be fine since we use numa=off by default for all other arches
> >>> > ie. x86, ppc64 and s390. Actually disabling numa in kdump kernel can save
> >>> > mm component memory usage.
> >>> >
> >>>
> >>> Forgot to say I means in RHEL and Fedora we use numa=off for kdump..
> >>
> >> Thank you for the clarification.
> >> (It might be better to make numa off automatically if maxcpus == 0 (and 1?).)
> >>
> >
> > Not sure if we can leave this to the distribution-specific kdump
> > scripts (as the crashkernel boot can be held up for sufficient time
> > and may appear stuck). The distribution scripts may be different (for
> > e.g. ubuntu and RHEL/fedora) across distributions and may have
> > different bootarg options.
> >
> > So how about considering a kernel fix only which doesn't require
> > relying on changing the distribution-specific kdump scripts, as we
> > should avoid introducing a regression while trying to fix a regression
> > :)
> >
> > Just my 2 cents.
> >
> 
> Sorry for the delay but I was on holidays in the last week.
> 
> Are you planning to send a patch to fix this issue or do you want me
> to send a RFC version instead?

I should have submitted my own patch before my new year holidays,
but I will do so as soon as possible.

Thanks,
-Takahiro AKASHI


> i think this is a blocking issue for aarch64 kdump support on newer
> kernels (v4.14) and we are already hearing about this issue from other
> users as well, so it would be great to get this fixed now that we have
> root-caused the issue and found a possible way around.
> 
> Regards,
> Bhupesh

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

WARNING: multiple messages have this Message-ID (diff)
From: AKASHI Takahiro <takahiro.akashi-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
To: Bhupesh Sharma <bhsharma-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
Cc: Dave Young <dyoung-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>,
	Ard Biesheuvel
	<ard.biesheuvel-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>,
	Bhupesh SHARMA
	<bhupesh.linux-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
	Matt Fleming
	<matt-mF/unelCI9GS6iBeEJttW/XRex20P6io@public.gmane.org>,
	"linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org"
	<linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org>,
	"linux-efi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
	<linux-efi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org>,
	James Morse <james.morse-5wv7dgnIgG8@public.gmane.org>,
	kexec-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org
Subject: Re: arm64 crashkernel fails to boot on acpi-only machines due to ACPI regions being no longer mapped as NOMAP
Date: Tue, 9 Jan 2018 13:42:53 +0900	[thread overview]
Message-ID: <20180109030717.GA18820@linaro.org> (raw)
In-Reply-To: <CACi5LpNeSNHoUcM9xOq0bjN_okaEUDbaz1qyuqAct7BSNLQqKQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>

Bhupesh,

On Tue, Jan 09, 2018 at 01:30:07AM +0530, Bhupesh Sharma wrote:
> Hello Akashi,
> 
> On Tue, Dec 26, 2017 at 8:26 AM, Bhupesh Sharma <bhsharma-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> wrote:
> > On Tue, Dec 26, 2017 at 7:58 AM, AKASHI Takahiro
> > <takahiro.akashi-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> wrote:
> >> On Tue, Dec 26, 2017 at 09:35:17AM +0800, Dave Young wrote:
> >>> [snip]
> >>> > > > Well, we may be able to change pr_warn() to pr_warn_once() here, but
> >>> > > > I hope that adding "numa=off" to kernel command line should also work.
> >>> > >
> >>> > > Hmm, adding "numa=off" to crashkernel bootargs works, and TBH it was
> >>> > > my initial thought process as well, but I am not sure if this will
> >>> > > cause any regressions on aarch64 systems which use crashdump feature.
> >>> >
> >>> > It should be fine since we use numa=off by default for all other arches
> >>> > ie. x86, ppc64 and s390. Actually disabling numa in kdump kernel can save
> >>> > mm component memory usage.
> >>> >
> >>>
> >>> Forgot to say I means in RHEL and Fedora we use numa=off for kdump..
> >>
> >> Thank you for the clarification.
> >> (It might be better to make numa off automatically if maxcpus == 0 (and 1?).)
> >>
> >
> > Not sure if we can leave this to the distribution-specific kdump
> > scripts (as the crashkernel boot can be held up for sufficient time
> > and may appear stuck). The distribution scripts may be different (for
> > e.g. ubuntu and RHEL/fedora) across distributions and may have
> > different bootarg options.
> >
> > So how about considering a kernel fix only which doesn't require
> > relying on changing the distribution-specific kdump scripts, as we
> > should avoid introducing a regression while trying to fix a regression
> > :)
> >
> > Just my 2 cents.
> >
> 
> Sorry for the delay but I was on holidays in the last week.
> 
> Are you planning to send a patch to fix this issue or do you want me
> to send a RFC version instead?

I should have submitted my own patch before my new year holidays,
but I will do so as soon as possible.

Thanks,
-Takahiro AKASHI


> i think this is a blocking issue for aarch64 kdump support on newer
> kernels (v4.14) and we are already hearing about this issue from other
> users as well, so it would be great to get this fixed now that we have
> root-caused the issue and found a possible way around.
> 
> Regards,
> Bhupesh

WARNING: multiple messages have this Message-ID (diff)
From: takahiro.akashi@linaro.org (AKASHI Takahiro)
To: linux-arm-kernel@lists.infradead.org
Subject: arm64 crashkernel fails to boot on acpi-only machines due to ACPI regions being no longer mapped as NOMAP
Date: Tue, 9 Jan 2018 13:42:53 +0900	[thread overview]
Message-ID: <20180109030717.GA18820@linaro.org> (raw)
In-Reply-To: <CACi5LpNeSNHoUcM9xOq0bjN_okaEUDbaz1qyuqAct7BSNLQqKQ@mail.gmail.com>

Bhupesh,

On Tue, Jan 09, 2018 at 01:30:07AM +0530, Bhupesh Sharma wrote:
> Hello Akashi,
> 
> On Tue, Dec 26, 2017 at 8:26 AM, Bhupesh Sharma <bhsharma@redhat.com> wrote:
> > On Tue, Dec 26, 2017 at 7:58 AM, AKASHI Takahiro
> > <takahiro.akashi@linaro.org> wrote:
> >> On Tue, Dec 26, 2017 at 09:35:17AM +0800, Dave Young wrote:
> >>> [snip]
> >>> > > > Well, we may be able to change pr_warn() to pr_warn_once() here, but
> >>> > > > I hope that adding "numa=off" to kernel command line should also work.
> >>> > >
> >>> > > Hmm, adding "numa=off" to crashkernel bootargs works, and TBH it was
> >>> > > my initial thought process as well, but I am not sure if this will
> >>> > > cause any regressions on aarch64 systems which use crashdump feature.
> >>> >
> >>> > It should be fine since we use numa=off by default for all other arches
> >>> > ie. x86, ppc64 and s390. Actually disabling numa in kdump kernel can save
> >>> > mm component memory usage.
> >>> >
> >>>
> >>> Forgot to say I means in RHEL and Fedora we use numa=off for kdump..
> >>
> >> Thank you for the clarification.
> >> (It might be better to make numa off automatically if maxcpus == 0 (and 1?).)
> >>
> >
> > Not sure if we can leave this to the distribution-specific kdump
> > scripts (as the crashkernel boot can be held up for sufficient time
> > and may appear stuck). The distribution scripts may be different (for
> > e.g. ubuntu and RHEL/fedora) across distributions and may have
> > different bootarg options.
> >
> > So how about considering a kernel fix only which doesn't require
> > relying on changing the distribution-specific kdump scripts, as we
> > should avoid introducing a regression while trying to fix a regression
> > :)
> >
> > Just my 2 cents.
> >
> 
> Sorry for the delay but I was on holidays in the last week.
> 
> Are you planning to send a patch to fix this issue or do you want me
> to send a RFC version instead?

I should have submitted my own patch before my new year holidays,
but I will do so as soon as possible.

Thanks,
-Takahiro AKASHI


> i think this is a blocking issue for aarch64 kdump support on newer
> kernels (v4.14) and we are already hearing about this issue from other
> users as well, so it would be great to get this fixed now that we have
> root-caused the issue and found a possible way around.
> 
> Regards,
> Bhupesh

  reply	other threads:[~2018-01-09  4:38 UTC|newest]

Thread overview: 130+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-11-10 12:09 arm64 crashkernel fails to boot on acpi-only machines due to ACPI regions being no longer mapped as NOMAP Bhupesh Sharma
2017-11-10 12:09 ` Bhupesh Sharma
     [not found] ` <CACi5LpM_95ebYFguPTyjWk+qHT5rDJVXiYDkNWbszo6Zw41zRA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2017-11-10 12:11   ` Bhupesh Sharma
2017-11-10 12:11     ` Bhupesh Sharma
     [not found]     ` <CACi5LpNV_E9pvhTwLcy6vtEj9qbL1ZEHe-5sv=iiW0k9JxPD1Q-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2017-11-13  9:27       ` AKASHI Takahiro
2017-11-13  9:27         ` AKASHI Takahiro
     [not found]         ` <20171113092730.GA29552-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2017-11-14 11:20           ` Ard Biesheuvel
2017-11-14 11:20             ` Ard Biesheuvel
     [not found]             ` <CAKv+Gu_eQ-s0J22tKeHKJme4qXcvxvDkS7vKrNW+o_XtMTkMhQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2017-11-15 10:58               ` Bhupesh Sharma
2017-11-15 10:58                 ` Bhupesh Sharma
     [not found]                 ` <3df4c6c5-0abe-01ee-730d-2edaa5f497d2-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2017-11-16  7:00                   ` AKASHI Takahiro
2017-11-16  7:00                     ` AKASHI Takahiro
     [not found]                     ` <20171116070005.GI29552-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2017-11-26  8:29                       ` Bhupesh SHARMA
2017-11-26  8:29                         ` Bhupesh SHARMA
     [not found]                         ` <CAFTCetQHmpprAVu6uYO+rc5Xi4EUVhmovbmSaU6nM1n1mAH62w-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2017-12-04 14:02                           ` Ard Biesheuvel
2017-12-04 14:02                             ` Ard Biesheuvel
     [not found]                             ` <CAKv+Gu9oda1Ee8AoXsCEw+Bjn-XF3wZA_CsxvqhjtT6_bmJ7uA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2017-12-12 21:51                               ` Bhupesh Sharma
2017-12-12 21:51                                 ` Bhupesh Sharma
     [not found]                                 ` <CACi5LpOZ=WOx14gTwH5jfLozepT2Jw8JSY5x+bfEZ_YaiQvFpw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2017-12-13 10:26                                   ` AKASHI Takahiro
2017-12-13 10:26                                     ` AKASHI Takahiro
     [not found]                                     ` <20171213102624.GC28046-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2017-12-13 10:49                                       ` Ard Biesheuvel
2017-12-13 10:49                                         ` Ard Biesheuvel
     [not found]                                         ` <CAKv+Gu_BmFN9Zg861SCS+R=V4khFykjuOzkmfEknsL=NvWW3Eg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2017-12-13 12:16                                           ` AKASHI Takahiro
2017-12-13 12:16                                             ` AKASHI Takahiro
     [not found]                                             ` <20171213121605.GE28046-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2017-12-13 12:17                                               ` Ard Biesheuvel
2017-12-13 12:17                                                 ` Ard Biesheuvel
     [not found]                                                 ` <CAKv+Gu_G8kBEAdAznVauZVAdJOFkr1vmu0Gf6tOwJfH2CgdufA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2017-12-13 19:22                                                   ` Bhupesh SHARMA
2017-12-13 19:22                                                     ` Bhupesh SHARMA
2017-12-15  8:59                                                   ` AKASHI Takahiro
2017-12-15  8:59                                                     ` AKASHI Takahiro
2017-12-15  9:35                                                     ` Ard Biesheuvel
2017-12-15  9:35                                                       ` Ard Biesheuvel
     [not found]                                                       ` <CAKv+Gu-W5VpVrgA=FVZCCevksaRGOVvPdE+B8WkpZc6AE1jOPw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2017-12-17 21:01                                                         ` Bhupesh Sharma
2017-12-17 21:01                                                           ` Bhupesh Sharma
2017-12-18  5:16                                                           ` Dave Young
2017-12-18  5:16                                                             ` Dave Young
2017-12-18  5:16                                                             ` Dave Young
2017-12-18  5:54                                                             ` AKASHI Takahiro
2017-12-18  5:54                                                               ` AKASHI Takahiro
2017-12-18  5:54                                                               ` AKASHI Takahiro
2017-12-18  8:59                                                               ` Bhupesh SHARMA
2017-12-18  8:59                                                                 ` Bhupesh SHARMA
2017-12-18  8:59                                                                 ` Bhupesh SHARMA
2017-12-18  8:59                                                                 ` Bhupesh SHARMA
2017-12-18 11:18                                                                 ` AKASHI Takahiro
2017-12-18 11:18                                                                   ` AKASHI Takahiro
2017-12-18 11:18                                                                   ` AKASHI Takahiro
2017-12-18 11:18                                                                   ` AKASHI Takahiro
2017-12-18 22:28                                                                   ` Bhupesh Sharma
2017-12-18 22:28                                                                     ` Bhupesh Sharma
2017-12-18 22:28                                                                     ` Bhupesh Sharma
2017-12-19  5:01                                                                 ` AKASHI Takahiro
2017-12-19  5:01                                                                   ` AKASHI Takahiro
2017-12-19  5:01                                                                   ` AKASHI Takahiro
2017-12-19  5:01                                                                   ` AKASHI Takahiro
2017-12-20 19:52                                                                   ` Bhupesh Sharma
2017-12-20 19:52                                                                     ` Bhupesh Sharma
2017-12-20 19:52                                                                     ` Bhupesh Sharma
2017-12-20 19:52                                                                     ` Bhupesh Sharma
2017-12-18 21:28                                                             ` Bhupesh Sharma
2017-12-18 21:28                                                               ` Bhupesh Sharma
2017-12-18 21:28                                                               ` Bhupesh Sharma
2017-12-19  5:25                                                               ` AKASHI Takahiro
2017-12-19  5:25                                                                 ` AKASHI Takahiro
2017-12-19  5:25                                                                 ` AKASHI Takahiro
2017-12-18  5:40                                                     ` Dave Young
2017-12-18  5:40                                                       ` Dave Young
2017-12-18  5:43                                                       ` Dave Young
2017-12-18  5:43                                                         ` Dave Young
2017-12-18  5:43                                                         ` Dave Young
2017-12-18  5:43                                                         ` Dave Young
     [not found]                                                       ` <20171218054009.GA6392-0VdLhd/A9Pl+NNSt+8eSiB/sF2h8X+2i0E9HWUfgJXw@public.gmane.org>
2017-12-19  6:09                                                         ` AKASHI Takahiro
2017-12-19  6:09                                                           ` AKASHI Takahiro
     [not found]                                                           ` <20171219060927.GH28046-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2017-12-19 13:09                                                             ` Ard Biesheuvel
2017-12-19 13:09                                                               ` Ard Biesheuvel
     [not found]                                                               ` <CAKv+Gu-gmbWdZ7rxp5qGrtSBQ7dM=3FqF-Pw=J0LaL=oKTMg4w-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2017-12-20 20:00                                                                 ` Bhupesh Sharma
2017-12-20 20:00                                                                   ` Bhupesh Sharma
2017-12-21 10:34                                                                   ` AKASHI Takahiro
2017-12-21 10:34                                                                     ` AKASHI Takahiro
2017-12-21 10:34                                                                     ` AKASHI Takahiro
2017-12-21 12:06                                                                     ` Bhupesh Sharma
2017-12-21 12:06                                                                       ` Bhupesh Sharma
2017-12-21 12:06                                                                       ` Bhupesh Sharma
2017-12-22  8:33                                                                       ` AKASHI Takahiro
2017-12-22  8:33                                                                         ` AKASHI Takahiro
2017-12-22  8:33                                                                         ` AKASHI Takahiro
2017-12-23 19:51                                                                         ` Bhupesh Sharma
2017-12-23 19:51                                                                           ` Bhupesh Sharma
2017-12-23 19:51                                                                           ` Bhupesh Sharma
2017-12-25  3:25                                                                           ` AKASHI Takahiro
2017-12-25  3:25                                                                             ` AKASHI Takahiro
2017-12-25  3:25                                                                             ` AKASHI Takahiro
2017-12-25 20:14                                                                             ` Bhupesh Sharma
2017-12-25 20:14                                                                               ` Bhupesh Sharma
2017-12-25 20:14                                                                               ` Bhupesh Sharma
2017-12-26  1:32                                                                               ` Dave Young
2017-12-26  1:32                                                                                 ` Dave Young
2017-12-26  1:32                                                                                 ` Dave Young
2017-12-26  1:35                                                                                 ` Dave Young
2017-12-26  1:35                                                                                   ` Dave Young
2017-12-26  1:35                                                                                   ` Dave Young
2017-12-26  2:28                                                                                   ` AKASHI Takahiro
2017-12-26  2:28                                                                                     ` AKASHI Takahiro
2017-12-26  2:28                                                                                     ` AKASHI Takahiro
2017-12-26  2:56                                                                                     ` Bhupesh Sharma
2017-12-26  2:56                                                                                       ` Bhupesh Sharma
2017-12-26  2:56                                                                                       ` Bhupesh Sharma
2017-12-26  6:58                                                                                       ` Dave Young
2017-12-26  6:58                                                                                         ` Dave Young
2017-12-26  6:58                                                                                         ` Dave Young
2018-01-09  5:22                                                                                         ` AKASHI Takahiro
2018-01-09  5:22                                                                                           ` AKASHI Takahiro
2018-01-09  5:22                                                                                           ` AKASHI Takahiro
2018-01-08 20:00                                                                                       ` Bhupesh Sharma
2018-01-08 20:00                                                                                         ` Bhupesh Sharma
2018-01-08 20:00                                                                                         ` Bhupesh Sharma
2018-01-09  4:42                                                                                         ` AKASHI Takahiro [this message]
2018-01-09  4:42                                                                                           ` AKASHI Takahiro
2018-01-09  4:42                                                                                           ` AKASHI Takahiro
2018-01-09 11:46                                                                                           ` Bhupesh Sharma
2018-01-09 11:46                                                                                             ` Bhupesh Sharma
2018-01-09 11:46                                                                                             ` Bhupesh Sharma
2017-12-26  6:56                                                                                     ` Dave Young
2017-12-26  6:56                                                                                       ` Dave Young
2017-12-26  6:56                                                                                       ` Dave Young
2018-01-09  5:02                                                                                       ` AKASHI Takahiro
2018-01-09  5:02                                                                                         ` AKASHI Takahiro
2018-01-09  5:02                                                                                         ` AKASHI Takahiro
2017-11-24  8:47                   ` Dave Young
2017-11-24  8:47                     ` Dave Young

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=20180109030717.GA18820@linaro.org \
    --to=takahiro.akashi@linaro.org \
    --cc=ard.biesheuvel@linaro.org \
    --cc=bhsharma@redhat.com \
    --cc=bhupesh.linux@gmail.com \
    --cc=dyoung@redhat.com \
    --cc=james.morse@arm.com \
    --cc=kexec@lists.infradead.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-efi@vger.kernel.org \
    --cc=mark.rutland@arm.com \
    --cc=matt@codeblueprint.co.uk \
    /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.