From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by dpdk.org (Postfix) with ESMTP id A4BC8567F; Tue, 28 Mar 2017 13:04:52 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=intel.com; i=@intel.com; q=dns/txt; s=intel; t=1490699093; x=1522235093; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=OSFvmhkfiAijQ29UH6uSVb5j+jvgJUp17zX6fSIJcdY=; b=nqe72tfa77Lw6RyKJadaWka88Spc6e0X4PzIZ0M5NLEZJUHHsnf+RVS/ ZLH8NEt2EpkCMEhXJkuap7AMTPc0Ow==; Received: from fmsmga004.fm.intel.com ([10.253.24.48]) by fmsmga101.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 28 Mar 2017 04:04:51 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.36,236,1486454400"; d="scan'208";a="241179099" Received: from bricha3-mobl3.ger.corp.intel.com ([10.237.221.140]) by fmsmga004.fm.intel.com with SMTP; 28 Mar 2017 04:04:49 -0700 Received: by (sSMTP sendmail emulation); Tue, 28 Mar 2017 12:04:49 +0100 Date: Tue, 28 Mar 2017 12:04:49 +0100 From: Bruce Richardson To: roy.fan.zhang@intel.com Cc: dev@dpdk.org, stable@dpdk.org Message-ID: <20170328110448.GA22460@bricha3-MOBL3.ger.corp.intel.com> References: <20170328105816.22502-1-bruce.richardson@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170328105816.22502-1-bruce.richardson@intel.com> Organization: Intel Research and =?iso-8859-1?Q?De=ACvel?= =?iso-8859-1?Q?opment?= Ireland Ltd. User-Agent: Mutt/1.8.0 (2017-02-23) Subject: Re: [dpdk-dev] [PATCH] crypto/scheduler: fix include of local headers 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: Tue, 28 Mar 2017 11:04:53 -0000 On Tue, Mar 28, 2017 at 11:58:16AM +0100, Bruce Richardson wrote: > When a C file for a library/driver is including the public header files for > that library, those need to be included as local includes using quotes > rather than angle-brackets. Without doing so, parallel builds can fail, as > the compiler will only look for those headers in the global include folder > rather than locally, and the build system does not enforce that the headers > for a lib are installed before the rest of the lib is compiled. > > Fixes: 097ab0bac017 ("crypto/scheduler: add API") > Fixes: 503e9c5afb38 ("crypto/scheduler: register as vdev driver") > Fixes: 31439ee72b2c ("crypto/scheduler: add API implementations") > > CC: stable@dpdk.org > Signed-off-by: Bruce Richardson > --- > drivers/crypto/scheduler/rte_cryptodev_scheduler.c | 2 +- > drivers/crypto/scheduler/rte_cryptodev_scheduler.h | 2 +- > drivers/crypto/scheduler/scheduler_pmd.c | 2 +- > drivers/crypto/scheduler/scheduler_pmd_private.h | 2 +- > 4 files changed, 4 insertions(+), 4 deletions(-) > Two follow-up points: * Even though this is a crypto patch, I think it should go in main tree as builds are broken right now if you turn on the PMD and use a large -j setting. (i.e. it's broken for me!! :-) ) * Is there a reason why this PMD is disabled by default? It doesn't appear to have any external dependencies and if we turn it on by default we should be able to catch issues like this a lot quicker. Regards, /Bruce