[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <20250829194908.24852-3-soci@c64.rulez.org>
Date: Fri, 29 Aug 2025 21:49:07 +0200
From: Zsolt Kajtar <soci@....rulez.org>
To: linux-serial@...r.kernel.org,
linux-kernel@...r.kernel.org,
gregkh@...uxfoundation.org,
jirislaby@...nel.org
Cc: Zsolt Kajtar <soci@....rulez.org>
Subject: [PATCH 2/3] tty/vt: Prevent 8th bit corruption with soft cursor
The attributes of the soft cursor are configurable and one would rightly
expect that only the attributes are to be affected. But that's not
guaranteed for a font with 512 glyphs as the 8th bit is in the attribute
byte. This patch makes sure that really only the attribute bits are
changed by the cursor and not the glyph's appearance.
Signed-off-by: Zsolt Kajtar <soci@....rulez.org>
---
drivers/tty/vt/vt.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/tty/vt/vt.c b/drivers/tty/vt/vt.c
index c6c931047..d54f4d24e 100644
--- a/drivers/tty/vt/vt.c
+++ b/drivers/tty/vt/vt.c
@@ -823,6 +823,7 @@ static void add_softcursor(struct vc_data *vc)
{
int i = scr_readw((u16 *) vc->vc_pos);
u32 type = vc->vc_cursor_type;
+ int mask = vc->vc_hi_font_mask | 0xff;
if (!(type & CUR_SW))
return;
@@ -836,6 +837,7 @@ static void add_softcursor(struct vc_data *vc)
i ^= CUR_BG;
if ((type & CUR_INVERT_FG_BG) && (i & CUR_FG) == ((i & CUR_BG) >> 4))
i ^= CUR_FG;
+ i = (i & ~mask) | (softcursor_original & mask);
scr_writew(i, (u16 *)vc->vc_pos);
if (con_should_update(vc))
con_putc(vc, i, vc->state.y, vc->state.x);
--
2.30.2
Powered by blists - more mailing lists