Messages in this thread | | | Subject | Re: [PATCH 08/10] mm/hugetlb: return non-isolated page in the loop instead of break and check | From | Mike Kravetz <> | Date | Mon, 10 Aug 2020 15:55:45 -0700 |
| |
On 8/7/20 2:12 AM, Wei Yang wrote: > Function dequeue_huge_page_node_exact() iterates the free list and > return the first non-isolated one. > > Instead of break and check the loop variant, we could return in the loop > directly. This could reduce some redundant check. > > Signed-off-by: Wei Yang <richard.weiyang@linux.alibaba.com>
I agree with Baoquan He in that this is more of a style change. Certainly there is the potential to avoid an extra check and that is always good.
The real value in this patch (IMO) is removal of the stale comment.
Reviewed-by: Mike Kravetz <mike.kravetz@oracle.com>
-- Mike Kravetz
| |