Re: [PATCH v3 5/9] mm, hugetlb: remove redundant list_empty check in gather_surplus_pages()

On Mon, Jul 29, 2013 at 1:28 PM, Joonsoo Kim wrote: > If list is empty, list_for_each_entry_safe() doesn't do anything. > So, this check is redundant. Remove it. > > Acked-by: Michal Hocko > Reviewed-by: Wanpeng Li > Reviewed-by: Aneesh Kumar K.V > Signed-off-by: Joonsoo Kim > Acked-by: Hillf

Re: [PATCH v3 5/9] mm, hugetlb: remove redundant list_empty check in gather_surplus_pages()

On Mon, Jul 29, 2013 at 1:28 PM, Joonsoo Kim iamjoonsoo@lge.com wrote: If list is empty, list_for_each_entry_safe() doesn't do anything. So, this check is redundant. Remove it. Acked-by: Michal Hocko mho...@suse.cz Reviewed-by: Wanpeng Li liw...@linux.vnet.ibm.com Reviewed-by: Aneesh

[PATCH v3 5/9] mm, hugetlb: remove redundant list_empty check in gather_surplus_pages()

If list is empty, list_for_each_entry_safe() doesn't do anything. So, this check is redundant. Remove it. Acked-by: Michal Hocko Reviewed-by: Wanpeng Li Reviewed-by: Aneesh Kumar K.V Signed-off-by: Joonsoo Kim diff --git a/mm/hugetlb.c b/mm/hugetlb.c index 87d7637..2e52afea 100644 ---

[PATCH v3 5/9] mm, hugetlb: remove redundant list_empty check in gather_surplus_pages()

If list is empty, list_for_each_entry_safe() doesn't do anything. So, this check is redundant. Remove it. Acked-by: Michal Hocko mho...@suse.cz Reviewed-by: Wanpeng Li liw...@linux.vnet.ibm.com Reviewed-by: Aneesh Kumar K.V aneesh.ku...@linux.vnet.ibm.com Signed-off-by: Joonsoo Kim