Re: [PATCH] 2.4.0-test6-pre8 - dark vga textmode console problem in RIVA TNT 2

From: Dirk Hohndel (hohndel@suse.de)
Date: Wed Aug 09 2000 - 00:55:58 EST


On Tue 08 Aug, Linus Torvalds wrote:
>
>
> On Tue, 8 Aug 2000, Linus Torvalds wrote:
> >
> > This seems to be a bug in the XF86 4.0.x code-base. The patch is wrong:
> > the new Linux kernel palette setting order is correct (we should set the
> > palette _after_ switching to the new console).
> >
> > So yes, the patch will fix the problem. But I don't think it's correct.
>
> It isn't correct, but after looking a bit at what the kernel does, I
> suspect it's not a bug in XFree86 either.
>
> The fact is, that we should only set the palette after we've switched to a
> new console, and only if that console is in text mode. That part of the
> code is right. Your patch messes that logic up, and sets the palette even
> in graphics mode and before doing the switch).

Please don't do that...

> However, that also misses a corner case: let's say that we switch modes
> (not VC's) on an existing console that _used_ to be in graphics mode (so
> we didn't set the palette when we switched to it) into text mode. We
> should now set the palette on that graphics->textmode switch because it
> was inhibited before by the graphics mode.

Correct.

        Dirk

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.rutgers.edu
Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Tue Aug 15 2000 - 21:00:17 EST