Skip to content

OpenAMP System Reference Meeting Notes 2025

Nathalie Chan King Choy edited this page Jan 9, 2025 · 1 revision

Table of Contents

2025-01-08

Attended

  • Bill, Tanmay, Arnaud, Iulia, Nathalie, Dan

Agenda

  • Recent previous assigned actions
  • Felipe messaged in Discord: if he can be added to reviewer list for OpenAMP stuff in Zephyr
  • https://github.com/OpenAMP/openamp-docs/pull/37
    • Who else would be good to pull into the docs review?
    • Sipke requested a meeting w/ Bill & Arnaud to discuss outstanding items from Arnaud's review comments as well as any other items or direction.
  • Pending patches in Xilinx OpenAMP that will become a big PR in system reference to modify Xilinx FW code to latest
  • Schedule of next meeting: Nathalie OOO on 22nd - would it work to move from 1/22 to 1/21 or does someone else want to host it?
  • Nothing is posted for GSOC 2025 yet
  • Individual updates / 2025 Task tracker - Please populate with tasks planned for 2025

Decisions/conclusions

  • Iulia and Felipe can submit PRs to be added as collaborators for OpenAMP in Zephyr w/ justification why, Arnaud can approve as support & Zephyr leads will accept the PRs if they agree.
  • Bill will let Sipke know to split up his work into smaller PRs. For PR37, it will probably be easiest to merge it to a branch, so that GitHub Issues can be filed for specific changes.

Action items

  • Iulia to send Zephyr PR to add herself w/ justification of why should be collaborator for OpenAMP in Zephyr & notify Arnaud on Discord to approve it
  • Iulia will create a GitHub issue regarding resource table sample for i.MX8ULP (Libmetal virtual to physical is where NXP sample crashes) & can discuss there
  • Dan, Tanmay, Iulia: Review Sipke's doc pull request in the next week or so, and share feedback in the PR or with Bill and Arnaud before the meeting with Sipke https://github.com/OpenAMP/openamp-docs/pull/37
  • Bill will try to take a look at Arnaud's POC (linked in the comments of row 7 in the new Google Sheet) in the next week or so
  • Arnaud to bring this up his POC in virtio-msg meeting w/ Edgar

Notes

  • Recent previous assigned actions
    • Getting Iulia added to the openamp-rp meeting series
      • Bill added Iulia today
    • https://github.com/OpenAMP/openamp-demo/issues/8
      • Lopper demo update is merged. Tanmay to work on the update to the readme & will submit to openamp-docs
  • Zephyr
    • Iulia tested the PR that Arnaud submitted 8M+ and another board
    • Iulia would like to get added as contributor for OpenAMP in Zephyr
      • Carlo C is no longer active
      • Iulia to send Zephyr PR to add herself w/ justification of why should be collaborator for OpenAMP in Zephyr & notify Arnaud on Discord to approve it
      • If Zephyr team agrees, they will merge it
    • Felipe messaged in Discord: if he can be added to collaborator list for OpenAMP stuff in Zephyr
      • Felipe's review helped Arnaud's PR get merged
  • OpenAMP resource table sample
    • i.MX8ULP
      • DSP addresses are remapped for Cortex-A
      • OpenAMP sample works w/ addresses that are same on both on main & secondary core
      • Linux will do the translation: PA2DA translation (PA is physical address from Linux POV & DA is physical address from coprocessor POV)
      • ST has DT node parent w/ bus that declares the DMA range & specify the translation at this level -> Can look at ST driver for Linux
      • Coprocessor will have not map on virtual address, so needs to work with physical address
      • Iulia will create a GitHub issue regarding resource table sample for i.MX8ULP (Libmetal virtual to physical is where NXP sample crashes) & can discuss there
      • Limitation for RPMsg virtio port: no translation for buffer itself. Put address for vring descriptor and there is no translation for this address, so may need something on DSP side.
  • How to split work on reviewing Sipke's docs PR?
    • Bill & Arnaud will probably meet w/ Sipke next week or the week after
    • Any feedback to Arnaud & Bill in the next week or so would be greatly appreciated
      • Can reply in PR or send email
    • Bill likes idea of creating a branch for docs-next where we can start building something up there b/c big PRs are hard to digest. Prefer if Sipke submits multiple PRs for his work. If we accepted this into a branch, then could make PRs for what we want changed
    • Bill: There is a diagram w/ wrong architecture
    • Iulia, Tanmay & Dan will review
  • Pending patches in Xilinx OpenAMP that will become a big PR in system reference to modify Xilinx FW code to latest
    • Were not putting anything into apps directory, and now would like to upstream some patches to system-reference
    • Attach-detach will be possible from baremetal firmware
    • Baremetal demos would be able to work with DT flow
    • 1 big commit & which commits are taken from downstream openamp repo b/c hard to have each change separately. Might be too big, but let's see the PR first.
  • Task tracker / individual updates
    • 2025 task tracker has the remaining 2024 tasks. Please add any new 2025 tasks to the new Google Sheet
    • Arnaud: virtio-msg: first POC is delivered on GitHub at these links. Do Bill or Edgar have time to look?
      • Bill will try to take a look at Arnaud's POC (linked in the comments of row 7 in the new Google Sheet) in the next week or so
      • Arnaud to bring this up his POC in virtio-msg meeting w/ Edgar
      • It's a different queue structure than what Edgar has today. Arnaud has Zephyr on the other side & Edgar has QEMU on the other side.
      • Arnaud: Maybe could propose an API that could also be used by application? Would be good to have some standardization of the system.
      • Bill: Alternative shared memory structure that could complement virtqueue structure
      • Arnaud: That's why put it in separate file to be able to expose it, but only implemented device part. POC - not mature code yet.
    • Bill:
      • Will see what progress can make on OpenAMP release best effort b/c demo upcoming + some OOO time
      • Arnaud already has the libraries tagged & that's most important. For the other stuff, don't want to tag without testing b/c that doesn't add value. Most ppl look at latest docs anyway & not tag.
    • Dan
      • No updates on existing items. Thinking of some new items
      • Bill: Viresh has done rust work & tested with Xen and FFA. Might be adaptable to co-processors. Is Dan a Rust person?
        • Not really.
      • Thinking of RTOS-RTOS scenario w/ running VxWorks on A and R cores on similar setup to ZCU102 Xilinx QEMU setup w. updated virtio-msg implementation (simple shared memory, no Xen stuff), and see how that would work
      • Would like to see rev up of version, TBD if will get time
    • Iulia
      • i.MX8ULP: Linux on the A core
      • Would be nice to have in Zephyr same like how we have Linux & remoteproc to start & stop the core. Want to run application on DSP and start from M core.
      • Bill: Library has that support in it, but doesn't get tested, so having a real use case will be helpful. What we had was probably the Zynq-7000 stuff that is no longer supported.
      • i.MX RT700 was announce by NXP at end of September
        • Will be helpful to start DSP form M33 core
        • If have support in OpenAMP, can try to test it
      • Tanmay: There is load firmware demo that shows how to use library to load firmware on the core.
      • Arnaud: Will need to create a remoteproc framework in Zephyr that relies on OpenAMP that gets the file from file system
      • Added 2 tasks to the tracking sheet
    • Tanmay
      • No change in statuses
      • Won't handle all of these in this release b/c have a lot of work in downstream repos
      • Added a task to the tracking sheet