linux-fbdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Ville Syrjälä" <syrjala@sci.fi>
To: linux-fbdev-devel@lists.sourceforge.net
Cc: Geert Uytterhoeven <geert@linux-m68k.org>
Subject: Re: matroxfb returning incorrect fix.smem_len?
Date: Thu, 1 Mar 2007 18:58:17 +0200	[thread overview]
Message-ID: <20070301165817.GB7907@sci.fi> (raw)
In-Reply-To: <45E703A6.6080200@us.ibm.com>

On Thu, Mar 01, 2007 at 08:47:34AM -0800, Ian Romanick wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
> 
> Geert Uytterhoeven wrote:
> > On Wed, 28 Feb 2007, Ian Romanick wrote:
> >> While doing some work on the pci-rework branch of X.org, I noticed
> >> something strange.  matroxfb correctly detects that my card has 16MiB,
> >> but fix.smem_len is only 8MiB.
> >>
> >> I looked around the matroxfb code and some of the other driver's code,
> >> and I believe that matroxfb is incorrectly setting fix.smem_len.
> >> Matroxfb sets fix.smem_len to (total_memory_size -
> >> memory_used_by_current_mode).  Other drivers, such as nvidiafb do not do
> >> this.
> > 
> > Did matroxfb detect a dual-head capable card?
> 
> I did some more digging on this yesterday.  It seems that ydstorg.bytes
> is always zero, so what I originally thought was the problem is not the
> problem.
> 
> Looking at the logs, it looks like it is detecting my card as dual-head,
> but it's not.  The weird thing is that I added a printk in
> matroxfb_update_fix to show what smem_len is begin set to.  Before I
> start X, it prints 16MiB twice.  As soon as X starts, it prints 8MiB and
> returns that value to the X server.
> 
> I'm not quite sure what to make of that.

IIRC someone using DirectFB noticed something similar. I didn't bother 
investigating it much though.

Also matroxfb will never report > 16MB. We have a patch in DirectFB to 
fix that but IIRC it was not accepted into matroxfb because the matroxfb 
accel code can't actually handle it.

-- 
Ville Syrjälä
syrjala@sci.fi
http://www.sci.fi/~syrjala/

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

      reply	other threads:[~2007-03-01 16:58 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-02-28 19:20 matroxfb returning incorrect fix.smem_len? Ian Romanick
2007-03-01 13:21 ` Geert Uytterhoeven
2007-03-01 16:47   ` Ian Romanick
2007-03-01 16:58     ` Ville Syrjälä [this message]

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=20070301165817.GB7907@sci.fi \
    --to=syrjala@sci.fi \
    --cc=geert@linux-m68k.org \
    --cc=linux-fbdev-devel@lists.sourceforge.net \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).