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]
next prev parent 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 ` [ammarfaizi2-block:testing/io_uring-sendto-recvfrom.v1] BUILD SUCCESS 68d110c39241b887ec388cd3316dbedb85b0cbcf 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] \
/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