Re: [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 <[email protected]> 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 [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

---

• References:
  • [PATCH] autofs4 needs to force fail return revalidate
    • From: Ian Kent <[email protected]>
  • Re: [PATCH] autofs4 needs to force fail return revalidate
    • From: Andrew Morton <[email protected]>
  • Re: [PATCH] autofs4 needs to force fail return revalidate
    • From: Ian Kent <[email protected]>
  • Re: [PATCH] autofs4 needs to force fail return revalidate
    • From: Andrew Morton <[email protected]>

• Prev by Date: Re: [PATCH 17/25] x86_64 irq: Remove the msi assumption that irq == vector
• Next by Date: Re: [PATCH -rt] NUMA-Q fix to __cache_free and reap_alien
• Previous by thread: Re: [PATCH] autofs4 needs to force fail return revalidate
• Next by thread: Re: [PATCH] autofs4 needs to force fail return revalidate
• Index(es):
  • Date
  • Thread

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]