[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <20070208215237.e5a48659.akpm@linux-foundation.org>
Date: Thu, 8 Feb 2007 21:52:37 -0800
From: Andrew Morton <akpm@...ux-foundation.org>
To: "Ananiev, Leonid I" <leonid.i.ananiev@...el.com>
Cc: <linux-kernel@...r.kernel.org>, "linux-aio" <linux-aio@...ck.org>,
"Zach Brown" <zach.brown@...cle.com>, <suparna@...ibm.com>,
"Chris Mason" <chris.mason@...cle.com>,
"Badari Pulavarty" <pbadari@...ibm.com>
Subject: Re: [PATCH] aio: fix kernel bug when page is temporally busy
On Fri, 9 Feb 2007 08:41:41 +0300 "Ananiev, Leonid I" <leonid.i.ananiev@...el.com> wrote:
>
> > invalidate_inode_pages2() has other callers. I suspect with this
> change
> > we'll end up leaking EIOCBRETRY back to userspace.
>
> EIOCBRETRY is used and caught already in do_sync_read() and
> do_sync_readv_writev().
To pick one example:
nfs_follow_link
->nfs_revalidate_mapping_nolock
->nfs_invalidate_mapping_nolock
->invalidate_inode_pages2
so that, I assume, affects open(), unlink(), etc.
> Below fixed patch against kernel 2.6.20.
The tab->spaces issue is fixed, but it's still all wordwrapped.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists