From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752516AbbEFQur (ORCPT ); Wed, 6 May 2015 12:50:47 -0400 Received: from mga03.intel.com ([134.134.136.65]:33329 "EHLO mga03.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752457AbbEFQul (ORCPT ); Wed, 6 May 2015 12:50:41 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.13,380,1427785200"; d="scan'208";a="721702377" Date: Wed, 6 May 2015 09:48:41 -0700 (PDT) From: Vikas Shivappa X-X-Sender: vikas@vshiva-Udesk To: Peter Zijlstra cc: Vikas Shivappa , Vikas Shivappa , linux-kernel@vger.kernel.org, x86@kernel.org, hpa@zytor.com, tglx@linutronix.de, mingo@kernel.org, tj@kernel.org, Matt Fleming , "Auld, Will" , peter.zijlstra@intel.com, h.peter.anvin@intel.com, "Juvva, Kanaka D" Subject: Re: [PATCH 3/7] x86/intel_rdt: Support cache bit mask for Intel CAT In-Reply-To: <20150506080914.GU23123@twins.programming.kicks-ass.net> Message-ID: References: <1430530601-16319-1-git-send-email-vikas.shivappa@linux.intel.com> <1430530601-16319-4-git-send-email-vikas.shivappa@linux.intel.com> <20150502184610.GH3007@worktop.Skamania.guest> <20150506080914.GU23123@twins.programming.kicks-ass.net> User-Agent: Alpine 2.10 (DEB 1266 2009-07-14) MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII; format=flowed Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, 6 May 2015, Peter Zijlstra wrote: > On Mon, May 04, 2015 at 10:30:15AM -0700, Vikas Shivappa wrote: >> On Sat, 2 May 2015, Peter Zijlstra wrote: >> >>> >>> There's CAT in your subject, make up your minds already on what you want >>> to call this stuff. >> >> We dont have control over the names.It is clear from the patch 0/7 where its > > If I read 0/n its _after_ I've read all the other patches. The thing is, > 0/n should not contain anything persistent. Patches should stand on > their own. > >> explained that RDT is the umbrella term and CAT is a part of it and this >> patch series is only for CAT ... It also mentions what exact section of the >> Intel manual this refers to. Is there still some lack of clarification here >> ? > > But we're not implementing an umbrella right? We're implementing Cache > QoS Enforcement (CQE aka. CAT). In some sense we are - The idea was that the same rdt cgroup would include other features in the rdt and cache allocation is one of them. Hence the cgroup name is RDT. Like Matt just commented we just found some naming issues with respect to the APIs whether to use cat or rdt. I can plan to remove the 'cat' altogether and use cache alloc as I just learnt it may be not liked because its cat(mean an animal.. in english :) ) Only reason to do it now is that we cant change the cgroup name later. > > Why confuse things with calling it random other names? > > From what I understand the whole RDT thing is the umbrella term for > Cache QoS Monitoring and Enforcement together. CQM is implemented > elsewhere, this part is only implementing CQE. > > So just call it that, calling it RDT is actively misleading, because it > explicitly does _NOT_ do the monitoring half of it. > >> If its just your disliking the term thats already known. > > I think its crazy to go CQE no CAT no RDT, but I could get over that in > time. But now it turns out you need _both_, and that's even more silly. I agree the changing of names have led to enough confusion and we can try to see that if we can make this better in coming features. I will send fixes where I will try to be more clear on names. Basically have rdt for things which would be common to cache alloc and all other features and keep cache alloc which is specific to cache alloc (like the cache bit mask is only for cache alloc).. > >