All of lore.kernel.org
 help / color / mirror / Atom feed
From: Gerd Knorr <kraxel@bytesex.org>
To: Dave Jones <davej@redhat.com>, Andrew Morton <akpm@osdl.org>,
	linux-kernel@vger.kernel.org
Subject: Re: MTRR vesafb and wrong X performance
Date: Mon, 29 Nov 2004 18:34:19 +0100	[thread overview]
Message-ID: <20041129173419.GC26190@bytesex> (raw)
In-Reply-To: <20041129165701.GA903@redhat.com>

> size_total is calculated thus:
> 
>     size_total = screen_info.lfb_size * 65536;

That comes almost directly from the BIOS, the screen_info struct is
filled by the real mode boot code (vga.S IIRC).

>     if (vram_total)
>         size_total = vram_total * 1024 * 1024;

The vesafb option to override stuff.

>     if (size_total < size_vmode)
>         size_total = size_vmode;

Thats kida silly, but as I've found some simliar construct in the old
code I left it in to avoid breaking stuff.  Guess there is a reason that
this was there.  I'll take that as proof that broken BIOSes exist which
don't fill screen_info.lfb_size correctly ;)

> or blacklist if there aren't too many perhaps?

Hmm, how identify them?  Map the BIOS and poke around there?
screen_info gives next to no info here.

Maybe it works better to walk the PCI device list, find the correct
gfx card using the framebuffer start address, then double-check the
size by looking at the PCI ressources?

  Gerd

-- 
#define printk(args...) fprintf(stderr, ## args)

  reply	other threads:[~2004-11-29 18:00 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-11-24 23:15 MTRR vesafb and wrong X performance Pawel Fengler
2004-11-25  1:18 ` Andrew Morton
2004-11-25 21:56   ` Pawel Fengler
2004-11-29 11:12     ` Gerd Knorr
2004-11-29 15:40       ` Dave Jones
2004-11-29 16:22         ` Gerd Knorr
2004-11-29 16:57           ` Dave Jones
2004-11-29 17:34             ` Gerd Knorr [this message]
2004-11-29 17:22               ` Alan Cox
2004-11-29 18:17               ` Dave Jones
2004-11-29 17:23                 ` Alan Cox
2004-11-29 16:24         ` Alan Cox
2004-11-25  8:49 ` Gerd Knorr
  -- strict thread matches above, loose matches on Subject: below --
2004-11-26 16:48 Pawel Fengler

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20041129173419.GC26190@bytesex \
    --to=kraxel@bytesex.org \
    --cc=akpm@osdl.org \
    --cc=davej@redhat.com \
    --cc=linux-kernel@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.