Re: [PATCH] mm/hugetlb: hide nr_nodes in the internal of for_each_node_mask_to_[alloc|free]

2020-07-14 Thread Wei Yang
On Tue, Jul 14, 2020 at 02:12:03PM -0700, Mike Kravetz wrote: >On 7/14/20 3:02 AM, Vlastimil Babka wrote: >> On 7/14/20 11:57 AM, Wei Yang wrote: >>> On Tue, Jul 14, 2020 at 11:22:03AM +0200, Vlastimil Babka wrote: On 7/14/20 11:13 AM, Vlastimil Babka wrote: > On 7/14/20 9:34 AM, Wei Yang

Re: [PATCH] mm/hugetlb: hide nr_nodes in the internal of for_each_node_mask_to_[alloc|free]

2020-07-14 Thread Mike Kravetz
On 7/14/20 3:02 AM, Vlastimil Babka wrote: > On 7/14/20 11:57 AM, Wei Yang wrote: >> On Tue, Jul 14, 2020 at 11:22:03AM +0200, Vlastimil Babka wrote: >>> On 7/14/20 11:13 AM, Vlastimil Babka wrote: On 7/14/20 9:34 AM, Wei Yang wrote: > The second parameter of

Re: [PATCH] mm/hugetlb: hide nr_nodes in the internal of for_each_node_mask_to_[alloc|free]

2020-07-14 Thread Vlastimil Babka
On 7/14/20 11:57 AM, Wei Yang wrote: > On Tue, Jul 14, 2020 at 11:22:03AM +0200, Vlastimil Babka wrote: >>On 7/14/20 11:13 AM, Vlastimil Babka wrote: >>> On 7/14/20 9:34 AM, Wei Yang wrote: The second parameter of for_each_node_mask_to_[alloc|free] is a loop variant, which is not used

Re: [PATCH] mm/hugetlb: hide nr_nodes in the internal of for_each_node_mask_to_[alloc|free]

2020-07-14 Thread Wei Yang
On Tue, Jul 14, 2020 at 11:22:03AM +0200, Vlastimil Babka wrote: >On 7/14/20 11:13 AM, Vlastimil Babka wrote: >> On 7/14/20 9:34 AM, Wei Yang wrote: >>> The second parameter of for_each_node_mask_to_[alloc|free] is a loop >>> variant, which is not used outside of loop iteration. >>> >>> Let's

Re: [PATCH] mm/hugetlb: hide nr_nodes in the internal of for_each_node_mask_to_[alloc|free]

2020-07-14 Thread Wei Yang
On Tue, Jul 14, 2020 at 11:22:03AM +0200, Vlastimil Babka wrote: >On 7/14/20 11:13 AM, Vlastimil Babka wrote: >> On 7/14/20 9:34 AM, Wei Yang wrote: >>> The second parameter of for_each_node_mask_to_[alloc|free] is a loop >>> variant, which is not used outside of loop iteration. >>> >>> Let's

Re: [PATCH] mm/hugetlb: hide nr_nodes in the internal of for_each_node_mask_to_[alloc|free]

2020-07-14 Thread Vlastimil Babka
On 7/14/20 11:13 AM, Vlastimil Babka wrote: > On 7/14/20 9:34 AM, Wei Yang wrote: >> The second parameter of for_each_node_mask_to_[alloc|free] is a loop >> variant, which is not used outside of loop iteration. >> >> Let's hide this. >> >> Signed-off-by: Wei Yang >> --- >> mm/hugetlb.c | 38

Re: [PATCH] mm/hugetlb: hide nr_nodes in the internal of for_each_node_mask_to_[alloc|free]

2020-07-14 Thread Vlastimil Babka
On 7/14/20 9:34 AM, Wei Yang wrote: > The second parameter of for_each_node_mask_to_[alloc|free] is a loop > variant, which is not used outside of loop iteration. > > Let's hide this. > > Signed-off-by: Wei Yang > --- > mm/hugetlb.c | 38 -- > 1 file

Re: [PATCH] mm/hugetlb: hide nr_nodes in the internal of for_each_node_mask_to_[alloc|free]

2020-07-14 Thread Pankaj Gupta
> The second parameter of for_each_node_mask_to_[alloc|free] is a loop > variant, which is not used outside of loop iteration. > > Let's hide this. > > Signed-off-by: Wei Yang > --- > mm/hugetlb.c | 38 -- > 1 file changed, 20 insertions(+), 18 deletions(-) >

[PATCH] mm/hugetlb: hide nr_nodes in the internal of for_each_node_mask_to_[alloc|free]

2020-07-14 Thread Wei Yang
The second parameter of for_each_node_mask_to_[alloc|free] is a loop variant, which is not used outside of loop iteration. Let's hide this. Signed-off-by: Wei Yang --- mm/hugetlb.c | 38 -- 1 file changed, 20 insertions(+), 18 deletions(-) diff --git