From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jerin Jacob Subject: Re: [PATCH] Patch introducing API to read/write Intel Architecture Model Specific Registers (MSR), rte_msr_read and rte_msr_write functions. Date: Wed, 6 Jan 2016 23:03:51 +0530 Message-ID: <20160106173348.GA23478@localhost.localdomain> References: <1450354363-102022-1-git-send-email-wojciechx.andralojc@intel.com> <3FD2C4106EAA5C43838688C653B6E2AFDB8422@IRSMSX103.ger.corp.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Cc: "dev@dpdk.org" To: "Andralojc, WojciechX" Return-path: Received: from na01-bl2-obe.outbound.protection.outlook.com (mail-bl2on0055.outbound.protection.outlook.com [65.55.169.55]) by dpdk.org (Postfix) with ESMTP id ECA9695C2 for ; Wed, 6 Jan 2016 18:34:16 +0100 (CET) Content-Disposition: inline In-Reply-To: <3FD2C4106EAA5C43838688C653B6E2AFDB8422@IRSMSX103.ger.corp.intel.com> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Wed, Jan 06, 2016 at 11:47:28AM +0000, Andralojc, WojciechX wrote: > > From: Andralojc, WojciechX > > Sent: Thursday, December 17, 2015 12:13 PM > > To: dev@dpdk.org > > Cc: Andralojc, WojciechX > > Subject: [PATCH] Patch introducing API to read/write Intel Architecture Model > > Specific Registers (MSR), rte_msr_read and rte_msr_write functions. > > > > There is work in progress to implement Intel Cache Allocation Technology (CAT) > > support in DPDK, this technology is programmed through MSRs. > > In the future it will be possible to program CAT through Linux cgroups and DPDK > > CAT implementation will take advantage of it. > > > > MSR R/W's are privileged ring 0 operations and they must be done in kernel > > space. For this reason implementation utilizes Linux MSR driver. > > > > Signed-off-by: Wojciech Andralojc > > I've got suggestion offline that as MSRs are IA specific, > I should not give the dummy APIs for the other arches > and move MSR access functions into the EAL specific APIs > or some place more arch specific. > Do you find submitted MSR patch OK? > or do you agree with the above feedback and patch should be re-worked? +1 IMO, No need to expose this function as EAL as other archs can't implement this.I think, a IA specific function under lib/librte_eal/common/include/arch/x86/ and removing rte_* from internal architecture functions looks more appropriate Jerin > I am looking forward to your feedback > > Thank you! > > Wojciech Andralojc > -------------------------------------------------------------- > Intel Research and Development Ireland Limited > Registered in Ireland > Registered Office: Collinstown Industrial Park, Leixlip, County Kildare > Registered Number: 308263 > > > This e-mail and any attachments may contain confidential material for the sole > use of the intended recipient(s). Any review or distribution by others is > strictly prohibited. If you are not the intended recipient, please contact the > sender and delete all copies. >