From: "Gioh Kim" <gioh.kim@lge.com>
To: 'Andrew Morton' <akpm@linux-foundation.org>,
'Joonsoo Kim' <iamjoonsoo.kim@lge.com>,
'Zhang Yanfei' <zhangyanfei@cn.fujitsu.com>,
linux-mm@kvack.org, linux-kernel@vger.kernel.org
Cc: 이건호 <gunho.lee@lge.com>,
chanho.min@lge.com, "Minchan Kim" <minchan.kim@lge.com>
Subject: [PATCH][RFC] mm: warning message for vm_map_ram about vm size
Date: Mon, 10 Mar 2014 13:57:07 +0900 [thread overview]
Message-ID: <001a01cf3c1d$310716a0$931543e0$@lge.com> (raw)
Hi,
I have a failure of allocation of virtual memory on ARMv7 based platform.
I called alloc_page()/vm_map_ram() for allocation/mapping pages.
Virtual memory space exhausting problem occurred.
I checked virtual memory space and found that there are too many 4MB chunks.
I thought that if just one page in the 4MB chunk lives long,
the entire chunk cannot be freed. Therefore new chunk is created again and again.
In my opinion, the vm_map_ram() function should be used for temporary mapping
and/or short term memory mapping. Otherwise virtual memory is wasted.
I am not sure if my opinion is correct. If it is, please add some warning message
about the vm_map_ram().
---8<---
Subject: [PATCH] mm: warning comment for vm_map_ram
vm_map_ram can occur locking of virtual memory space
because if only one page lives long in one vmap_block,
it takes 4MB (1024-times more than one page) space.
Change-Id: I6f5919848cf03788b5846b7d850d66e4d93ac39a
Signed-off-by: Gioh Kim <gioh.kim@lge.com>
---
mm/vmalloc.c | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/mm/vmalloc.c b/mm/vmalloc.c
index 0fdf968..2de1d1b 100644
--- a/mm/vmalloc.c
+++ b/mm/vmalloc.c
@@ -1083,6 +1083,10 @@ EXPORT_SYMBOL(vm_unmap_ram);
* @node: prefer to allocate data structures on this node
* @prot: memory protection to use. PAGE_KERNEL for regular RAM
*
+ * This function should be used for TEMPORARY mapping. If just one page lives i
+ * long, it would occupy 4MB vm size permamently. 100 pages (just 400KB) could
+ * takes 400MB with bad luck.
+ *
* Returns: a pointer to the address that has been mapped, or %NULL on failure
*/
void *vm_map_ram(struct page **pages, unsigned int count, int node, pgprot_t prot)
--
1.7.9.5
Gioh Kim / 김 기 오
Research Engineer
Advanced OS Technology Team
Software Platform R&D Lab.
Mobile: 82-10-7322-5548
E-mail: gioh.kim@lge.com
19, Yangjae-daero 11gil
Seocho-gu, Seoul 137-130, Korea
--
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/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>
next reply other threads:[~2014-03-10 4:57 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-03-10 4:57 Gioh Kim [this message]
2014-03-10 5:47 ` Minchan Kim
2014-03-10 6:28 ` Zhang Yanfei
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='001a01cf3c1d$310716a0$931543e0$@lge.com' \
--to=gioh.kim@lge.com \
--cc=akpm@linux-foundation.org \
--cc=chanho.min@lge.com \
--cc=gunho.lee@lge.com \
--cc=iamjoonsoo.kim@lge.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=minchan.kim@lge.com \
--cc=zhangyanfei@cn.fujitsu.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox