summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDan Carpenter <dan.carpenter@linaro.org>2024-11-04 20:16:30 +0300
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2024-12-05 14:02:13 +0100
commit673f503cfe82ba94c9dd749944e9ab511d711ded (patch)
treea4ef1968ae885eac9f2fca1e6c99dceb7547bb76
parente691826a3d076abb8b5ff705bb6269b2e93e4f7c (diff)
kunit: skb: use "gfp" variable instead of hardcoding GFP_KERNEL
[ Upstream commit fd0a5afb5455b4561bfc6dfb0c4b2d8226f9ccfe ] 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> Signed-off-by: Sasha Levin <sashal@kernel.org>
-rw-r--r--include/kunit/skbuff.h2
1 files changed, 1 insertions, 1 deletions
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;