Paths

Table of Contentst

Differential D30542

evdev: Extend EVIOCGRAB ioctl scope to cover kbd/sysmouse interfaces
ClosedPublic
Actions

Authored by wulf on May 29 2021, 1:12 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, Nov 3, 9:48 PM

	Unknown Object (File)
	Sun, Nov 3, 9:48 PM

	Unknown Object (File)
	Sun, Nov 3, 9:48 PM

	Unknown Object (File)
	Sun, Nov 3, 9:26 PM

	Unknown Object (File)
	Oct 4 2024, 7:19 PM

	Unknown Object (File)
	Oct 4 2024, 1:53 PM

	Unknown Object (File)
	Oct 4 2024, 5:08 AM

	Unknown Object (File)
	Oct 3 2024, 5:31 AM

View All 75 Files

Subscribers

Details

Reviewers

imp
zeising

Commits

rG183088934a6c: evdev: Extend EVIOCGRAB ioctl scope to cover kbd interface
rG4a0db5e2920c: evdev: Extend EVIOCGRAB ioctl scope to cover sysmouse interface

Summary

EVIOCGRAB ioctl execution on /dev/input/event# device node gains exclusive
access to this device. It is used mostly for development purposes and
remote control software. See e.g. https://reviews.freebsd.org/D30020
which is the reason of creation of this patch.

Extending EVIOCGRAB to kbd interface increases chances of foot shooting
as terminal control commands in K_CODE and K_XLATE modes are ignored.
So runnig of libinput debug-events --grab will effectively break all
inputs, but that behavior is consistent with what we have in Xorg server.

To reduce negative effects grabbing is disabled in KDB and during panics.

Test Plan

run in vt console # libinput debug-events --grab
than ensure that no symbols are echoed to the screen on keypresses.
Only libinput events are shown.
Don't forget to open remote ssh session to kill libinput or you will need
to push reset button otherwise.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

wulf requested review of this revision.May 29 2021, 1:12 PM

wulf created this revision.

Harbormaster completed remote builds in B39561: Diff 90103.May 29 2021, 1:12 PM

wulf mentioned this in D30020: bhyve: add virtio-input emulation.May 29 2021, 1:25 PM

corvink added a subscriber: corvink.Jun 8 2021, 9:45 AM

Ping. What's the state of this commit? The issue still exists.
I've tested this commit some time ago and it worked. I'm going to test it again.

This revision was not accepted when it landed; it landed in state Needs Review.Nov 17 2022, 9:52 PM

Closed by commit rG4a0db5e2920c: evdev: Extend EVIOCGRAB ioctl scope to cover sysmouse interface (authored by wulf). · Explain Why

This revision was automatically updated to reflect the committed changes.

wulf added a commit: rG4a0db5e2920c: evdev: Extend EVIOCGRAB ioctl scope to cover sysmouse interface.

I forgot about this revision. Sorry.
Mouse part has been committed. Let me know if keyboard part is needed too.

In D30542#850947, @wulf wrote:

I forgot about this revision. Sorry.
Mouse part has been committed. Let me know if keyboard part is needed too.

Thanks. Yes, the keyboard part is even more important.

wulf added a commit: rG183088934a6c: evdev: Extend EVIOCGRAB ioctl scope to cover kbd interface.Nov 18 2022, 11:57 PM

Revision Contents
Changeset List

Path

Size

sys/

dev/

atkbdc/

12 lines

evdev/

1 line

11 lines

syscons/

2 lines

usb/

input/

4 lines

vt/

2 lines

Diff 113256

sys/dev/atkbdc/psm.c

Loading...

sys/dev/evdev/evdev.h

Loading...

sys/dev/evdev/evdev.c

Loading...

sys/dev/syscons/sysmouse.c

Loading...

sys/dev/usb/input/ums.c

Loading...

sys/dev/vt/vt_sysmouse.c

Loading...