lkml.org 
[lkml]   [2013]   [Aug]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH v2 04/20] mm, hugetlb: remove useless check about mapping type
From
Date
On Fri, 2013-08-09 at 18:26 +0900, Joonsoo Kim wrote:
> is_vma_resv_set(vma, HPAGE_RESV_OWNER) implys that this mapping is
> for private. So we don't need to check whether this mapping is for
> shared or not.
>
> This patch is just for clean-up.
>
> Signed-off-by: Joonsoo Kim <iamjoonsoo.kim@lge.com>

Reviewed-by: Davidlohr Bueso <davidlohr@hp.com>

>
> diff --git a/mm/hugetlb.c b/mm/hugetlb.c
> index ea1ae0a..c017c52 100644
> --- a/mm/hugetlb.c
> +++ b/mm/hugetlb.c
> @@ -2544,8 +2544,7 @@ retry_avoidcopy:
> * at the time of fork() could consume its reserves on COW instead
> * of the full address range.
> */
> - if (!(vma->vm_flags & VM_MAYSHARE) &&
> - is_vma_resv_set(vma, HPAGE_RESV_OWNER) &&
> + if (is_vma_resv_set(vma, HPAGE_RESV_OWNER) &&
> old_page != pagecache_page)
> outside_reserve = 1;
>




\
 
 \ /
  Last update: 2013-08-12 16:01    [W:0.338 / U:0.036 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site