From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wi0-f179.google.com (mail-wi0-f179.google.com [209.85.212.179]) by dpdk.org (Postfix) with ESMTP id 13B7AC410 for ; Mon, 11 May 2015 11:47:58 +0200 (CEST) Received: by wizk4 with SMTP id k4so98290388wiz.1 for ; Mon, 11 May 2015 02:47:57 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id:organization :user-agent:in-reply-to:references:mime-version :content-transfer-encoding:content-type; bh=UhCUhkrVyVxXpSkavjk8J/jVsgt97HeqdtTPCPkhS4M=; b=Y/sJlbitlMUSSg5+fpQg730f2TslDVfIz8IbUxCllRouDdlwUkOErjmb+2efmcp2iz ZkmcD0jBwn1ww8JVcCiXCf5XYVCras0j1lUXNdEgqoDfCHnLe1fOlrv3H0LnJb5I25Ld SxxaGHo+o9QckncI5EaMEbSiDREkPucjo+xVlfL+LXmLhFjZwJUBLKpw23qcm80D4gSE qGM+/rcXX+RyBAB/iIxLFxs5LqyFZ7D5Ym8h/kNz4GqJz6jZfu/ROYJhL3+9xOGd1SN4 4sLO3M1k5XGwsEdRZS7rpxXkRQdiiRJYUDxFFpm+hdEI4ZqOqB6x2wrmigb7Xm7Tc8n0 ca6w== X-Gm-Message-State: ALoCoQksNvIAniWMLnkjJ2O0N5rmDxoNbqiEhc8l1U0y5PUkGxOpmyvCnam4UBZsDLlsMM4Bj4CZ X-Received: by 10.180.24.65 with SMTP id s1mr17436158wif.66.1431337677903; Mon, 11 May 2015 02:47:57 -0700 (PDT) Received: from xps13.localnet (136-92-190-109.dsl.ovh.fr. [109.190.92.136]) by mx.google.com with ESMTPSA id l3sm12399125wik.16.2015.05.11.02.47.56 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Mon, 11 May 2015 02:47:57 -0700 (PDT) From: Thomas Monjalon To: Bruce Richardson Date: Mon, 11 May 2015 11:47:17 +0200 Message-ID: <1471925.FRZW63X2uB@xps13> Organization: 6WIND User-Agent: KMail/4.14.7 (Linux/4.0.1-1-ARCH; KDE/4.14.7; x86_64; ; ) In-Reply-To: <1431012951-6423-1-git-send-email-bruce.richardson@intel.com> References: <1431012951-6423-1-git-send-email-bruce.richardson@intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" 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: Mon, 11 May 2015 09:47:58 -0000 2015-05-07 16:35, Bruce Richardson: > 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? When someone is looking for a driver implementation and check what is done in DPDK, it will be easier to open a directory named "drivers" rather than "pmds". I agree that they are not really libs as they are used as plugins. So they deserve a separate directory at the top level. Moreover, I suspect that the dataplane managed by DPDK can be extended to crypto and storage devices in the near future. So, I would suggest drivers/net drivers/crypto drivers/storage This kind of split could help to clearly define the responsibilities of some new git subtrees. Don't you think we could also remove the librte_pmd_ prefix for these new directories? Ultimately, I'd like to see the subdirectories for e1000, ixgbe and i40e renamed to base/.