From: Randolph Sapp <rs@ti.com>
To: Randolph Sapp <rs@ti.com>,
Robert Nelson <robertcnelson@gmail.com>,
"Erik Welsh" <Erik.Welsh@octavosystems.com>
Cc: Anshul Dalal <anshuld@ti.com>, <bb@ti.com>, <nm@ti.com>,
<trini@konsulko.com>, <afd@ti.com>, <u-boot@lists.denx.de>
Subject: Re: [PATCH 2/3] k3-am62-pocketbeagle2: add initial board support
Date: Thu, 26 Mar 2026 18:23:20 -0500 [thread overview]
Message-ID: <DHD3N5DPHPVI.159WWX3H4U3Q3@ti.com> (raw)
In-Reply-To: <DHCAJ6L5XC77.9F2NK2OOEAT5@ti.com>
On Wed Mar 25, 2026 at 7:34 PM CDT, Randolph Sapp wrote:
> On Mon Mar 23, 2026 at 2:46 PM CDT, Robert Nelson wrote:
>> On Mon, Mar 23, 2026 at 2:37 PM Randolph Sapp <rs@ti.com> wrote:
>>> On Fri Mar 20, 2026 at 10:32 AM CDT, Robert Nelson wrote:
>>> >> > +++ b/configs/am62_pocketbeagle2_a53_defconfig
>>> >> > @@ -6,11 +6,11 @@ CONFIG_SPL_LIBCOMMON_SUPPORT=y
>>> >> > CONFIG_SPL_LIBGENERIC_SUPPORT=y
>>> >> > CONFIG_NR_DRAM_BANKS=1
>>> >> > CONFIG_SOC_K3_AM625=y
> [snip]
>>> >> > @@ -120,7 +115,8 @@ CONFIG_SYSRESET_TI_SCI=y
>>> >> > CONFIG_EXT4_WRITE=y
>>> >> > CONFIG_FS_FAT_MAX_CLUSTSIZE=16384
>>> >> > CONFIG_LZO=y
>>> >> > -CONFIG_EFI_SET_TIME=y
>>> >> > +CONFIG_SYS_MEM_TOP_HIDE=0x4000000
>>> >>
>>> >> Any reason why we are using TOP_HIDE here instead of just moving OPTEE
>>> >> lower in DDR like we do on the 512MiB AM6254atl EVM?
>>> >
>>> > Sorry, that is now a legacy setting before OPTEE finally got moved in
>>> > v2026.01, as this had been developed thru v2025 u-boot releases..
>>> >
>>>
>>> Well, it's worth noting that this change was not done in the usual way, and
>>> involves user interaction during the build beyond selecting a defconfig.
>>>
>>> https://texasinstruments.github.io/processor-sdk-doc/processor-sdk-linux-AM62X/esd/docs/master/linux/Foundational_Components_ATF.html
>>> https://git.ti.com/cgit/arago-project/meta-ti/tree/meta-ti-bsp/conf/machine/am62xxsip-evm.conf
>>>
>>> Robert, are you alright with me making the requested changes?
>>
>> I guess as long as we document it. I know the Octavo OSD62-sip @Erik
>> Welsh will also be building on both am62xxsip and pocketbeagle2 based
>> on the 512MB and then larger memory sizes (1G, 2G, etc.).
>>
>> Regards,
>
> Oh boy, adjusting the memory maps kept getting me out of memory errors in the
> EFI flow that I knew should not be true. Found something fun: LMB reserved
> memory regions do not match EFI reserved memory regions. EFI's
> efi_carve_out_dt_rsv is setting regions to be more strict that LMB's base
> requirements. When this occurs and an allocation runs into this discrepancy,
> that allocation and all future allocation requests in the EFI flow will begin to
> fail as they are repeatedly given the same LMB start address in the unapproved
> region.
>
> Randolph
Alright, looking into the allocation helpers it seems that
EFI_CONVENTIONAL_MEMORY can be remapped in efi_allocate_pages so long as LMB
agrees that it's free. This aligns with my understanding of the UEFI spec as
well. I dumped the EFI memory map and noticed there were 2 fragmented sections
of EFI_CONVENTIONAL_MEMORY that it could still use.
Wired up efi_allocate_pages to go to those regions and attempt to allocate from
there in the even an LMB_MEM_ALLOC_MAX or LMB_MEM_ALLOC_ANY start failing. Seems
to have worked, but now I'm seeing the following reported in the kernel:
[ 0.048167] [Firmware Bug]: Unable to handle paging request in EFI runtime service
[ 0.048246] ------------[ cut here ]------------
[ 0.048249] WARNING: CPU: 0 PID: 1 at /usr/src/kernel/drivers/firmware/efi/runtime-wrappers.c:341 __efi_queue_work+0xd4/0x108
[ 0.048270] Modules linked in:
[ 0.048285] CPU: 0 UID: 0 PID: 1 Comm: swapper/0 Tainted: G I 6.18.13-ti-00636-g30182cf3ac7d-dirty #1 PREEMPT
[ 0.048296] Tainted: [I]=FIRMWARE_WORKAROUND
[ 0.048299] Hardware name: beagle BeagleBoard.org PocketBeagle2/BeagleBoard.org PocketBeagle2, BIOS 2026.04-rc5-00003-gf1dace477fb8-dirty 04/01/2026
[ 0.048305] pstate: 60000005 (nZCv daif -PAN -UAO -TCO -DIT -SSBS BTYPE=--)
[ 0.048312] pc : __efi_queue_work+0xd4/0x108
[ 0.048318] lr : __efi_queue_work+0xc0/0x108
[ 0.048323] sp : ffff80008003bc80
[ 0.048326] x29: ffff80008003bc80 x28: ffffad05b176bad4 x27: ffffad05b16b00ac
[ 0.048336] x26: ffffad05b1622e60 x25: ffffad05b1953000 x24: ffffad05b1749050
[ 0.048345] x23: 0000000000000004 x22: ffff80008003bd1e x21: ffff80008003bd20
[ 0.048353] x20: ffff80008003bd28 x19: ffffad05b19cd5d8 x18: fffffffffffe2f20
[ 0.048362] x17: 0000000000007000 x16: ffff000001c065a0 x15: 0000000000000000
[ 0.048370] x14: 000000000000008a x13: ffff000001cf8090 x12: 0000000000000001
[ 0.048379] x11: 00000000000000c0 x10: 00000000000009f0 x9 : ffff80008003bad0
[ 0.048387] x8 : ffff000001cf8a50 x7 : 0000000000000001 x6 : 0000000000000001
[ 0.048395] x5 : 0000000000000000 x4 : 0000000000000000 x3 : 0000000000000001
[ 0.048403] x2 : 0000000000000000 x1 : 8000000000000015 x0 : 8000000000000015
[ 0.048413] Call trace:
[ 0.048417] __efi_queue_work+0xd4/0x108 (P)
[ 0.048426] virt_efi_get_next_variable+0x5c/0xac
[ 0.048434] efisubsys_init+0x148/0x390
[ 0.048444] do_one_initcall+0x60/0x1d4
[ 0.048457] kernel_init_freeable+0x248/0x2c4
[ 0.048468] kernel_init+0x20/0x140
[ 0.048478] ret_from_fork+0x10/0x20
[ 0.048489] ---[ end trace 0000000000000000 ]---
U-boot itself doesn't report any errors, so I want to assume this is probably a
configuration issue regarding something else I've forgotten to add.
Anyone got any quick/obvious comments before I start digging through more of
that? Was my assumption about EFI_CONVENTIONAL_MEMORY incorrect? Is this, as
unlikely as I think it is, actually just a known issue right now?
next prev parent reply other threads:[~2026-03-26 23:23 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-03-17 22:37 [PATCH 0/3] k3-am62-pocketbeagle2: add board and variant support rs
2026-03-17 22:37 ` [PATCH 1/3] arm: mach-k3: am62: add &main_uart6 to clock and pwr tree rs
2026-03-17 22:37 ` [PATCH 2/3] k3-am62-pocketbeagle2: add initial board support rs
2026-03-20 9:29 ` Anshul Dalal
2026-03-20 15:32 ` Robert Nelson
2026-03-23 19:37 ` Randolph Sapp
2026-03-23 19:46 ` Robert Nelson
2026-03-26 0:34 ` Randolph Sapp
2026-03-26 23:23 ` Randolph Sapp [this message]
2026-03-27 14:25 ` Marko Mäkelä
2026-03-27 14:53 ` Robert Nelson
2026-03-27 19:15 ` Marko Mäkelä
2026-03-28 8:18 ` Marko Mäkelä
2026-03-28 16:39 ` Robert Nelson
2026-03-29 17:13 ` Marko Mäkelä
2026-03-30 14:41 ` Robert Nelson
2026-03-30 23:57 ` Randolph Sapp
2026-03-31 16:13 ` Robert Nelson
2026-03-31 22:54 ` Randolph Sapp
2026-03-17 22:38 ` [PATCH 3/3] k3-am62-pocketbeagle2: add support for 1GB variant rs
2026-03-20 10:10 ` Anshul Dalal
2026-03-20 15:26 ` Robert Nelson
2026-03-23 18:38 ` Randolph Sapp
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=DHD3N5DPHPVI.159WWX3H4U3Q3@ti.com \
--to=rs@ti.com \
--cc=Erik.Welsh@octavosystems.com \
--cc=afd@ti.com \
--cc=anshuld@ti.com \
--cc=bb@ti.com \
--cc=nm@ti.com \
--cc=robertcnelson@gmail.com \
--cc=trini@konsulko.com \
--cc=u-boot@lists.denx.de \
/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