[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <5ec7b561-551b-f02a-9b7b-65d05ccdabbc@quicinc.com>
Date: Sat, 18 Jun 2022 11:25:43 +0530
From: Charan Teja Kalla <quic_charante@...cinc.com>
To: kernel test robot <lkp@...el.com>, Michal Hocko <mhocko@...e.com>,
"Minchan Kim" <minchan@...nel.org>
CC: <kbuild-all@...ts.01.org>, <linux-kernel@...r.kernel.org>,
Andrew Morton <akpm@...ux-foundation.org>,
Linux Memory Management List <linux-mm@...ck.org>
Subject: Re: mm/madvise.c:1438:6: warning: Redundant assignment of 'ret' to
itself. [selfAssignment]
Hello Andrew,
On 6/18/2022 4:34 AM, kernel test robot wrote:
> tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
> head: 4b35035bcf80ddb47c0112c4fbd84a63a2836a18
> commit: 5bd009c7c9a9e888077c07535dc0c70aeab242c3 mm: madvise: return correct bytes advised with process_madvise
> date: 3 months ago
> compiler: mips-linux-gcc (GCC) 11.3.0
> reproduce (cppcheck warning):
> # apt-get install cppcheck
> git checkout 5bd009c7c9a9e888077c07535dc0c70aeab242c3
> cppcheck --quiet --enable=style,performance,portability --template=gcc FILE
>
> If you fix the issue, kindly add following tag where applicable
> Reported-by: kernel test robot <lkp@...el.com>
>
>
> cppcheck warnings: (new ones prefixed by >>)
>>> mm/madvise.c:1438:6: warning: Redundant assignment of 'ret' to itself. [selfAssignment]
> ret = (total_len - iov_iter_count(&iter)) ? : ret;
Other way to avoid this warning is by creating another local variable
that holds the total bytes processed. Having another local variable to
get rid off some compilation warning doesn't seem proper to me. So,
leaving this warning unless you ask me to fix this.
Thanks,
Charan
Powered by blists - more mailing lists