public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Ingo Molnar <mingo@elte.hu>
To: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Frans Pop <elendil@planet.nl>, Eric Anholt <eric@anholt.net>,
	nix.or.die@googlemail.com, linux-kernel@vger.kernel.org,
	rjw@sisk.pl, Arjan van de Ven <arjan@infradead.org>
Subject: Re: Linux 2.6.28-rc8
Date: Thu, 11 Dec 2008 17:35:48 +0100	[thread overview]
Message-ID: <20081211163548.GA11859@elte.hu> (raw)
In-Reply-To: <alpine.LFD.2.00.0812110814360.3340@localhost.localdomain>


* Linus Torvalds <torvalds@linux-foundation.org> wrote:

> 
> 
> On Thu, 11 Dec 2008, Frans Pop wrote:
> 
> > Eric Anholt wrote:
> > > My recommended solution, of course, is to remove vesafb.
> > 
> > How is taking away useful functionality from users a better option than 
> > just fixing the bug?
> 
> Well, just to clarify: it's not a _bug_. It's a benign warnign that two 
> subsystems are trying to map the same memory differently.
> 
> In this case, we have:
> 
> 	resource map sanity check conflict: 0xd0000000 0xdfffffff 0xd0000000 0xd07effff vesafb
> 
> and what it means is that the caller (which is i915_gem_entervt_ioctl) is 
> trying to apparently ioremap the _whole_ graphics card MMIO resource 
> (0xd0000000-0xdfffffff), but the vesafb driver has already registered the 
> fact that it uses _part_ of that resource (0xd0000000-0xd07effff).
> 
> There's no bug there. It's a warning. It's usually a very odd situation 
> when somebody tries to ioremap something that crosses resource reservation 
> boundaries, but the thing is, in this case it's not really a problem.
> 
> It's triggered by a couple of oddities:
> 
>  - fbcon (vesafb) is odd and only requests a partial resource, because it 
>    only uses part of the MMIO window.
> 
>  - the interaction between fbcon and X is odd to begin with, since they 
>    both use the same physical resource.
> 
> so it's a generic warning that triggers because these things 
> _shouldn't_ happen, but it's not actually an error in this case. We 
> could just remove the warning. Or leave it in, in case it finds other 
> (real) issues, and just ignore it.

hm, the warning caught a couple of real bugs already. (one in some cpq 
driver, another was in some networking driver iirc)

	Ingo

  reply	other threads:[~2008-12-11 16:36 UTC|newest]

Thread overview: 40+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-12-11  1:04 Linux 2.6.28-rc8 Linus Torvalds
2008-12-11  2:37 ` Gabriel C
2008-12-11  7:19   ` Eric Anholt
2008-12-11 16:07     ` Frans Pop
2008-12-11 16:22       ` Linus Torvalds
2008-12-11 16:35         ` Ingo Molnar [this message]
2008-12-11 17:05           ` Linus Torvalds
2008-12-11 20:36             ` Ingo Molnar
2008-12-11 20:46               ` Pekka Enberg
2008-12-11 21:34                 ` Suresh Siddha
2008-12-12  8:24                 ` Ingo Molnar
2008-12-12 15:57                   ` Arjan van de Ven
2008-12-12 16:11                     ` Linus Torvalds
2008-12-13 17:15                       ` Arjan van de Ven
2008-12-16 22:34                         ` Ingo Molnar
2008-12-11  5:09 ` Arjan van de Ven
2008-12-11  7:57 ` Nick Piggin
2008-12-11  8:07   ` Andrew Morton
2008-12-11  8:45     ` Nick Piggin
2008-12-12  3:02       ` Andrew Morton
2008-12-12  3:07         ` Nick Piggin
2008-12-12  3:39           ` Andrew Morton
2008-12-11  8:06 ` Willy Tarreau
2008-12-11  8:40 ` Joerg Roedel
2008-12-11 22:57   ` Theodore Tso
2008-12-11 23:12     ` Mike Travis
2008-12-11  9:26 ` Jean Delvare
2008-12-11 10:38 ` Frederik Deweerdt
2008-12-11 16:59   ` Andrew Morton
2008-12-11 13:00 ` Sam Ravnborg
2008-12-11 13:23   ` Paolo Ciarrocchi
2008-12-11 13:44 ` Alexey Zaytsev
2008-12-11 13:48 ` Ingo Molnar
2008-12-11 15:20 ` Pavel Machek
2008-12-11 15:29 ` David Howells
2008-12-12  3:19   ` David Miller
2008-12-12  5:40     ` Linus Torvalds
2008-12-12  7:54       ` Joerg Roedel
2008-12-12 15:48     ` Alan Cox
2008-12-11 20:12 ` Rafael J. Wysocki

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=20081211163548.GA11859@elte.hu \
    --to=mingo@elte.hu \
    --cc=arjan@infradead.org \
    --cc=elendil@planet.nl \
    --cc=eric@anholt.net \
    --cc=linux-kernel@vger.kernel.org \
    --cc=nix.or.die@googlemail.com \
    --cc=rjw@sisk.pl \
    --cc=torvalds@linux-foundation.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox