tree: https://git.kernel.org/pub/scm/linux/kernel/git/axboe/linux-block.git for-5.20/io_uring-zerocopy-send
head: fbe6f6bc3210e853aab74f20da776c15c5b052fe
commit: fbe6f6bc3210e853aab74f20da776c15c5b052fe [23/23] io_uring/zc: notification completion optimisation
config: x86_64-randconfig-a013 (https://download.01.org/0day-ci/archive/20220727/[email protected]/config)
compiler: gcc-11 (Debian 11.3.0-3) 11.3.0
reproduce (this is a W=1 build):
# https://git.kernel.org/pub/scm/linux/kernel/git/axboe/linux-block.git/commit/?id=fbe6f6bc3210e853aab74f20da776c15c5b052fe
git remote add axboe-block https://git.kernel.org/pub/scm/linux/kernel/git/axboe/linux-block.git
git fetch --no-tags axboe-block for-5.20/io_uring-zerocopy-send
git checkout fbe6f6bc3210e853aab74f20da776c15c5b052fe
# save the config file
mkdir build_dir && cp config build_dir/.config
make W=1 O=build_dir ARCH=x86_64 SHELL=/bin/bash
If you fix the issue, kindly add following tag where applicable
Reported-by: kernel test robot <[email protected]>
All error/warnings (new ones prefixed by >>):
In file included from include/linux/export.h:33,
from include/linux/linkage.h:7,
from include/linux/kernel.h:17,
from io_uring/notif.c:1:
io_uring/notif.c: In function 'io_alloc_notif':
>> io_uring/notif.c:52:23: error: implicit declaration of function 'io_alloc_req_refill'; did you mean 'io_rsrc_refs_refill'? [-Werror=implicit-function-declaration]
52 | if (unlikely(!io_alloc_req_refill(ctx)))
| ^~~~~~~~~~~~~~~~~~~
include/linux/compiler.h:78:45: note: in definition of macro 'unlikely'
78 | # define unlikely(x) __builtin_expect(!!(x), 0)
| ^
>> io_uring/notif.c:54:17: error: implicit declaration of function 'io_alloc_req'; did you mean 'xa_alloc_irq'? [-Werror=implicit-function-declaration]
54 | notif = io_alloc_req(ctx);
| ^~~~~~~~~~~~
| xa_alloc_irq
>> io_uring/notif.c:54:15: warning: assignment to 'struct io_kiocb *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
54 | notif = io_alloc_req(ctx);
| ^
cc1: some warnings being treated as errors
vim +52 io_uring/notif.c
> 1 #include <linux/kernel.h>
2 #include <linux/errno.h>
3 #include <linux/file.h>
4 #include <linux/slab.h>
5 #include <linux/net.h>
6 #include <linux/io_uring.h>
7
8 #include "io_uring.h"
9 #include "notif.h"
10 #include "rsrc.h"
11
12 static void __io_notif_complete_tw(struct io_kiocb *notif, bool *locked)
13 {
14 struct io_notif_data *nd = io_notif_to_data(notif);
15 struct io_ring_ctx *ctx = notif->ctx;
16
17 if (nd->account_pages && ctx->user) {
18 __io_unaccount_mem(ctx->user, nd->account_pages);
19 nd->account_pages = 0;
20 }
21 io_req_task_complete(notif, locked);
22 }
23
24 static inline void io_notif_complete(struct io_kiocb *notif)
25 __must_hold(¬if->ctx->uring_lock)
26 {
27 bool locked = true;
28
29 __io_notif_complete_tw(notif, &locked);
30 }
31
32 static void io_uring_tx_zerocopy_callback(struct sk_buff *skb,
33 struct ubuf_info *uarg,
34 bool success)
35 {
36 struct io_notif_data *nd = container_of(uarg, struct io_notif_data, uarg);
37 struct io_kiocb *notif = cmd_to_io_kiocb(nd);
38
39 if (refcount_dec_and_test(&uarg->refcnt)) {
40 notif->io_task_work.func = __io_notif_complete_tw;
41 io_req_task_work_add(notif);
42 }
43 }
44
45 struct io_kiocb *io_alloc_notif(struct io_ring_ctx *ctx,
46 struct io_notif_slot *slot)
47 __must_hold(&ctx->uring_lock)
48 {
49 struct io_kiocb *notif;
50 struct io_notif_data *nd;
51
> 52 if (unlikely(!io_alloc_req_refill(ctx)))
53 return NULL;
> 54 notif = io_alloc_req(ctx);
55 notif->opcode = IORING_OP_NOP;
56 notif->flags = 0;
57 notif->file = NULL;
58 notif->task = current;
59 io_get_task_refs(1);
60 notif->rsrc_node = NULL;
61 io_req_set_rsrc_node(notif, ctx, 0);
62 notif->cqe.user_data = slot->tag;
63 notif->cqe.flags = slot->seq++;
64 notif->cqe.res = 0;
65
66 nd = io_notif_to_data(notif);
67 nd->account_pages = 0;
68 nd->uarg.flags = SKBFL_ZEROCOPY_FRAG | SKBFL_DONT_ORPHAN;
69 nd->uarg.callback = io_uring_tx_zerocopy_callback;
70 /* master ref owned by io_notif_slot, will be dropped on flush */
71 refcount_set(&nd->uarg.refcnt, 1);
72 return notif;
73 }
74
--
0-DAY CI Kernel Test Service
https://01.org/lkp
On 7/27/22 05:31, kernel test robot wrote:
> tree: https://git.kernel.org/pub/scm/linux/kernel/git/axboe/linux-block.git for-5.20/io_uring-zerocopy-send
> head: fbe6f6bc3210e853aab74f20da776c15c5b052fe
> commit: fbe6f6bc3210e853aab74f20da776c15c5b052fe [23/23] io_uring/zc: notification completion optimisation
> config: x86_64-randconfig-a013 (https://download.01.org/0day-ci/archive/20220727/[email protected]/config)
> compiler: gcc-11 (Debian 11.3.0-3) 11.3.0
> reproduce (this is a W=1 build):
> # https://git.kernel.org/pub/scm/linux/kernel/git/axboe/linux-block.git/commit/?id=fbe6f6bc3210e853aab74f20da776c15c5b052fe
> git remote add axboe-block https://git.kernel.org/pub/scm/linux/kernel/git/axboe/linux-block.git
> git fetch --no-tags axboe-block for-5.20/io_uring-zerocopy-send
> git checkout fbe6f6bc3210e853aab74f20da776c15c5b052fe
> # save the config file
> mkdir build_dir && cp config build_dir/.config
> make W=1 O=build_dir ARCH=x86_64 SHELL=/bin/bash
>
> If you fix the issue, kindly add following tag where applicable
> Reported-by: kernel test robot <[email protected]>
Looks like I forgot to send a patch it depends on, thanks!
> All error/warnings (new ones prefixed by >>):
>
> In file included from include/linux/export.h:33,
> from include/linux/linkage.h:7,
> from include/linux/kernel.h:17,
> from io_uring/notif.c:1:
> io_uring/notif.c: In function 'io_alloc_notif':
>>> io_uring/notif.c:52:23: error: implicit declaration of function 'io_alloc_req_refill'; did you mean 'io_rsrc_refs_refill'? [-Werror=implicit-function-declaration]
> 52 | if (unlikely(!io_alloc_req_refill(ctx)))
> | ^~~~~~~~~~~~~~~~~~~
> include/linux/compiler.h:78:45: note: in definition of macro 'unlikely'
> 78 | # define unlikely(x) __builtin_expect(!!(x), 0)
> | ^
>>> io_uring/notif.c:54:17: error: implicit declaration of function 'io_alloc_req'; did you mean 'xa_alloc_irq'? [-Werror=implicit-function-declaration]
> 54 | notif = io_alloc_req(ctx);
> | ^~~~~~~~~~~~
> | xa_alloc_irq
>>> io_uring/notif.c:54:15: warning: assignment to 'struct io_kiocb *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
> 54 | notif = io_alloc_req(ctx);
> | ^
> cc1: some warnings being treated as errors
>
>
> vim +52 io_uring/notif.c
>
> > 1 #include <linux/kernel.h>
> 2 #include <linux/errno.h>
> 3 #include <linux/file.h>
> 4 #include <linux/slab.h>
> 5 #include <linux/net.h>
> 6 #include <linux/io_uring.h>
> 7
> 8 #include "io_uring.h"
> 9 #include "notif.h"
> 10 #include "rsrc.h"
> 11
> 12 static void __io_notif_complete_tw(struct io_kiocb *notif, bool *locked)
> 13 {
> 14 struct io_notif_data *nd = io_notif_to_data(notif);
> 15 struct io_ring_ctx *ctx = notif->ctx;
> 16
> 17 if (nd->account_pages && ctx->user) {
> 18 __io_unaccount_mem(ctx->user, nd->account_pages);
> 19 nd->account_pages = 0;
> 20 }
> 21 io_req_task_complete(notif, locked);
> 22 }
> 23
> 24 static inline void io_notif_complete(struct io_kiocb *notif)
> 25 __must_hold(¬if->ctx->uring_lock)
> 26 {
> 27 bool locked = true;
> 28
> 29 __io_notif_complete_tw(notif, &locked);
> 30 }
> 31
> 32 static void io_uring_tx_zerocopy_callback(struct sk_buff *skb,
> 33 struct ubuf_info *uarg,
> 34 bool success)
> 35 {
> 36 struct io_notif_data *nd = container_of(uarg, struct io_notif_data, uarg);
> 37 struct io_kiocb *notif = cmd_to_io_kiocb(nd);
> 38
> 39 if (refcount_dec_and_test(&uarg->refcnt)) {
> 40 notif->io_task_work.func = __io_notif_complete_tw;
> 41 io_req_task_work_add(notif);
> 42 }
> 43 }
> 44
> 45 struct io_kiocb *io_alloc_notif(struct io_ring_ctx *ctx,
> 46 struct io_notif_slot *slot)
> 47 __must_hold(&ctx->uring_lock)
> 48 {
> 49 struct io_kiocb *notif;
> 50 struct io_notif_data *nd;
> 51
> > 52 if (unlikely(!io_alloc_req_refill(ctx)))
> 53 return NULL;
> > 54 notif = io_alloc_req(ctx);
> 55 notif->opcode = IORING_OP_NOP;
> 56 notif->flags = 0;
> 57 notif->file = NULL;
> 58 notif->task = current;
> 59 io_get_task_refs(1);
> 60 notif->rsrc_node = NULL;
> 61 io_req_set_rsrc_node(notif, ctx, 0);
> 62 notif->cqe.user_data = slot->tag;
> 63 notif->cqe.flags = slot->seq++;
> 64 notif->cqe.res = 0;
> 65
> 66 nd = io_notif_to_data(notif);
> 67 nd->account_pages = 0;
> 68 nd->uarg.flags = SKBFL_ZEROCOPY_FRAG | SKBFL_DONT_ORPHAN;
> 69 nd->uarg.callback = io_uring_tx_zerocopy_callback;
> 70 /* master ref owned by io_notif_slot, will be dropped on flush */
> 71 refcount_set(&nd->uarg.refcnt, 1);
> 72 return notif;
> 73 }
> 74
>
--
Pavel Begunkov