On Fri, Jul 22, 2022 at 02:11:33PM +0800, Xingrui Yi wrote:
> As was demonstrated by commit ff2047fb755d ("vt: drop old FONT ioctls"),
> old font ioctls like PIO_FONT have been deleted and KDFONTOP ioctl is used
> for years instead.
>
> However, unused definitions of these ioctl numbers and "strut
> consolefontdesc" still exist in a uapi header. They could have been removed
> since no userspace was using them. Otherwise they will become a
> misleading for users, and users will fail with ENOTTY with wrong call.
>
> We are moving old font ioctl numbers definitions here to guide users to
> use KDFONTOP.
>
> Signed-off-by: Xingrui Yi <yixingrui@xxxxxxxxxxxxxxxxx>
> ---
> include/uapi/linux/kd.h | 13 -------------
> 1 file changed, 13 deletions(-)
>
> diff --git a/include/uapi/linux/kd.h b/include/uapi/linux/kd.h
> index ee929ece4112..d739b17fc942 100644
> --- a/include/uapi/linux/kd.h
> +++ b/include/uapi/linux/kd.h
> @@ -6,19 +6,6 @@
>
> /* 0x4B is 'K', to avoid collision with termios and vt */
>
> -#define GIO_FONT 0x4B60 /* gets font in expanded form */
> -#define PIO_FONT 0x4B61 /* use font in expanded form */
> -
> -#define GIO_FONTX 0x4B6B /* get font using struct consolefontdesc */
> -#define PIO_FONTX 0x4B6C /* set font using struct consolefontdesc */
> -struct consolefontdesc {
> - unsigned short charcount; /* characters in font (256 or 512) */
> - unsigned short charheight; /* scan lines per character (1-32) */
> - char __user *chardata; /* font data in expanded form */
> -};
> -
> -#define PIO_FONTRESET 0x4B6D /* reset to default font */
> -
> #define GIO_CMAP 0x4B70 /* gets colour palette on VGA+ */
> #define PIO_CMAP 0x4B71 /* sets colour palette on VGA+ */
>
Did you rebuild a distribution like Debian to verify that these values
and structure are not used anywhere?
thanks,
greg k-h
