Dave Hansen <dave.hansen@xxxxxxxxxxxxxxx> writes: > On 07/09/2018 06:19 PM, Huang, Ying wrote: >> Dave Hansen <dave.hansen@xxxxxxxxxxxxxxx> writes: >> >>>> config THP_SWAP >>>> def_bool y >>>> - depends on TRANSPARENT_HUGEPAGE && ARCH_WANTS_THP_SWAP >>>> + depends on TRANSPARENT_HUGEPAGE && ARCH_WANTS_THP_SWAP && SWAP >>>> help >>> >>> This seems like a bug-fix. Is there a reason this didn't cause problems >>> up to now? >> Yes. The original code has some problem in theory, but not in practice >> because all code enclosed by >> >> #ifdef CONFIG_THP_SWAP >> #endif >> >> are in swapfile.c. But that will be not true in this patchset. > > That's great info for the changelog. Sure. Will add it. Best Regards, Huang, Ying
