From: Erik de Castro Lopo <erikd+lkml@sensorynetworks.com>
To: linux-kernel@vger.kernel.org
Cc: Hugh Dickins <hugh@veritas.com>
Subject: Re: 2.6.10 : kernel BUG at mm/rmap.c:483!
Date: Thu, 17 Feb 2005 09:59:32 +1100 [thread overview]
Message-ID: <20050217095932.196c2f8b.erikd+lkml@sensorynetworks.com> (raw)
In-Reply-To: <Pine.LNX.4.61.0502160655330.16056@goblin.wat.veritas.com>
On Wed, 16 Feb 2005 08:07:11 +0000 (GMT)
Hugh Dickins <hugh@veritas.com> wrote:
> Yes, I believe /dev/mem handling on 2.4 and 2.6 is much the same,
> but the rmap.c BUG you're hitting in 2.6 had no equivalent in 2.4 -
> maybe the 2.4 case has actually been unsafe too, or maybe not.
OK, that makes sense.
> Though there have been other reports of this BUG, some remaining
> mysterious, some down to bad ram, your case is almost certainly
> due to mismatched PageReserved accounting.
>
> mmap of /dev/mem only works usefully when the mapped pages are
> outside of ordinary ram,
This is memory on a PCI card which is definitely not part of
ordinary ram.
> or temporarily marked PageReserved, normally by the relevant driver.
>
> I may be wrong, but I think it's fair to say that usually when
> there's to be cooperation between driver and user space of this
> kind, the driver would offer its own device and mmap entry point,
> user space would be mapping that rather than /dev/mem, and file
Since I wrote the driver, thats definitely a possibility.
> reference counting would avoid the possibility that driver clears
> PageReserved while the page is mapped.
Ahh, here's a clue. My driver uses ioremap_nocache() to get virtual
pointers to physical register/memory (addresses obtain using
pci_resource_start etc) on the card and calls iounmap() when the
driver is unloaded. However, while the driver is loaded, the user
space program mmaps a superset of the same memory. Is it possible
that this is screwing up the page accounting?
> Beyond this particular BUG, clearing PageReserved too soon is
> likely to lead to freeing an already freed page - noticed with
> a warning if the page is still free, but mayhem if by then reused.
The driver is very strict about mapping memory on load and
unmapping memory on unload. I'm pretty confident that this is
working correctly because the driver can be repeatedly loaded
and unloaded without errors. During normal operation, it does
not fiddle with mmapping.
> Which is the driver involved?
Its a driver I wrote for a piece of hardware designed by the
company I work for.
> Or, might the user space application be (arguably) at fault,
> mmap'ing an indeterminate uncontrolled area of /dev/mem, some
> pages of which may or may not be PageReserved at any instant?
Nope, very, very unlikely. My driver is not doing anything
complicated with memory mapping and I know the user space code
is reading the right memory location because the read data is
valid.
Thanks Hugh. Your input was very helpful.
Erik
--
-------------------------------------------------------
[N] Erik de Castro Lopo, Senior Computer Engineer
[E] erik.de.castro.lopo@sensorynetworks.com
[W] http://www.sensorynetworks.com
[T] +61 2 83022726
[F] +61 2 94750316
[A] L6/140 William St, East Sydney NSW 2011, Australia
-------------------------------------------------------
A good debugger is no substitute for a good test suite.
prev parent reply other threads:[~2005-02-16 22:59 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2005-02-16 1:07 Erik de Castro Lopo
2005-02-16 8:07 ` Hugh Dickins
2005-02-16 22:59 ` Erik de Castro Lopo [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=20050217095932.196c2f8b.erikd+lkml@sensorynetworks.com \
--to=erikd+lkml@sensorynetworks.com \
--cc=hugh@veritas.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®