docs: Include instructions on enable Remoteproc Runtime in userspace#15
Merged
Conversation
yejseo01
force-pushed
the
userspace_enable
branch
from
October 15, 2025 13:54
adcd1a6 to
c5f6801
Compare
muchzill4
reviewed
Oct 15, 2025
th3james
requested changes
Oct 15, 2025
yejseo01
changed the title
muchzill4
reviewed
Oct 15, 2025
muchzill4
reviewed
Oct 15, 2025
3 tasks
th3james
reviewed
Oct 21, 2025
yejseo01
force-pushed
the
userspace_enable
branch
from
October 21, 2025 12:05
edc99ee to
bc92a6f
Compare
muchzill4
previously requested changes
Oct 22, 2025
th3james
requested changes
Oct 27, 2025
th3james
reviewed
Oct 27, 2025
Contributor
Author
|
The CI test of this PR is dependent on this change - arm/remoteproc-simulator#2 |
th3james
requested changes
Oct 28, 2025
3 tasks
yejseo01
commented
Oct 30, 2025
muchzill4
reviewed
Oct 31, 2025
muchzill4
left a comment
muchzill4
left a comment
Contributor
There was a problem hiding this comment.
Everywhere we use fmt.Println we should aim to use logger instead.
In order to make some incremental progress here - it's been a bit of a drag - I suggest we split this PR. I worry that tackling the logging properly will balloon this further. We should also consider how we test the features added here.
Something we could land as a separate change is respecting the /sys/.../parameters/path. This does not rely on root or non-root.
yejseo01
marked this pull request as draft
November 3, 2025 12:15
3 tasks
Remoteproc Runtime uses Remoteproc, which usually requires root permissions to write to its filesystem. To ensure that Remoteproc Runtime has root access, the container engines that spawn containers with Remoteproc Runtime should be run with root access. Signed-off-by: Yejin Seo <yejseo01@arm.com>
Signed-off-by: Yejin Seo <yejseo01@arm.com>
Signed-off-by: Yejin Seo <yejseo01@arm.com>
Signed-off-by: Yejin Seo <yejseo01@arm.com>
Signed-off-by: Yejin Seo <yejseo01@arm.com>
Signed-off-by: Yejin Seo <yejseo01@arm.com>
Signed-off-by: Yejin Seo <yejseo01@arm.com>
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR enables Remoteproc Runtime to be usable by non-root users by removing hard-coded root-only paths and providing configuration guidance. The changes focus on documentation to help users set up proper permissions for running the runtime in user space.
- Added permission setting documentation for non-root user access
- Updated usage documentation to reference new permission requirements
- Provided instructions for configuring remoteproc driver access and firmware paths
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| docs/USAGE.md | Added note about elevated permissions requirement with link to permission setup guide |
| docs/PERMISSION_SETTING.md | New comprehensive guide for configuring non-root user access to remoteproc driver and firmware paths |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
yejseo01
changed the title
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: yejseo01 <yejseo01@arm.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: yejseo01 <yejseo01@arm.com>
yejseo01
marked this pull request as ready for review
November 11, 2025 13:33
Luke-Parkin
requested changes
Nov 11, 2025
Co-authored-by: Luke Parkin <65398091+Luke-Parkin@users.noreply.github.com> Signed-off-by: yejseo01 <yejseo01@arm.com>
Co-authored-by: Luke Parkin <65398091+Luke-Parkin@users.noreply.github.com> Signed-off-by: yejseo01 <yejseo01@arm.com>
Co-authored-by: Luke Parkin <65398091+Luke-Parkin@users.noreply.github.com> Signed-off-by: yejseo01 <yejseo01@arm.com>
Signed-off-by: Yejin Seo <yejseo01@arm.com>
Signed-off-by: Yejin Seo <yejseo01@arm.com>
Signed-off-by: Yejin Seo <yejseo01@arm.com>
Signed-off-by: Yejin Seo <yejseo01@arm.com>
Signed-off-by: Yejin Seo <yejseo01@arm.com>
th3james
previously requested changes
Nov 11, 2025
Co-authored-by: James Cox-Morton <Th3james@fastmail.fm> Signed-off-by: yejseo01 <yejseo01@arm.com>
Co-authored-by: James Cox-Morton <Th3james@fastmail.fm> Signed-off-by: yejseo01 <yejseo01@arm.com>
Co-authored-by: James Cox-Morton <Th3james@fastmail.fm> Signed-off-by: yejseo01 <yejseo01@arm.com>
Signed-off-by: Yejin Seo <yejseo01@arm.com>
Signed-off-by: Yejin Seo <yejseo01@arm.com>
Luke-Parkin
approved these changes
Nov 12, 2025
yejseo01
dismissed stale reviews from th3james and muchzill4
November 12, 2025 15:31
All requested changes applied
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Changes
Update document to include information on enabling Remoteproc Runtime run in userspace
Checklist