[PATCH 22/23] SUNRPC: simplify auth_unix.

2018-12-02 Thread NeilBrown
1/ discard 'struct unx_cred'. We don't need any data that is not already in 'struct rpc_cred'. 2/ Don't keep these creds in a hash table. When a credential is needed, simply allocate it. When not needed, discard it. This can easily be faster than performing a lookup on a shared hash

Re: [PATCH 22/23] SUNRPC: simplify auth_unix.

2018-11-08 Thread NeilBrown
On Thu, Nov 08 2018, Chuck Lever wrote: > > Point taken, but having a single mempool for all RPC transports > and users is also going to be a shared resource that can > bottleneck. Agreed. mempools will only access the pre-allocated memory if a regular kmalloc(GFP_NOWAIT) fails. I asked an mm dev

Re: [PATCH 22/23] SUNRPC: simplify auth_unix.

2018-11-08 Thread Chuck Lever
> On Nov 7, 2018, at 8:41 PM, NeilBrown wrote: > > On Wed, Nov 07 2018, Chuck Lever wrote: > >> Hi Neil- >> >> >>> On Nov 6, 2018, at 11:12 PM, NeilBrown wrote: >>> >>> 1/ discard 'struct unx_cred'. We don't need any data that >>> is not already in 'struct rpc_cred'. >>> 2/ Don't keep t

Re: [PATCH 22/23] SUNRPC: simplify auth_unix.

2018-11-07 Thread NeilBrown
On Wed, Nov 07 2018, Chuck Lever wrote: > Hi Neil- > > >> On Nov 6, 2018, at 11:12 PM, NeilBrown wrote: >> >> 1/ discard 'struct unx_cred'. We don't need any data that >> is not already in 'struct rpc_cred'. >> 2/ Don't keep these creds in a hash table. When a credential >> is needed, simp

Re: [PATCH 22/23] SUNRPC: simplify auth_unix.

2018-11-07 Thread Chuck Lever
Hi Neil- > On Nov 6, 2018, at 11:12 PM, NeilBrown wrote: > > 1/ discard 'struct unx_cred'. We don't need any data that > is not already in 'struct rpc_cred'. > 2/ Don't keep these creds in a hash table. When a credential > is needed, simply allocate it. When not needed, discard it. > T

[PATCH 22/23] SUNRPC: simplify auth_unix.

2018-11-06 Thread NeilBrown
1/ discard 'struct unx_cred'. We don't need any data that is not already in 'struct rpc_cred'. 2/ Don't keep these creds in a hash table. When a credential is needed, simply allocate it. When not needed, discard it. This can easily be faster than performing a lookup on a shared hash