Skip to content

VT: Add ability to get font requirements

Alexey Gladkov kbd linux-api linux-fbdev linux-serial lkml 6 versions – In review

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v5 Latest3 patches 11 repliesIn review
  2. v43 patches 4 repliesSuperseded
  3. v3 posting 2/20 of 2 patches 0 repliesSuperseded
Earlier versions · 3
VersionPostedPatchesRepliesStatus
v3 posting 1/22 patches9 repliesSuperseded
v2 RFC5 patches2 repliesSuperseded
v1 RFC5 patches7 repliesSuperseded

Changes in v5 from the cover letter

* Use data types that are compatible with the uapi structure.
* Use _IOR to define a new ioctl as required for new ioctls.

Patches in v5 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/3VT: Use macros to define ioctlsinclude/uapi/linux/kd.h+49 -474 replies
2/3VT: Add KDFONTINFO ioctldrivers/tty/vt/vt.c drivers/tty/vt/vt_ioctl.c include/linux/console.h +2 more+55 -07 replies
3/3VT: Allow to get max font width and heightdrivers/video/console/newport_con.c drivers/video/console/sticon.c drivers/video/console/vgacon.c +1 more+76 -70 replies

Reviews trailers per patch

2 of 3 patches have a Reviewed-by or Acked-by.

PatchHelge Deller
1/3
2/3A-b
3/3A-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v5 thread

Cover letter [PATCH v5 0/3] VT: Add ability to get font requirements

We now have KD_FONT_OP_SET_TALL, but in fact such large fonts cannot be loaded. No console driver supports tall fonts. Unfortunately, userspace cannot distinguish the lack of support in the driver from errors in the font itself. In all cases, EINVAL will be returned.

This patchset adds a separate ioctl to obtain the font parameters supported by the console driver.

v5:
* Use data types that are compatible with the uapi structure.
* Use _IOR to define a new ioctl as required for new ioctls.

v4:
* Rebased on v6.9-rc1 and conflicts have been fixed.
* Do not copy KDFONTINFO data from the userspace.
* Header include/uapi/linux/kd.h uses _IOC macros to define ioctls.

v3:
* Added the use of the in_range macro.
* Squashed the commits that add ioctl to console divers.

v2:
* Instead of the KDFONTOP extension, a new ioctl has been added to obtain font information.

Alexey Gladkov (3):
VT: Use macros to define ioctls
VT: Add KDFONTINFO ioctl
VT: Allow to get max font width and height

 drivers/tty/vt/vt.c                 |  24 ++++++
 drivers/tty/vt/vt_ioctl.c           |  13 ++++
 drivers/video/console/newport_con.c |  21 +++++-
 drivers/video/console/sticon.c      |  25 ++++++-
 drivers/video/console/vgacon.c      |  21 +++++-
 drivers/video/fbdev/core/fbcon.c    |  16 ++++
 include/linux/console.h             |   3 +
 include/linux/vt_kern.h             |   1 +
 include/uapi/linux/kd.h             | 110 ++++++++++++++++------------
 9 files changed, 180 insertions(+), 54 deletions(-)
Signature · 1 line
2.44.0