Tea Inside Mailing List <[email protected]>
 help / color / mirror / Atom feed
From: "Chen, Rong A" <[email protected]>
To: Ammar Faizi <[email protected]>, kernel test robot <[email protected]>
Cc: GNU/Weeb Mailing List <[email protected]>,
	Tea Inside Mailing List <[email protected]>
Subject: Re: [ammarfaizi2-block:testing/io_uring-sendto-recvfrom.v1] BUILD SUCCESS 68d110c39241b887ec388cd3316dbedb85b0cbcf
Date: Mon, 7 Feb 2022 14:13:17 +0800	[thread overview]
Message-ID: <[email protected]> (raw)
In-Reply-To: <[email protected]>

[-- Attachment #1: Type: text/plain, Size: 11142 bytes --]

Hi,

On 1/30/2022 5:11 PM, Ammar Faizi wrote:
> Hello,
> 
> On 1/30/22 3:26 AM, kernel test robot wrote:
>> tree/branch: https://github.com/ammarfaizi2/linux-block 
>> testing/io_uring-sendto-recvfrom.v1
>> branch HEAD: 68d110c39241b887ec388cd3316dbedb85b0cbcf  io_uring: Add 
>> `sendto(2)` and `recvfrom(2)` support
>>
>> possible Warning in current branch (please contact us if interested):
>>
>> fs/io_uring.c:5238 io_recvfrom() error: uninitialized symbol 'flags'.
>>
>> Warning ids grouped by kconfigs:
>>
>> gcc_recent_errors
>> `-- x86_64-randconfig-m001
>>      `-- fs-io_uring.c-io_recvfrom()-error:uninitialized-symbol-flags-.
> 
> 
> Interesting...
> Can I have the reproducer for this warning?

Sorry for the delay, I attached the report for your reference.

Best Regards,
Rong Chen

> 
>> elapsed time: 722m
>>
>> configs tested: 137
>> configs skipped: 3
>>
>> The following configs have been built successfully.
>> More configs may be tested in the coming days.
>>
>> gcc tested configs:
>> arm                                 defconfig
>> arm64                            allyesconfig
>> arm64                               defconfig
>> arm                              allyesconfig
>> arm                              allmodconfig
>> i386                 randconfig-c001-20220124
>> sh                             sh03_defconfig
>> arm                         axm55xx_defconfig
>> powerpc                      pcm030_defconfig
>> xtensa                           alldefconfig
>> powerpc                      ep88xc_defconfig
>> powerpc                     ep8248e_defconfig
>> m68k                        stmark2_defconfig
>> powerpc                 mpc834x_itx_defconfig
>> arm                        shmobile_defconfig
>> mips                     decstation_defconfig
>> arm                         lubbock_defconfig
>> arm                          iop32x_defconfig
>> arc                           tb10x_defconfig
>> s390                                defconfig
>> sh                           se7751_defconfig
>> openrisc                         alldefconfig
>> arc                 nsimosci_hs_smp_defconfig
>> powerpc                     sequoia_defconfig
>> m68k                           sun3_defconfig
>> powerpc                 mpc834x_mds_defconfig
>> sh                           se7705_defconfig
>> xtensa                    smp_lx200_defconfig
>> arm                        mini2440_defconfig
>> sh                           se7343_defconfig
>> sh                          polaris_defconfig
>> sh                           se7712_defconfig
>> xtensa                  audio_kc705_defconfig
>> mips                        vocore2_defconfig
>> arm                           h5000_defconfig
>> arc                        nsimosci_defconfig
>> sh                           se7722_defconfig
>> powerpc                     asp8347_defconfig
>> mips                           ip32_defconfig
>> arm                  randconfig-c002-20220127
>> arm                  randconfig-c002-20220124
>> arm                  randconfig-c002-20220129
>> ia64                             allmodconfig
>> ia64                                defconfig
>> ia64                             allyesconfig
>> m68k                             allmodconfig
>> m68k                                defconfig
>> m68k                             allyesconfig
>> nios2                               defconfig
>> arc                              allyesconfig
>> nds32                             allnoconfig
>> nds32                               defconfig
>> csky                                defconfig
>> alpha                               defconfig
>> alpha                            allyesconfig
>> nios2                            allyesconfig
>> arc                                 defconfig
>> sh                               allmodconfig
>> h8300                            allyesconfig
>> xtensa                           allyesconfig
>> parisc                              defconfig
>> s390                             allyesconfig
>> s390                             allmodconfig
>> parisc                           allyesconfig
>> i386                             allyesconfig
>> sparc                            allyesconfig
>> sparc                               defconfig
>> i386                                defconfig
>> i386                   debian-10.3-kselftests
>> i386                              debian-10.3
>> mips                             allyesconfig
>> mips                             allmodconfig
>> powerpc                          allyesconfig
>> powerpc                          allmodconfig
>> powerpc                           allnoconfig
>> x86_64               randconfig-a002-20220124
>> x86_64               randconfig-a003-20220124
>> x86_64               randconfig-a001-20220124
>> x86_64               randconfig-a004-20220124
>> x86_64               randconfig-a005-20220124
>> x86_64               randconfig-a006-20220124
>> i386                 randconfig-a002-20220124
>> i386                 randconfig-a005-20220124
>> i386                 randconfig-a003-20220124
>> i386                 randconfig-a004-20220124
>> i386                 randconfig-a001-20220124
>> i386                 randconfig-a006-20220124
>> x86_64                        randconfig-a011
>> x86_64                        randconfig-a013
>> x86_64                        randconfig-a015
>> riscv                    nommu_k210_defconfig
>> riscv                            allyesconfig
>> riscv                    nommu_virt_defconfig
>> riscv                             allnoconfig
>> riscv                               defconfig
>> riscv                          rv32_defconfig
>> riscv                            allmodconfig
>> x86_64                    rhel-8.3-kselftests
>> um                           x86_64_defconfig
>> um                             i386_defconfig
>> x86_64                              defconfig
>> x86_64                               rhel-8.3
>> x86_64                                  kexec
>> x86_64                           allyesconfig
>> x86_64                          rhel-8.3-func
>>
>> clang tested configs:
>> arm                  randconfig-c002-20220124
>> riscv                randconfig-c006-20220124
>> i386                 randconfig-c001-20220124
>> powerpc              randconfig-c003-20220124
>> mips                 randconfig-c004-20220124
>> x86_64               randconfig-c007-20220124
>> x86_64                        randconfig-c007
>> riscv                randconfig-c006-20220129
>> arm                  randconfig-c002-20220129
>> s390                 randconfig-c005-20220129
>> powerpc              randconfig-c003-20220129
>> mips                 randconfig-c004-20220129
>> i386                          randconfig-c001
>> arm                        mvebu_v5_defconfig
>> x86_64                        randconfig-a012
>> x86_64                        randconfig-a014
>> x86_64                        randconfig-a016
>> x86_64               randconfig-a011-20220124
>> x86_64               randconfig-a013-20220124
>> x86_64               randconfig-a015-20220124
>> x86_64               randconfig-a016-20220124
>> x86_64               randconfig-a014-20220124
>> x86_64               randconfig-a012-20220124
>> i386                 randconfig-a011-20220124
>> i386                 randconfig-a016-20220124
>> i386                 randconfig-a013-20220124
>> i386                 randconfig-a014-20220124
>> i386                 randconfig-a015-20220124
>> i386                 randconfig-a012-20220124
>> riscv                randconfig-r042-20220124
>> s390                 randconfig-r044-20220124
>> hexagon              randconfig-r045-20220124
>> hexagon              randconfig-r045-20220127
>> hexagon              randconfig-r041-20220124
>> hexagon              randconfig-r041-20220127
>>
>> ---
>> 0-DAY CI Kernel Test Service, Intel Corporation
>> https://lists.01.org/hyperkitty/list/[email protected]
> 
> 

[-- Attachment #2: [kbuild] [ammarfaizi2-block testing_io_uring-sendto-recvfrom.v1 3_5] fs_io_uring.c 5238 io_recvfrom() error  uninitialized symbol 'flags'..eml --]
[-- Type: message/rfc822, Size: 21004 bytes --]

From: kernel test robot <[email protected]>
To: [email protected]
Cc: [email protected], Dan Carpenter <[email protected]>
Subject: [kbuild] [ammarfaizi2-block:testing/io_uring-sendto-recvfrom.v1 3/5] fs/io_uring.c:5238 io_recvfrom() error: uninitialized symbol 'flags'.
Date: Sat, 29 Jan 2022 19:46:23 +0800
Message-ID: <[email protected]>

CC: [email protected]
CC: "GNU/Weeb Mailing List" <[email protected]>
CC: [email protected]
TO: Ammar Faizi <[email protected]>

tree:   https://github.com/ammarfaizi2/linux-block testing/io_uring-sendto-recvfrom.v1
head:   68d110c39241b887ec388cd3316dbedb85b0cbcf
commit: 8d597f720657a5156c4b3cbced8d1571ba151f62 [3/5] io_uring: Rename `io_{send,recv}` to `io_{sendto,recvfrom}`
:::::: branch date: 6 hours ago
:::::: commit date: 6 hours ago
config: x86_64-randconfig-m001 (https://download.01.org/0day-ci/archive/20220129/[email protected]/config)
compiler: gcc-9 (Debian 9.3.0-22) 9.3.0

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <[email protected]>
Reported-by: Dan Carpenter <[email protected]>

New smatch warnings:
fs/io_uring.c:5238 io_recvfrom() error: uninitialized symbol 'flags'.

Old smatch warnings:
fs/io_uring.c:8422 io_sqe_files_register() error: we previously assumed 'ctx->file_data' could be null (see line 8394)

vim +/flags +5238 fs/io_uring.c

0fa03c624d8fc9 Jens Axboe        2019-04-19  5189  
8d597f720657a5 Ammar Faizi       2022-01-07  5190  static int io_recvfrom(struct io_kiocb *req, unsigned int issue_flags)
fddafacee287b3 Jens Axboe        2020-01-04  5191  {
6b754c8b912a16 Pavel Begunkov    2020-07-16  5192  	struct io_buffer *kbuf;
fddafacee287b3 Jens Axboe        2020-01-04  5193  	struct io_sr_msg *sr = &req->sr_msg;
fddafacee287b3 Jens Axboe        2020-01-04  5194  	struct msghdr msg;
7a7cacba8b4560 Pavel Begunkov    2020-07-16  5195  	void __user *buf = sr->buf;
7a7cacba8b4560 Pavel Begunkov    2020-07-16  5196  	struct socket *sock;
fddafacee287b3 Jens Axboe        2020-01-04  5197  	struct iovec iov;
fddafacee287b3 Jens Axboe        2020-01-04  5198  	unsigned flags;
d1fd1c201d7507 Pavel Begunkov    2021-12-05  5199  	int ret, min_ret = 0;
45d189c6062922 Pavel Begunkov    2021-02-10  5200  	bool force_nonblock = issue_flags & IO_URING_F_NONBLOCK;
7a7cacba8b4560 Pavel Begunkov    2020-07-16  5201  
dba4a9256bb4d7 Florent Revest    2020-12-04  5202  	sock = sock_from_file(req->file);
7a7cacba8b4560 Pavel Begunkov    2020-07-16  5203  	if (unlikely(!sock))
dba4a9256bb4d7 Florent Revest    2020-12-04  5204  		return -ENOTSOCK;
fddafacee287b3 Jens Axboe        2020-01-04  5205  
bc02ef3325e3ef Pavel Begunkov    2020-07-16  5206  	if (req->flags & REQ_F_BUFFER_SELECT) {
51aac424aef980 Pavel Begunkov    2021-10-14  5207  		kbuf = io_recv_buffer_select(req, issue_flags);
bcda7baaa3f15c Jens Axboe        2020-02-23  5208  		if (IS_ERR(kbuf))
bcda7baaa3f15c Jens Axboe        2020-02-23  5209  			return PTR_ERR(kbuf);
bcda7baaa3f15c Jens Axboe        2020-02-23  5210  		buf = u64_to_user_ptr(kbuf->addr);
bcda7baaa3f15c Jens Axboe        2020-02-23  5211  	}
fddafacee287b3 Jens Axboe        2020-01-04  5212  
7a7cacba8b4560 Pavel Begunkov    2020-07-16  5213  	ret = import_single_range(READ, buf, sr->len, &iov, &msg.msg_iter);
14c32eee928662 Pavel Begunkov    2020-07-16  5214  	if (unlikely(ret))
14c32eee928662 Pavel Begunkov    2020-07-16  5215  		goto out_free;
fddafacee287b3 Jens Axboe        2020-01-04  5216  
fddafacee287b3 Jens Axboe        2020-01-04  5217  	msg.msg_name = NULL;
fddafacee287b3 Jens Axboe        2020-01-04  5218  	msg.msg_control = NULL;
fddafacee287b3 Jens Axboe        2020-01-04  5219  	msg.msg_controllen = 0;
fddafacee287b3 Jens Axboe        2020-01-04  5220  	msg.msg_namelen = 0;
fddafacee287b3 Jens Axboe        2020-01-04  5221  	msg.msg_iocb = NULL;
fddafacee287b3 Jens Axboe        2020-01-04  5222  	msg.msg_flags = 0;
fddafacee287b3 Jens Axboe        2020-01-04  5223  
044118069a23fd Pavel Begunkov    2021-04-01  5224  	flags = req->sr_msg.msg_flags;
044118069a23fd Pavel Begunkov    2021-04-01  5225  	if (force_nonblock)
fddafacee287b3 Jens Axboe        2020-01-04  5226  		flags |= MSG_DONTWAIT;
0031275d119efe Stefan Metzmacher 2021-03-20  5227  	if (flags & MSG_WAITALL)
0031275d119efe Stefan Metzmacher 2021-03-20  5228  		min_ret = iov_iter_count(&msg.msg_iter);
0031275d119efe Stefan Metzmacher 2021-03-20  5229  
0b7b21e42ba2d6 Jens Axboe        2020-01-31  5230  	ret = sock_recvmsg(sock, &msg, flags);
7297ce3d59449d Pavel Begunkov    2021-11-23  5231  out_free:
7297ce3d59449d Pavel Begunkov    2021-11-23  5232  	if (ret < min_ret) {
7297ce3d59449d Pavel Begunkov    2021-11-23  5233  		if (ret == -EAGAIN && force_nonblock)
fddafacee287b3 Jens Axboe        2020-01-04  5234  			return -EAGAIN;
fddafacee287b3 Jens Axboe        2020-01-04  5235  		if (ret == -ERESTARTSYS)
fddafacee287b3 Jens Axboe        2020-01-04  5236  			ret = -EINTR;
93d2bcd2cbfed2 Pavel Begunkov    2021-05-16  5237  		req_set_fail(req);
7297ce3d59449d Pavel Begunkov    2021-11-23 @5238  	} else if ((flags & MSG_WAITALL) && (msg.msg_flags & (MSG_TRUNC | MSG_CTRUNC))) {
7297ce3d59449d Pavel Begunkov    2021-11-23  5239  		req_set_fail(req);
7297ce3d59449d Pavel Begunkov    2021-11-23  5240  	}
d1fd1c201d7507 Pavel Begunkov    2021-12-05  5241  
d1fd1c201d7507 Pavel Begunkov    2021-12-05  5242  	__io_req_complete(req, issue_flags, ret, io_put_kbuf(req));
fddafacee287b3 Jens Axboe        2020-01-04  5243  	return 0;
fddafacee287b3 Jens Axboe        2020-01-04  5244  }
fddafacee287b3 Jens Axboe        2020-01-04  5245  

:::::: The code at line 5238 was first introduced by commit
:::::: 7297ce3d59449de49d3c9e1f64ae25488750a1fc io_uring: improve send/recv error handling

:::::: TO: Pavel Begunkov <[email protected]>
:::::: CC: Jens Axboe <[email protected]>

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/[email protected]
_______________________________________________
kbuild mailing list -- [email protected]
To unsubscribe send an email to [email protected]

  reply	other threads:[~2022-02-07  6:13 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <61f5a2fa./zZ+BkSwdkmvxv7x%[email protected]>
2022-01-30  9:11 ` Ammar Faizi
2022-02-07  6:13   ` Chen, Rong A [this message]
2022-02-07 11:43     ` [PATCH io_uring-5.17] io_uring: Fix build error potential reading uninitialized value Ammar Faizi
2022-02-07 13:45       ` Jens Axboe
2022-02-07 14:05         ` [PATCH io_uring-5.17 v2] io_uring: Clean up a false-positive warning from GCC 9.3.0 Ammar Faizi
2022-02-07 15:38           ` Jens Axboe
2022-02-07 14:20         ` [PATCH io_uring-5.17] io_uring: Fix build error potential reading uninitialized value Dan Carpenter
2022-02-07 14:33           ` Alviro Iskandar Setiawan
2022-02-07 15:37             ` Jens Axboe

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 \
    [email protected] \
    [email protected] \
    [email protected] \
    [email protected] \
    [email protected] \
    [email protected] \
    --subject='Re: [ammarfaizi2-block:testing/io_uring-sendto-recvfrom.v1] BUILD SUCCESS 68d110c39241b887ec388cd3316dbedb85b0cbcf' \
    /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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox