From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759854AbcAKNjz (ORCPT ); Mon, 11 Jan 2016 08:39:55 -0500 Received: from mail-wm0-f52.google.com ([74.125.82.52]:35033 "EHLO mail-wm0-f52.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1759808AbcAKNjy (ORCPT ); Mon, 11 Jan 2016 08:39:54 -0500 Date: Mon, 11 Jan 2016 14:39:52 +0100 From: Michal Hocko To: Xishi Qiu Cc: cl@linux.com, Pekka Enberg , David Rientjes , iamjoonsoo.kim@lge.com, Andrew Morton , zhong jiang , Linux MM , LKML Subject: Re: [PATCH] mm: add ratio in slabinfo print Message-ID: <20160111133950.GC27317@dhcp22.suse.cz> References: <56932791.3080502@huawei.com> <20160111122553.GB27317@dhcp22.suse.cz> <5693AAD5.6090101@huawei.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <5693AAD5.6090101@huawei.com> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon 11-01-16 21:15:01, Xishi Qiu wrote: > On 2016/1/11 20:25, Michal Hocko wrote: > > > On Mon 11-01-16 11:54:57, Xishi Qiu wrote: > >> Add ratio(active_objs/num_objs) in /proc/slabinfo, it is used to show > >> the availability factor in each slab. > > > > What is the reason to add such a new value when it can be trivially > > calculated from the userspace? > > > > Besides that such a change would break existing parsers no? > > Oh, maybe it is. > > How about adjustment the format because some names are too long? Parsers should be clever enough to process white spaces properly but there is no guarantee this will be the case. A more important question is whether it really makes sense to change this in the first place. What would be the benefit? Somehow nicer output? Does this justify a potential breakage of tool processing this file? To me this all sounds like such a change is not worth it. -- Michal Hocko SUSE Labs