Re: [PATCH] fix find_next_system_ram comments
From: Randy Dunlap <hidden>
Date: 2011-09-26 16:58:58
Also in:
lkml
On 09/26/2011 06:58 AM, Wizard wrote:
The purpose of find_next_system_ram() is to find a the lowest memory resource which contain or overlap the [res->start, res->end), not just contain. In this patch, I make this comment more exact and fix one typo. Signed-off-by: Wizard <redacted>
For Signed-off-by: Documentation/SubmittingPatches says: using your real name (sorry, no pseudonyms or anonymous contributions.)
quoted hunk ↗ jump to hunk
--- kernel/resource.c | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-)diff --git a/kernel/resource.c b/kernel/resource.c index 3b3cedc..2751a8c 100644 --- a/kernel/resource.c +++ b/kernel/resource.c@@ -279,7 +279,8 @@ EXPORT_SYMBOL(release_resource); #if !defined(CONFIG_ARCH_HAS_WALK_MEMORY) /* - * Finds the lowest memory reosurce exists within [res->start.res->end) + * Finds the lowest memory resource which contains or overlaps + * [res->start.res->end)
Your patch description uses ", " (comma) here. I think that's better than keeping the ".".
* the caller must specify res->start, res->end, res->flags and "name". * If found, returns 0, res is overwritten, if not found, returns -1. */
-- ~Randy *** Remember to use Documentation/SubmitChecklist when testing your code *** -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/ Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>