From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pg0-f72.google.com (mail-pg0-f72.google.com [74.125.83.72]) by kanga.kvack.org (Postfix) with ESMTP id CA5CF6B025F for ; Tue, 19 Sep 2017 18:41:15 -0400 (EDT) Received: by mail-pg0-f72.google.com with SMTP id d8so1711241pgt.1 for ; Tue, 19 Sep 2017 15:41:15 -0700 (PDT) Received: from mail-sor-f41.google.com (mail-sor-f41.google.com. [209.85.220.41]) by mx.google.com with SMTPS id v41sor178514plg.72.2017.09.19.15.41.13 for (Google Transport Security); Tue, 19 Sep 2017 15:41:14 -0700 (PDT) Date: Tue, 19 Sep 2017 15:41:12 -0700 (PDT) From: David Rientjes Subject: Re: [PATCH 2/2] mm: oom: show unreclaimable slab info when kernel panic In-Reply-To: <01f4cce4-d7a3-2fcb-06e0-382eff8e83e5@alibaba-inc.com> Message-ID: References: <1505759209-102539-1-git-send-email-yang.s@alibaba-inc.com> <1505759209-102539-3-git-send-email-yang.s@alibaba-inc.com> <01f4cce4-d7a3-2fcb-06e0-382eff8e83e5@alibaba-inc.com> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: owner-linux-mm@kvack.org List-ID: To: Yang Shi Cc: cl@linux.com, penberg@kernel.org, iamjoonsoo.kim@lge.com, akpm@linux-foundation.org, mhocko@kernel.org, linux-mm@kvack.org, linux-kernel@vger.kernel.org On Wed, 20 Sep 2017, Yang Shi wrote: > > > --- a/mm/slab_common.c > > > +++ b/mm/slab_common.c > > > @@ -35,6 +35,8 @@ > > > static DECLARE_WORK(slab_caches_to_rcu_destroy_work, > > > slab_caches_to_rcu_destroy_workfn); > > > +#define K(x) ((x)/1024) > > > + > > > /* > > > * Set of flags that will prevent slab merging > > > */ > > > @@ -1272,6 +1274,34 @@ static int slab_show(struct seq_file *m, void *p) > > > return 0; > > > } > > > +void show_unreclaimable_slab() > > > +{ > > > + struct kmem_cache *s = NULL; > > > + struct slabinfo sinfo; > > > + > > > + memset(&sinfo, 0, sizeof(sinfo)); > > > + > > > + printk("Unreclaimable slabs:\n"); > > > + > > > + /* > > > + * Here acquiring slab_mutex is unnecessary since we don't prefer to > > > + * get sleep in oom path right before kernel panic, and avoid race > > > condition. > > > + * Since it is already oom, so there should be not any big allocation > > > + * which could change the statistics significantly. > > > + */ > > > + list_for_each_entry(s, &slab_caches, list) { > > > + if (!is_root_cache(s)) > > > + continue; > > > + > > > + get_slabinfo(s, &sinfo); > > > + > > > + if (!is_reclaimable(s) && sinfo.num_objs > 0) > > > + printk("%-17s %luKB\n", cache_name(s), > > > K(sinfo.num_objs * s->size)); > > > + } > > > > I like this, but could we be even more helpful by giving the user more > > information from sinfo beyond just the total size of objects allocated? > > Sure, we definitely can. But, the question is what info is helpful to users to > diagnose oom other than the size. > > I think of the below: > - the number of active objs, the number of total objs, the percentage > of active objs per cache > - the number of active slabs, the number of total slabs, the > percentage of active slabs per cache > > Anything else? > Right now it's a useful tool to find out what unreclaimable slab is sitting around that is causing the system to run out of memory. If we knew how much of this slab is actually in use vs free, it can determine if its stranding or if there's a bug in the slab allocator itself. We wouldn't need percentages, we can calculate that directly from the data if necessary. -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org