Re: [PATCH] x86: mm: clean up probe_memory_block_size()

2015-11-26 Thread Seth Jennings
On Thu, Nov 26, 2015 at 10:12:01AM +0100, Ingo Molnar wrote: > > * Seth Jennings wrote: > > > The cumulative effect of bdee237c and 982792c7 is some pretty convoluted > > code. This commit has no (intended) functional change; just seeks to > > simplify and make the code more understandable. > >

Re: [PATCH] x86: mm: clean up probe_memory_block_size()

2015-11-26 Thread Ingo Molnar
* Seth Jennings wrote: > The cumulative effect of bdee237c and 982792c7 is some pretty convoluted > code. This commit has no (intended) functional change; just seeks to > simplify and make the code more understandable. > > The whole section with the "tail size" doesn't seem to be reachable, >

[PATCH] x86: mm: clean up probe_memory_block_size()

2015-11-24 Thread Seth Jennings
The cumulative effect of bdee237c and 982792c7 is some pretty convoluted code. This commit has no (intended) functional change; just seeks to simplify and make the code more understandable. The whole section with the "tail size" doesn't seem to be reachable, since both the >= 64GB and < 64GB case