- Openamp-rp - lists.openampproject.org

[OpenAMP/openamp-system-reference] 877039: examples: zephyr: dual_qemu_ivhmem: remove $ promp...

by Arnaud Pouliquen

Branch: refs/heads/main Home: https://github.com/OpenAMP/openamp-system-reference Commit: 877039581b38baa11180104eaf7ffa8706af4f74 https://github.com/OpenAMP/openamp-system-reference/commit/877039581b38baa1… Author: Sipke Vriend <sipke(a)direktembedded.com> Date: 2026-09-21 (Mon, 21 Sep 2026) Changed paths: M examples/zephyr/dual_qemu_ivshmem/README.rst Log Message: ----------- examples: zephyr: dual_qemu_ivhmem: remove $ prompt from code blocks Remove the prompt $ symbol from console block command lines, so it is not copied as well when using github or IDE copy button. Signed-off-by: Sipke Vriend <sipke(a)direktembedded.com> Commit: 13d0cd249b00d12002bc3b126cc820d7c16fb4c2 https://github.com/OpenAMP/openamp-system-reference/commit/13d0cd249b00d120… Author: Sipke Vriend <sipke(a)direktembedded.com> Date: 2026-09-21 (Mon, 21 Sep 2026) Changed paths: M examples/zephyr/dual_qemu_ivshmem/README.rst Log Message: ----------- examples: zephyr: dual_qemu_ivhmem: split Zephyr uart code blocks Split the zephyr console blocks into command line and output blocks to allow the command line to be copied when using github or IDE. Signed-off-by: Sipke Vriend <sipke(a)direktembedded.com> Commit: b24346cb17bd5ed5e00f8f24e1813ee978ce287e https://github.com/OpenAMP/openamp-system-reference/commit/b24346cb17bd5ed5… Author: Sipke Vriend <sipke(a)direktembedded.com> Date: 2026-09-21 (Mon, 21 Sep 2026) Changed paths: M examples/legacy_apps/machine/xlnx/microblaze_generic/README.md M examples/linux/rpmsg-echo-test/README.md M examples/linux/rpmsg-mat-mul/README.md M examples/linux/rpmsg-proxy-app/README.md Log Message: ----------- examples: readme: use shell block for command lines Use the markdown shell block so that users can copy the block, or execute lines as supported by IDEs. Signed-off-by: Sipke Vriend <sipke(a)direktembedded.com> Commit: 797f19f9c186e9c73b44c6953cc6e6dac541c63e https://github.com/OpenAMP/openamp-system-reference/commit/797f19f9c186e9c7… Author: Sipke Vriend <sipke(a)direktembedded.com> Date: 2026-09-21 (Mon, 21 Sep 2026) Changed paths: R examples/linux/rpmsg-utils/README A examples/linux/rpmsg-utils/README.md Log Message: ----------- examples: linux: rpmsg-utils: rename to make README a Markdown file To allow for formatting of readme rename to README.md Signed-off-by: Sipke Vriend <sipke(a)direktembedded.com> Commit: 0d54a0b29d2cd01387aea14668ab962037cd3a70 https://github.com/OpenAMP/openamp-system-reference/commit/0d54a0b29d2cd013… Author: Sipke Vriend <sipke(a)direktembedded.com> Date: 2026-09-21 (Mon, 21 Sep 2026) Changed paths: M examples/linux/rpmsg-utils/README.md Log Message: ----------- examples: linux: rpmsg-utils: Format with markdown code block and heading Use a code block for the udev rules and extend the heading underline for eptdestroy. Signed-off-by: Sipke Vriend <sipke(a)direktembedded.com> Compare: https://github.com/OpenAMP/openamp-system-reference/compare/4ca6a085a7e2...… To unsubscribe from these emails, change your notification settings at https://github.com/OpenAMP/openamp-system-reference/settings/notifications

1 hour, 12 minutes

1
0
0 0

[OpenAMP/libmetal] 9aa2f1: linux: remove sudo modprobe fallback from metal_li...

by Chirag Shilwant

