On Tue, 3 Mar 2015, Michal Hocko wrote:

> I think we can safely remove the following code as well, because it is
> anon_vma_clone which is responsible to do all the cleanups.
> 
> diff --git a/mm/mmap.c b/mm/mmap.c
> index 943c6ad18b1d..06a6076c92e5 100644
> --- a/mm/mmap.c
> +++ b/mm/mmap.c
> @@ -774,10 +774,8 @@ again:                   remove_next = 1 + (end > 
> next->vm_end);
>  
>                       importer->anon_vma = exporter->anon_vma;
>                       error = anon_vma_clone(importer, exporter);
> -                     if (error) {
> -                             importer->anon_vma = NULL;
> +                     if (error)
>                               return error;
> -                     }
>               }
>       }
>  

When Michal proposes this on top of -mm, feel free to add my

Acked-by: David Rientjes <[email protected]>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to