On 4/20/21 2:10 AM, Dan Carpenter wrote:
> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/axboe/linux-block.git  for-5.13/drivers
> head:   b09729cbbbf28ecdc61f00335684f7566eb740ec
> commit: 97c62b647fcabd86fe703f9869be7dd91e685f50 [46/61] bcache: initialization 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 problems)
> 
>>> drivers/md/bcache/nvm-pages.c:156:34: warning: Unsigned variable 'i' can't be negative so it is unnecessary to test it. [unsignedPositive]
>       for (i = BCH_MAX_ORDER - 1; i >= 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.

-- 
Jens Axboe