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

Unify driver protobuf protocol to SolarXR #895

Open
3 tasks
ImUrX opened this issue Nov 5, 2023 · 0 comments
Open
3 tasks

Unify driver protobuf protocol to SolarXR #895

ImUrX opened this issue Nov 5, 2023 · 0 comments
Labels
Area: SteamVR Driver Related to the SteamVR Driver Priority: Normal The default priority Type: Discussion Further information is requested Type: Enhancement Adds or improves a feature

Comments

@ImUrX
Copy link
Member

ImUrX commented Nov 5, 2023

We kinda have 2 different schemas currently being used, this needs an overhaul.

  • Figure out how the whole protocol should look in flatbuffer
  • Use only unix sockets in all OS for the driver IPC (supported since Insider Build 17063 on Windows 10)
  • Work out if any extra is needed for Monado to have an easy time using this

Stuff that needs to be thought of:

  • Because we are gonna still use sockets/pipes, we should use TCP directly, it will make everything easier by making sure messages are in order (which should still happen in UDP tbh) and take care of the concept of a "connection" which UDP doesn't have. The current implementation in Linux is TCP, so it won't require much change.
@ImUrX ImUrX added Type: Discussion Further information is requested Priority: Normal The default priority Type: Enhancement Adds or improves a feature Area: SteamVR Driver Related to the SteamVR Driver labels Nov 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: SteamVR Driver Related to the SteamVR Driver Priority: Normal The default priority Type: Discussion Further information is requested Type: Enhancement Adds or improves a feature
Development

No branches or pull requests

1 participant