Re: [PATCH 1/6] mm: get rid of zone_is_initialized

2017-03-30 Thread Michal Hocko
On Fri 31-03-17 08:43:01, Michal Hocko wrote: > On Fri 31-03-17 11:39:07, Hillf Danton wrote: > > > > On March 30, 2017 7:55 PM Michal Hocko wrote: > > > > > > @@ -5535,9 +5535,6 @@ int __meminit init_currently_empty_zone(struct zone > > > *zone, > > > zone_start_pfn, (zone_sta

Re: [PATCH 1/6] mm: get rid of zone_is_initialized

2017-03-30 Thread Michal Hocko
On Fri 31-03-17 11:39:07, Hillf Danton wrote: > > On March 30, 2017 7:55 PM Michal Hocko wrote: > > > > @@ -5535,9 +5535,6 @@ int __meminit init_currently_empty_zone(struct zone > > *zone, > > zone_start_pfn, (zone_start_pfn + size)); > > > > zone_init_free_lists(zone);

Re: [PATCH 1/6] mm: get rid of zone_is_initialized

2017-03-30 Thread Hillf Danton
On March 30, 2017 7:55 PM Michal Hocko wrote: > > @@ -5535,9 +5535,6 @@ int __meminit init_currently_empty_zone(struct zone > *zone, > zone_start_pfn, (zone_start_pfn + size)); > > zone_init_free_lists(zone); > - zone->initialized = 1; > - > - return 0; > }

[PATCH 1/6] mm: get rid of zone_is_initialized

2017-03-30 Thread Michal Hocko
From: Michal Hocko There shouldn't be any reason to add initialized when we can tell the same thing from checking whether there are any pages spanned to the zone. Remove zone_is_initialized() and replace it by zone_is_empty which can be used for the same set of tests. This shouldn't have any vis