[NET]: Cleanup the allocation/freeing of the sock object
authorPavel Emelyanov <xemul@openvz.org>
Thu, 1 Nov 2007 07:33:50 +0000 (00:33 -0700)
committerDavid S. Miller <davem@davemloft.net>
Thu, 1 Nov 2007 07:33:50 +0000 (00:33 -0700)
The sock object is allocated either from the generic cache with
the kmalloc, or from the proc->slab cache.

Move this logic into an isolated set of helpers and make the
sk_alloc/sk_free look a bit nicer.

Signed-off-by: Pavel Emelyanov <xemul@openvz.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/core/sock.c

index 9c2dbfaca60d6f91c90d1315e7204307b76c083b..6ee2ed104a83329bf06d3137af078244f73cdcc4 100644 (file)
@@ -870,6 +870,31 @@ static void sock_copy(struct sock *nsk, const struct sock *osk)
 #endif
 }
 
+static struct sock *sk_prot_alloc(struct proto *prot, gfp_t priority)
+{
+       struct sock *sk;
+       struct kmem_cache *slab;
+
+       slab = prot->slab;
+       if (slab != NULL)
+               sk = kmem_cache_alloc(slab, priority);
+       else
+               sk = kmalloc(prot->obj_size, priority);
+
+       return sk;
+}
+
+static void sk_prot_free(struct proto *prot, struct sock *sk)
+{
+       struct kmem_cache *slab;
+
+       slab = prot->slab;
+       if (slab != NULL)
+               kmem_cache_free(slab, sk);
+       else
+               kfree(sk);
+}
+
 /**
  *     sk_alloc - All socket objects are allocated here
  *     @net: the applicable net namespace
@@ -881,14 +906,9 @@ static void sock_copy(struct sock *nsk, const struct sock *osk)
 struct sock *sk_alloc(struct net *net, int family, gfp_t priority,
                      struct proto *prot, int zero_it)
 {
-       struct sock *sk = NULL;
-       struct kmem_cache *slab = prot->slab;
-
-       if (slab != NULL)
-               sk = kmem_cache_alloc(slab, priority);
-       else
-               sk = kmalloc(prot->obj_size, priority);
+       struct sock *sk;
 
+       sk = sk_prot_alloc(prot, priority);
        if (sk) {
                if (zero_it) {
                        memset(sk, 0, prot->obj_size);
@@ -911,10 +931,7 @@ struct sock *sk_alloc(struct net *net, int family, gfp_t priority,
        return sk;
 
 out_free:
-       if (slab != NULL)
-               kmem_cache_free(slab, sk);
-       else
-               kfree(sk);
+       sk_prot_free(prot, sk);
        return NULL;
 }
 
@@ -940,10 +957,7 @@ void sk_free(struct sock *sk)
 
        security_sk_free(sk);
        put_net(sk->sk_net);
-       if (sk->sk_prot_creator->slab != NULL)
-               kmem_cache_free(sk->sk_prot_creator->slab, sk);
-       else
-               kfree(sk);
+       sk_prot_free(sk->sk_prot_creator, sk);
        module_put(owner);
 }