Branch: refs/heads/main Home: https://github.com/OpenAMP/libmetal Commit: 9aa2f15556c5fe95e7c4d3ea33d15f3a113eaba8 https://github.com/OpenAMP/libmetal/commit/9aa2f15556c5fe95e7c4d3ea33d15f3a… Author: Chirag Shilwant <c-shilwant(a)ti.com> Date: 2026-09-21 (Mon, 21 Sep 2026) Changed paths: M lib/system/linux/device.c Log Message: ----------- linux: remove sudo modprobe fallback from metal_linux_probe_driver A library must not attempt to escalate its own privileges. The sudo modprobe fallback was incorrect for three reasons: 1. A library silently attempting privilege escalation on behalf of its caller is unexpected behavior and a security concern. 2. If modprobe failed due to missing CAP_SYS_MODULE, sudo will also fail unless passwordless sudo is explicitly configured in sudoers making the fallback redundant in practice. 3. On embedded and production Linux systems, sudo is often not installed, causing system() to fail silently or hang waiting for a password prompt. If the driver is not found after modprobe, return -ENODEV and let the caller handle it. Fixes: commit 7bda1db4a90d ("Initial commit") Signed-off-by: Chirag Shilwant <c-shilwant(a)ti.com> Commit: f05a041762532320a5e8d8532b118c22fc3e8e1e https://github.com/OpenAMP/libmetal/commit/f05a041762532320a5e8d8532b118c22… Author: Chirag Shilwant <c-shilwant(a)ti.com> Date: 2026-09-21 (Mon, 21 Sep 2026) Changed paths: M .github/workflows/continuous-integration.yml Log Message: ----------- ci: load UIO kernel modules before running nonreg tests The nr_tests job was relying on libmetal's internal sudo modprobe fallback in metal_linux_probe_driver() to load UIO modules at runtime. Now that the sudo escalation has been removed from the library, the CI environment must explicitly load the required modules before invoking the test suite. Add a "sudo modprobe" step for uio, uio_pdrv_genirq, and uio_dmem_genirq prior to make test. Module loading with sudo privileges is the responsibility of the environment, not the library. Signed-off-by: Chirag Shilwant <c-shilwant(a)ti.com> Compare: https://github.com/OpenAMP/libmetal/compare/02e46d91d3d8...f05a04176253 To unsubscribe from these emails, change your notification settings at https://github.com/OpenAMP/libmetal/settings/notifications

1 hour, 14 minutes

1
0
0 0

[OpenAMP/open-amp] 372048: rpmsg: rpc: Validate and initialize server requests

by ben

