From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-qk0-f198.google.com (mail-qk0-f198.google.com [209.85.220.198]) by kanga.kvack.org (Postfix) with ESMTP id AD7C06B0273 for ; Wed, 13 Jun 2018 14:53:50 -0400 (EDT) Received: by mail-qk0-f198.google.com with SMTP id g2-v6so2830846qkm.13 for ; Wed, 13 Jun 2018 11:53:50 -0700 (PDT) Received: from mx1.redhat.com (mx3-rdu2.redhat.com. [66.187.233.73]) by mx.google.com with ESMTPS id u1-v6si3202663qka.174.2018.06.13.11.53.49 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Wed, 13 Jun 2018 11:53:49 -0700 (PDT) Date: Wed, 13 Jun 2018 14:53:47 -0400 (EDT) From: Mikulas Patocka Subject: Re: [PATCH RESEND] slab: introduce the flag SLAB_MINIMIZE_WASTE In-Reply-To: <20180613181654.GA24315@infradead.org> Message-ID: References: <20180613181654.GA24315@infradead.org> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: owner-linux-mm@kvack.org List-ID: To: Christoph Hellwig Cc: Christopher Lameter , Mike Snitzer , Vlastimil Babka , Matthew Wilcox , Pekka Enberg , linux-mm@kvack.org, dm-devel@redhat.com, David Rientjes , Joonsoo Kim , Andrew Morton , linux-kernel@vger.kernel.org On Wed, 13 Jun 2018, Christoph Hellwig wrote: > On Wed, Jun 13, 2018 at 01:01:22PM -0400, Mikulas Patocka wrote: > > Hi > > > > I'd like to ask about this patch - will you commit it, or do you want to > > make some more changes to it? > > How about you resend it with the series adding an actual user once > ready? I haven't actually seen patches using it posted on any list yet. dm-bufio is already using it. Starting with the kernel 4.17 (f51f2e0a7fb1 - "dm bufio: support non-power-of-two block sizes"), dm-bufio has the capability to use non-power-of-two buffers. It uses slab cache for its buffers - so we would like to have this slab optimization - to avoid excessive memory wasting. Originally, the slub patch used a new flag SLAB_MINIMIZE_WASTE, but after a suggestion from others, I reworked the patch so that it minimizes waste of all slub caches and doesn't need an extra flag to activate. Mikulas