Hi,

> So the agreement in first stage is that:
> 1. Accept all memory below 4G.

Yes.

> 2. The PcdLazyAcceptPartialMemorySize is not needed in current stage.

Question is whenever we ever need PcdLazyAcceptPartialMemorySize.  When
accepting all memory below 4G works fast enough (which I think is the
case when using 2M large pages) I don't see a reason to ever change
that.

> 3. The patch of "Pool and page functions accept memory when OOM
> occurs" is not needed. Because OOM will not occur in the situation of
> accepting all memory below 4G.

Probably not in the vast majority of use cases, but there are
exceptions.  When unlocking luks-encrypted partitions the grub boot
loader might need alot of memory for decryption (depends on encryption
parameters).  So being able to accept memory above 4G if needed would be
good.  Skipping that for the first stage and do it later is fine I
think.

take care,
  Gerd



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#93086): https://edk2.groups.io/g/devel/message/93086
Mute This Topic: https://groups.io/mt/93285605/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to