On 6/18/21 9:47 PM, kernel test robot wrote:
> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/axboe/linux-block.git for-5.14/io_uring
> head:   7a778f9dc32deae4f748903f6f9169dc01cbcd28
> commit: a3dbdf54da80326fd12bc11ad75ecd699a82374f [59/64] io_uring: refactor io_get_sequence()
> compiler: hppa-linux-gcc (GCC) 9.3.0
> 
> If you fix the issue, kindly add following tag as appropriate
> Reported-by: kernel test robot <lkp@intel.com>
> 
> 
> cppcheck warnings: (new ones prefixed by >>)
>>> fs/io_uring.c:5999:2: warning: Redundant assignment of 'req' to itself. [selfAssignment]
>     io_for_each_link(req, req)
>     ^
> 
> cppcheck possible warnings: (new ones prefixed by >>, may not real problems)
> 
>    fs/io_uring.c:5305:19: warning: Uninitialized variable: req [uninitvar]
>      if (sqe_addr != req->user_data)
>                      ^
>    fs/io_uring.c:5309:10: warning: Uninitialized variable: req [uninitvar]
>      return req;
>             ^
>    fs/io_uring.c:9491:46: warning: Uninitialized variable: req [uninitvar]
>       seq_printf(m, "  op=%d, task_works=%dn", req->opcode,
>                                                 ^
>    fs/io_uring.c:9492:6: warning: Uninitialized variable: req [uninitvar]
>         req->task->task_works != NULL);
>         ^
> 
> vim +/req +5999 fs/io_uring.c
> 
>   5993	
>   5994	static u32 io_get_sequence(struct io_kiocb *req)
>   5995	{
>   5996		u32 seq = req->ctx->cached_sq_head;
>   5997	
>   5998		/* need original cached_sq_head, but it was increased for each req */
>> 5999		io_for_each_link(req, req)

Thanks for reporting, it was intentional. May delete it
in the future to not trigger bots.

-- 
Pavel Begunkov