Received: by 2002:ad5:474a:0:0:0:0:0 with SMTP id i10csp3223162imu; Sat, 24 Nov 2018 00:28:57 -0800 (PST) X-Google-Smtp-Source: AJdET5dQHX3hk2bn8gx6GT+sco03UPlKYlIPES8yBmnRVaV2Gfr7nfwZyZYffu0eZgKI/iChOa7X X-Received: by 2002:a62:380e:: with SMTP id f14-v6mr19201772pfa.203.1543048137371; Sat, 24 Nov 2018 00:28:57 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1543048137; cv=none; d=google.com; s=arc-20160816; b=vYfZBmyk4R0mcWdJTAlhZfSbGFGdB0mYBRlZVFBVqGJV4wwx3mehnpvvmwecxCfihc bVbybYEI3QgGzl3xMT8hvJ8z2ry0Gf+8fIlNs4JUo/5w8y0wTzgJeINNYrHHv2ZpgTUB 9qrRxFNaqc6JMpCuSo7XmiYNy2JAxVcfm7mu0zgfUlXwSOZxcVx8nhsvqyXfBJLMFGeM XeoYVR+yjLEs3ajvcF/wSOdJElyMKe7fZ4Zn7szZ/7vgvAqWCWLIGyxqU8j4cAaPrq7/ UU0ZGQm8FAu8EncjlEZOJArrTTAPmCHZB48D7NmnMhbOMUZt3x7sxqpd57xasxbXleA0 qiYw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date:dkim-signature; bh=AthpPSnMAj/0Mvf97hAuWLGCUd5G7KbAhX2rxPoNpu4=; b=gq3ARBStA2n8SqzA8jzjtAHlPZHOpvQdXyK0MAB+zDnHUSp1MD/yJaJpmLckXMeNwZ wDMN3OV+Wl9EtUZewp471U5YGJEbhOtMA7NDFjw42UGgR/Z0hwImyPXSba8IR3b1OApo 5fv7OyX3y1MdldBGLkJVSkH4gdfnQGMBMgtUQlnT3ryGsukBKcYFB53fFvwWipoV3Uir MGxd0fbHqGOQbpR86kCKq2d02c5PMJsvxxIvtYJt+uMJxB9soAf8golLd52dxl3AFk2V OuDGPDJTh6Wj2UL2fdPiVrgi6rNnVTJiVH6xn07lsetxxSm6gyx7UMDShPZ0IsdKunbF Pb5Q== ARC-Authentication-Results: i=1; mx.google.com; dkim=fail header.i=@gmail.com header.s=20161025 header.b=kaBVFIuH; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id ay4si24156498plb.235.2018.11.24.00.28.42; Sat, 24 Nov 2018 00:28:57 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=fail header.i=@gmail.com header.s=20161025 header.b=kaBVFIuH; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2503850AbeKWWZD (ORCPT + 99 others); Fri, 23 Nov 2018 17:25:03 -0500 Received: from mail-wr1-f68.google.com ([209.85.221.68]:46600 "EHLO mail-wr1-f68.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S2390529AbeKWWZC (ORCPT ); Fri, 23 Nov 2018 17:25:02 -0500 Received: by mail-wr1-f68.google.com with SMTP id l9so12026425wrt.13; Fri, 23 Nov 2018 03:41:07 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=sender:date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=AthpPSnMAj/0Mvf97hAuWLGCUd5G7KbAhX2rxPoNpu4=; b=kaBVFIuHfFsSqdJcV8a70viD4Xi+MVArFPLsgmrFgio+XdVaB19LP1YwQbv6qr6GaW TerJO0nqZ704G4/gy+A8J93/jPH/lnLd7LFcAq70OLjMBIAW9UAWsFYWuZAsvtn83xUF gRXvXTaYoLFSbClkATEvZJ9NvEANKIQHdscxsrleWwwBn+wNV+pxaV0IM/7wIsxnCcKq Qq26KNXlzXFMFd5KDT47vVUzHvfssYkIhdZVFr5qVVKs/FbTDe77YEk0vZU+NZRqlR2j tcYH93MWmLEkPrzVrFt9AySLU3yFJYLxwPSVKdwWqCh0+bD5aAFqwoIcwA128QjVl5V1 p85A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:sender:date:from:to:cc:subject:message-id :references:mime-version:content-disposition:in-reply-to:user-agent; bh=AthpPSnMAj/0Mvf97hAuWLGCUd5G7KbAhX2rxPoNpu4=; b=Qhsl4vsCPX8nCOl1tPTZ0OMGJiTQ02G2fmCwhVQlUjbxoiatwYfhm0pICO/7TBVWno qpmZRkUGZodTC9L8EutGYCatxeH2rYA31OjLKlDrxAEPuEpIhFCW4XMEWa6eZ50pgh9d lenulBpQyIqwGlu7JM7OGS2FTWobIxtvQOit6g80WkYavYwTCqxyVT8ijb+S8uE9yQBu asz2j3pBvHH0LRQzb4oZj60SwDSXKY4K2YjSIT3LqAjcVzRyMOaDltgOskUiTFoEE+UB rcJ0+H8BKiiHKsjmBn8DfeUyoE7zma1PcKyD5LmPyM6w/pWjqwohUl07vGtN5A7ICkCB mB4w== X-Gm-Message-State: AA+aEWa1wOFsZAimrHbTSq+ZyZCO8TMXGwzG4BXJCNh7Sht6ZHCjG+ZC VZYi2omtuTag9UikBYp5f9M= X-Received: by 2002:a5d:63c3:: with SMTP id c3mr13239829wrw.215.1542973266055; Fri, 23 Nov 2018 03:41:06 -0800 (PST) Received: from gmail.com (2E8B0CD5.catv.pool.telekom.hu. [46.139.12.213]) by smtp.gmail.com with ESMTPSA id n8sm19157222wrw.59.2018.11.23.03.41.03 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Fri, 23 Nov 2018 03:41:05 -0800 (PST) Date: Fri, 23 Nov 2018 12:41:02 +0100 From: Ingo Molnar To: Borislav Petkov Cc: "Moger, Babu" , "tglx@linutronix.de" , "mingo@redhat.com" , "corbet@lwn.net" , "fenghua.yu@intel.com" , "reinette.chatre@intel.com" , "peterz@infradead.org" , "gregkh@linuxfoundation.org" , "davem@davemloft.net" , "akpm@linux-foundation.org" , "hpa@zytor.com" , "x86@kernel.org" , "mchehab+samsung@kernel.org" , "arnd@arndb.de" , "kstewart@linuxfoundation.org" , "pombredanne@nexb.com" , "rafael@kernel.org" , "kirill.shutemov@linux.intel.com" , "tony.luck@intel.com" , "qianyue.zj@alibaba-inc.com" , "xiaochen.shen@intel.com" , "pbonzini@redhat.com" , "Singh, Brijesh" , "Hurwitz, Sherry" , "dwmw2@infradead.org" , "Lendacky, Thomas" , "luto@kernel.org" , "joro@8bytes.org" , "jannh@google.com" , "vkuznets@redhat.com" , "rian@alum.mit.edu" , "jpoimboe@redhat.com" , "linux-kernel@vger.kernel.org" , "linux-doc@vger.kernel.org" Subject: Re: [PATCH v9 01/13] x86/resctrl: Rename and move rdt files to new directory Message-ID: <20181123114102.GB130228@gmail.com> References: <20181121202811.4492-1-babu.moger@amd.com> <20181121202811.4492-2-babu.moger@amd.com> <20181123072839.GA51597@gmail.com> <20181123082258.GA30697@zn.tnic> <20181123084117.GA130228@gmail.com> <20181123090011.GB30697@zn.tnic> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20181123090011.GB30697@zn.tnic> User-Agent: Mutt/1.9.4 (2018-02-28) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org * Borislav Petkov wrote: > On Fri, Nov 23, 2018 at 09:41:17AM +0100, Ingo Molnar wrote: > > Then at least make the directory name resource_control/, which is only > > marginally longer and a lot more readable. > > > > We really don't have to fit directly names into the 8 character DOS limit > > anymore. ;-) > > How about > > resource_ctl > > ? The thing is, thinking about this as a 'CPU resource' is really a misnomer on the conceptual level, which is why it's bothering me: RDT is not really about 'CPU resources', because registers are CPU resources, ioports and iomem are CPU resources, APICs are CPU resources and PMU events are resources - none of which is part of RDT. The key difference in RDT is that they are *shared* resources - caches really - where the ad-hoc cache sharing might be causing security and scalability problems so there's partitioning and throttling (bandwidth control) support in the hardware. Is there any other resource handled than caches by RDT or by the AMD variant? So how about "cache_control"? It's shorter and a lot closer to what the code actually does. > resource_control/ is kinda long-ish and the other names we have there > are nice and short, see below. > > BTW, while we're talking renaming, I have a patch which renames the MCE > pile and am planning to slap it in around -rc6 timeframe since we don't > have a lot of RAS commits this time around, see also the end of this > mail. It makes the naming there all nicely regular. :) That's cool - these IMHO need to be done periodically to keep overall namespace complexity low enough. (As long as can be done without breaking any ABI that is.) > 15 files changed, 14 insertions(+), 14 deletions(-) > rename arch/x86/kernel/cpu/{mcheck => mce}/Makefile (52%) > rename arch/x86/kernel/cpu/{mcheck/mce_amd.c => mce/amd.c} (99%) > rename arch/x86/kernel/cpu/{mcheck/mce-apei.c => mce/apei.c} (99%) > rename arch/x86/kernel/cpu/{mcheck/mce.c => mce/core.c} (99%) > rename arch/x86/kernel/cpu/{mcheck => mce}/dev-mcelog.c (99%) > rename arch/x86/kernel/cpu/{mcheck/mce-genpool.c => mce/genpool.c} (99%) > rename arch/x86/kernel/cpu/{mcheck/mce-inject.c => mce/inject.c} (99%) > rename arch/x86/kernel/cpu/{mcheck/mce_intel.c => mce/intel.c} (99%) > rename arch/x86/kernel/cpu/{mcheck/mce-internal.h => mce/internal.h} (100%) > rename arch/x86/kernel/cpu/{mcheck => mce}/p5.c (100%) > rename arch/x86/kernel/cpu/{mcheck/mce-severity.c => mce/severity.c} (99%) > rename arch/x86/kernel/cpu/{mcheck => mce}/therm_throt.c (100%) > rename arch/x86/kernel/cpu/{mcheck => mce}/threshold.c (100%) > rename arch/x86/kernel/cpu/{mcheck => mce}/winchip.c (100%) Standardizing around 'MCE' sounds good to me! Thanks, Ingo