* [PATCH] sunrpc: drop "select NETVM"
@ 2013-03-09 16:02 Paul Bolle
2013-03-09 17:51 ` J. Bruce Fields
0 siblings, 1 reply; 3+ messages in thread
From: Paul Bolle @ 2013-03-09 16:02 UTC (permalink / raw)
To: J. Bruce Fields, Trond Myklebust, David S. Miller
Cc: linux-nfs, netdev, linux-kernel
The Kconfig entry for SUNRPC_SWAP selects NETVM. That select statement
was added in commit a564b8f0398636ba30b07c0eaebdef7ff7837249 ("nfs:
enable swap on NFS"). But there's no Kconfig symbol NETVM. It apparently
was only in used in development versions of the swap over nfs
functionality but never entered mainline. Anyhow, it is a nop and can
safely be dropped.
Signed-off-by: Paul Bolle <pebolle@tiscali.nl>
---
net/sunrpc/Kconfig | 1 -
1 file changed, 1 deletion(-)
diff --git a/net/sunrpc/Kconfig b/net/sunrpc/Kconfig
index 516fe2c..262caf0 100644
--- a/net/sunrpc/Kconfig
+++ b/net/sunrpc/Kconfig
@@ -24,7 +24,6 @@ config SUNRPC_XPRT_RDMA
config SUNRPC_SWAP
bool
depends on SUNRPC
- select NETVM
config RPCSEC_GSS_KRB5
tristate "Secure RPC: Kerberos V mechanism"
--
1.7.11.7
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] sunrpc: drop "select NETVM"
2013-03-09 16:02 [PATCH] sunrpc: drop "select NETVM" Paul Bolle
@ 2013-03-09 17:51 ` J. Bruce Fields
2013-03-13 10:08 ` Mel Gorman
0 siblings, 1 reply; 3+ messages in thread
From: J. Bruce Fields @ 2013-03-09 17:51 UTC (permalink / raw)
To: Paul Bolle
Cc: Trond Myklebust, David S. Miller, linux-nfs, netdev, linux-kernel,
Mel Gorman
ACK.
On Sat, Mar 09, 2013 at 05:02:31PM +0100, Paul Bolle wrote:
> The Kconfig entry for SUNRPC_SWAP selects NETVM. That select statement
> was added in commit a564b8f0398636ba30b07c0eaebdef7ff7837249 ("nfs:
> enable swap on NFS").
Cc'ing Mel Gorman just in case there was something else going on there.
--b.
> But there's no Kconfig symbol NETVM. It apparently
> was only in used in development versions of the swap over nfs
> functionality but never entered mainline. Anyhow, it is a nop and can
> safely be dropped.
>
> Signed-off-by: Paul Bolle <pebolle@tiscali.nl>
> ---
> net/sunrpc/Kconfig | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/net/sunrpc/Kconfig b/net/sunrpc/Kconfig
> index 516fe2c..262caf0 100644
> --- a/net/sunrpc/Kconfig
> +++ b/net/sunrpc/Kconfig
> @@ -24,7 +24,6 @@ config SUNRPC_XPRT_RDMA
> config SUNRPC_SWAP
> bool
> depends on SUNRPC
> - select NETVM
>
> config RPCSEC_GSS_KRB5
> tristate "Secure RPC: Kerberos V mechanism"
> --
> 1.7.11.7
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] sunrpc: drop "select NETVM"
2013-03-09 17:51 ` J. Bruce Fields
@ 2013-03-13 10:08 ` Mel Gorman
0 siblings, 0 replies; 3+ messages in thread
From: Mel Gorman @ 2013-03-13 10:08 UTC (permalink / raw)
To: J. Bruce Fields
Cc: Paul Bolle, Trond Myklebust, David S. Miller, linux-nfs, netdev,
linux-kernel
On Sat, Mar 09, 2013 at 12:51:18PM -0500, J. Bruce Fields wrote:
> ACK.
>
> On Sat, Mar 09, 2013 at 05:02:31PM +0100, Paul Bolle wrote:
> > The Kconfig entry for SUNRPC_SWAP selects NETVM. That select statement
> > was added in commit a564b8f0398636ba30b07c0eaebdef7ff7837249 ("nfs:
> > enable swap on NFS").
>
> Cc'ing Mel Gorman just in case there was something else going on there.
>
> --b.
>
> > But there's no Kconfig symbol NETVM. It apparently
> > was only in used in development versions of the swap over nfs
> > functionality but never entered mainline. Anyhow, it is a nop and can
> > safely be dropped.
> >
Correct, the NETVM symbol existed in an early version of swap-over-nfs
that I failed to clean up properly. Thanks.
--
Mel Gorman
SUSE Labs
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2013-03-13 10:08 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-03-09 16:02 [PATCH] sunrpc: drop "select NETVM" Paul Bolle
2013-03-09 17:51 ` J. Bruce Fields
2013-03-13 10:08 ` Mel Gorman
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).