Re: [PATCH v11 19/25] erofs: Convert compressed files from readpages to readahead

From: Andrew Morton
Date: Tue Apr 21 2020 - 01:42:18 EST


On Tue, 14 Apr 2020 08:02:27 -0700 Matthew Wilcox <willy@xxxxxxxxxxxxx> wrote:

>
> Use the new readahead operation in erofs.
>

Well this is exciting.

fs/erofs/data.c: In function erofs_raw_access_readahead:
fs/erofs/data.c:149:18: warning: last_block may be used uninitialized in this function [-Wmaybe-uninitialized]
*last_block + 1 != current_block) {

It seems to be a preexisting bug, which your patch prompted gcc-7.2.0
to notice.

erofs_read_raw_page() goes in and uses *last_block, but neither of its
callers has initialized it. Could the erofs maintainers please take a
look?