Branch: refs/heads/main Home: https://github.com/OpenAMP/open-amp Commit: 372048d49dc46a58586af3b62e94d4952bf92d59 https://github.com/OpenAMP/open-amp/commit/372048d49dc46a58586af3b62e94d495… Author: Ben Levinsky <ben.levinsky(a)amd.com> Date: 2026-09-21 (Mon, 21 Sep 2026) Changed paths: M lib/service/rpmsg/rpc/rpmsg_rpc_server.c Log Message: ----------- rpmsg: rpc: Validate and initialize server requests The RPC callback does not receive the request length, so it may inspect bytes beyond a short message. Those bytes currently come from an uninitialized stack buffer. A message shorter than the function ID can also make the dispatch path read uninitialized data. Reject messages that do not contain a complete function ID and zero-initialize the request buffer so callbacks never consume stale stack contents from bytes omitted by the remote peer. Signed-off-by: Ben Levinsky <ben.levinsky(a)amd.com> Assisted-by: Codex:GPT-5 Commit: 2b7013e7335aa62de8e62f2084e668fa6c6995c2 https://github.com/OpenAMP/open-amp/commit/2b7013e7335aa62de8e62f2084e668fa… Author: Ben Levinsky <ben.levinsky(a)amd.com> Date: 2026-09-21 (Mon, 21 Sep 2026) Changed paths: M lib/proxy/rpmsg_retarget.c Log Message: ----------- proxy: Validate retarget write length _write() copies a caller-controlled length into a fixed-size stack buffer without checking that the RPC header and payload fit. Negative lengths also become large unsigned memcpy() sizes, and the stdout NUL terminator is written one byte beyond its intended position. Reject invalid lengths before constructing the request. Include the optional terminator in the capacity check and place the terminator immediately after the copied payload. Signed-off-by: Ben Levinsky <ben.levinsky(a)amd.com> Assisted-by: Codex:GPT-5 Commit: 49780d11b49f3e2bea558d9ccd22e61487ed9e85 https://github.com/OpenAMP/open-amp/commit/49780d11b49f3e2bea558d9ccd22e614… Author: Ben Levinsky <ben.levinsky(a)amd.com> Date: 2026-09-21 (Mon, 21 Sep 2026) Changed paths: M lib/service/rpmsg/rpc/rpmsg_rpc_client.c Log Message: ----------- rpmsg: rpc: Validate client reply lengths The RPC client reads the reply ID and status without checking that the remote message contains the fixed reply header. It also passes the total message length to callbacks that receive a parameters pointer, making the reported length include the header bytes. Reject replies shorter than the fixed header and pass callbacks only the number of bytes that follow it. Signed-off-by: Ben Levinsky <ben.levinsky(a)amd.com> Assisted-by: Codex:GPT-5 Commit: f7c918bf07e3ad884247d06aad84b892c958e065 https://github.com/OpenAMP/open-amp/commit/f7c918bf07e3ad884247d06aad84b892… Author: Ben Levinsky <ben.levinsky(a)amd.com> Date: 2026-09-21 (Mon, 21 Sep 2026) Changed paths: M lib/include/openamp/rpmsg_rpc_client_server.h M lib/service/rpmsg/rpc/rpmsg_rpc_client.c Log Message: ----------- rpmsg: rpc: Validate client request lengths The RPC client copies caller-provided parameters into a fixed-size stack buffer without checking whether the complete request fits. Oversized requests can therefore overwrite the caller's stack frame. Reject requests that exceed the remaining parameter capacity before copying them, validate nonempty parameter pointers, and document the public API limit. Signed-off-by: Ben Levinsky <ben.levinsky(a)amd.com> Assisted-by: Codex:GPT-5 Commit: d0e5e0d376a320026211312df890fedf46ec4bc4 https://github.com/OpenAMP/open-amp/commit/d0e5e0d376a320026211312df890fedf… Author: Ben Levinsky <ben.levinsky(a)amd.com> Date: 2026-09-21 (Mon, 21 Sep 2026) Changed paths: M lib/proxy/rpmsg_retarget.c Log Message: ----------- proxy: Validate retarget read response length _read() converts the peer-provided unsigned data length to a signed integer and bounds it only against the caller's buffer size. Large values can bypass that comparison or make memcpy() read beyond the fixed response buffer. Reject nonpositive destination sizes, retain the peer length as an unsigned value, and clamp it to both the response payload capacity and the caller's buffer before copying. Signed-off-by: Ben Levinsky <ben.levinsky(a)amd.com> Assisted-by: Codex:GPT-5 Compare: https://github.com/OpenAMP/open-amp/compare/80284e3ac0e9...d0e5e0d376a3 To unsubscribe from these emails, change your notification settings at https://github.com/OpenAMP/open-amp/settings/notifications

1 hour, 18 minutes

1
0
0 0

[OpenAMP/open-amp] 80284e: virtqueue: add writable buffer helper

by Francesco Valla

Branch: refs/heads/main Home: https://github.com/OpenAMP/open-amp Commit: 80284e3ac0e94d2e9f15bab79e84f57d4cf77976 https://github.com/OpenAMP/open-amp/commit/80284e3ac0e94d2e9f15bab79e84f57d… Author: Francesco Valla <francesco(a)valla.it> Date: 2026-09-21 (Mon, 21 Sep 2026) Changed paths: M lib/include/openamp/virtqueue.h M lib/virtio/virtqueue.c Log Message: ----------- virtqueue: add writable buffer helper An application at device side might need to know if a buffer has been marked as device-writable by the driver. An example is the virtio SPI protocol, in which the direction of an half-duplex communication can be determined by the device only by checking if the buffer containing the data is writable or not. Add a simple helper to test if a buffer is device-writable, that is, if the VRING_DESC_F_WRITE flag is present on it. Signed-off-by: Francesco Valla <francesco(a)valla.it> To unsubscribe from these emails, change your notification settings at https://github.com/OpenAMP/open-amp/settings/notifications

