On 09/17/2018 07:29 AM, Thomas Gleixner wrote: > Avoid the extra variable and gotos by splitting the function into the > actual algorithm and a callable function which contains the lock > protection. > > Rename it to should_split_large_page() while at it so the return values make > actually sense. > > Clean up the code flow, comments and general whitespace damage while at it. No > functional change. > > Signed-off-by: Thomas Gleixner <[email protected]>
This one is getting a wee bit hard to follow from the diff, but the cleanup and rename looks nice. Reviewed-by: Dave Hansen <[email protected]>
