-
Notifications
You must be signed in to change notification settings - Fork 295
OpenAMP remoteproc Subgroup Meeting Notes 2023
Bill Mills edited this page Jun 15, 2023
·
5 revisions
This sub-group covers areas such as remoteproc, rp-message, virtio, big buffers, etc. The meeting cadence is every 2 weeks on Thursdays at 11 am Eastern US time.
- Bill Mills (Linaro)
- Arnaud Pouliquen (ST)
- Ed Mooring (Independent)
- Mathieu Poirier (Linaro)
- Tanmay
- Tammy
- MP: no update to list as still keeping up and queue is empty
- MP: looking at Arnaud's patch to add a TEE mode to remoteproc
- AP: Main focus is on OP-TEE support, then will look at kernel
https://lore.kernel.org/linux-remoteproc/[email protected]/T/#maca0a1fc897aadd54c7deac432e11473fe970d1d
- MP: also looking at tracepoint
https://lore.kernel.org/linux-remoteproc/[email protected]/T/#t
- Buf size
- AP: PR check of virtio status #387
- AP: OA PR #491 & LM # XX, zynqmp specific platform updates
- TS: Presentation
- BM: Why limit to just one end point??
- HN & AP: concern about vdev detach needed before kernel start
- BM: How do we know detach happens before kernel start?
- TS: U-boot device model destroys drivers before kernel start, take care in there
- Bill Mills (Linaro)
- Jiafei Pan (NXP)
- Ed Mooring (Independent)
- Mathieu Poirier (Linaro)
- Tanmay
- Tammy
- Sergei Kornei
- MP: keeping up with queue, nothing pending so no email to list
- TS: RPMSG is 512 bytes wants configurable
- TS: Ok to
- ept reserved numbers
- dynamic > 1024
- < 128 reserve for standard usage
- 128 to 1023 platform or app specific
- EM: release update has gone out, same level of pull requests etc
- SK: Move "libmetal examples" to system reference
- Platform support move to system reference
- PR #231
- Bill Mills (Linaro)
- Jiafei Pan (NXP)
- Arnaud Pouliquen (ST)
- Ed Mooring (Independent)
- Mathieu Poirier (Linaro)
- MP: Tanmay's get rproc devices for cluster
- ping Bjorn, will look at it, still not reviewed
- MP will give Bjorn until after that the v6.4 merge Window
- rpmsg signaling/flowcontrol v6
- Bjorn merged one already but it is working
- There are 4 different V5 series! and 2 V6s
- Need to see a clean v7 series
- Feedback to MediaTek
- follow the examples of TI and Xilinx
- JP: Asked for the links:
- Mediatek:
- https://patchwork.kernel.org/project/linux-remoteproc/list/?series=734403
- good examples:
- https://elixir.bootlin.com/linux/v6.3-rc7/source/drivers/remoteproc/ti_k3_r5_remoteproc.c
- https://elixir.bootlin.com/linux/v6.3-rc7/source/drivers/remoteproc/xlnx_r5_remoteproc.c
- AP: Heads up to MP about authenticated firmware presentation
- BM: Heads up to MP about AMP BOF
- open-amp PR 478 for doxygen
- open-amp PR 469 use generated version_def.h
- libmetal PR 234 fix CI for Zephyr (already merged)
- Bill Mills (Linaro)
- Mathieu Poirier (Linaro)
- Arnaud Pouliquen (ST)
- Ed Mooring (Independent)
- Jiafei Pan (NXP)
- Xiaobo Xie (NXP)
- sergeik
- Hari
- Mingkai Hu (NXP)
- MP: update yesterday
- TI PRU patch is only thing pending
- Tanmay get-rproc patch
- Arnaud
- JP: kernel
- JP: we use rpmsg-lite on RTOS side
- JP: resource sharing peripherals
- Invites to virtio call for NXP
- AP: Not a ton of activities
- AP: fixes
- AP:
- SK: Trying to upstream some fixes
- JP: rpmsg on Zephyr on new platform
- mostly in libmetal and app
- call remoteproc vdev
- There is a PR for NXP platform for open-amp rpmsg
- Jailhouse Linux + RTOS
- Zephyr RTOS + Linux
- Using HW partitioning for separation
- AP: flow control
- AP: secure side OP-TEE based remote proc image and resources
- Optionally encrypt firmware
- will load multiple images
- Will effect U-boot, OP-TEE, and Linux kernel
- Bill Mills (Linaro) - Tanmay Shah (AMD Xilinx) - Ben Levinsky (AMD Xilinx) - Tammy Leino (Siemens) - Ed Mooring (Independent)
- MP: no review queue update as it is empty right now
- Sent patch to release device-nodes
- Still waiting for RENESAS
- Looked at Tanmay's multi cluster rproc-get-by-handle
- will clean up a bit
- V5 PRU consumer review (will check for Rodger's review )
- Tony Lingrine has already looked at everything and is happy
- BL are there virtio tracers?
- MP: no not today
- Release in April, code freeze by April 15
- TL will have the doc PRs ready
- Arnaud tests on ST platforms
- AP has check list
- Bare-metal FreeRTOS Zephyr
- EM testing is mostly virtual
- Linux PC can test open-amp
- Bill Mills (Linaro)
- Tanmay Shah (AMD Xilinx)
- Arnaud Pouliquen (ST)
- Ben Levinsky (AMD Xilinx)
- Tammy Leino (Siemens)
- Ed Mooring (Independent)
- Don Harbin (Linaro)
- DOC open-map libmetal libs
- RPMSG over other transports
- Markup text
- Lets fix one file as an RFC in open-amp
- Need to add libmetal
- Bill is looking at the automation
- Tammy will have the RFC
- Bill will create a discord core-team channel
- DOC open-map libmetal libs
- ST RPMSG over UART was proof of concept
- No UART errors handled
- BL: No update
- TS: U-boot has virtio already, probably better to do RPMGS over virtio
- BM & AP: the need was only for load
- rpmsg will be poll based on command
- TS: remoteproc attach is needed
- AP: maybe you don't need the full attach, just a sync point
- MP: incoming patch rate is increasing but I am keeping up
- [PATCH] TI K3 M4F support on AM64x and AM62x SoCs
- [PATCH] Add support for MT8195 SCP 2nd core
- [PATCH] Introduce PRU platform consumer AP
- Zynqmp rpmsg support is pretty close but should go in next time, target v6.4
Will appove PR today
- Mathieu Poirier (Linaro)
- Bill Mills (Linaro)
- Ed Mooring (Independent)
- Tanmay Shah (AMD Xilinx)
- Ben Levinsky (AMD Xilinx)
- Tammy Leino (Siemens)
- Hari
- Two patches from NXP team in Europe
- ELF segment is store in HW
- segments can be 32 bit aligned but kernel did not handle that
- Want to load w/o Mailbox or rpmsg
- was not working
- ELF segment is store in HW
- Xilinx mailbox support
- Good progress
- TS: next rev will be sent shortly
- MP: expect next rev to be ready for merge
- Xilinx get by phandle
- TS: will be working on that shortly
- BL: put* needs to changed as well?? Will verify.
- EM: no specific topics today
- TL: lets talk about doc when AP is back next time
- Mathieu Poirier (Linaro)
- Bill Mills (Linaro)
- Ed Mooring (Independent)
- Tanmay Shah (AMD Xilinx)
- Ben Levinsky (AMD Xilinx)
- Hari
- MP: picking up again, 2 more came in after status review
- happy to see other people reviewing
- NXP: reintoduce things Pang took out
- TS: rpmsg
- TS: what is the status of rproc by handle patch
- extra patches: send them and include MP's patch in series
- libmetal ST micro, GCC 12 broke atomics but no patch yet
- BL: do we really need
- virtio is only way for RX/TX
- mediatek rpmsg over mailbox
- Bill Mills (Linaro)
- Arnaud Pouliquen (ST)
- Ed Mooring (Independent)
- Tanmay Shah (AMD Xilinx)
- Ben Levinsky (AMD Xilinx)
- Tammy Leino (Siemens)
- (Mathieu had a conflict)
- Ben: GLINK seems to be simpler for init
- Simplify rpmsg in constrained env
- Want RPMGS over mailbox (payload fits into rpmsg)
- AP: may want to look at my private RPMSG over UART
- AP: Virtio RPMSG interrupt for I2C
- AP: tried to move rpmsg to interrupt context
- Mailbox can call back in interrupt or in normal
- AP: did choice based on Mailbox ID
- Two RPMSG instances one for interrupt and one for normal
- WAM: How does Vitio MMIO or PCI do this (assume interrupt context)
- MMIO will need adaptation for mailbox notify
- Mailbox is associated to one virtio
- How do we demix the mailbox?
- AP: remoteproc interrupt controller
- BM: Maybe look at TI PRU INTC??
Meeting was hosted by Mathieu as Bill was still on vacation. No notes where taken as this is normally Bill's job.