Re: [PATCH 2/3] hugetlb: alloc_bootmem_huge_page(): use IS_ALIGNED()

2014-11-19 Thread David Rientjes
On Wed, 12 Nov 2014, Luiz Capitulino wrote: > No reason to duplicate the code of an existing macro. > > Signed-off-by: Luiz Capitulino Acked-by: David Rientjes -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More

Re: [PATCH 2/3] hugetlb: alloc_bootmem_huge_page(): use IS_ALIGNED()

2014-11-19 Thread David Rientjes
On Wed, 12 Nov 2014, Luiz Capitulino wrote: No reason to duplicate the code of an existing macro. Signed-off-by: Luiz Capitulino lcapitul...@redhat.com Acked-by: David Rientjes rient...@google.com -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a

Re: [PATCH 2/3] hugetlb: alloc_bootmem_huge_page(): use IS_ALIGNED()

2014-11-16 Thread Naoya Horiguchi
On Wed, Nov 12, 2014 at 05:33:12PM -0500, Luiz Capitulino wrote: > No reason to duplicate the code of an existing macro. > > Signed-off-by: Luiz Capitulino I think that we can apply the same macro for the following two lines in __unmap_hugepage_range(): BUG_ON(start &

Re: [PATCH 2/3] hugetlb: alloc_bootmem_huge_page(): use IS_ALIGNED()

2014-11-16 Thread Naoya Horiguchi
On Wed, Nov 12, 2014 at 05:33:12PM -0500, Luiz Capitulino wrote: No reason to duplicate the code of an existing macro. Signed-off-by: Luiz Capitulino lcapitul...@redhat.com I think that we can apply the same macro for the following two lines in __unmap_hugepage_range(): BUG_ON(start

[PATCH 2/3] hugetlb: alloc_bootmem_huge_page(): use IS_ALIGNED()

2014-11-12 Thread Luiz Capitulino
No reason to duplicate the code of an existing macro. Signed-off-by: Luiz Capitulino --- mm/hugetlb.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/mm/hugetlb.c b/mm/hugetlb.c index 9fd7227..a10fd57 100644 --- a/mm/hugetlb.c +++ b/mm/hugetlb.c @@ -1457,7 +1457,7 @@ int

[PATCH 2/3] hugetlb: alloc_bootmem_huge_page(): use IS_ALIGNED()

2014-11-12 Thread Luiz Capitulino
No reason to duplicate the code of an existing macro. Signed-off-by: Luiz Capitulino lcapitul...@redhat.com --- mm/hugetlb.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/mm/hugetlb.c b/mm/hugetlb.c index 9fd7227..a10fd57 100644 --- a/mm/hugetlb.c +++ b/mm/hugetlb.c @@