net: remove osize variable in __alloc_skb()
authorEric Dumazet <edumazet@google.com>
Mon, 6 Feb 2023 17:31:01 +0000 (17:31 +0000)
committerJakub Kicinski <kuba@kernel.org>
Tue, 7 Feb 2023 18:59:52 +0000 (10:59 -0800)
This is a cleanup patch, to prepare following change.

Signed-off-by: Eric Dumazet <edumazet@google.com>
Acked-by: Soheil Hassas Yeganeh <soheil@google.com>
Acked-by: Paolo Abeni <pabeni@redhat.com>
Reviewed-by: Alexander Duyck <alexanderduyck@fb.com>
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
net/core/skbuff.c

index 4abfc3ba6898d89f4df97bf5f069b291dd5e420f..333f793f9cdba9946e0bd014e9a0f18bae20771d 100644 (file)
@@ -533,7 +533,6 @@ struct sk_buff *__alloc_skb(unsigned int size, gfp_t gfp_mask,
 {
        struct kmem_cache *cache;
        struct sk_buff *skb;
-       unsigned int osize;
        bool pfmemalloc;
        u8 *data;
 
@@ -559,16 +558,15 @@ struct sk_buff *__alloc_skb(unsigned int size, gfp_t gfp_mask,
         * Both skb->head and skb_shared_info are cache line aligned.
         */
        size = SKB_HEAD_ALIGN(size);
-       osize = kmalloc_size_roundup(size);
-       data = kmalloc_reserve(osize, gfp_mask, node, &pfmemalloc);
+       size = kmalloc_size_roundup(size);
+       data = kmalloc_reserve(size, gfp_mask, node, &pfmemalloc);
        if (unlikely(!data))
                goto nodata;
        /* kmalloc_size_roundup() might give us more room than requested.
         * Put skb_shared_info exactly at the end of allocated zone,
         * to allow max possible filling before reallocation.
         */
-       size = SKB_WITH_OVERHEAD(osize);
-       prefetchw(data + size);
+       prefetchw(data + SKB_WITH_OVERHEAD(size));
 
        /*
         * Only clear those fields we need to clear, not those that we will
@@ -576,7 +574,7 @@ struct sk_buff *__alloc_skb(unsigned int size, gfp_t gfp_mask,
         * the tail pointer in struct sk_buff!
         */
        memset(skb, 0, offsetof(struct sk_buff, tail));
-       __build_skb_around(skb, data, osize);
+       __build_skb_around(skb, data, size);
        skb->pfmemalloc = pfmemalloc;
 
        if (flags & SKB_ALLOC_FCLONE) {