1 hour, 21 minutes

1
0
0 0

[OpenAMP/open-amp] 2ad032: remoteproc: add checked arithmetic helpers

by ben

Branch: refs/heads/main Home: https://github.com/OpenAMP/open-amp Commit: 2ad0328fe688978b33c59251dff38de44739f6bd https://github.com/OpenAMP/open-amp/commit/2ad0328fe688978b33c59251dff38de4… Author: Ben Levinsky <ben.levinsky(a)amd.com> Date: 2026-09-21 (Mon, 21 Sep 2026) Changed paths: M lib/include/internal/utilities.h M lib/remoteproc/elf_loader.c Log Message: ----------- remoteproc: add checked arithmetic helpers Add generic checked addition, subtraction, and multiplication helpers with consistent input validation across builtin and fallback implementations. Add a generic range-containment helper which reports arithmetic overflow separately from non-containment. This lets the ELF loader distinguish malformed ranges from data absent from the current image chunk. Signed-off-by: Ben Levinsky <ben.levinsky(a)amd.com> Commit: f3b110faff3df0f36c0f5126b9d9ecdfca2575e5 https://github.com/OpenAMP/open-amp/commit/f3b110faff3df0f36c0f5126b9d9ecdf… Author: Ben Levinsky <ben.levinsky(a)amd.com> Date: 2026-09-21 (Mon, 21 Sep 2026) Changed paths: M lib/remoteproc/elf_loader.c Log Message: ----------- remoteproc: validate ELF header shape before table loading Reject malformed ELF headers before using program or section table metadata from the firmware image. Validate the ELF class, ELF header size, program header entry size, section header entry size, and section string-table index. Release an image-info object allocated by this call if validation fails, while leaving caller-owned objects untouched. Signed-off-by: Ben Levinsky <ben.levinsky(a)amd.com> Commit: 8d245caca221297ff395bce00a2331430506b04e https://github.com/OpenAMP/open-amp/commit/8d245caca221297ff395bce00a233143… Author: Ben Levinsky <ben.levinsky(a)amd.com> Date: 2026-09-21 (Mon, 21 Sep 2026) Changed paths: M lib/remoteproc/elf_loader.c Log Message: ----------- remoteproc: check ELF program and section table sizes Use checked multiplication when computing ELF program and section header table sizes from e_phnum/e_phentsize and e_shnum/e_shentsize. Also replace wrapping offset-plus-length range checks with overflow-safe image chunk validation before allocation and memcpy. Malformed firmware images with impossible table sizes now fail with -RPROC_EINVAL. Signed-off-by: Ben Levinsky <ben.levinsky(a)amd.com> Commit: 872a4fcae7e3a2ead1366a52dd8e044534043186 https://github.com/OpenAMP/open-amp/commit/872a4fcae7e3a2ead1366a52dd8e0445… Author: Ben Levinsky <ben.levinsky(a)amd.com> Date: 2026-09-21 (Mon, 21 Sep 2026) Changed paths: M lib/remoteproc/elf_loader.c Log Message: ----------- remoteproc: harden ELF section string-table range checks Use overflow-safe range validation before copying the ELF section string table from the firmware image. The section string-table offset and size are read from untrusted section headers. Validate that the full table is present in the current image chunk without relying on wrapping offset arithmetic. Signed-off-by: Ben Levinsky <ben.levinsky(a)amd.com> Commit: 08a45aca334d49133abdf89b21817af1217595fc https://github.com/OpenAMP/open-amp/commit/08a45aca334d49133abdf89b21817af1… Author: Ben Levinsky <ben.levinsky(a)amd.com> Date: 2026-09-21 (Mon, 21 Sep 2026) Changed paths: M lib/include/openamp/elf_loader.h M lib/remoteproc/elf_loader.c Log Message: ----------- remoteproc: track loaded ELF section string-table size Store the loaded section string-table size in the ELF image information for both ELF32 and ELF64 images. Keeping the size alongside the string-table pointer allows later section name lookups to validate sh_name offsets against the actual loaded table bounds. Signed-off-by: Ben Levinsky <ben.levinsky(a)amd.com> Commit: 28b0a020c633715d7b90341b3b9ec44277d030d4 https://github.com/OpenAMP/open-amp/commit/28b0a020c633715d7b90341b3b9ec442… Author: Ben Levinsky <ben.levinsky(a)amd.com> Date: 2026-09-21 (Mon, 21 Sep 2026) Changed paths: M lib/remoteproc/elf_loader.c Log Message: ----------- remoteproc: bound ELF section name lookups Make ELF section-name lookup validate sh_name before reading from the loaded section string table. Skip malformed section names whose sh_name offset is outside the table or whose string is not NUL-terminated within the remaining table bytes. Use bounded comparison for valid candidates so .resource_table lookup cannot read past the loaded string table. Signed-off-by: Ben Levinsky <ben.levinsky(a)amd.com> Commit: 3ea751021e4f157c561b8bf2598d263ac072a8ef https://github.com/OpenAMP/open-amp/commit/3ea751021e4f157c561b8bf2598d263a… Author: Ben Levinsky <ben.levinsky(a)amd.com> Date: 2026-09-21 (Mon, 21 Sep 2026) Changed paths: M lib/remoteproc/remoteproc.c Log Message: ----------- remoteproc: avoid wrapping loader offset comparison Replace the non-seekable loader offset comparison that used offset + len with an overflow-safe equivalent. This preserves existing behavior for normal ranges while avoiding a wraparound case when deciding whether required image data is contiguous with the current chunk. Signed-off-by: Ben Levinsky <ben.levinsky(a)amd.com> Compare: https://github.com/OpenAMP/open-amp/compare/1e2cae3a2edc...3ea751021e4f To unsubscribe from these emails, change your notification settings at https://github.com/OpenAMP/open-amp/settings/notifications

