Commit abd69b9e authored Apr 01, 2020 by Wang Wenhu Committed by Linus Torvalds Apr 02, 2020

mm/memory.c: clarify a confusing comment for vm_iomap_memory



The param "start" actually referes to the physical memory start, which is
to be mapped into virtual area vma.  And it is the field vma->vm_start
which stands for the start of the area.

Most of the time, we do not read through whole implementation of a
function but only the definition and essential comments.  Accurate
comments are definitely the base stone.

Signed-off-by: Wang Wenhu <wenhu.wang@vivo.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Reviewed-by: Andrew Morton <akpm@linux-foundation.org>
Link: http://lkml.kernel.org/r/20200318052206.105104-1-wenhu.wang@vivo.com


Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>

parent 86a76331

mm/memory.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -2009,7 +2009,7 @@ EXPORT_SYMBOL(remap_pfn_range);
		/**
		* vm_iomap_memory - remap memory to userspace
		* @vma: user vma to map to
		* @start: start of area
		* @start: start of the physical memory to be mapped
		* @len: size of area
		*
		* This is a simplified io_remap_pfn_range() for common driver use. The

Admin message