From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 1079DA04DC; Tue, 20 Oct 2020 17:39:08 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 51BA0BCCE; Tue, 20 Oct 2020 17:39:03 +0200 (CEST) Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) by dpdk.org (Postfix) with ESMTP id 60F59BCCC for ; Tue, 20 Oct 2020 17:39:02 +0200 (CEST) IronPort-SDR: wJxWZmZ0lQkgDwTilpo9pInjoiphG2izh70+T2O/gDbNmCeIgv3USVnF1xg8pNJURmSoiZtW94 hel8otdPyXjw== X-IronPort-AV: E=McAfee;i="6000,8403,9779"; a="231424811" X-IronPort-AV: E=Sophos;i="5.77,397,1596524400"; d="scan'208";a="231424811" X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga004.jf.intel.com ([10.7.209.38]) by orsmga105.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 20 Oct 2020 08:38:57 -0700 IronPort-SDR: 5M/GZSJElo8BZUtcwIWeADTqC7Uy2RKhlS8Igqz+I3YPDaH997vuAar7Ps1DcqlCRYhGWjtJAo +kt4vi1yI0UQ== X-IronPort-AV: E=Sophos;i="5.77,397,1596524400"; d="scan'208";a="465970669" Received: from bricha3-mobl.ger.corp.intel.com ([10.213.248.1]) by orsmga004-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-SHA; 20 Oct 2020 08:38:55 -0700 Date: Tue, 20 Oct 2020 16:38:51 +0100 From: Bruce Richardson To: "McDaniel, Timothy" Cc: Thomas Monjalon , "Eads, Gage" , Jerin Jacob , "Mcnamara, John" , "Kovacevic, Marko" , Ray Kinsella , Neil Horman , dpdk-dev , "Carrillo, Erik G" , "Van Haaren, Harry" , Jerin Jacob , "david.marchand@redhat.com" Message-ID: <20201020153851.GJ558@bricha3-MOBL.ger.corp.intel.com> References: <1599855987-25976-2-git-send-email-timothy.mcdaniel@intel.com> <20201019083337.GA649@bricha3-MOBL.ger.corp.intel.com> <44534720.QNPfj4FMZr@thomas> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Subject: Re: [dpdk-dev] [PATCH v2 01/22] event/dlb2: add documentation and meson build infrastructure 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: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Tue, Oct 20, 2020 at 04:33:42PM +0100, McDaniel, Timothy wrote: > > > > -----Original Message----- > > From: Thomas Monjalon > > Sent: Tuesday, October 20, 2020 10:21 AM > > To: McDaniel, Timothy ; Eads, Gage > > > > Cc: Richardson, Bruce ; Jerin Jacob > > ; Mcnamara, John ; > > Kovacevic, Marko ; Ray Kinsella > > ; Neil Horman ; dpdk-dev > > ; Carrillo, Erik G ; Van Haaren, Harry > > ; Jerin Jacob ; > > david.marchand@redhat.com > > Subject: Re: [dpdk-dev] [PATCH v2 01/22] event/dlb2: add documentation and > > meson build infrastructure > > > > 20/10/2020 17:17, McDaniel, Timothy: > > > From: Bruce Richardson > > > > On Sun, Oct 18, 2020 at 02:18:32PM +0530, Jerin Jacob wrote: > > > > > On Sat, Oct 17, 2020 at 11:50 PM Timothy McDaniel > > > > > wrote: > > > > > > > > > > > > Adds the meson build infrastructure, which includes > > > > > > compile-time constants in rte_config.h. DLB2 is > > > > > > only supported on Linux X86 platforms at this time. > > > > > > > > > > > > Signed-off-by: Timothy McDaniel > > > > > > Reviewed-by: Gage Eads > > > > > > --- > > > > > > --- a/drivers/event/meson.build > > > > > > +++ b/drivers/event/meson.build > > > > > > @@ -10,6 +10,9 @@ if not (toolchain == 'gcc' and > > > > cc.version().version_compare('<4.8.6') and > > > > > > dpdk_conf.has('RTE_ARCH_ARM64')) > > > > > > drivers += 'octeontx' > > > > > > endif > > > > > > +if (dpdk_conf.has('RTE_ARCH_X86_64') and is_linux) > > > > > > + drivers += 'dlb2' > > > > > > +endif > > > > > > > > > > Please add the message in "Content Skipped" section, > > > > > Reference: grep "reason" in drivers/vdpa/mlx5/meson.build > > > > > > > > > > > > > The octeontx case is also wrong in this file, IMHO. Rather than checking > > > > things in the event level and adding things to the list, the list should > > > > just be static. If something should be optionally compiled, then check the > > > > conditions in the driver meson.build file itself and add "build=false" to > > > > disable, setting "reason" to the cause of it being disabled. This keeps all > > > > the logic about a driver in the other file, rather than someone having to > > > > look in multiple places for why something is or isn't getting built > > > > properly. > > > > > > > > Regards, > > > > /Bruce > > > > > > Due to time constraints, I would prefer to take these issues up in a future > > release. > > > > If you have strong time constraints to complete everything, > > then it is better to postpone the feature to the next release. > > > > You read too much into my comment. > My point is that there does not seem to be community consensus on this issue, There is consensus here, and the way the octeon event driver is doing things is wrong, and was just missed in previous reviews. > so why are we debating it at this point in the release cycle. There are plenty of examples > doing exactly what I am doing. > Looking through the driver/*/meson.build files this appears to be the only incidence where a driver is optionally adding itself to the list. All other drivers are always processed so that when enabled/disabled they can be properly listed in the configuration summary and a reason given. Regards, /Bruce