From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) by dpdk.org (Postfix) with ESMTP id 486881B44C for ; Wed, 20 Jun 2018 09:26:02 +0200 (CEST) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by orsmga102.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 20 Jun 2018 00:26:01 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.51,246,1526367600"; d="scan'208";a="65677715" Received: from irsmsx151.ger.corp.intel.com ([163.33.192.59]) by fmsmga001.fm.intel.com with ESMTP; 20 Jun 2018 00:25:59 -0700 Received: from irsmsx108.ger.corp.intel.com ([169.254.11.139]) by IRSMSX151.ger.corp.intel.com ([169.254.4.216]) with mapi id 14.03.0319.002; Wed, 20 Jun 2018 08:25:58 +0100 From: "De Lara Guarch, Pablo" To: "Verma, Shally" CC: "Trahe, Fiona" , "dev@dpdk.org" , "Athreya, Narayana Prasad" , "Challa, Mahipal" , "Gupta, Ashish" , "Sahu, Sunila" Thread-Topic: [PATCH v1 7/7] drivers/compress: add meson.build support Thread-Index: AQHT/LkXWvD1C4ljTkirijIKQicpNaRoPrdAgABvjQCAACa7AA== Date: Wed, 20 Jun 2018 07:25:58 +0000 Message-ID: References: <1528194913-25893-1-git-send-email-shally.verma@caviumnetworks.com> <1528194913-25893-8-git-send-email-shally.verma@caviumnetworks.com> In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiYTI4M2UwMGItNWU4My00NzVhLWFjMTUtZDBiYWVlMmZmYzcxIiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX05UIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE3LjEwLjE4MDQuNDkiLCJUcnVzdGVkTGFiZWxIYXNoIjoidmt4blp6UUlTYWpGcHNsNXZUXC91bjhFTlFWS2RZOWpFeXM1bGRQXC80XC9QRjlCWkVPdGNxZVpnWXRueHhZdTM1QSJ9 x-ctpclassification: CTP_NT dlp-product: dlpe-windows dlp-version: 11.0.200.100 dlp-reaction: no-action x-originating-ip: [163.33.239.182] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Subject: Re: [dpdk-dev] [PATCH v1 7/7] drivers/compress: add meson.build support X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 20 Jun 2018 07:26:02 -0000 Hi Shally, > -----Original Message----- > From: Verma, Shally [mailto:Shally.Verma@cavium.com] > Sent: Wednesday, June 20, 2018 7:02 AM > To: De Lara Guarch, Pablo > Cc: Trahe, Fiona ; dev@dpdk.org; Athreya, Narayana > Prasad ; Challa, Mahipal > ; Gupta, Ashish ; > Sahu, Sunila > Subject: RE: [PATCH v1 7/7] drivers/compress: add meson.build support >=20 >=20 >=20 > >-----Original Message----- > >From: De Lara Guarch, Pablo [mailto:pablo.de.lara.guarch@intel.com] > >Sent: 20 June 2018 04:00 > >To: Verma, Shally > >Cc: Trahe, Fiona ; dev@dpdk.org; Athreya, > >Narayana Prasad ; > >Challa, Mahipal ; Gupta, Ashish > >; Sahu, Sunila > >Subject: RE: [PATCH v1 7/7] drivers/compress: add meson.build support > > > >External Email > > > >> -----Original Message----- > >> From: Shally Verma [mailto:shally.verma@caviumnetworks.com] > >> Sent: Tuesday, June 5, 2018 11:35 AM > >> To: De Lara Guarch, Pablo > >> Cc: Trahe, Fiona ; dev@dpdk.org; > >> pathreya@caviumnetworks.com; mchalla@caviumnetworks.com; Ashish > Gupta > >> ; Sunila Sahu > >> > >> Subject: [PATCH v1 7/7] drivers/compress: add meson.build support > >> > >> Add octeontx in drivers/compress/meson.build. > > > >This should be added in patch 1. > > > >> > >> Also change the config and drv format flags to append _compress to > >> use compression specific format for global filenames. This also > >> require change in compression PMDs to rename their version.map files > >> as rte_pmd_ >> name>_compress.map ex. Isal should rename as > >> rte_pmd_isal_compress_version.map. > >> > >> Currently the drivers/compress/meson.build use the generic config > >> names with no compress specific specifier. This breaks meson build > >> when vendor uses same name for different PMDs, where all belongs to sa= me > platform. > >> Ex. drivers/net/octeontx, drivers/event/octeontx, > >> drivers/compress/octeontx where, net, event and compress carry PMD > >> for eth, event and compression devices for Octeontx platform. Thus, > >> differentiate globally used files using module specific specifier ex. > >> rte_pmd_octeontx_compress_version.map > >> and rte_pmd_octeontx_event_version.map > > > >I think instead of forcing all the PMDs to use that suffix, if there > >are drivers with the same PMD, they can add the "compres" suffix, withou= t > imposing it to others. > > > [Shally] Meson build force us to use then directory name as octeontx_comp= ress > where as we want to keep it as compress/octeontx (on the similar lines of > net/octeontx, event/octeontx). Also its already under compress, so having= it as > compress/octeontx_compress looks unnecessary, For null crypto, the directory is called crypto/null and it is built with m= eson (and there is also a net/null). Maybe doing something like that will work? If it doesn't, then you can add this, but make sure that you fix the isal export map name in the Makefile too. Thanks, Pablo >=20 > Thanks > Shally