From: Yu Zhao <yuzhao@google.com> To: "Kirill A. Shutemov" <kirill@shutemov.name> Cc: Christoph Lameter <cl@linux.com>, Pekka Enberg <penberg@kernel.org>, David Rientjes <rientjes@google.com>, Joonsoo Kim <iamjoonsoo.kim@lge.com>, Andrew Morton <akpm@linux-foundation.org>, Tetsuo Handa <penguin-kernel@i-love.sakura.ne.jp>, linux-mm@kvack.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2 1/4] mm: correct mask size for slub page->objects Date: Thu, 12 Sep 2019 15:11:14 -0600 Message-ID: <20190912211114.GA146974@google.com> (raw) In-Reply-To: <20190912094035.vkqnj24bwh33yvia@box> On Thu, Sep 12, 2019 at 12:40:35PM +0300, Kirill A. Shutemov wrote: > On Wed, Sep 11, 2019 at 08:31:08PM -0600, Yu Zhao wrote: > > Mask of slub objects per page shouldn't be larger than what > > page->objects can hold. > > > > It requires more than 2^15 objects to hit the problem, and I don't > > think anybody would. It'd be nice to have the mask fixed, but not > > really worth cc'ing the stable. > > > > Fixes: 50d5c41cd151 ("slub: Do not use frozen page flag but a bit in the page counters") > > Signed-off-by: Yu Zhao <yuzhao@google.com> > > I don't think the patch fixes anything. Technically it does. It makes no sense for a mask to have more bits than the variable that holds the masked value. I had to look up the commit history to find out why and go through the code to make sure it doesn't actually cause any problem. My hope is that nobody else would have to go through the same trouble. > Yes, we have one spare bit between order and number of object that is not > used and always zero. So what? > > I can imagine for some microarchitecures accessing higher 16 bits of int > is cheaper than shifting by 15. Well, I highly doubt the inconsistency is intended for such optimization, even it existed.
next prev parent reply index Thread overview: 40+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-09-09 6:10 [PATCH] mm: avoid slub allocation while holding list_lock Yu Zhao 2019-09-09 16:00 ` Kirill A. Shutemov [not found] ` <e5e25aa3-651d-92b4-ac82-c5011c66a7cb@I-love.SAKURA.ne.jp> 2019-09-09 21:39 ` Yu Zhao [not found] ` <201909100141.x8A1fVdu048305@www262.sakura.ne.jp> 2019-09-10 2:16 ` Yu Zhao 2019-09-10 9:16 ` Kirill A. Shutemov 2019-09-11 14:13 ` Andrew Morton 2019-09-12 0:29 ` [PATCH 1/3] mm: correct mask size for slub page->objects Yu Zhao 2019-09-12 0:29 ` [PATCH 2/3] mm: avoid slub allocation while holding list_lock Yu Zhao 2019-09-12 0:44 ` Kirill A. Shutemov 2019-09-12 1:31 ` Yu Zhao 2019-09-12 2:31 ` [PATCH v2 1/4] mm: correct mask size for slub page->objects Yu Zhao 2019-09-12 2:31 ` [PATCH v2 2/4] mm: clean up validate_slab() Yu Zhao 2019-09-12 9:46 ` Kirill A. Shutemov 2019-09-12 2:31 ` [PATCH v2 3/4] mm: avoid slub allocation while holding list_lock Yu Zhao 2019-09-12 10:04 ` Kirill A. Shutemov 2019-09-12 2:31 ` [PATCH v2 4/4] mm: lock slub page when listing objects Yu Zhao 2019-09-12 10:06 ` Kirill A. Shutemov 2019-09-12 21:12 ` Yu Zhao 2019-09-13 14:58 ` Christopher Lameter 2019-09-12 9:40 ` [PATCH v2 1/4] mm: correct mask size for slub page->objects Kirill A. Shutemov 2019-09-12 21:11 ` Yu Zhao [this message] 2019-09-12 22:03 ` Kirill A. Shutemov 2019-09-14 0:07 ` [PATCH v3 1/2] mm: clean up validate_slab() Yu Zhao 2019-09-14 0:07 ` [PATCH v3 2/2] mm: avoid slub allocation while holding list_lock Yu Zhao 2019-09-16 8:39 ` [PATCH v3 1/2] mm: clean up validate_slab() Kirill A. Shutemov 2019-11-08 19:39 ` [PATCH v4 " Yu Zhao 2019-11-08 19:39 ` [PATCH v4 2/2] mm: avoid slub allocation while holding list_lock Yu Zhao 2019-11-09 20:52 ` Christopher Lameter 2019-11-09 23:01 ` Yu Zhao 2019-11-09 23:16 ` Christopher Lameter 2019-11-10 18:47 ` Yu Zhao 2019-11-11 15:47 ` Christopher Lameter 2019-11-11 15:55 ` [FIX] slub: Remove kmalloc under list_lock from list_slab_objects() V2 Christopher Lameter 2019-11-30 23:09 ` Andrew Morton 2019-12-02 15:12 ` Christopher Lameter 2019-12-07 22:03 ` Yu Zhao 2020-01-10 14:11 ` Vlastimil Babka [not found] ` <e0ed44ae-8dae-e8db-9d14-2b09b239af8e@i-love.sakura.ne.jp> 2020-01-13 1:34 ` Christopher Lameter 2019-11-11 18:15 ` [PATCH v4 2/2] mm: avoid slub allocation while holding list_lock Shakeel Butt 2019-09-12 0:29 ` [PATCH 3/3] mm: lock slub page when listing objects Yu Zhao
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20190912211114.GA146974@google.com \ --to=yuzhao@google.com \ --cc=akpm@linux-foundation.org \ --cc=cl@linux.com \ --cc=iamjoonsoo.kim@lge.com \ --cc=kirill@shutemov.name \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-mm@kvack.org \ --cc=penberg@kernel.org \ --cc=penguin-kernel@i-love.sakura.ne.jp \ --cc=rientjes@google.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
Linux-mm Archive on lore.kernel.org Archives are clonable: git clone --mirror https://lore.kernel.org/linux-mm/0 linux-mm/git/0.git # If you have public-inbox 1.1+ installed, you may # initialize and index your mirror using the following commands: public-inbox-init -V2 linux-mm linux-mm/ https://lore.kernel.org/linux-mm \ linux-mm@kvack.org public-inbox-index linux-mm Example config snippet for mirrors Newsgroup available over NNTP: nntp://nntp.lore.kernel.org/org.kvack.linux-mm AGPL code for this site: git clone https://public-inbox.org/public-inbox.git