Linux filesystem development
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: Joanne Koong <joannelkoong@gmail.com>, axboe@kernel.dk
Cc: oe-kbuild-all@lists.linux.dev, miklos@szeredi.hu,
	bschubert@ddn.com, linux-fsdevel@vger.kernel.org
Subject: Re: [PATCH v1 8/9] fuse: add zero-copy over io-uring
Date: Sat, 28 Mar 2026 14:02:20 +0100	[thread overview]
Message-ID: <202603281328.m7ticaDV-lkp@intel.com> (raw)
In-Reply-To: <20260324224532.3733468-9-joannelkoong@gmail.com>

Hi Joanne,

kernel test robot noticed the following build errors:

[auto build test ERROR on mszeredi-fuse/for-next]
[also build test ERROR on linus/master v7.0-rc5 next-20260327]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]

url:    https://github.com/intel-lab-lkp/linux/commits/Joanne-Koong/fuse-separate-next-request-fetching-from-sending-logic/20260328-142620
base:   https://git.kernel.org/pub/scm/linux/kernel/git/mszeredi/fuse.git for-next
patch link:    https://lore.kernel.org/r/20260324224532.3733468-9-joannelkoong%40gmail.com
patch subject: [PATCH v1 8/9] fuse: add zero-copy over io-uring
config: x86_64-rhel-9.4 (https://download.01.org/0day-ci/archive/20260328/202603281328.m7ticaDV-lkp@intel.com/config)
compiler: gcc-14 (Debian 14.2.0-19) 14.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260328/202603281328.m7ticaDV-lkp@intel.com/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202603281328.m7ticaDV-lkp@intel.com/

All errors (new ones prefixed by >>):

   fs/fuse/dev_uring.c: In function 'fuse_uring_req_end':
   fs/fuse/dev_uring.c:132:17: error: implicit declaration of function 'io_buffer_unregister'; did you mean 'io_buffer_unregister_bvec'? [-Wimplicit-function-declaration]
     132 |                 io_buffer_unregister(ent->cmd, ent->id, issue_flags);
         |                 ^~~~~~~~~~~~~~~~~~~~
         |                 io_buffer_unregister_bvec
   fs/fuse/dev_uring.c: In function 'fuse_uring_get_buf_ring_headers':
   fs/fuse/dev_uring.c:317:24: error: implicit declaration of function 'io_uring_registered_mem_region_get' [-Wimplicit-function-declaration]
     317 |         mem_reg_addr = io_uring_registered_mem_region_get(cmd, &nr_pages,
         |                        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   fs/fuse/dev_uring.c:317:22: error: assignment to 'void *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
     317 |         mem_reg_addr = io_uring_registered_mem_region_get(cmd, &nr_pages,
         |                      ^
   fs/fuse/dev_uring.c: In function 'fuse_uring_buf_ring_setup':
   fs/fuse/dev_uring.c:356:15: error: implicit declaration of function 'io_uring_buf_ring_pin'; did you mean 'fuse_uring_buf_ring_setup'? [-Wimplicit-function-declaration]
     356 |         err = io_uring_buf_ring_pin(cmd, FUSE_URING_BUFRING_GROUP, issue_flags,
         |               ^~~~~~~~~~~~~~~~~~~~~
         |               fuse_uring_buf_ring_setup
   fs/fuse/dev_uring.c:361:14: error: implicit declaration of function 'io_uring_is_kmbuf_ring' [-Wimplicit-function-declaration]
     361 |         if (!io_uring_is_kmbuf_ring(cmd, FUSE_URING_BUFRING_GROUP, issue_flags))
         |              ^~~~~~~~~~~~~~~~~~~~~~
   fs/fuse/dev_uring.c:372:9: error: implicit declaration of function 'io_uring_buf_ring_unpin'; did you mean 'fuse_uring_buf_ring_setup'? [-Wimplicit-function-declaration]
     372 |         io_uring_buf_ring_unpin(cmd, FUSE_URING_BUFRING_GROUP, issue_flags);
         |         ^~~~~~~~~~~~~~~~~~~~~~~
         |         fuse_uring_buf_ring_setup
   fs/fuse/dev_uring.c: In function 'fuse_uring_set_up_zero_copy':
   fs/fuse/dev_uring.c:859:25: error: 'IO_BUF_DEST' undeclared (first use in this function)
     859 |                 ddir |= IO_BUF_DEST;
         |                         ^~~~~~~~~~~
   fs/fuse/dev_uring.c:859:25: note: each undeclared identifier is reported only once for each function it appears in
   fs/fuse/dev_uring.c:861:25: error: 'IO_BUF_SOURCE' undeclared (first use in this function); did you mean 'ITER_SOURCE'?
     861 |                 ddir |= IO_BUF_SOURCE;
         |                         ^~~~~~~~~~~~~
         |                         ITER_SOURCE
   fs/fuse/dev_uring.c:881:49: error: passing argument 2 of 'io_buffer_register_bvec' from incompatible pointer type [-Wincompatible-pointer-types]
     881 |         err = io_buffer_register_bvec(ent->cmd, bvs, ap->num_folios,
         |                                                 ^~~
         |                                                 |
         |                                                 struct bio_vec *
   In file included from fs/fuse/dev_uring.c:14:
   include/linux/io_uring/cmd.h:185:71: note: expected 'struct request *' but argument is of type 'struct bio_vec *'
     185 | int io_buffer_register_bvec(struct io_uring_cmd *cmd, struct request *rq,
         |                                                       ~~~~~~~~~~~~~~~~^~
   fs/fuse/dev_uring.c:881:56: error: passing argument 3 of 'io_buffer_register_bvec' makes pointer from integer without a cast [-Wint-conversion]
     881 |         err = io_buffer_register_bvec(ent->cmd, bvs, ap->num_folios,
         |                                                      ~~^~~~~~~~~~~~
         |                                                        |
         |                                                        unsigned int
   include/linux/io_uring/cmd.h:186:36: note: expected 'void (*)(void *)' but argument is of type 'unsigned int'
     186 |                             void (*release)(void *), unsigned int index,
         |                             ~~~~~~~^~~~~~~~~~~~~~~~
   fs/fuse/dev_uring.c:882:39: error: passing argument 4 of 'io_buffer_register_bvec' makes integer from pointer without a cast [-Wint-conversion]
     882 |                                       fuse_zero_copy_release, zc_bvs,
         |                                       ^~~~~~~~~~~~~~~~~~~~~~
         |                                       |
         |                                       void (*)(void *)
   include/linux/io_uring/cmd.h:186:67: note: expected 'unsigned int' but argument is of type 'void (*)(void *)'
     186 |                             void (*release)(void *), unsigned int index,
         |                                                      ~~~~~~~~~~~~~^~~~~
   fs/fuse/dev_uring.c:882:63: error: passing argument 5 of 'io_buffer_register_bvec' makes integer from pointer without a cast [-Wint-conversion]
     882 |                                       fuse_zero_copy_release, zc_bvs,
         |                                                               ^~~~~~
         |                                                               |
         |                                                               struct fuse_zero_copy_bvs *
   include/linux/io_uring/cmd.h:187:42: note: expected 'unsigned int' but argument is of type 'struct fuse_zero_copy_bvs *'
     187 |                             unsigned int issue_flags);
         |                             ~~~~~~~~~~~~~^~~~~~~~~~~
>> fs/fuse/dev_uring.c:881:15: error: too many arguments to function 'io_buffer_register_bvec'
     881 |         err = io_buffer_register_bvec(ent->cmd, bvs, ap->num_folios,
         |               ^~~~~~~~~~~~~~~~~~~~~~~
   include/linux/io_uring/cmd.h:185:5: note: declared here
     185 | int io_buffer_register_bvec(struct io_uring_cmd *cmd, struct request *rq,
         |     ^~~~~~~~~~~~~~~~~~~~~~~
   fs/fuse/dev_uring.c: In function 'fuse_uring_select_buffer':
   fs/fuse/dev_uring.c:1013:15: error: implicit declaration of function 'io_ring_buffer_select'; did you mean 'io_uring_cmd_buffer_select'? [-Wimplicit-function-declaration]
    1013 |         sel = io_ring_buffer_select(cmd_to_io_kiocb(ent->cmd), &len,
         |               ^~~~~~~~~~~~~~~~~~~~~
         |               io_uring_cmd_buffer_select
   fs/fuse/dev_uring.c:1013:15: error: incompatible types when assigning to type 'struct io_br_sel' from type 'int'
   fs/fuse/dev_uring.c:1017:18: error: 'struct io_br_sel' has no member named 'kaddr'; did you mean 'addr'?
    1017 |         if (!sel.kaddr)
         |                  ^~~~~
         |                  addr
   fs/fuse/dev_uring.c:1020:42: error: 'struct io_br_sel' has no member named 'kaddr'; did you mean 'addr'?
    1020 |         ent->payload_kvec.iov_base = sel.kaddr;
         |                                          ^~~~~
         |                                          addr
   fs/fuse/dev_uring.c:1022:27: error: 'struct io_br_sel' has no member named 'buf_id'; did you mean 'buf_list'?
    1022 |         ent->buf_id = sel.buf_id;
         |                           ^~~~~~
         |                           buf_list
   In file included from arch/x86/include/asm/bug.h:193,
                    from arch/x86/include/asm/alternative.h:9,
                    from arch/x86/include/asm/segment.h:6,
                    from arch/x86/include/asm/ptrace.h:5,
                    from arch/x86/include/asm/math_emu.h:5,
                    from arch/x86/include/asm/processor.h:13,
                    from arch/x86/include/asm/timex.h:5,
                    from include/linux/timex.h:67,
                    from include/linux/time32.h:13,
                    from include/linux/time.h:60,
                    from include/linux/stat.h:19,
                    from include/linux/fs_dirent.h:5,
                    from include/linux/fs/super_types.h:5,
                    from include/linux/fs/super.h:5,
                    from include/linux/fs.h:5,
                    from fs/fuse/fuse_i.h:17,
                    from fs/fuse/dev_uring.c:7:
   fs/fuse/dev_uring.c: In function 'fuse_uring_recycle_buffer':
   fs/fuse/dev_uring.c:1038:22: error: implicit declaration of function 'io_uring_kmbuf_recycle' [-Wimplicit-function-declaration]
    1038 |         WARN_ON_ONCE(io_uring_kmbuf_recycle(ent->cmd, FUSE_URING_BUFRING_GROUP,
         |                      ^~~~~~~~~~~~~~~~~~~~~~
   include/asm-generic/bug.h:120:32: note: in definition of macro 'WARN_ON_ONCE'
     120 |         int __ret_warn_on = !!(condition);                              \
         |                                ^~~~~~~~~


vim +/io_buffer_register_bvec +881 fs/fuse/dev_uring.c

   847	
   848	static int fuse_uring_set_up_zero_copy(struct fuse_ring_ent *ent,
   849					       struct fuse_req *req,
   850					       unsigned issue_flags)
   851	{
   852		struct fuse_args_pages *ap;
   853		int err, i, ddir = 0;
   854		struct fuse_zero_copy_bvs *zc_bvs;
   855		struct bio_vec *bvs;
   856	
   857		/* out_pages indicates a read, in_pages indicates a write */
   858		if (req->args->out_pages)
   859			ddir |= IO_BUF_DEST;
   860		if (req->args->in_pages)
   861			ddir |= IO_BUF_SOURCE;
   862	
   863		WARN_ON_ONCE(!ddir);
   864	
   865		ap = container_of(req->args, typeof(*ap), args);
   866	
   867		zc_bvs = kmalloc(struct_size(zc_bvs, bvs, ap->num_folios),
   868				 GFP_KERNEL_ACCOUNT);
   869		if (!zc_bvs)
   870			return -ENOMEM;
   871	
   872		zc_bvs->nr_bvs = ap->num_folios;
   873		bvs = zc_bvs->bvs;
   874		for (i = 0; i < ap->num_folios; i++) {
   875			bvs[i].bv_page = folio_page(ap->folios[i], 0);
   876			bvs[i].bv_offset = ap->descs[i].offset;
   877			bvs[i].bv_len = ap->descs[i].length;
   878			folio_get(page_folio(bvs[i].bv_page));
   879		}
   880	
 > 881		err = io_buffer_register_bvec(ent->cmd, bvs, ap->num_folios,
 > 882					      fuse_zero_copy_release, zc_bvs,
   883					      ddir, ent->id,
   884					      issue_flags);
   885		if (err) {
   886			fuse_zero_copy_release(zc_bvs);
   887			return err;
   888		}
   889	
   890		ent->zero_copied = true;
   891	
   892		return 0;
   893	}
   894	

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki

  reply	other threads:[~2026-03-28 13:03 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-03-24 22:45 [PATCH v1 0/9] fuse: add io-uring buffer rings and zero-copy Joanne Koong
2026-03-24 22:45 ` [PATCH v1 1/9] fuse: separate next request fetching from sending logic Joanne Koong
2026-03-24 22:45 ` [PATCH v1 2/9] fuse: refactor io-uring header copying to ring Joanne Koong
2026-03-24 22:45 ` [PATCH v1 3/9] fuse: refactor io-uring header copying from ring Joanne Koong
2026-03-24 22:45 ` [PATCH v1 4/9] fuse: use enum types for header copying Joanne Koong
2026-03-24 22:45 ` [PATCH v1 5/9] fuse: refactor setting up copy state for payload copying Joanne Koong
2026-03-24 22:45 ` [PATCH v1 6/9] fuse: support buffer copying for kernel addresses Joanne Koong
2026-03-24 22:45 ` [PATCH v1 7/9] fuse: add io-uring kernel-managed buffer ring Joanne Koong
2026-03-28 12:21   ` kernel test robot
2026-03-28 12:54   ` kernel test robot
2026-03-28 13:27   ` kernel test robot
2026-03-30 21:28   ` Bernd Schubert
2026-03-30 23:19     ` Joanne Koong
2026-03-31 21:30       ` Bernd Schubert
2026-03-24 22:45 ` [PATCH v1 8/9] fuse: add zero-copy over io-uring Joanne Koong
2026-03-28 13:02   ` kernel test robot [this message]
2026-03-28 13:53   ` kernel test robot
2026-03-30 15:43     ` Joanne Koong
2026-03-31  3:35       ` Philip Li
2026-03-24 22:45 ` [PATCH v1 9/9] docs: fuse: add io-uring bufring and zero-copy documentation Joanne Koong

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=202603281328.m7ticaDV-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=axboe@kernel.dk \
    --cc=bschubert@ddn.com \
    --cc=joannelkoong@gmail.com \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=miklos@szeredi.hu \
    --cc=oe-kbuild-all@lists.linux.dev \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox