Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <c4ed4e2b-ea79-70e8-2722-137dfa9bbab3@kernel.dk>
Date:   Sat, 5 Nov 2022 09:29:19 -0600
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux aarch64; rv:102.0) Gecko/20100101
 Thunderbird/102.4.0
Subject: Re: [PATCH v2] io_uring: fix two assignments in if conditions
Content-Language: en-US
To:     Xinghui Li <korantwork@gmail.com>, asml.silence@gmail.com
Cc:     linux-kernel@vger.kernel.org, io-uring@vger.kernel.org,
        Xinghui Li <korantli@tencent.com>,
        kernel test robot <lkp@intel.com>
References: <20221102082503.32236-1-korantwork@gmail.com>
 <630269a9-29a3-6b75-0a67-449ec36e7e5e@kernel.dk>
 <EF99381C-E842-49AE-A683-B8EABD1612A0@gmail.com>
From:   Jens Axboe <axboe@kernel.dk>
In-Reply-To: <EF99381C-E842-49AE-A683-B8EABD1612A0@gmail.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: bulk

On 11/2/22 8:50 PM, Xinghui Li wrote:
> 
> 
> ?? 2022/11/2 22:19??Jens Axboe?<axboe@kernel.dk> ??:
> 
>>    On 11/2/22 2:25 AM, korantwork@gmail.com wrote:
>>    > From: Xinghui Li <korantli@tencent.com>
>>    > 
>>    > Fixs two error:
>>    > 
>>    > "ERROR: do not use assignment in if condition
>>    > 130: FILE: io_uring/net.c:130:
>>    > +       if (!(issue_flags & IO_URING_F_UNLOCKED) &&
>>    > 
>>    > ERROR: do not use assignment in if condition
>>    > 599: FILE: io_uring/poll.c:599:
>>    > +       } else if (!(issue_flags & IO_URING_F_UNLOCKED) &&"
>>    > reported by checkpatch.pl in net.c and poll.c . 
>>
>>    I'm not super excited about this patch as the previous one
>>    wasn't even compiled? Did you test this one, compile and runtime?
>>
> 
> First of all, I'm really really really SORRY for the last stupid
> warning and my careless. But I did make kernel and install it in my
> device. Also, I did the fio test with polling. However, there is no
> warning be reported by 'make' command. But when I check the patch code
> with 'smatch', I did find that warning. I realized how stupid that
> mistake was.... So, in addition to compiling and installing this
> kernel, I also checked the code using the static analysis tool this
> time. There is no new error or warning.
> "
> make -j64 CHECK="~/workspace/smatch/smatch" C=1
>   DESCEND objtool
>   CALL    scripts/checksyscalls.sh
>   CC      io_uring/net.o
>   CC      io_uring/poll.o
>   CHECK   io_uring/poll.c
> io_uring/poll.c: note: in included file (through io_uring/io_uring.h):
> io_uring/slist.h:138:29: warning: no newline at end of file
>   CHECK   io_uring/net.c
> io_uring/poll.c: note: in included file (through include/trace/events/io_uring.h):
> ./include/linux/io_uring_types.h:151:37: warning: array of flexible structures
> io_uring/net.c: note: in included file (through io_uring/io_uring.h):
> io_uring/slist.h:138:29: warning: no newline at end of file
> io_uring/net.c: note: in included file (through io_uring/io_uring.h):
> ./include/linux/io_uring_types.h:151:37: warning: array of flexible structures
> ...
>   BUILD   arch/x86/boot/bzImage
> Kernel: arch/x86/boot/bzImage is ready  (#10)
> " 
> 
> Again, I am ashamed of the previous mistake. And I totally understand
> if you reject this one.

All good, thanks for doing the due diligence this time around for v2.
I'll get it applied for 6.2, thanks.

-- 
Jens Axboe