Re: [PATCH v3 04/15] mm: update print_bad_page_map() to show anonymous page index
From: Gregory Price
Date: Wed Aug 05 2026 - 04:48:05 EST
On Mon, Aug 03, 2026 at 11:19:17AM +0100, Lorenzo Stoakes (ARM) wrote:
> On Mon, Aug 03, 2026 at 11:39:11AM +0200, David Hildenbrand (Arm) wrote:
> > On 7/30/26 08:32, Gregory Price wrote:
> > > On Wed, Jul 29, 2026 at 05:48:39PM +0100, Lorenzo Stoakes (ARM) wrote:
> > >> This is potentially useful debugging information and matches the existing
> > >> page offset provided.
> > >>
> > >> Use the raw __linear_anon_page_index() function so as to always output this
> > >> value regardless of whether the mapping is file-backed or not.
> > >>
> > >
> > > Begs the question whether we should mark the value based on whether
> > > it is file backed or not, but i suppose you can gleen that from the
> > > other printed values (anon_mem, mapping, vm_ops, etc).
> >
> > You mean something anon_pgoff_t or what did you have in mind?
>
> I suspect he means e.g. adding a (A) suffix or something? Like page offset xxx
> (F) or yyy (A) I guess?
>
Yeah just an F/A in the print I suppose. Sorry if that was unclear.
If it saves someone the head-scratch of whether some bug is because it's
"file" or "true anon", given the ambiguity here, seems like maybe useful
debug information.
~Gregory