1 hour, 23 minutes

1
0
0 0

[OpenAMP/open-amp] 1e2cae: docs: clarify release_cb initialization requirement

by Atul Akella

Branch: refs/heads/main Home: https://github.com/OpenAMP/open-amp Commit: 1e2cae3a2edccfe5a50c255bc2f078e7cae148d8 https://github.com/OpenAMP/open-amp/commit/1e2cae3a2edccfe5a50c255bc2f078e7… Author: Atul Akella <atul.akella(a)gmail.com> Date: 2026-09-10 (Thu, 10 Sep 2026) Changed paths: M lib/include/openamp/rpmsg.h Log Message: ----------- docs: clarify release_cb initialization requirement Document that callers must zero-initialize the endpoint structure before calling rpmsg_create_ept, including the release_cb field. Fixes #659 Signed-off-by: Atul Akella <atul.akella(a)gmail.com> To unsubscribe from these emails, change your notification settings at https://github.com/OpenAMP/open-amp/settings/notifications

1 week, 4 days

1
0
0 0

[OpenAMP/open-amp] bb4585: README: use shell block for commands in Markdown f...

by Arnaud Pouliquen

Branch: refs/heads/main Home: https://github.com/OpenAMP/open-amp Commit: bb45852c584be0746f1eb3c62052f214606b4bac https://github.com/OpenAMP/open-amp/commit/bb45852c584be0746f1eb3c62052f214… Author: Sipke Vriend <sipke(a)direktembedded.com> Date: 2026-09-09 (Wed, 09 Sep 2026) Changed paths: M README.md Log Message: ----------- README: use shell block for commands in Markdown files Use the markdown shell block instead of prepending $ to command so that users can copy the block, as supported in github and IDEs. Signed-off-by: Sipke Vriend <sipke(a)direktembedded.com> Commit: c0359e4ae5bfa71a09654c78abf0049d6b06720a https://github.com/OpenAMP/open-amp/commit/c0359e4ae5bfa71a09654c78abf0049d… Author: Sipke Vriend <sipke(a)direktembedded.com> Date: 2026-09-09 (Wed, 09 Sep 2026) Changed paths: M .github/actions/build_ci/README.md M README.md Log Message: ----------- github actions:README: use shell block for commands in Markdown files Use the markdown shell block instead of prepending $ to command so that users can copy the block, as supported in github and IDEs. Signed-off-by: Sipke Vriend <sipke(a)direktembedded.com> Commit: 2b172cef30f1563d01a7a8ba7ca0027964c78dd6 https://github.com/OpenAMP/open-amp/commit/2b172cef30f1563d01a7a8ba7ca00279… Author: Sipke Vriend <sipke(a)direktembedded.com> Date: 2026-09-09 (Wed, 09 Sep 2026) Changed paths: M .github/actions/build_ci/README.md Log Message: ----------- github actions:README: add some new lines and code block for clarity Multiple lines of text are concatenated so use a code block for the supported targets so they are on individual lines. Signed-off-by: Sipke Vriend <sipke(a)direktembedded.com> Compare: https://github.com/OpenAMP/open-amp/compare/2e83e405d3a0...2b172cef30f1 To unsubscribe from these emails, change your notification settings at https://github.com/OpenAMP/open-amp/settings/notifications

