Re: [Fwd: [PATCH 3/5] NFS: Abstract out namespace initialisation [try #2]]

Christoph Hellwig <[email protected]> wrote:

> > The attached patch abstracts out the namespace initialisation so that 
> > temporary namespaces can be set up elsewhere.
> 
> Definitily shouldb't be inline.  And until you have a second caller
> it's utterly pointless.  So NACK for now.

I presume from that you didn't notice that the second caller was in patch 5/5?

David
-
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/

Follow-Ups:
- Re: [Fwd: [PATCH 3/5] NFS: Abstract out namespace initialisation [try #2]]
  - From: David Howells <[email protected]>
- Re: [Fwd: [PATCH 3/5] NFS: Abstract out namespace initialisation [try #2]]
  - From: Sam Vilain <[email protected]>

References:
- Re: [Fwd: [PATCH 3/5] NFS: Abstract out namespace initialisation [try #2]]
  - From: Christoph Hellwig <[email protected]>
- [Fwd: [PATCH 3/5] NFS: Abstract out namespace initialisation [try #2]]
  - From: Sam Vilain <[email protected]>

Prev by Date: Re: [PATCH] pcmcia: add another ide-cs CF card id
Next by Date: Re: [PATCH] pcmcia: add another ide-cs CF card id
Previous by thread: Re: [Fwd: [PATCH 3/5] NFS: Abstract out namespace initialisation [try #2]]
Next by thread: Re: [Fwd: [PATCH 3/5] NFS: Abstract out namespace initialisation [try #2]]
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]