Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752305AbaGaQf5 (ORCPT ); Thu, 31 Jul 2014 12:35:57 -0400 Received: from mx1.redhat.com ([209.132.183.28]:42831 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752234AbaGaQfy (ORCPT ); Thu, 31 Jul 2014 12:35:54 -0400 Date: Thu, 31 Jul 2014 12:35:46 -0400 (EDT) From: Mikulas Patocka X-X-Sender: mpatocka@file01.intranet.prod.int.rdu2.redhat.com To: device-mapper development cc: Alasdair Kergon , Mike Snitzer , Vladimir Davydov , linux-kernel@vger.kernel.org, linux-raid@vger.kernel.org, linux-mm@kvack.org, Dave Chinner , Greg Thelen , Andrew Morton Subject: Re: [dm-devel] [PATCH] dm bufio: fully initialize shrinker In-Reply-To: <1406822839-2423-1-git-send-email-gthelen@google.com> Message-ID: References: <1406822839-2423-1-git-send-email-gthelen@google.com> User-Agent: Alpine 2.02 (LRH 1266 2009-07-14) MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, 31 Jul 2014, Greg Thelen wrote: > 1d3d4437eae1 ("vmscan: per-node deferred work") added a flags field to > struct shrinker assuming that all shrinkers were zero filled. The dm > bufio shrinker is not zero filled, which leaves arbitrary kmalloc() data > in flags. So far the only defined flags bit is SHRINKER_NUMA_AWARE. > But there are proposed patches which add other bits to shrinker.flags > (e.g. memcg awareness). > > Rather than simply initializing the shrinker, this patch uses kzalloc() > when allocating the dm_bufio_client to ensure that the embedded shrinker > and any other similar structures are zeroed. > > This fixes theoretical over aggressive shrinking of dm bufio objects. > If the uninitialized dm_bufio_client.shrinker.flags contains > SHRINKER_NUMA_AWARE then shrink_slab() would call the dm shrinker for > each numa node rather than just once. This has been broken since 3.12. > > Signed-off-by: Greg Thelen Acked-by: Mikulas Patocka Cc: stable@vger.kernel.org #v3.12 > --- > drivers/md/dm-bufio.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/md/dm-bufio.c b/drivers/md/dm-bufio.c > index 4e84095833db..d724459860d9 100644 > --- a/drivers/md/dm-bufio.c > +++ b/drivers/md/dm-bufio.c > @@ -1541,7 +1541,7 @@ struct dm_bufio_client *dm_bufio_client_create(struct block_device *bdev, unsign > BUG_ON(block_size < 1 << SECTOR_SHIFT || > (block_size & (block_size - 1))); > > - c = kmalloc(sizeof(*c), GFP_KERNEL); > + c = kzalloc(sizeof(*c), GFP_KERNEL); > if (!c) { > r = -ENOMEM; > goto bad_client; > -- > 2.0.0.526.g5318336 > > -- > dm-devel mailing list > dm-devel@redhat.com > https://www.redhat.com/mailman/listinfo/dm-devel > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/