On Thu, 2024-02-15 at 15:55 -0800, Andrew Morton wrote: > + * This function marks the beginning of the __GFP_MEMALLOC allocation scope. > > ... > > > > + * This function marks the beginning of the ~__GFP_MOVABLE allocation scope. > > pet peeves. We can assume that the reader knows that this is a > function! Perhaps we should teach checkpatch to whine. > > Like "the function foo" and "the function foo()" in changelogs. > "foo()" is enough! IMO: checkpatch should not be a grammar checker. It's stupid enough as is let alone trying to get it to analyze sentence structure.