src/sys/dev/kbd, branch releng/12.4

MFC r355796-r355797, r355799: kbd: defaults for get_fkeystr/diag

2019-12-22T17:15:48Z

The genkbd version of these remains exposed for stable branches, but keyboard drivers that just want to use the defaults can simply not provide their own. There shouldn't be any unset in the wild. r355796: kbd: provide default implementations of get_fkeystr/diag Most keyboard drivers are using the genkbd implementations as it is; formally use them for any that aren't set. r355797: chrome_kb: remove default get_fkeystr/diag implementations This file was missed in r355796, but no harm would have come from this. r355799: kbd: patch linker set methods, too This is needed after r355796. Some double-registration of kbd drivers needs to be sorted out, then this sysinit will simply add these drivers into the normal list and kill off any other bits in the driver that are aware of the linker set, for simplicity.

Fix kbdd_diag definition

2019-12-22T17:04:16Z

This is a direct commit to stable branches as these have been converted to inline functions in head, which will likely not get MFC'd due to limited utility -- follow up locking assertions likely cannot be done in stable branches. The use of 'level' in the macro definition was truncated to 'leve', which makes the macro unusable.

MFC r355842: kbd: const'ify a couple of keyboard_driver fields

2019-12-22T16:57:19Z

evdev: Remove evdev.ko linkage dependency on kbd driver

2018-08-13T19:05:53Z

Move evdev_ev_kbd_event() helper from evdev to kbd.c as otherwise evdev unconditionally requires all keyboard and console stuff to be compiled into the kernel. This dependency happens as evdev_ev_kbd_event() helper references kbdsw global variable defined in kbd.c through use of kbdd_ioctl() macro. While here make all keyboard drivers respect evdev_rcpt_mask while setting typematic rate and LEDs with evdev interface. Requested by: Milan Obuch Reviewed by: hselasky, gonzo Differential Revision: https://reviews.freebsd.org/D16614

Revert r327828, r327949, r327953, r328016-r328026, r328041:

2018-01-21T15:42:36Z

Uses of mallocarray(9). The use of mallocarray(9) has rocketed the required swap to build FreeBSD. This is likely caused by the allocation size attributes which put extra pressure on the compiler. Given that most of these checks are superfluous we have to choose better where to use mallocarray(9). We still have more uses of mallocarray(9) but hopefully this is enough to bring swap usage to a reasonable level. Reported by: wosch PR: 225197

dev: make some use of mallocarray(9).

2018-01-13T22:30:30Z

Focus on code where we are doing multiplications within malloc(9). None of these is likely to overflow, however the change is still useful as some static checkers can benefit from the allocation attributes we use for mallocarray. This initial sweep only covers malloc(9) calls with M_NOWAIT. No good reason but I started doing the changes before r327796 and at that time it was convenient to make sure the sorrounding code could handle NULL values.

sys/dev: further adoption of SPDX licensing ID tags.

2017-11-27T14:52:40Z

Mainly focus on files that use BSD 2-Clause license, however the tool I was using misidentified many licenses so this was mostly a manual - error prone - task. The Software Package Data Exchange (SPDX) group provides a specification to make it easier for automated tools to detect and summarize well known opensource licenses. We are gradually adopting the specification, noting that the tags are considered only advisory and do not, in any way, superceed or replace the license texts.

Remove pc98 support completely.

2017-01-28T02:22:15Z

I thank all developers and contributors for pc98. Relnotes: yes

Restore priority value for OGIO_KEYMAP

2017-01-07T15:58:57Z

PR: 206678 Submitted by: ecturt@gmail.com Reviewed by: cem Approved by: cem, matthew (mentor) MFC after: 1 week Differential Revision: https://reviews.freebsd.org/D5095

Use M_WAITOK in PIO_KEYMAP ioctl

2016-10-12T12:56:18Z

The malloc return value is not checked. Submitted by: CTurt MFC after: 1 week