From: Dan Carpenter <dan.carpenter@linaro.org>
Date: Mon, 4 Nov 2024 17:16:30 +0000 (+0300)
Subject: kunit: skb: use "gfp" variable instead of hardcoding GFP_KERNEL
X-Git-Tag: microblaze-v6.16~578^2~10
X-Git-Url: http://git.monstr.eu/?a=commitdiff_plain;h=fd0a5afb5455b4561bfc6dfb0c4b2d8226f9ccfe;p=linux-2.6-microblaze.git

kunit: skb: use "gfp" variable instead of hardcoding GFP_KERNEL

The intent here was clearly to use the gfp variable flags instead of
hardcoding GFP_KERNEL.  All the callers pass GFP_KERNEL as the gfp
flags so this doesn't affect runtime.

Fixes: b3231d353a51 ("kunit: add a convenience allocation wrapper for SKBs")
Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org>
Reviewed-by: David Gow <davidgow@google.com>
Reviewed-by: Kuan-Wei Chiu <visitorckw@gmail.com>
Signed-off-by: Shuah Khan <skhan@linuxfoundation.org>
---

diff --git a/include/kunit/skbuff.h b/include/kunit/skbuff.h
index 44d12370939a..345e1e8f0312 100644
--- a/include/kunit/skbuff.h
+++ b/include/kunit/skbuff.h
@@ -29,7 +29,7 @@ static void kunit_action_kfree_skb(void *p)
 static inline struct sk_buff *kunit_zalloc_skb(struct kunit *test, int len,
 					       gfp_t gfp)
 {
-	struct sk_buff *res = alloc_skb(len, GFP_KERNEL);
+	struct sk_buff *res = alloc_skb(len, gfp);
 
 	if (!res || skb_pad(res, len))
 		return NULL;