From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933764Ab0HLPSo (ORCPT ); Thu, 12 Aug 2010 11:18:44 -0400 Received: from mx2.netapp.com ([216.240.18.37]:5639 "EHLO mx2.netapp.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933766Ab0HLPSk (ORCPT ); Thu, 12 Aug 2010 11:18:40 -0400 X-IronPort-AV: E=Sophos;i="4.55,358,1278313200"; d="scan'208";a="427213023" Message-ID: <4C6410D4.6050100@netapp.com> Date: Thu, 12 Aug 2010 11:18:44 -0400 From: Bryan Schumaker User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.8) Gecko/20100806 Lightning/1.0b2 Lanikai/3.1.2 ThunderBrowse/3.3.1 MIME-Version: 1.0 To: Stephen Rothwell CC: Steve French , linux-cifs@vger.kernel.org, linux-next@vger.kernel.org, linux-kernel@vger.kernel.org, Wang Lei , David Howells Subject: Re: linux-next: build failure after merge of the cifs tree References: <20100812131741.f3a800d7.sfr@canb.auug.org.au> <20100812174621.b1dbc20f.sfr@canb.auug.org.au> In-Reply-To: <20100812174621.b1dbc20f.sfr@canb.auug.org.au> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi I updated my NFS DNS patch. The new version will set CONFIG_KEYS when CONFIG_NFS_USE_KERNEL_DNS is set. Thanks for letting me know about this! Bryan Schumaker Use the kernel DNS resolver to translate hostnames to IP addresses. Create a new config option to choose between the legacy DNS resolver and the new resolver. Signed-off-by: Bryan Schumaker Acked-by: Trond Myklebust --- Fix for KEYS dependency fs/nfs/Kconfig | 16 ++++++++++++++++ fs/nfs/dns_resolve.c | 24 ++++++++++++++++++++++++ fs/nfs/dns_resolve.h | 12 ++++++++++++ 3 files changed, 52 insertions(+), 0 deletions(-) diff --git a/fs/nfs/Kconfig b/fs/nfs/Kconfig index cc1bb33..c5bbdca 100644 --- a/fs/nfs/Kconfig +++ b/fs/nfs/Kconfig @@ -100,3 +100,19 @@ config NFS_FSCACHE help Say Y here if you want NFS data to be cached locally on disc through the general filesystem cache manager + +config NFS_USE_LEGACY_DNS + bool "Use the legacy NFS DNS resolver" + depends on NFS_V4 + help + The kernel now provides a method for translating a host name into an + IP address. Select Y here if you would rather use your own DNS + resolver script. + + If unsure, say N + +config NFS_USE_KERNEL_DNS + bool + depends on NFS_V4 && !NFS_USE_LEGACY_DNS + select DNS_RESOLVER + select KEYS + default y diff --git a/fs/nfs/dns_resolve.c b/fs/nfs/dns_resolve.c index 76fd235..dba50a5 100644 --- a/fs/nfs/dns_resolve.c +++ b/fs/nfs/dns_resolve.c @@ -6,6 +6,29 @@ * Resolves DNS hostnames into valid ip addresses */ +#ifdef CONFIG_NFS_USE_KERNEL_DNS + +#include +#include + +ssize_t nfs_dns_resolve_name(char *name, size_t namelen, + struct sockaddr *sa, size_t salen) +{ + ssize_t ret; + char *ip_addr = NULL; + int ip_len; + + ip_len = dns_query(NULL, name, namelen, NULL, &ip_addr, NULL); + if (ip_len > 0) + ret = rpc_pton(ip_addr, ip_len, sa, salen); + else + ret = -ESRCH; + kfree(ip_addr); + return ret; +} + +#else + #include #include #include @@ -346,3 +369,4 @@ void nfs_dns_resolver_destroy(void) nfs_cache_unregister(&nfs_dns_resolve); } +#endif diff --git a/fs/nfs/dns_resolve.h b/fs/nfs/dns_resolve.h index a3f0938..199bb55 100644 --- a/fs/nfs/dns_resolve.h +++ b/fs/nfs/dns_resolve.h @@ -6,8 +6,20 @@ #define NFS_DNS_HOSTNAME_MAXLEN (128) + +#ifdef CONFIG_NFS_USE_KERNEL_DNS +static inline int nfs_dns_resolver_init(void) +{ + return 0; +} + +static inline void nfs_dns_resolver_destroy(void) +{} +#else extern int nfs_dns_resolver_init(void); extern void nfs_dns_resolver_destroy(void); +#endif + extern ssize_t nfs_dns_resolve_name(char *name, size_t namelen, struct sockaddr *sa, size_t salen); On 08/12/2010 03:46 AM, Stephen Rothwell wrote: > Hi Steve, > > On Wed, 11 Aug 2010 23:04:36 -0500 Steve French wrote: >> >> Are you sure that this is a valid config file? CONFIG_KEYS should be >> set if CONFIG_DNS_RESOLVER is set? > > It is a ppc64_defconfig. It has: > > # CONFIG_KEYS is not set > CONFIG_DNS_RESOLVER=y > > CONFIG_NFS_USE_KERNEL_DNS just selects CONFIG_DNS_RESOLVER but there is > no dependency on CONFIG_KEYS (select does not care about the > dependencies of the selected config option). > >> key.h looks like it is included in all arch in this path >> >> net/dns_resolver/dns_key.c includes >> include/keys/dns_resolver-type.h which includes >> include/linux/key-type.h which includes >> include/linux/key.h >> >> the line 258 "key_instantiate_and_link" error could occur if >> "CONFIG_KEYS" is not set in .config > > Which t isn't. > >> net/dns_resolver/Kconfig config DNS_RESOLVER "depends on NET && KEYS" >> so ... CONFIG_KEYS should be set .... > > See above. You need CONFIG_NFS_USE_KERNEL_DNS to also depend on > CONFIG_KEYS ... >