Messages in this thread | | | Subject | Re: [PATCH v2 7/8] mm/memory_hotplug: Make unregister_memory_block_under_nodes() never fail | From | David Hildenbrand <> | Date | Wed, 8 May 2019 09:21:43 +0200 |
| |
>> drivers/base/node.c | 18 +++++------------- >> include/linux/node.h | 5 ++--- >> 2 files changed, 7 insertions(+), 16 deletions(-) >> >> diff --git a/drivers/base/node.c b/drivers/base/node.c >> index 04fdfa99b8bc..9be88fd05147 100644 >> --- a/drivers/base/node.c >> +++ b/drivers/base/node.c >> @@ -803,20 +803,14 @@ int register_mem_sect_under_node(struct memory_block *mem_blk, void *arg) >> >> /* >> * Unregister memory block device under all nodes that it spans. >> + * Has to be called with mem_sysfs_mutex held (due to unlinked_nodes). > > Given this comment can bitrot relative to the implementation lets > instead add an explicit: > > lockdep_assert_held(&mem_sysfs_mutex);
That would require to make the mutex non-static. Is that what you suggest, or any other alternative?
Thanks Dan!
> > With that you can add: > > Reviewed-by: Dan Williams <dan.j.williams@intel.com> >
--
Thanks,
David / dhildenb
| |