diff options
Diffstat (limited to 'www/firefox-esr/files/patch-bug1680982')
-rw-r--r-- | www/firefox-esr/files/patch-bug1680982 | 388 |
1 files changed, 388 insertions, 0 deletions
diff --git a/www/firefox-esr/files/patch-bug1680982 b/www/firefox-esr/files/patch-bug1680982 new file mode 100644 index 000000000000..c9fc2344bb44 --- /dev/null +++ b/www/firefox-esr/files/patch-bug1680982 @@ -0,0 +1,388 @@ +commit 3204512f58a1 +Author: Greg V <greg@unrelenting.technology> +Date: Sun Dec 6 22:07:00 2020 +0000 + + Bug 1680982 - Use evdev instead of the Linux legacy joystick API for gamepads + + Using evdev is a prerequisite for adding rumble (haptic feedback) and LED support. + + - BTN_GAMEPAD semantic buttons are interpreted directly, + since all kernel drivers are supposed to use them correctly: + https://www.kernel.org/doc/html/latest/input/gamepad.html + - BTN_JOYSTICK legacy style numbered buttons use the model specific remappers + - we support even strange devices that combine both styles in one device + - the Linux gamepad module is enabled on FreeBSD and DragonFly, because + these kernels provide evdev, and libudev-devd provides enough of libudev + (evdev headers are provided by the devel/evdev-proto package) + + Differential Revision: https://phabricator.services.mozilla.com/D98868 +--- + dom/gamepad/linux/LinuxGamepad.cpp | 262 ++++++++++++++++++++++++++++++++----- + dom/gamepad/moz.build | 2 +- + 2 files changed, 229 insertions(+), 35 deletions(-) + +diff --git dom/gamepad/linux/LinuxGamepad.cpp dom/gamepad/linux/LinuxGamepad.cpp +index deee47b9d267..31f0aad7ae4a 100644 +--- dom/gamepad/linux/LinuxGamepad.cpp ++++ dom/gamepad/linux/LinuxGamepad.cpp +@@ -5,15 +5,16 @@ + * file, You can obtain one at http://mozilla.org/MPL/2.0/. */ + + /* +- * LinuxGamepadService: A Linux backend for the GamepadService. +- * Derived from the kernel documentation at +- * http://www.kernel.org/doc/Documentation/input/joystick-api.txt ++ * LinuxGamepadService: An evdev backend for the GamepadService. ++ * ++ * Ref: https://www.kernel.org/doc/html/latest/input/gamepad.html + */ + #include <algorithm> ++#include <unordered_map> + #include <cstddef> + + #include <glib.h> +-#include <linux/joystick.h> ++#include <linux/input.h> + #include <stdio.h> + #include <stdint.h> + #include <sys/ioctl.h> +@@ -21,10 +22,14 @@ + #include "nscore.h" + #include "mozilla/dom/GamepadHandle.h" + #include "mozilla/dom/GamepadPlatformService.h" ++#include "mozilla/dom/GamepadRemapping.h" + #include "mozilla/Tainting.h" + #include "mozilla/UniquePtr.h" + #include "udev.h" + ++#define LONG_BITS (sizeof(long) * 8) ++#define NLONGS(x) (((x) + LONG_BITS - 1) / LONG_BITS) ++ + namespace { + + using namespace mozilla::dom; +@@ -36,19 +41,29 @@ using mozilla::udev_list_entry; + using mozilla::udev_monitor; + using mozilla::UniquePtr; + +-static const float kMaxAxisValue = 32767.0; +-static const char kJoystickPath[] = "/dev/input/js"; ++static const char kEvdevPath[] = "/dev/input/event"; ++ ++static inline bool TestBit(const unsigned long* arr, int bit) { ++ return !!(arr[bit / LONG_BITS] & (1LL << (bit % LONG_BITS))); ++} ++ ++static inline double ScaleAxis(const input_absinfo& info, int value) { ++ return 2.0 * (value - info.minimum) / (double)(info.maximum - info.minimum) - ++ 1.0; ++} + + // TODO: should find a USB identifier for each device so we can + // provide something that persists across connect/disconnect cycles. +-typedef struct { ++struct Gamepad { + GamepadHandle handle; +- guint source_id; +- int numAxes; +- int numButtons; +- char idstring[256]; +- char devpath[PATH_MAX]; +-} Gamepad; ++ RefPtr<GamepadRemapper> remapper = nullptr; ++ guint source_id = UINT_MAX; ++ char idstring[256] = {0}; ++ char devpath[PATH_MAX] = {0}; ++ uint8_t key_map[KEY_MAX] = {0}; ++ uint8_t abs_map[ABS_MAX] = {0}; ++ std::unordered_map<uint16_t, input_absinfo> abs_info; ++}; + + class LinuxGamepadService { + public: +@@ -66,7 +81,7 @@ class LinuxGamepadService { + bool is_gamepad(struct udev_device* dev); + void ReadUdevChange(); + +- // handler for data from /dev/input/jsN ++ // handler for data from /dev/input/eventN + static gboolean OnGamepadData(GIOChannel* source, GIOCondition condition, + gpointer data); + +@@ -114,8 +129,14 @@ void LinuxGamepadService::AddDevice(struct udev_device* dev) { + g_io_channel_set_encoding(channel, nullptr, nullptr); + g_io_channel_set_buffered(channel, FALSE); + int fd = g_io_channel_unix_get_fd(channel); ++ ++ struct input_id id {}; ++ if (ioctl(fd, EVIOCGID, &id) == -1) { ++ return; ++ } ++ + char name[128]; +- if (ioctl(fd, JSIOCGNAME(sizeof(name)), &name) == -1) { ++ if (ioctl(fd, EVIOCGNAME(sizeof(name)), &name) == -1) { + strcpy(name, "unknown"); + } + const char* vendor_id = +@@ -131,20 +152,86 @@ void LinuxGamepadService::AddDevice(struct udev_device* dev) { + model_id = mUdev.udev_device_get_sysattr_value(parent, "id/product"); + } + } ++ if (!vendor_id && id.vendor != 0) { ++ vendor_id = (const char*)alloca(5); ++ snprintf((char*)vendor_id, 5, "%04x", id.vendor); ++ } ++ if (!model_id && id.product != 0) { ++ model_id = (const char*)alloca(5); ++ snprintf((char*)model_id, 5, "%04x", id.product); ++ } + snprintf(gamepad->idstring, sizeof(gamepad->idstring), "%s-%s-%s", + vendor_id ? vendor_id : "unknown", model_id ? model_id : "unknown", + name); + + char numAxes = 0, numButtons = 0; +- ioctl(fd, JSIOCGAXES, &numAxes); +- gamepad->numAxes = numAxes; +- ioctl(fd, JSIOCGBUTTONS, &numButtons); +- gamepad->numButtons = numButtons; ++ unsigned long key_bits[NLONGS(KEY_CNT)] = {0}; ++ unsigned long abs_bits[NLONGS(ABS_CNT)] = {0}; ++ ioctl(fd, EVIOCGBIT(EV_KEY, sizeof(key_bits)), key_bits); ++ ioctl(fd, EVIOCGBIT(EV_ABS, sizeof(abs_bits)), abs_bits); ++ ++ /* Here, we try to support even strange cases where proper semantic ++ * BTN_GAMEPAD button are combined with arbitrary extra buttons. */ ++ for (uint16_t i = BTN_JOYSTICK; i < KEY_MAX; i++) { ++ /* Do not map semantic buttons, they are handled directly */ ++ if (i == BTN_GAMEPAD) { ++ i = BTN_THUMBR + 1; ++ continue; ++ } ++ if (i == BTN_DPAD_UP) { ++ i = BTN_DPAD_RIGHT + 1; ++ continue; ++ } ++ if (TestBit(key_bits, i)) { ++ gamepad->key_map[i] = numButtons++; ++ } ++ } ++ for (uint16_t i = 0; i < BTN_JOYSTICK; i++) { ++ if (TestBit(key_bits, i)) { ++ gamepad->key_map[i] = numButtons++; ++ } ++ } ++ for (uint16_t i = BTN_GAMEPAD; i <= BTN_THUMBR; i++) { ++ /* But if any semantic event exists, count them all */ ++ if (TestBit(key_bits, i)) { ++ numButtons += BUTTON_INDEX_COUNT; ++ break; ++ } ++ } ++ for (uint16_t i = 0; i < ABS_MAX; ++i) { ++ if (TestBit(abs_bits, i)) { ++ gamepad->abs_info.emplace(i, input_absinfo{}); ++ if (ioctl(fd, EVIOCGABS(i), &gamepad->abs_info[i]) < 0) { ++ continue; ++ } ++ if (gamepad->abs_info[i].minimum == gamepad->abs_info[i].maximum) { ++ gamepad->abs_info.erase(i); ++ continue; ++ } ++ gamepad->abs_map[i] = numAxes++; ++ } ++ } ++ ++ if (numAxes == 0) { ++ NS_WARNING("Gamepad with zero axes detected?"); ++ } ++ if (numButtons == 0) { ++ NS_WARNING("Gamepad with zero buttons detected?"); ++ } ++ ++ bool defaultRemapper = false; ++ RefPtr<GamepadRemapper> remapper = ++ GetGamepadRemapper(id.vendor, id.product, defaultRemapper); ++ MOZ_ASSERT(remapper); ++ remapper->SetAxisCount(numAxes); ++ remapper->SetButtonCount(numButtons); + + gamepad->handle = service->AddGamepad( +- gamepad->idstring, mozilla::dom::GamepadMappingType::_empty, +- mozilla::dom::GamepadHand::_empty, gamepad->numButtons, gamepad->numAxes, +- 0, 0, 0); // TODO: Bug 680289, implement gamepad haptics for Linux. ++ gamepad->idstring, remapper->GetMappingType(), GamepadHand::_empty, ++ remapper->GetButtonCount(), remapper->GetAxisCount(), 0, ++ remapper->GetLightIndicatorCount(), remapper->GetTouchEventCount()); ++ gamepad->remapper = remapper.forget(); ++ // TODO: Bug 680289, implement gamepad haptics for Linux. + // TODO: Bug 1523355, implement gamepad lighindicator and touch for Linux. + + gamepad->source_id = +@@ -257,7 +344,7 @@ bool LinuxGamepadService::is_gamepad(struct udev_device* dev) { + if (!devpath) { + return false; + } +- if (strncmp(kJoystickPath, devpath, sizeof(kJoystickPath) - 1) != 0) { ++ if (strncmp(kEvdevPath, devpath, sizeof(kEvdevPath) - 1) != 0) { + return false; + } + +@@ -292,7 +379,7 @@ gboolean LinuxGamepadService::OnGamepadData(GIOChannel* source, + if (condition & G_IO_ERR || condition & G_IO_HUP) return FALSE; + + while (true) { +- struct js_event event; ++ struct input_event event {}; + gsize count; + GError* err = nullptr; + if (g_io_channel_read_chars(source, (gchar*)&event, sizeof(event), &count, +@@ -301,18 +388,125 @@ gboolean LinuxGamepadService::OnGamepadData(GIOChannel* source, + break; + } + +- // TODO: store device state? +- if (event.type & JS_EVENT_INIT) { +- continue; +- } +- + switch (event.type) { +- case JS_EVENT_BUTTON: +- service->NewButtonEvent(gamepad->handle, event.number, !!event.value); ++ case EV_KEY: ++ switch (event.code) { ++ /* The gamepad events are meaningful, and according to ++ * https://www.kernel.org/doc/html/latest/input/gamepad.html ++ * "No other devices, that do not look/feel like a gamepad, shall ++ * report these events" */ ++ case BTN_SOUTH: ++ service->NewButtonEvent(gamepad->handle, BUTTON_INDEX_PRIMARY, ++ !!event.value); ++ break; ++ case BTN_EAST: ++ service->NewButtonEvent(gamepad->handle, BUTTON_INDEX_SECONDARY, ++ !!event.value); ++ break; ++ case BTN_NORTH: ++ service->NewButtonEvent(gamepad->handle, BUTTON_INDEX_QUATERNARY, ++ !!event.value); ++ break; ++ case BTN_WEST: ++ service->NewButtonEvent(gamepad->handle, BUTTON_INDEX_TERTIARY, ++ !!event.value); ++ break; ++ case BTN_TL: ++ service->NewButtonEvent(gamepad->handle, BUTTON_INDEX_LEFT_SHOULDER, ++ !!event.value); ++ break; ++ case BTN_TR: ++ service->NewButtonEvent(gamepad->handle, ++ BUTTON_INDEX_RIGHT_SHOULDER, !!event.value); ++ break; ++ case BTN_TL2: ++ service->NewButtonEvent(gamepad->handle, BUTTON_INDEX_LEFT_TRIGGER, ++ !!event.value); ++ break; ++ case BTN_TR2: ++ service->NewButtonEvent(gamepad->handle, BUTTON_INDEX_RIGHT_TRIGGER, ++ !!event.value); ++ break; ++ case BTN_SELECT: ++ service->NewButtonEvent(gamepad->handle, BUTTON_INDEX_BACK_SELECT, ++ !!event.value); ++ break; ++ case BTN_START: ++ service->NewButtonEvent(gamepad->handle, BUTTON_INDEX_START, ++ !!event.value); ++ break; ++ case BTN_MODE: ++ service->NewButtonEvent(gamepad->handle, BUTTON_INDEX_META, ++ !!event.value); ++ break; ++ case BTN_THUMBL: ++ service->NewButtonEvent( ++ gamepad->handle, BUTTON_INDEX_LEFT_THUMBSTICK, !!event.value); ++ break; ++ case BTN_THUMBR: ++ service->NewButtonEvent( ++ gamepad->handle, BUTTON_INDEX_RIGHT_THUMBSTICK, !!event.value); ++ break; ++ case BTN_DPAD_UP: ++ service->NewButtonEvent(gamepad->handle, BUTTON_INDEX_DPAD_UP, ++ !!event.value); ++ break; ++ case BTN_DPAD_DOWN: ++ service->NewButtonEvent(gamepad->handle, BUTTON_INDEX_DPAD_DOWN, ++ !!event.value); ++ break; ++ case BTN_DPAD_LEFT: ++ service->NewButtonEvent(gamepad->handle, BUTTON_INDEX_DPAD_LEFT, ++ !!event.value); ++ break; ++ case BTN_DPAD_RIGHT: ++ service->NewButtonEvent(gamepad->handle, BUTTON_INDEX_DPAD_RIGHT, ++ !!event.value); ++ break; ++ default: ++ /* For non-gamepad events, this is the "anything goes" numbered ++ * handling that should be handled with remappers. */ ++ gamepad->remapper->RemapButtonEvent( ++ gamepad->handle, gamepad->key_map[event.code], !!event.value); ++ break; ++ } + break; +- case JS_EVENT_AXIS: +- service->NewAxisMoveEvent(gamepad->handle, event.number, +- ((float)event.value) / kMaxAxisValue); ++ case EV_ABS: ++ if (!gamepad->abs_info.count(event.code)) continue; ++ switch (event.code) { ++ case ABS_HAT0X: ++ service->NewButtonEvent( ++ gamepad->handle, BUTTON_INDEX_DPAD_LEFT, ++ AxisNegativeAsButton( ++ ScaleAxis(gamepad->abs_info[event.code], event.value))); ++ service->NewButtonEvent( ++ gamepad->handle, BUTTON_INDEX_DPAD_RIGHT, ++ AxisPositiveAsButton( ++ ScaleAxis(gamepad->abs_info[event.code], event.value))); ++ break; ++ case ABS_HAT0Y: ++ service->NewButtonEvent( ++ gamepad->handle, BUTTON_INDEX_DPAD_UP, ++ AxisNegativeAsButton( ++ ScaleAxis(gamepad->abs_info[event.code], event.value))); ++ service->NewButtonEvent( ++ gamepad->handle, BUTTON_INDEX_DPAD_DOWN, ++ AxisPositiveAsButton( ++ ScaleAxis(gamepad->abs_info[event.code], event.value))); ++ break; ++ case ABS_HAT1X: ++ case ABS_HAT1Y: ++ case ABS_HAT2X: ++ case ABS_HAT2Y: ++ case ABS_HAT3X: ++ case ABS_HAT3Y: ++ break; ++ default: ++ gamepad->remapper->RemapAxisMoveEvent( ++ gamepad->handle, gamepad->abs_map[event.code], ++ ScaleAxis(gamepad->abs_info[event.code], event.value)); ++ break; ++ } + break; + } + } +diff --git dom/gamepad/moz.build dom/gamepad/moz.build +index 5f55d5a95e96..544b7f927736 100644 +--- dom/gamepad/moz.build ++++ dom/gamepad/moz.build +@@ -59,7 +59,7 @@ elif CONFIG["MOZ_WIDGET_TOOLKIT"] == "windows": + UNIFIED_SOURCES += ["windows/WindowsGamepad.cpp"] + elif CONFIG["MOZ_WIDGET_TOOLKIT"] == "android": + UNIFIED_SOURCES += ["android/AndroidGamepad.cpp"] +-elif CONFIG["OS_ARCH"] == "Linux": ++elif CONFIG["OS_ARCH"] in ("Linux", "FreeBSD", "DragonFly"): + UNIFIED_SOURCES += ["linux/LinuxGamepad.cpp"] + else: + UNIFIED_SOURCES += ["fallback/FallbackGamepad.cpp"] |