[SeaBIOS] [PATCH] vga: Don't advertise multiple pages for lfb modes.

Kevin O'Connor kevin at koconnor.net
Fri Sep 13 22:37:56 CEST 2013


On Wed, Sep 11, 2013 at 12:45:27PM +0200, Gerd Hoffmann wrote:
> Retested, figured with seavgabios win8 modeswitching works just fine,
> even without the patch.  /me is puzzled, I'm sure I've tried that before
> and it didn't work.

Okay, so no patch is necessary?  I think it might be worthwhile to
clamp the pages value to a number between 1-127 - see the patch I sent
separately.

> 
> > +        dprintf(1, "mode=%02x x=%d y=%d d=%d pages=%d\n", videomode
> > +                , mode_info->xres, mode_info->yres, mode_info->bits_per_pixel
> > +                , mode_info->pages);
> 
> While wading through the source I've noticed mode_info has not only
> "pages", but also "linear_pages" and "bank_pages" fields (new for vbe3).
> seavgabios doesn't set them.  Should it?

I don't know.  The VBE3 support is fairly limited - if the spec calls
for it then I'd say it makes sense to fill it in.  In practice, I
suspect very few programs (if any) care what VBE reports.

-Kevin



More information about the SeaBIOS mailing list