All of lore.kernel.org
 help / color / mirror / Atom feed
From: Patrick Steinhardt <ps@pks.im>
To: The development of GNU GRUB <grub-devel@gnu.org>
Cc: Stefan Berger <stefanb@linux.ibm.com>,
	Daniel Kiper <dkiper@net-space.pl>,
	Stefan Berger <stefanb@linux.vnet.ibm.com>,
	Daniel Axtens <dja@axtens.net>
Subject: Re: [PATCH v3 2/4] ieee1275: claim more memory
Date: Sun, 8 Aug 2021 14:53:31 +0200	[thread overview]
Message-ID: <YQ/Tvyt2DUSauHPM@ncase> (raw)
In-Reply-To: <YQ6wnY+gjUr2bIyF@ncase>

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

On Sat, Aug 07, 2021 at 06:11:09PM +0200, Patrick Steinhardt wrote:
> On Wed, Aug 04, 2021 at 08:40:01AM -0400, Stefan Berger wrote:
> > 
> > On 8/4/21 7:19 AM, Daniel Kiper wrote:
> > > CC-ing Patrick.
> > >
> > > On Fri, Jul 30, 2021 at 11:45:38AM -0400, Stefan Berger wrote:
> > >> From: Daniel Axtens <dja@axtens.net>
> > >>
> > >> On powerpc-ieee1275, we are running out of memory trying to verify
> > >> anything. This is because:
> > >>
> > >>   - we have to load an entire file into memory to verify it. This is
> > >>     extremely difficult to change with appended signatures.
> > >>   - We only have 32MB of heap.
> > >>   - Distro kernels are now often around 30MB.
> > >>
> > >> So we want to claim more memory from OpenFirmware for our heap.
> > > Stefan, Patrick, Daniel, may I ask one of you to work on the [1]
> > > proposal. I want to have this solution implemented.
> > 
> > Is the design in [1] correct just that the patches need forward-porting?
> > 
> >    Stefan
> 
> Honestly, I don't quite recall the state back when I was working on it.
> Forward-porting the patch set is trivial to do, there aren't really any
> conflicts. I may try to invest some time into it tomorrow to see whether
> it did work as advertised.
> 
> Patrick

So I've checked with my LUKS2/Argon2 patch set, and it does work as
expected with a memory hardness parameter of 1GB. I'll send out a
revised patch series of those patches for review.

Patrick

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

  reply	other threads:[~2021-08-08 12:53 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-07-30 15:45 [PATCH v3 0/4] Add support for trusted boot on IBM PPC platform Stefan Berger
2021-07-30 15:45 ` [PATCH v3 1/4] ieee1275: Move #defines into common ieee1275.h header Stefan Berger
2021-08-04 10:42   ` Daniel Kiper
2021-08-05 22:22   ` Stefan Berger
2021-08-09 12:17     ` Daniel Kiper
2021-07-30 15:45 ` [PATCH v3 2/4] ieee1275: claim more memory Stefan Berger
2021-08-04 11:19   ` Daniel Kiper
2021-08-04 12:40     ` Stefan Berger
2021-08-05 13:15       ` Daniel Kiper
2021-08-07 16:11       ` Patrick Steinhardt
2021-08-08 12:53         ` Patrick Steinhardt [this message]
2021-07-30 15:45 ` [PATCH v3 3/4] ieee1275: request memory with ibm, client-architecture-support Stefan Berger
2021-07-30 15:45 ` [PATCH v3 4/4] ibmvtpm: Add support for trusted boot using a vTPM 2.0 Stefan Berger
2021-08-04 11:09   ` 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=YQ/Tvyt2DUSauHPM@ncase \
    --to=ps@pks.im \
    --cc=dja@axtens.net \
    --cc=dkiper@net-space.pl \
    --cc=grub-devel@gnu.org \
    --cc=stefanb@linux.ibm.com \
    --cc=stefanb@linux.vnet.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.