diff options
author | Wei Yang <richard.weiyang@gmail.com> | 2017-05-03 17:53:51 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2017-05-03 18:52:09 -0400 |
commit | 60a7a88dbb9fc9adcca78a10a3ecf36966b5a45c (patch) | |
tree | 67bc60057a09cbe1082c83cbd85ba4b9d9a27f11 /mm/sparse.c | |
parent | 822519634142cb66614514ac8ee9221868f764bb (diff) |
mm/sparse: refine usemap_size() a little
The current implementation calculates usemap_size in two steps:
* calculate number of bytes to cover these bits
* calculate number of "unsigned long" to cover these bytes
It would be more clear by:
* calculate number of "unsigned long" to cover these bits
* multiple it with sizeof(unsigned long)
This patch refine usemap_size() a little to make it more easy to
understand.
Link: http://lkml.kernel.org/r/20170310043713.96871-1-richard.weiyang@gmail.com
Signed-off-by: Wei Yang <richard.weiyang@gmail.com>
Cc: Tejun Heo <tj@kernel.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'mm/sparse.c')
-rw-r--r-- | mm/sparse.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/mm/sparse.c b/mm/sparse.c index db6bf3c97ea2..6903c8fc3085 100644 --- a/mm/sparse.c +++ b/mm/sparse.c | |||
@@ -248,10 +248,7 @@ static int __meminit sparse_init_one_section(struct mem_section *ms, | |||
248 | 248 | ||
249 | unsigned long usemap_size(void) | 249 | unsigned long usemap_size(void) |
250 | { | 250 | { |
251 | unsigned long size_bytes; | 251 | return BITS_TO_LONGS(SECTION_BLOCKFLAGS_BITS) * sizeof(unsigned long); |
252 | size_bytes = roundup(SECTION_BLOCKFLAGS_BITS, 8) / 8; | ||
253 | size_bytes = roundup(size_bytes, sizeof(unsigned long)); | ||
254 | return size_bytes; | ||
255 | } | 252 | } |
256 | 253 | ||
257 | #ifdef CONFIG_MEMORY_HOTPLUG | 254 | #ifdef CONFIG_MEMORY_HOTPLUG |