On Wed, Apr 27, 2011 at 9:29 AM, Jeff Layton <[email protected]> wrote:
> On Wed, 27 Apr 2011 15:22:27 +0100
> Mark Brown <[email protected]> wrote:
>
>> There's no point in supporting NFS export of filesystems if we don't have
>> nfsd in the first place.
>>
>> Signed-off-by: Mark Brown <[email protected]>
>> ---
>>  fs/cifs/Kconfig |    2 +-
>>  1 files changed, 1 insertions(+), 1 deletions(-)
>>
>> diff --git a/fs/cifs/Kconfig b/fs/cifs/Kconfig
>> index 75c47cd..61e5f95 100644
>> --- a/fs/cifs/Kconfig
>> +++ b/fs/cifs/Kconfig
>> @@ -175,6 +175,6 @@ config CIFS_SMB2
>>
>>  config CIFS_NFSD_EXPORT
>>         bool "Allow nfsd to export CIFS file system (EXPERIMENTAL)"
>> -       depends on CIFS && EXPERIMENTAL
>> +       depends on CIFS && NFSD && EXPERIMENTAL
>>         help
>>          Allows NFS server to export a CIFS mounted share (nfsd over cifs)
>
> This looks ok, though I'd still prefer to see all of the nfs exporting
> code removed. The cifs protocol has no consistent way to look up inodes
> by filehandle and so it simply can never work.
>
> Acked-by: Jeff Layton <[email protected]>

Patch seems reasonable



-- 
Thanks,

Steve
--
To unsubscribe from this list: send the line "unsubscribe linux-cifs" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to