From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH v5 17/19] crypto/ccp: support cpu based md5 and sha2 family authentication algo Date: Mon, 23 Apr 2018 10:05:46 +0200 Message-ID: <1698373.c9XDPgIzTm@xps> References: <1520584520-130522-1-git-send-email-Ravi1.kumar@amd.com> <14946114.ASIPbkMQcp@xps> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: "pablo.de.lara.guarch@intel.com" , "dev@dpdk.org" , "hemant.agrawal@nxp.com" To: "Kumar, Ravi1" Return-path: Received: from out3-smtp.messagingengine.com (out3-smtp.messagingengine.com [66.111.4.27]) by dpdk.org (Postfix) with ESMTP id F090E1DBF for ; Mon, 23 Apr 2018 10:05:48 +0200 (CEST) In-Reply-To: List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 23/04/2018 08:41, Kumar, Ravi1: > >> +CONFIG_RTE_LIBRTE_PMD_CCP_CPU_AUTH=n > > > >Why introducing a compile-time option? > >Can it be a run-time option of the device? > >We must not add compile-time device option if not well justified. [...] > By default, all the crypto operations (cipher + auth) are offloaded to CCP engines. When user enables CONFIG_RTE_LIBRTE_PMD_CCP_CPU_AUTH=y, the auth operations are not offloaded to CCP and rather performed over CPU. We kept this feature as a compile time option in order to let user decide whether to run auth operations on CCP or CPU as some of the auth operations performs faster on CPU as compared to their performance on CCP. No, you do not let the user decide. The compilation options are for the packager to decide. The user can rely on pre-compiled packages and use only runtime options. That's why we forbid compile-time options for such features.