kexec.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: Baoquan He <bhe@redhat.com>
To: "Goel, Sameer" <sgoel@codeaurora.org>
Cc: Geoff Levand <geoff@infradead.org>, kexec@lists.infradead.org
Subject: Re: [PATCH] kexec: Increase the upper limit for RAM segments
Date: Tue, 1 Nov 2016 09:37:14 +0800	[thread overview]
Message-ID: <20161101013714.GA20838@x1> (raw)
In-Reply-To: <43d2add1-15f5-7e62-b8ce-9a8afeff71f3@codeaurora.org>

On 10/31/16 at 01:11pm, Goel, Sameer wrote:
> This change is needed in case when the System Ram listings are retrieved
> from /proc/iomem. On a newer UEFI based Qualcomm system, the number of
> regions listed were close to 34. So, the actual system ram region was not
> covered and the kexec tool failed to load the kernel image.
> 
> So, I have requested an increase in the number of memory regions.
> Thanks,

It would be great to add this to your patch log so that reviewers can
understand it better. Better repost with an updated patch log, copying
these as patch log is good enough.

> Sameer
> 
> On 10/21/2016 11:05 AM, Geoff Levand wrote:
> > Hi Sameer.
> > 
> > On 10/20/2016 04:14 PM, Sameer Goel wrote:
> > > Change the current hardcoded values to 64 from 16.
> > 
> > Please add an explanation of why this change is needed.
> > 
> > -Geoff
> > 
> > _______________________________________________
> > kexec mailing list
> > kexec@lists.infradead.org
> > http://lists.infradead.org/mailman/listinfo/kexec
> > 
> 
> -- 
> Qualcomm Innovation Center, Inc.
> The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
> a Linux Foundation Collaborative Project.
> 
> _______________________________________________
> kexec mailing list
> kexec@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/kexec

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

  reply	other threads:[~2016-11-01  1:37 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-10-20 23:14 [PATCH] kexec: Increase the upper limit for RAM segments Sameer Goel
2016-10-21 17:05 ` Geoff Levand
2016-10-31 19:11   ` Goel, Sameer
2016-11-01  1:37     ` Baoquan He [this message]
2016-11-01 19:24     ` Geoff Levand

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=20161101013714.GA20838@x1 \
    --to=bhe@redhat.com \
    --cc=geoff@infradead.org \
    --cc=kexec@lists.infradead.org \
    --cc=sgoel@codeaurora.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).