On Tue, Feb 11, 2020 at 08:18:25PM -0800, Matthew Wilcox wrote: > - * @fgp_flags: PCG flags > + * @fgp_flags: FGP flags > * @gfp_mask: gfp mask to use for the page cache data page allocation > * > * Looks up the page cache slot at @mapping & @offset. > * > - * PCG flags modify how the page is returned. > + * FGP flags modify how the page is returned. This still looks weird. Why not just a single line: * @fgp_flags: FGP_* flags that control how the page is returned.