lkml.org 
[lkml]   [2005]   [Aug]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: Kernel bug: Bad page state: related to generic symlink code and mmap
On Fri, Aug 19, 2005 at 07:00:38PM +0100, Christoph Hellwig wrote:
> On Fri, Aug 19, 2005 at 07:02:18PM +0100, Al Viro wrote:
> > On Fri, Aug 19, 2005 at 05:53:32PM +0100, Al Viro wrote:
> > > I'm taking NFS helpers to libfs.c and switching ncpfs to them. IMO that's
> > > better than copying the damn thing and other network filesystems might have
> > > the same needs eventually...
> >
> > [something like this - completely untested]
> >
> > * stray_page_get_link(inode, filler) - returns ERR_PTR(error) or pointer
> > to symlink body. Said symlink body sits in a page at offset equal to
> > offsetof(page, struct stray_page_link). filler() is expected to put it
> > at such offset. Page is cached.
> >
> > * stray_page_put_link() - ->put_link() suitable for links obtained from
> > stray_page_get_link(). Unlike the usual pagecache-based variants, this
> > sucker does _not_ rely on page staying cached.
> >
> > * nfs and ncpfs switched to the helpers above.
>
> Can you add some kerneldoc comments to describe them? Especially as
> the name is not very descriptive.

Hey, if anybody has suggestions on names - they are very welcome ;-)

FWIW, I'd rather take page_symlink(), page_symlink_inode_operations,
page_put_link(), page_follow_link_light(), page_readlink(), page_getlink(),
generic_readlink() and vfs_readlink() to the same place where these guys
would live. They all belong together and none of them has any business
in fs/namei.c. Options: fs/libfs.c or separate library since fs/libfs.c
is getting crowded. Linus, do you have any objections to that or suggestions
on filename here?
-
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-08-19 21:38    [W:0.720 / U:0.092 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site