lkml.org 
[lkml]   [2006]   [Jun]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] autofs4 needs to force fail return revalidate
On Thu, 22 Jun 2006, Andrew Morton wrote:

> On Fri, 23 Jun 2006 12:14:09 +0800 (WST)
> Ian Kent <raven@themaw.net> wrote:
>
> > >
> > > Also, did you consider broadening the ->d_revalidate() semantics? It
> > > appears that all implementations return 0 or 1. You could teach the VFS to
> > > also recognise and act upon a -ve return value, and do this trickery within
> > > the autofs d_revalidate(), perhaps?
> > >
> >
> > Yep and I've now done this.
> > I think this is in fact the only way to do it.
>
> Below is the combined patch for reviewing purposes.
>
> Removing that __always_inline saves 30-odd bytes. I suspect that removing
> it would be a performance loss in this case. But I'll make it just
> `inline' because __always_inline is peculiar, and people will wonder what's
> special about this function.
>

Thanks Andrew.
The combined diff is exactly what I'm trying to achieve.

I marked the function __always_inline because several other functions
that appear to be frequently called from this code path are also declared
this way. Hopefully someone with more experience of this area of the VFS
will recommend the best choice.

Ian

-
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: 2006-06-23 08:47    [W:0.085 / U:0.104 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site