From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by dpdk.org (Postfix) with ESMTP id 347EF56AB for ; Thu, 7 May 2015 23:11:19 +0200 (CEST) Received: from orsmga001.jf.intel.com ([10.7.209.18]) by orsmga103.jf.intel.com with ESMTP; 07 May 2015 14:11:18 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.13,386,1427785200"; d="scan'208";a="691600297" Received: from orsmsx104.amr.corp.intel.com ([10.22.225.131]) by orsmga001.jf.intel.com with ESMTP; 07 May 2015 14:11:17 -0700 Received: from fmsmsx119.amr.corp.intel.com (10.18.124.207) by ORSMSX104.amr.corp.intel.com (10.22.225.131) with Microsoft SMTP Server (TLS) id 14.3.224.2; Thu, 7 May 2015 14:11:16 -0700 Received: from fmsmsx113.amr.corp.intel.com ([169.254.13.26]) by FMSMSX119.amr.corp.intel.com ([169.254.14.249]) with mapi id 14.03.0224.002; Thu, 7 May 2015 14:11:16 -0700 From: "Wiles, Keith" To: "Richardson, Bruce" , Marc Sune Thread-Topic: [dpdk-dev] [RFC PATCH 0/2] Move PMDs out of lib directory Thread-Index: AQHQiNu6eQrwNh7biEqu/oZQVO/0sZ1xHUYAgAAFWAD//+BcgA== Date: Thu, 7 May 2015 21:11:15 +0000 Message-ID: References: <1431012951-6423-1-git-send-email-bruce.richardson@intel.com> <554B8890.3070502@bisdn.de> <20150507160428.GB5972@bricha3-MOBL3> In-Reply-To: <20150507160428.GB5972@bricha3-MOBL3> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.3.128.16] Content-Type: text/plain; charset="us-ascii" Content-ID: <831DDA295005F24EBEA5D3F3F009FD98@intel.com> Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Cc: "dev@dpdk.org" Subject: Re: [dpdk-dev] [RFC PATCH 0/2] Move PMDs out of lib directory X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 07 May 2015 21:11:20 -0000 On 5/7/15, 9:04 AM, "Bruce Richardson" wrote: >On Thu, May 07, 2015 at 05:45:20PM +0200, Marc Sune wrote: >>=20 >>=20 >> On 07/05/15 17:35, Bruce Richardson wrote: >> >The "lib" directory is getting very crowded, with both general libs and >> >poll mode drivers in it. This patch set proposes to move the PMDs out >>of the >> >lib folder and to put them in a separate "pmds" folder. This should >>help >> >with code browse-ability as the number of libs, and pmds increases. >> > >> >Comments or objections? >> > >> >Bruce Richardson (2): >> > pmds: Use relative rather than absolute paths >> > pmds: move pmds from lib to separate pmd dir >> > >> > create mode 100644 pmds/librte_pmd_xenvirt/rte_mempool_gntalloc.c >> > create mode 100644 pmds/librte_pmd_xenvirt/rte_xen_lib.c >> > create mode 100644 pmds/librte_pmd_xenvirt/rte_xen_lib.h >> > create mode 100644 pmds/librte_pmd_xenvirt/virtio_logs.h >> > create mode 100644 pmds/librte_pmd_xenvirt/virtqueue.h >> > >>=20 >> But at the end they are also libraries. What about something like: >>=20 >> * libs/core <=3D fundamental libraries (eal, mbuf rings...) >> * libs/pmds <=3D all pmds >>=20 >> And other feature-group oriented, higher level lib, directories (not >>sure >> right now how to better classify them right now): >> * libs/processing <=3D packet processing >> * libs/utils >> ... >>=20 >Yes, they are all just libs, so we could make "pmds" be a sub-dir of the >lib >folder. I prefer the shorter path myself, but if others want a multi-level >hierarchy it's no big deal. I like the dpdk/pmds as dpdk/lib/pmds is a bit longer, but I also see if we want to move the pmds to other repo(s) in the future it would be easier (I think) to have the subtree at the top. To me pmds are not really libraries as I think of libc or libcrypto or something along that path. The PMDs need to be plug able and they maybe more like loadable modules then libraries in the future. > >For the other libs, I'm not sure we need to split them up, and I also >think >that trying to divide them into categories - and what those categories >should >be could - cause endless discussion. However, maybe I'm overly >pessimistic... :-) I agree with Bruce here we just need the PMDS split out for now. > >/Bruce >