From mboxrd@z Thu Jan  1 00:00:00 1970
Content-Type: multipart/mixed; boundary="===============7000983229270000602=="
MIME-Version: 1.0
From: Coly Li <colyli@suse.de>
To: kbuild-all@lists.01.org
Subject: Re: [kbuild] [block:for-5.13/drivers 46/61] drivers/md/bcache/nvm-pages.c:156:34: warning: Unsigned variable 'i' can't be negative so it is unnecessary to test it. [unsignedPositive]
Date: Tue, 20 Apr 2021 22:59:05 +0800
Message-ID: <bb32530b-cfa8-406a-ff1f-85d03a65160c@suse.de>
In-Reply-To: <1a770d8f-a51a-5b68-7931-24f6e95e07d3@kernel.dk>
List-Id: <oe-kbuild-all.lists.linux.dev>

--===============7000983229270000602==
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: quoted-printable

On 4/20/21 9:40 PM, Jens Axboe wrote:
> On 4/20/21 2:10 AM, Dan Carpenter wrote:
>> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/axboe/linux-bloc=
k.git  for-5.13/drivers
>> head:   b09729cbbbf28ecdc61f00335684f7566eb740ec
>> commit: 97c62b647fcabd86fe703f9869be7dd91e685f50 [46/61] bcache: initial=
ization of the buddy
>> compiler: microblaze-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 possible warnings: (new ones prefixed by >>, may not real probl=
ems)
>>
>>>> drivers/md/bcache/nvm-pages.c:156:34: warning: Unsigned variable 'i' c=
an't be negative so it is unnecessary to test it. [unsignedPositive]
>>       for (i =3D BCH_MAX_ORDER - 1; i >=3D 0 ; i--) {
> =

> Coly, how could this ever have worked?
> =

> I'm staring to get less and less impressed with this nvm changeset, it
> seems broken in various ways. Might not be a bad idea to pull it from
> the next queue and give it some more time to turn into something that is
> acceptable.
> =


Hi Jens,

Copied, please do.

We will prepare a more stable code base for 5.14 merge window.

Thanks.

Coly Li
--===============7000983229270000602==--