Message ID | 20180815144219.6014-5-osalvador@techadventures.net (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | Refactoring for remove_memory_section/unregister_mem_sect_under_nodes | expand |
On 15.08.2018 16:42, Oscar Salvador wrote: > From: Oscar Salvador <osalvador@suse.de> > > We are getting the nid from the pages that are not yet removed, > but a node can only be offline when its memory/cpu's have been removed. > Therefore, we know that the node is still online. > > Signed-off-by: Oscar Salvador <osalvador@suse.de> > --- > drivers/base/node.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/drivers/base/node.c b/drivers/base/node.c > index 81b27b5b1f15..b23769e4fcbb 100644 > --- a/drivers/base/node.c > +++ b/drivers/base/node.c > @@ -465,8 +465,6 @@ void unregister_mem_sect_under_nodes(struct memory_block *mem_blk, > > if (nid < 0) > continue; > - if (!node_online(nid)) > - continue; > /* > * It is possible that NODEMASK_ALLOC fails due to memory > * pressure. > Sounds reasonable to me Reviewed-by: David Hildenbrand <david@redhat.com>
On 18-08-15 16:42:19, Oscar Salvador wrote: > From: Oscar Salvador <osalvador@suse.de> > > We are getting the nid from the pages that are not yet removed, > but a node can only be offline when its memory/cpu's have been removed. > Therefore, we know that the node is still online. Reviewed-by: Pavel Tatashin <pavel.tatashin@microsoft.com> > > Signed-off-by: Oscar Salvador <osalvador@suse.de> > --- > drivers/base/node.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/drivers/base/node.c b/drivers/base/node.c > index 81b27b5b1f15..b23769e4fcbb 100644 > --- a/drivers/base/node.c > +++ b/drivers/base/node.c > @@ -465,8 +465,6 @@ void unregister_mem_sect_under_nodes(struct memory_block *mem_blk, > > if (nid < 0) > continue; > - if (!node_online(nid)) > - continue; > /* > * It is possible that NODEMASK_ALLOC fails due to memory > * pressure. > -- > 2.13.6 >
diff --git a/drivers/base/node.c b/drivers/base/node.c index 81b27b5b1f15..b23769e4fcbb 100644 --- a/drivers/base/node.c +++ b/drivers/base/node.c @@ -465,8 +465,6 @@ void unregister_mem_sect_under_nodes(struct memory_block *mem_blk, if (nid < 0) continue; - if (!node_online(nid)) - continue; /* * It is possible that NODEMASK_ALLOC fails due to memory * pressure.