Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 29 Aug 2009 13:47:25 +0000
From:      Alexey Dokuchaev <danfe@FreeBSD.org>
To:        Xin LI <delphij@FreeBSD.org>
Cc:        svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org
Subject:   Re: svn commit: r196524 - in head/sys: dev/syscons i386/isa
Message-ID:  <20090829134724.GA35926@FreeBSD.org>
In-Reply-To: <200908242235.n7OMZrs6008394@svn.freebsd.org>
References:  <200908242235.n7OMZrs6008394@svn.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Mon, Aug 24, 2009 at 10:35:53PM +0000, Xin LI wrote:
> Author: delphij
> Date: Mon Aug 24 22:35:53 2009
> New Revision: 196524
> URL: http://svn.freebsd.org/changeset/base/196524
> 
> Log:
>   Fix VESA modes and allow 8bit depth modes.
>   
>   PR:		i386/124902
>   Submitted by:	paradox <ddkprog yahoo com>
>   MFC after:	2 months
> 
> Modified:
>   head/sys/dev/syscons/scvgarndr.c
>   head/sys/dev/syscons/scvidctl.c
>   head/sys/i386/isa/vesa.c

This commit broke logo screensaver with vesa on my NEC Versa S950
laptop.  Reverting and rebuilding/reloading vesa module restored correct
behavior.  I'm not familiar with our VESA implementation internals to
give you sufficient analysis right now, but will take a look.

Who did review these changes, BTW?

./danfe



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20090829134724.GA35926>