lkml.org 
[lkml]   [2002]   [Jun]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [2.5.20-BUG] 3c59x + highmem + acpi + nfs -> kernel panic
At 12:39 04/06/02, Trond Myklebust wrote:
>On Tuesday 04 June 2002 08:37, Andrew Morton wrote:
>
> > The problem was that pagecache data on the NFS client was showing
> > incorrect chunks of several k's of zeroes. No particuar alignment,
> > either. And it only happened when the machine is under page-replacement
> > pressure. And only when the machine has highmem.
> >
> > It'd be nice to understand _why_ it fixed it. Do we know why NFS
> > was losing data when using KM_USER0? As far as I can see the new
> > and old code look pretty darn similar. Interested.
>
>Anton's Oops showed that kmap_atomic(page, KM_USER0) is sometimes failing to
>return an address, something that we cannot accept in a networking bottom
>half (performance and reliability would suck if we had to delay and retry).
>That indicates that something is calling kmap_atomic() and then getting
>interrupted before it can call kunmap_atomic().
>
> >From what I can see, the only other place where KM_USER0 is employed
> would be
>in the *_highpage() helper routines in include/linux/highmem.h. These
>routines are used in various places, but are usually not protected against
>(soft and hard) interrupts or kernel pre-emption. Could it be that the latter
>is what is causing trouble?

That would make sense as any user of kmap_atomic() could be preempted at
any time. And this is definitely something we should keep in mind for the
future. Perhaps we should even go as far to disable preemption in
kmap_atomic() and reenable it in kunmap_atomic()? The atomic kmaps are
usually only done in critical sections which will execute for a short
amount of time so disabling preemption should be ok.

However, I had the preemptible option disabled in the kernel I reported the
Oops on, so it can't be responsible for this particular case.

Best regards,

Anton


--
"I've not lost my mind. It's backed up on tape somewhere." - Unknown
--
Anton Altaparmakov <aia21 at cantab.net> (replace at with @)
Linux NTFS Maintainer / IRC: #ntfs on irc.openprojects.net
WWW: http://linux-ntfs.sf.net/ & http://www-stu.christs.cam.ac.uk/~aia21/

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 13:22    [W:0.071 / U:3.020 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site