[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <Y1IPqhVpiehRBGNS@casper.infradead.org>
Date: Fri, 21 Oct 2022 04:19:06 +0100
From: Matthew Wilcox <willy@...radead.org>
To: Maria Yu <quic_aiquny@...cinc.com>
Cc: akpm@...ux-foundation.org, ziy@...dia.com, david@...hat.com,
linux-kernel@...r.kernel.org, linux-mm@...ck.org,
mike.kravetz@...cle.com, opendmb@...il.com, stable@...r.kernel.org
Subject: Re: [PATCH] mm/page_isolation: fix clang deadcode warning
On Fri, Oct 21, 2022 at 11:09:53AM +0800, Maria Yu wrote:
> When !CONFIG_VM_BUG_ON, there is warning of
> clang-analyzer-deadcode.DeadStores:
> Value stored to 'mt' during its initialization
> is never read.
Honestly, the cure is worse than the disease. I'd rather not have a
line that's this long.
> - int mt = get_pageblock_migratetype(pfn_to_page(isolate_pageblock));
> -
> - VM_BUG_ON(!is_migrate_isolate(mt));
> + VM_BUG_ON(!is_migrate_isolate(get_pageblock_migratetype(pfn_to_page(isolate_pageblock))));
Powered by blists - more mailing lists