From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH v2 09/33] crypto/octeontx: adds symmetric capabilities Date: Mon, 22 Oct 2018 08:51:42 +0200 Message-ID: <3230031.dhaMjhjEo9@xps> References: <1528476325-15585-1-git-send-email-anoob.joseph@caviumnetworks.com> <348A99DA5F5B7549AA880327E580B4358965228A@IRSMSX101.ger.corp.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: "Trahe, Fiona" , "dev@dpdk.org" , Akhil Goyal , "De Lara Guarch, Pablo" , "Murthy, Nidadavolu" , "Jacob, Jerin" , "Athreya, Narayana Prasad" , "Dwivedi, Ankur" , "Dabilpuram, Nithin" , "Jayaraman, Ragothaman" , "Srinivasan, Srisivasubramanian" , "Tejasree, Kondoj" To: "Joseph, Anoob" Return-path: Received: from out3-smtp.messagingengine.com (out3-smtp.messagingengine.com [66.111.4.27]) by dpdk.org (Postfix) with ESMTP id BF5AA25B3 for ; Mon, 22 Oct 2018 08:51:45 +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" 22/10/2018 05:49, Joseph, Anoob: > Hi Fiona, > > I do agree that your solution seems to be a neat way for organizing capabilities. But Akhil & Thomas were against that idea and we had to come up with one array with all capabilities. This would not scale well when we start supporting devices with varying capabilities. > > If your plan is to follow the same approach for asym support, maybe we will also follow suit and submit the required patches. > > @Akhil, Thomas, thoughts? Generally speaking, macros are bad. Why cannot you just write functions? I don't understand your issue.