diff options
author | Dan Carpenter <dan.carpenter@linaro.org> | 2024-11-04 20:16:30 +0300 |
---|---|---|
committer | Shuah Khan <skhan@linuxfoundation.org> | 2024-11-11 10:25:40 -0700 |
commit | fd0a5afb5455b4561bfc6dfb0c4b2d8226f9ccfe (patch) | |
tree | 6e61cf06031973f0b6df7f63e89ff5ac14ddad07 | |
parent | 2d5404caa8c7bb5c4e0435f94b28834ae5456623 (diff) | |
download | lwn-fd0a5afb5455b4561bfc6dfb0c4b2d8226f9ccfe.tar.gz lwn-fd0a5afb5455b4561bfc6dfb0c4b2d8226f9ccfe.zip |
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>
-rw-r--r-- | include/kunit/skbuff.h | 2 |
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; |