Hi Andrew, Can you please apply this?
Thanks a lot in advance! Hi Zheng, Thank you for the patch! Best regards, Anton > On 12 Mar 2020, at 04:13, Zheng Zengkai <zhengzeng...@huawei.com> wrote: > > Fixes gcc '-Wunused-but-set-variable' warning: > > fs/ntfs/logfile.c: In function ntfs_check_logfile: > fs/ntfs/logfile.c:481:21: > warning: variable log_page_mask set but not used [-Wunused-but-set-variable] > > Actually log_page_mask can be used to replace 'log_page_size - 1' as it is > set. > > Signed-off-by: Zheng Zengkai <zhengzeng...@huawei.com> > Acked-by: Anton Altaparmakov <an...@tuxera.com> > --- > fs/ntfs/logfile.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/fs/ntfs/logfile.c b/fs/ntfs/logfile.c > index a0c40f1be7ac..c35fcf389369 100644 > --- a/fs/ntfs/logfile.c > +++ b/fs/ntfs/logfile.c > @@ -507,7 +507,7 @@ bool ntfs_check_logfile(struct inode *log_vi, > RESTART_PAGE_HEADER **rp) > * optimize log_page_size and log_page_bits into constants. > */ > log_page_bits = ntfs_ffs(log_page_size) - 1; > - size &= ~(s64)(log_page_size - 1); > + size &= ~(s64)(log_page_mask); > /* > * Ensure the log file is big enough to store at least the two restart > * pages and the minimum number of log record pages. > -- > 2.20.1 > -- Anton Altaparmakov <anton at tuxera.com> (replace at with @) Lead in File System Development, Tuxera Inc., http://www.tuxera.com/ Linux NTFS maintainer