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

systemd service: paranoia mode #90

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Tatsh
Copy link

@Tatsh Tatsh commented Dec 13, 2021

Makes the service run with extreme restrictions.

@ShadowsFriend
Copy link

As I have pointed out in the Gentoo bug report below[1], access to /dev/uinput is also needed. Otherwise, pairing the Pro Controller as virtual device does not work.

[1] https://bugs.gentoo.org/867385

@Tatsh
Copy link
Author

Tatsh commented Sep 22, 2022

Does that only affect Bluetooth?

Also do you have a udev rule for uinput for the Pro Controller?

@ShadowsFriend
Copy link

Does that only affect Bluetooth?

No, my controller is connected via cable.

Also do you have a udev rule for uinput for the Pro Controller?

Yes, I have the following rule for uinput:
KERNEL=="uinput", SUBSYSTEM=="misc", TAG+="uaccess", OPTIONS+="static_node=uinput"

@Tatsh
Copy link
Author

Tatsh commented Sep 23, 2022

Sounds like the udev rule should be submitted to upstream? That OPTIONS part is not in the files here.

@ShadowsFriend
Copy link

Maybe it should. The rule is part of the Steam Input rules[1] as Steam makes quite some use of uinput. Steam is also the primary reason why pairing the Pro Controller as a virtual controller is even a thing in joycond because Steam Input is fighting with the kernel driver until [2] is fixed.

[1] ValveSoftware/steam-for-linux#4794
[2] ValveSoftware/steam-for-linux#6651

@Tatsh
Copy link
Author

Tatsh commented Oct 1, 2022

Pushed the fix.

After the uinput device exists, what is the output of cat /proc/devices? I am wondering if there is uinput under character devices. I would prefer to use the device group name if possible.

@ShadowsFriend
Copy link

When I first played around with the issue, I also tried finding the corresponding device group name. I was, however, not knowledgeable enough to figure it out. Here are the character devices shown by cat /proc/devices :

  1 mem
  4 /dev/vc/0
  4 tty
  5 /dev/tty
  5 /dev/console
  5 /dev/ptmx
  7 vcs
 10 misc
 13 input
 14 sound
 21 sg
 29 fb
116 alsa
128 ptm
136 pts
180 usb
189 usb_device
202 cpu/msr
203 cpu/cpuid
226 drm
246 hidraw
247 usbmon
248 kfd
249 bsg
250 watchdog
251 ptp
252 pps
253 rtc
254 dax

@Tatsh
Copy link
Author

Tatsh commented Oct 1, 2022

I"m going to leave it as is with the device path.

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

Successfully merging this pull request may close these issues.

2 participants