Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make registers writeable by all users #30

Open
tvannoy opened this issue Jun 23, 2020 · 2 comments
Open

Make registers writeable by all users #30

tvannoy opened this issue Jun 23, 2020 · 2 comments

Comments

@tvannoy
Copy link
Member

tvannoy commented Jun 23, 2020

Currently, the permissions on our device attribute files in sysfs are 0664 (rw-rw-r--), and root is both the owner and the group for these files. This is fine right now, as we are running everything as root (which in and of itself is actually not fine...). When we start using a user that isn't root, we will no longer be able to write those files. Changing the permissions to 0666 would fix that, and I don't believe doing so will have any major drawbacks/implications, even if we end up creating a different interface.

@fe-wickham
Copy link
Member

fe-wickham commented Aug 7, 2020

Changing the permissions to 0666 would fix that

Just a note on this. That can't be done as it is not allowed. Attempting to do so literally makes the device drivers unable to compile to intentionally prevent those permissions.

@audiologic-vannoy
Copy link
Member

Just a note on this. That can't be done as it is not allowed. Attempting to do so literally makes the device drivers unable to compile to intentionally prevent those permissions.

That makes sense. They did the right thing.

It seems like giving sudo permissions for specific files is a good approach. If we can set the group to something other than root, we could avoid messing with the sudoers file, but I'd be surprised if we could automatically change the group ownership.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants