Re: [PATCH 0/2] NFSD: fix races in service per-net resources allocation

2013-02-12 Thread Stanislav Kinsbursky
rond.mykleb...@netapp.com; linux-kernel@vger.kernel.org; de...@openvz.org Subject: Re: [PATCH 0/2] NFSD: fix races in service per-net resources allocation On Tue, Feb 12, 2013 at 01:52:32PM +0400, Stanislav Kinsbursky wrote: 12.02.2013 00:58, J. Bruce Fields пишет: void svc_close_net(struct s

RE: [PATCH 0/2] NFSD: fix races in service per-net resources allocation

2013-02-12 Thread Peter Staubach
g Subject: Re: [PATCH 0/2] NFSD: fix races in service per-net resources allocation On Tue, Feb 12, 2013 at 01:52:32PM +0400, Stanislav Kinsbursky wrote: > 12.02.2013 00:58, J. Bruce Fields пишет: > > > void svc_close_net(struct svc_serv *serv, struct net *net) > > { > >-

Re: [PATCH 0/2] NFSD: fix races in service per-net resources allocation

2013-02-12 Thread J. Bruce Fields
On Tue, Feb 12, 2013 at 01:52:32PM +0400, Stanislav Kinsbursky wrote: > 12.02.2013 00:58, J. Bruce Fields пишет: > > > void svc_close_net(struct svc_serv *serv, struct net *net) > > { > >-svc_close_list(serv, &serv->sv_tempsocks, net); > >-svc_close_list(serv, &serv->sv_permsocks, net);

Re: [PATCH 0/2] NFSD: fix races in service per-net resources allocation

2013-02-12 Thread Stanislav Kinsbursky
12.02.2013 00:58, J. Bruce Fields пишет: void svc_close_net(struct svc_serv *serv, struct net *net) { - svc_close_list(serv, &serv->sv_tempsocks, net); - svc_close_list(serv, &serv->sv_permsocks, net); - - svc_clear_pools(serv, net); - /* -* At this point the

Re: [PATCH 0/2] NFSD: fix races in service per-net resources allocation

2013-02-11 Thread Stanislav Kinsbursky
11.02.2013 20:37, J. Bruce Fields пишет: On Mon, Feb 11, 2013 at 10:18:18AM +0400, Stanislav Kinsbursky wrote: This one looks a bit complicated and confusing to me. Probably because I'm not that familiar with service transports processing logic. So, as I can see, we now try to run over all per-

Re: [PATCH 0/2] NFSD: fix races in service per-net resources allocation

2013-02-11 Thread J. Bruce Fields
On Mon, Feb 11, 2013 at 11:37:15AM -0500, J. Bruce Fields wrote: > On Mon, Feb 11, 2013 at 10:18:18AM +0400, Stanislav Kinsbursky wrote: > > This one looks a bit complicated and confusing to me. Probably because > > I'm not that familiar with service transports processing logic. So, > > as I can s

Re: [PATCH 0/2] NFSD: fix races in service per-net resources allocation

2013-02-11 Thread J. Bruce Fields
On Mon, Feb 11, 2013 at 10:18:18AM +0400, Stanislav Kinsbursky wrote: > This one looks a bit complicated and confusing to me. Probably because > I'm not that familiar with service transports processing logic. So, > as I can see, we now try to run over all per-net pool-assigned > transports, remove

Re: [PATCH 0/2] NFSD: fix races in service per-net resources allocation

2013-02-10 Thread Stanislav Kinsbursky
11.02.2013 04:25, J. Bruce Fields пишет: On Fri, Feb 01, 2013 at 02:28:21PM +0300, Stanislav Kinsbursky wrote: After "NFS" (SUNRPC + NFSd actually) containerization work some basic principles of SUNRPC service initialization and deinitialization has been changed: now one service can be shared be

Re: [PATCH 0/2] NFSD: fix races in service per-net resources allocation

2013-02-10 Thread J. Bruce Fields
On Fri, Feb 01, 2013 at 02:28:21PM +0300, Stanislav Kinsbursky wrote: > After "NFS" (SUNRPC + NFSd actually) containerization work some basic > principles of SUNRPC service initialization and deinitialization has been > changed: now one service can be shared between different network namespaces > a

[PATCH 0/2] NFSD: fix races in service per-net resources allocation

2013-02-01 Thread Stanislav Kinsbursky
After "NFS" (SUNRPC + NFSd actually) containerization work some basic principles of SUNRPC service initialization and deinitialization has been changed: now one service can be shared between different network namespaces and network "resources" can be attached or detached from the running service. T