From: Mike Kravetz <mike.kravetz@oracle.com> To: linux-mm@kvack.org, linux-kernel@vger.kernel.org Cc: Joonsoo Kim <iamjoonsoo.kim@lge.com>, Vlastimil Babka <vbabka@suse.cz>, Michal Nazarewicz <mina86@mina86.com>, Michal Hocko <mhocko@suse.com>, Mel Gorman <mgorman@techsingularity.net>, Johannes Weiner <hannes@cmpxchg.org>, Andrew Morton <akpm@linux-foundation.org>, Mike Kravetz <mike.kravetz@oracle.com> Subject: [PATCH 0/1] mm/cma: fix alloc_contig_range ret code/potential leak Date: Mon, 20 Nov 2017 11:39:29 -0800 [thread overview] Message-ID: <20171120193930.23428-1-mike.kravetz@oracle.com> (raw) In an attempt to make contiguous allocation routines more available to drivers, I have been experimenting with code similar to that used by alloc_gigantic_page(). While stressing this code with many other allocations and frees in progress, I would sometimes notice large 'leaks' of page ranges. I traced this down to the routine alloc_contig_range() itself. In commit 8ef5849fa8a2 the code was changed so that an -EBUSY returned by __alloc_contig_migrate_range() would not immediately return to the caller. Rather, processing continues so that test_pages_isolated() is eventually called. This is done because test_pages_isolated() has a tracepoint to identify the busy pages. However, it is possible (observed in my testing) that pages which were busy when __alloc_contig_migrate_range was called may become available by the time test_pages_isolated is called. Further, it is possible that the entire range can actually be allocated. Unfortunately, in this case the return code originally set by __alloc_contig_migrate_range (-EBUSY) is returned to the calller. Therefore, the caller assumes the range was not allocated and the pages are essentially leaked. The following patch simply updates the return code based on the value returned from test_pages_isolated. It is unlikely that we will hit this issue today based on the limited number of callers to alloc_contig_range. However, I have Cc'ed stable because if we do hit this issue it has the potential to leak a large number of pages. Mike Kravetz (1): mm/cma: fix alloc_contig_range ret code/potential leak mm/page_alloc.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) -- 2.13.6
WARNING: multiple messages have this Message-ID (diff)
From: Mike Kravetz <mike.kravetz@oracle.com> To: linux-mm@kvack.org, linux-kernel@vger.kernel.org Cc: Joonsoo Kim <iamjoonsoo.kim@lge.com>, Vlastimil Babka <vbabka@suse.cz>, Michal Nazarewicz <mina86@mina86.com>, Michal Hocko <mhocko@suse.com>, Mel Gorman <mgorman@techsingularity.net>, Johannes Weiner <hannes@cmpxchg.org>, Andrew Morton <akpm@linux-foundation.org>, Mike Kravetz <mike.kravetz@oracle.com> Subject: [PATCH 0/1] mm/cma: fix alloc_contig_range ret code/potential leak Date: Mon, 20 Nov 2017 11:39:29 -0800 [thread overview] Message-ID: <20171120193930.23428-1-mike.kravetz@oracle.com> (raw) In an attempt to make contiguous allocation routines more available to drivers, I have been experimenting with code similar to that used by alloc_gigantic_page(). While stressing this code with many other allocations and frees in progress, I would sometimes notice large 'leaks' of page ranges. I traced this down to the routine alloc_contig_range() itself. In commit 8ef5849fa8a2 the code was changed so that an -EBUSY returned by __alloc_contig_migrate_range() would not immediately return to the caller. Rather, processing continues so that test_pages_isolated() is eventually called. This is done because test_pages_isolated() has a tracepoint to identify the busy pages. However, it is possible (observed in my testing) that pages which were busy when __alloc_contig_migrate_range was called may become available by the time test_pages_isolated is called. Further, it is possible that the entire range can actually be allocated. Unfortunately, in this case the return code originally set by __alloc_contig_migrate_range (-EBUSY) is returned to the calller. Therefore, the caller assumes the range was not allocated and the pages are essentially leaked. The following patch simply updates the return code based on the value returned from test_pages_isolated. It is unlikely that we will hit this issue today based on the limited number of callers to alloc_contig_range. However, I have Cc'ed stable because if we do hit this issue it has the potential to leak a large number of pages. Mike Kravetz (1): mm/cma: fix alloc_contig_range ret code/potential leak mm/page_alloc.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) -- 2.13.6 -- 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:[~2017-11-20 19:40 UTC|newest] Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top 2017-11-20 19:39 Mike Kravetz [this message] 2017-11-20 19:39 ` [PATCH 0/1] mm/cma: fix alloc_contig_range ret code/potential leak Mike Kravetz 2017-11-20 19:39 ` [PATCH 1/1] " Mike Kravetz 2017-11-20 19:39 ` Mike Kravetz 2017-11-21 7:53 ` Vlastimil Babka 2017-11-21 7:53 ` Vlastimil Babka 2017-11-21 13:06 ` Michal Hocko 2017-11-21 13:06 ` Michal Hocko 2017-11-21 13:04 ` Michal Hocko 2017-11-21 13:04 ` Michal Hocko 2017-11-21 14:20 ` Michal Nazarewicz 2017-11-21 14:20 ` Michal Nazarewicz 2017-11-21 14:20 ` Michal Nazarewicz 2017-11-22 12:00 ` Johannes Weiner 2017-11-22 12:00 ` Johannes Weiner 2017-11-22 17:46 ` Mike Kravetz 2017-11-22 17:46 ` Mike Kravetz 2017-11-22 18:52 ` [PATCH v2] " Mike Kravetz 2017-11-22 18:52 ` Mike Kravetz 2017-11-23 8:05 ` Vlastimil Babka 2017-11-23 8:05 ` Vlastimil Babka 2017-11-23 9:19 ` Michal Hocko 2017-11-23 9:19 ` Michal Hocko 2017-11-27 16:59 ` Johannes Weiner 2017-11-27 16:59 ` Johannes Weiner 2017-11-29 5:38 ` Joonsoo Kim 2017-11-29 5:38 ` Joonsoo Kim
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=20171120193930.23428-1-mike.kravetz@oracle.com \ --to=mike.kravetz@oracle.com \ --cc=akpm@linux-foundation.org \ --cc=hannes@cmpxchg.org \ --cc=iamjoonsoo.kim@lge.com \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-mm@kvack.org \ --cc=mgorman@techsingularity.net \ --cc=mhocko@suse.com \ --cc=mina86@mina86.com \ --cc=vbabka@suse.cz \ /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: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.