1 week, 4 days

1
0
0 0

[OpenAMP/open-amp] c2c26f: rpmsg: Fix RPMSG_BUF_INDEX macro to use parameter

by Chirag Shilwant

Branch: refs/heads/main Home: https://github.com/OpenAMP/open-amp Commit: c2c26fce51cd0f20719187db5640d8bf3a02bf2b https://github.com/OpenAMP/open-amp/commit/c2c26fce51cd0f20719187db5640d8bf… Author: Chirag Shilwant <c-shilwant(a)ti.com> Date: 2026-09-09 (Wed, 09 Sep 2026) Changed paths: M lib/rpmsg/rpmsg_virtio.c Log Message: ----------- rpmsg: Fix RPMSG_BUF_INDEX macro to use parameter The macro body references 'rp_hdr' instead of using the declared parameter 'rphdr'. This causes the macro to ignore its argument and always reference whatever 'rp_hdr' variable exists in the calling scope. Fix by using the parameter name in the macro body. Fixes: commit b32187e4fb89 ("openamp: change rx/tx buffer hold flag to count") Signed-off-by: Chirag Shilwant <c-shilwant(a)ti.com> Commit: 2e83e405d3a0c2d0ae53385e14e6332af79a6339 https://github.com/OpenAMP/open-amp/commit/2e83e405d3a0c2d0ae53385e14e6332a… Author: Chirag Shilwant <c-shilwant(a)ti.com> Date: 2026-09-09 (Wed, 09 Sep 2026) Changed paths: M lib/rpmsg/rpmsg_virtio.c Log Message: ----------- rpmsg: virtio: Change release_rx_buffer_nolock() return type to void The function unconditionally returns true and neither call site checks the return value. Remove the dead return and change the return type to void. Fixes: commit b32187e4fb89 ("openamp: change rx/tx buffer hold flag to count") Signed-off-by: Chirag Shilwant <c-shilwant(a)ti.com> Compare: https://github.com/OpenAMP/open-amp/compare/4ae383d32de3...2e83e405d3a0 To unsubscribe from these emails, change your notification settings at https://github.com/OpenAMP/open-amp/settings/notifications

1 week, 4 days

1
0
0 0

Unique Device Status value to indicate reset request

by Bill Mills

TLDR: virtio-remoteproc device reset To request a device reset (w/ acknowledgement), the driver should val = Status field // read current status field val |= 0x82 // set bits 7 and 1 (FAILED and DRIVER) val &= 0xfe // clear bit 0 (ACKNOWLEDGE) // leave other bits unchanged Status field = val // write the new status field value The driver should then poll the status field waiting for it to be == 0. Note: writing the status field to 0 when it is already non-zero, should continue to work as a device reset but the driver will not know when it is complete. Background: From the virtio spec [1] with extra clarity added by me. Status Field bits: (Spec section 2.1, html [2]) bit 0, 0x01, dec 1 ACKNOWLEDGE bit 1, 0x02, dec 2 DRIVER bit 2, 0x04, dec 4 DRIVER_OK (also used in suspend protocol) bit 3, 0x08, dec 8 FEATURES_OK bit 4, 0x10, dec 16 SUSPEND only used if negotiated bit 5, 0x20, dec 32 (currently unused) bit 6, 0x40, dec 64 DEVICE_NEEDS_RESET bit 7, 0x80, dec 128 FAILED All bits except DEVICE_NEEDS_RESET are set (and sometimes cleared) by the device. DEVICE_NEEDS_RESET is set by the device and should only be cleared by a reset. Logically the driver sets ACKNOWLEDGE, then DRIVER, then DRIVER_OK, then FEATURES_OK. Some of these could be set at the same time but never higher bits before lower bits. Once these bits are set, the driver in not allowed to clear them except by issuing a device reset. The one exception to this is that the DRIVER_OK bit is cleared and set as part of the SUSPEND protocol but only if it has been negotiated. The driver sets the FAILED bit when it has given up on the device. It is valid for FAILED with any of the following settings: 0b1xxx_0000 Fail before ack 0b1xxx_0001 Fail before driver found 0b1xxx_0011 Fail before driver OK 0b1xxx_0111 Fail before features OK 0b1xxx_1111 Fail after features OK And if SUSPEND has been negotiated 0b1xx1_0011 Fail while suspend active before features OK 0b1xx1_1011 Fail while Suspend active after features OK FAILED(0x80) and DRIVER (0x02) active with ACKNOWLEDGE(0x01) cleared is not a value that is possible in the base spec. Therefore we use this setting to request a reset w/ acknowledgement. 0b1xxx_xx10 Reset Request w/ acknowledgement TBD: Does/Should the driver send a notification on Status field change? Does/Should the device send a notification on Status field change? Sending the notice eliminates the need for a time on the other side to poll. Do we already support this? If not, probibly not worth it just for this protocol. Recent virtio-mmio enhancement: The patch in [3] is now in the virtio spec master branch but is not part of any released version yet. The patch adds a new V3 version of virtio-mmio where the driver can issue a non-blocking reset request and poll the status field to know when it is done. So we are not the only ones to notice this issue. However the solution used for virtio-mmio still requires trap and emulate to work and wont work if the Status field in located in normal memory of any type. In the patch the driver still writes 0 to the status field but the device is allowed to return from the MMR write with the status filed non-zero. The driver then waits for the status field to become zero to know when this operation is complete. This solution will not work for us as we use a real memory to store the status field and the driver writing that to 0 will take immediate effect and the driver then polling for zero would succeed immediately. Bill [1] https://docs.oasis-open.org/virtio/virtio/v1.4/virtio-v1.4.html [2] https://docs.oasis-open.org/virtio/virtio/v1.4/virtio-v1.4.html#x1-110001 [3] https://github.com/Linaro/virtio-msg-spec/commit/a1883517ee44cc03d1b621a331… -- Bill Mills Principal Technical Consultant, Linaro +1-240-643-0836 TZ: US Eastern Work Schedule: Tues/Wed/Thur

2 weeks, 3 days

2
2
0 0

[OpenAMP/openamp-system-reference] 4ca6a0: examples: linux: add 'rpmsg_' prefix to userspace ...

by Tanmay Shah

Branch: refs/heads/main Home: https://github.com/OpenAMP/openamp-system-reference Commit: 4ca6a085a7e2a2ebd8e7ae77f9e5997401be4661 https://github.com/OpenAMP/openamp-system-reference/commit/4ca6a085a7e2a2eb… Author: Tanmay Shah <tanmay.shah(a)amd.com> Date: 2026-08-28 (Fri, 28 Aug 2026) Changed paths: M examples/linux/rpmsg-echo-test/Makefile M examples/linux/rpmsg-echo-test/README.md M examples/linux/rpmsg-mat-mul/Makefile M examples/linux/rpmsg-mat-mul/README.md M examples/linux/rpmsg-proxy-app/Makefile M examples/linux/rpmsg-proxy-app/README.md Log Message: ----------- examples: linux: add 'rpmsg_' prefix to userspace app names Current linux userspace app names are vague and do not show the true purpose of the application. Add 'rpmsg_' prefix to the application binary's name. This prefix clearly shows that the respective binary will be used to send/receive rpmsg protocol payload. Signed-off-by: Tanmay Shah <tanmay.shah(a)amd.com> To unsubscribe from these emails, change your notification settings at https://github.com/OpenAMP/openamp-system-reference/settings/notifications

3 weeks, 2 days

1
0
0 0

2026

2025

2024

2023

2022

2021

2020

2019

Openamp-rp