Re: [PATCH 3/9] mm, hugetlb: clean-up alloc_huge_page()

2013-07-23 Thread Joonsoo Kim
On Mon, Jul 22, 2013 at 04:51:50PM +0200, Michal Hocko wrote: > On Mon 15-07-13 18:52:41, Joonsoo Kim wrote: > > We can unify some codes for succeed allocation. > > This makes code more readable. > > There is no functional difference. > > "This patch unifies successful allocation paths to make the

Re: [PATCH 3/9] mm, hugetlb: clean-up alloc_huge_page()

2013-07-22 Thread Michal Hocko
On Mon 15-07-13 18:52:41, Joonsoo Kim wrote: > We can unify some codes for succeed allocation. > This makes code more readable. > There is no functional difference. "This patch unifies successful allocation paths to make the code more readable. There are no functional changes." Better? > Signed-

[PATCH 3/9] mm, hugetlb: clean-up alloc_huge_page()

2013-07-15 Thread Joonsoo Kim
We can unify some codes for succeed allocation. This makes code more readable. There is no functional difference. Signed-off-by: Joonsoo Kim diff --git a/mm/hugetlb.c b/mm/hugetlb.c index d21a33a..0067cf4 100644 --- a/mm/hugetlb.c +++ b/mm/hugetlb.c @@ -1144,29 +1144,25 @@ static struct page *al