fix(linux): keep virtual Xbox controllers on hid-generic

[ReenigneArcher]

Description

Mark the upper word of the UHID vendor ID to prevent xpadneo and its udev rules from claiming virtual Xbox One and Series devices. Preserve the consumer-visible Xbox identity, Guide chords, trigger rumble, and battery reporting.

Add kernel identity assertions and SDL2/SDL3 Guide chord regression coverage.

Screenshot

Issues Fixed or Closed

• Fixes Xbox 360 controllers GUIDE+<some key> fails #191

Roadmap Issues

Testing

Validated with 144 unit tests and five native consumer tests with xpadneo Guide shift mode enabled.

Type of Change

• feat: New feature (non-breaking change which adds functionality)
• fix: Bug fix (non-breaking change which fixes an issue)
• docs: Documentation only changes
• style: Changes that do not affect the meaning of the code (white-space, formatting, missing semicolons, etc.)
• refactor: Code change that neither fixes a bug nor adds a feature
• perf: Code change that improves performance
• test: Adding missing tests or correcting existing tests
• build: Changes that affect the build system or external dependencies
• ci: Changes to CI configuration files and scripts
• chore: Other changes that don't modify src or test files
• revert: Reverts a previous commit
• BREAKING CHANGE: Introduces a breaking change (can be combined with any type above)

Checklist

• Code follows the style guidelines of this project
• Code has been self-reviewed
• Code has been commented, particularly in hard-to-understand areas
• Code docstring/documentation-blocks for new or existing methods/components have been added or updated
• Tests have been added or updated for new or modified functionality, including regression tests for bug fixes

AI Usage

See our AI usage policy.

• AI tools were used while preparing this PR

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.77%. Comparing base (13342c2) to head (ea46aa1).
✅ All tests successful. No failed tests found.

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #192      +/-   ##
==========================================
- Coverage   73.77%   73.77%   -0.01%     
==========================================
  Files          43       43              
  Lines       10697    10702       +5     
  Branches     3693     3695       +2     
==========================================
+ Hits         7892     7895       +3     
+ Misses       2442     2159     -283     
- Partials      363      648     +285     

Flag	Coverage Δ
FreeBSD	53.85% <ø> (-0.05%)	⬇️
Linux-Clang	68.41% <83.33%> (+0.05%)	⬆️
Linux-GCC	80.94% <100.00%> (+0.05%)	⬆️
PowerShell	75.60% <ø> (ø)
Windows-MSVC	74.17% <ø> (-0.07%)	⬇️
Windows-MinGW-UCRT64	66.16% <ø> (+0.23%)	⬆️
macOS	50.96% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines	Coverage Δ
src/platform/linux/uhid_backend.cpp	81.46% <100.00%> (+0.04%)	⬆️

... and 11 files with indirect coverage changes

---

Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 13342c2...ea46aa1. Read the comment docs.

[PVermeer]

This works.

Applied this patch to stable sunshine (2026.914.233613).

Tested with controllers:

• xbox360 controller;
• xbox one;
• GameSir Nova on xinput.

Steam and Linux now shows the Xbox Series X controller instead of the Xbox 360.

Thanks!

[PVermeer]

Also one small note:

When setting up the DualSense controller for RetroArch via sunshine / moonlight, I needed to unset "Map Back/Select to Touchpad Click" because the 'select' button (my hotkey) would register as a mouse click (mouse event is faster). This was very confusing and took me a little while to figure out. In my experience/opinion, this setting should be disabled by default.

I think it can also be fixed in RetroArch by delaying mouse events, but be aware that this can happen in certain applications.