On Wed, Jun 06, 2012 at 12:55:11PM +0200, Sasha Levin wrote: Could you explain in the git commit why it is unnecessary? I am pretty sure I know - it is b/c frontswap_init already does the check, but the git commit should mention it. > Signed-off-by: Sasha Levin <levinsasha928@xxxxxxxxx> > --- > mm/frontswap.c | 3 +-- > 1 files changed, 1 insertions(+), 2 deletions(-) > > diff --git a/mm/frontswap.c b/mm/frontswap.c > index f2f4685..bf99c7d 100644 > --- a/mm/frontswap.c > +++ b/mm/frontswap.c > @@ -89,8 +89,7 @@ void __frontswap_init(unsigned type) > BUG_ON(sis == NULL); > if (sis->frontswap_map == NULL) > return; > - if (frontswap_enabled) > - frontswap_ops.init(type); > + frontswap_ops.init(type); > } > EXPORT_SYMBOL(__frontswap_init); > > -- > 1.7.8.6 -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>