All of lore.kernel.org
 help / color / mirror / Atom feed
From: Robbie Harwood <rharwood@redhat.com>
To: Stefan Berger <stefanb@linux.ibm.com>,
	grub-devel@gnu.org, dkiper@net-space.pl
Cc: development@efficientek.com, dja@axtens.net,
	sudhakar@linux.ibm.com, nasastry@in.ibm.com,
	avnish@imap.linux.ibm.com, Stefan Berger <stefanb@linux.ibm.com>
Subject: Re: [PATCH v4 3/6] ieee1275: support runtime memory claiming
Date: Mon, 06 Feb 2023 15:09:15 -0500	[thread overview]
Message-ID: <jlgv8keftec.fsf@redhat.com> (raw)
In-Reply-To: <20230206150325.3756840-4-stefanb@linux.ibm.com>

[-- Attachment #1: Type: text/plain, Size: 682 bytes --]

Stefan Berger <stefanb@linux.ibm.com> writes:

> -/* The maximum heap size we're going to claim */
> +/* The maximum heap size we're going to claim at boot. Not used by sparc. */
>  #ifdef __i386__
>  #define HEAP_MAX_SIZE		(unsigned long) (64 * 1024 * 1024)
> -#else
> +#else /* __powerpc__ */
>  #define HEAP_MAX_SIZE		(unsigned long) (32 * 1024 * 1024)
>  #endif

In the version of this patch I've been carrying, the max heap is
increased a good bit:

> #define HEAP_MAX_SIZE          (unsigned long) (1 * 1024 * 1024 * 1024)

Should it be made larger here as well?

Full commit:
https://github.com/rhboot/grub2/commit/e572ccc0edb3423976e8301978617382994ed52a

Be well,
--Robbie

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 861 bytes --]

  reply	other threads:[~2023-02-06 20:09 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-02-06 15:03 [PATCH v4 0/6] Dynamic allocation of memory regions and IBM vTPM 2.0 Stefan Berger
2023-02-06 15:03 ` [PATCH v4 1/6] ieee1275: request memory with ibm, client-architecture-support Stefan Berger
2023-02-06 15:03 ` [PATCH v4 2/6] ieee1275: drop len -= 1 quirk in heap_init Stefan Berger
2023-02-16 14:27   ` Robbie Harwood
2023-02-06 15:03 ` [PATCH v4 3/6] ieee1275: support runtime memory claiming Stefan Berger
2023-02-06 20:09   ` Robbie Harwood [this message]
2023-02-06 20:59     ` Stefan Berger
2023-02-16 14:27   ` Robbie Harwood
2023-02-06 15:03 ` [PATCH v4 4/6] ieee1275: implement vec5 for cas negotiation Stefan Berger
2023-02-06 15:03 ` [PATCH v4 5/6] Add memtool module with memory allocation stress-test Stefan Berger
2023-02-06 15:03 ` [PATCH v4 6/6] ibmvtpm: Add support for trusted boot using a vTPM 2.0 Stefan Berger
2023-02-06 20:12   ` Robbie Harwood
2023-02-13 12:58 ` [PATCH v4 0/6] Dynamic allocation of memory regions and IBM " Nageswara R Sastry
2023-02-28 17:50 ` Robbie Harwood
2023-03-01 13:11   ` Daniel Kiper

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=jlgv8keftec.fsf@redhat.com \
    --to=rharwood@redhat.com \
    --cc=avnish@imap.linux.ibm.com \
    --cc=development@efficientek.com \
    --cc=dja@axtens.net \
    --cc=dkiper@net-space.pl \
    --cc=grub-devel@gnu.org \
    --cc=nasastry@in.ibm.com \
    --cc=stefanb@linux.ibm.com \
    --cc=sudhakar@linux.ibm.com \
    /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.