On Mon, Feb 15, 2021 at 09:02:03PM +0000, Matthew Wilcox (Oracle) wrote: > Sphinx interprets the Return section as a list and complains about it. > Turn it into a sentence and move it to the end of the kernel-doc to > fit the kernel-doc style. > > Signed-off-by: Matthew Wilcox (Oracle) <willy@xxxxxxxxxxxxx> Acked-by: Mike Rapoport <rppt@xxxxxxxxxxxxx> > --- > Documentation/core-api/mm-api.rst | 1 + > mm/mempolicy.c | 11 ++++------- > 2 files changed, 5 insertions(+), 7 deletions(-) ... > diff --git a/mm/mempolicy.c b/mm/mempolicy.c > index 96c98ce16727..577f59c8f327 100644 > --- a/mm/mempolicy.c > +++ b/mm/mempolicy.c > @@ -2456,14 +2456,11 @@ static void sp_free(struct sp_node *n) > * @addr: virtual address where page mapped > * > * Lookup current policy node id for vma,addr and "compare to" page's > - * node id. > - * > - * Returns: > - * -1 - not misplaced, page is in the right node > - * node - node id where the page should be > - * > - * Policy determination "mimics" alloc_page_vma(). > + * node id. Policy determination "mimics" alloc_page_vma(). > * Called from fault path where we know the vma and faulting address. > + * > + * Return: -1 if the page is in a node that is valid for this policy, or a > + * suitable node ID to allocate a replacement page from. I think it's possible to use lists in the Return: descriptions with the right combination of spaces, asterisks and dashes, but this description is way better anyway. -- Sincerely yours, Mike.
