From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by dpdk.org (Postfix) with ESMTP id 811592BE0 for ; Mon, 22 Feb 2016 17:44:11 +0100 (CET) Received: from int-mx14.intmail.prod.int.phx2.redhat.com (int-mx14.intmail.prod.int.phx2.redhat.com [10.5.11.27]) by mx1.redhat.com (Postfix) with ESMTPS id D14D2C000716; Mon, 22 Feb 2016 16:44:10 +0000 (UTC) Received: from sopuli.koti.laiskiainen.org (vpn1-6-141.ams2.redhat.com [10.36.6.141]) by int-mx14.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id u1MGi9cL020379; Mon, 22 Feb 2016 11:44:10 -0500 To: "Wiles, Keith" , "Trahe, Fiona" , "dev@dpdk.org" References: <1456149217-24833-1-git-send-email-keith.wiles@intel.com> <348A99DA5F5B7549AA880327E580B43588F70F25@IRSMSX101.ger.corp.intel.com> <3158B4E6-B902-47B7-A7DA-68AEC8C53A84@intel.com> From: Panu Matilainen Message-ID: <56CB3AD8.1040807@redhat.com> Date: Mon, 22 Feb 2016 18:44:08 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.5.0 MIME-Version: 1.0 In-Reply-To: <3158B4E6-B902-47B7-A7DA-68AEC8C53A84@intel.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit X-Scanned-By: MIMEDefang 2.68 on 10.5.11.27 Subject: Re: [dpdk-dev] [PATCH] config: remove duplicate configuration information 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, 22 Feb 2016 16:44:11 -0000 On 02/22/2016 06:02 PM, Wiles, Keith wrote: >> Hi Keith, >> >> What makes a param common? >> >> e.g. cryptodev QAT PMD is supported in linux, but currently not supported in bsd. >> So typically I disable it in the bsd file and enable it in the linux file. >> >> Couldn't the same apply to any other parameter, i.e. there may be users who want to have differences in config for different OSs? >> >> So why not just leave as is and give users the option to choose? > > The problem is the major configs are all common, in this design we have the common_base all configs are placed then as you stated they are disable in the common_OS files. Plus some are enabled/disabled in the deconfig_XXX files as well. > > The goal is to move all of the configs into one file then we do not have to keep updating all of the common_OS files, but only enable/disable that option. > > I have common_osxapp that I want to add later to build and run DPDK on OS X, which is another place to have these same configs. Later we may add another OS too, which means more copies :-) > My +1 for eliminating config redundancy. In addition to improving overall sanity, having the common options in a common file makes the few actually OS-dependent items stand out, which is only a good thing. - Panu -