From mboxrd@z Thu Jan 1 00:00:00 1970
Return-Path: <dev-bounces@dpdk.org>
Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124])
by inbox.dpdk.org (Postfix) with ESMTP id 2CBFDA034E;
Mon, 14 Feb 2022 16:03:53 +0100 (CET)
Received: from [217.70.189.124] (localhost [127.0.0.1])
by mails.dpdk.org (Postfix) with ESMTP id F1B874114E;
Mon, 14 Feb 2022 16:03:52 +0100 (CET)
Received: from mga11.intel.com (mga11.intel.com [192.55.52.93])
by mails.dpdk.org (Postfix) with ESMTP id 916CF41144
for <dev@dpdk.org>; Mon, 14 Feb 2022 16:03:50 +0100 (CET)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple;
d=intel.com; i=@intel.com; q=dns/txt; s=Intel;
t=1644851030; x=1676387030;
h=date:from:to:cc:subject:message-id:references:
mime-version:in-reply-to;
bh=QqFcC8PD1PDNV45GgyA3K0jQa3yQWoBDHwfRFYemnyY=;
b=i6qQl+y2BwQ6gc2tX/rgJvq/wuEeIedXTrMnls71/9n8/OECkKeEMNGn
+y+3dduFw8+lDuqRQPk/Gk7jVfxW6SELsVQndtSFi+lXL3YigBypnLJ54
d8U1LGVx/yYb3ewcf7mIlobGEVDbR92850pyU0szJgz4T0fUVSeEH+F9b
7v4dOKLdpWRkppoTbMgCg2BFJk+J9dsNLdZE2Hj2tqNCLfV2o7jjFCfXV
B8RLVah0NAOLBFvM8ePr7+MpYGS9pR/viwE1GQJqA0wSy1AUjjx366Kuv
hBerpaOtu6heEksnlkyvYfxsYm43ctngvv6jHI2nZfQmkACLRA85nfdzQ A==;
X-IronPort-AV: E=McAfee;i="6200,9189,10257"; a="247700199"
X-IronPort-AV: E=Sophos;i="5.88,368,1635231600"; d="scan'208";a="247700199"
Received: from orsmga005.jf.intel.com ([10.7.209.41])
by fmsmga102.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384;
14 Feb 2022 07:03:48 -0800
X-IronPort-AV: E=Sophos;i="5.88,368,1635231600"; d="scan'208";a="703123025"
Received: from bricha3-mobl.ger.corp.intel.com ([10.252.13.94])
by orsmga005-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-SHA;
14 Feb 2022 07:03:46 -0800
Date: Mon, 14 Feb 2022 15:03:43 +0000
From: Bruce Richardson <bruce.richardson@intel.com>
To: Thomas Monjalon <thomas@monjalon.net>
Cc: dev@dpdk.org, Tyler Retzlaff <roretzla@linux.microsoft.com>,
ocardona@microsoft.com, roretzla@microsoft.com,
david.marchand@redhat.com, ferruh.yigit@intel.com
Subject: Re: out of tree driver builds broken with C++
Message-ID: <YgpvT2l7mdDuJE42@bricha3-MOBL.ger.corp.intel.com>
References: <20220214091350.GA2793@linuxonhyperv3.guj3yctzbm1etfxqx2vob5hsef.xx.internal.cloudapp.net>
<3450334.eFTFzoEnKi@thomas>
<Ygoy4Fy4X84N1Iqs@bricha3-MOBL.ger.corp.intel.com>
<3617079.VQhiAETyHQ@thomas>
<YgpAZ1rJ+DOq1wYl@bricha3-MOBL.ger.corp.intel.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <YgpAZ1rJ+DOq1wYl@bricha3-MOBL.ger.corp.intel.com>
X-BeenThere: dev@dpdk.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: DPDK patches and discussions <dev.dpdk.org>
List-Unsubscribe: <https://mails.dpdk.org/options/dev>,
<mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://mails.dpdk.org/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <https://mails.dpdk.org/listinfo/dev>,
<mailto:dev-request@dpdk.org?subject=subscribe>
Errors-To: dev-bounces@dpdk.org
On Mon, Feb 14, 2022 at 11:43:35AM +0000, Bruce Richardson wrote:
> On Mon, Feb 14, 2022 at 12:19:24PM +0100, Thomas Monjalon wrote:
> > 14/02/2022 11:45, Bruce Richardson:
> > > On Mon, Feb 14, 2022 at 10:22:08AM +0100, Thomas Monjalon wrote:
> > > > 14/02/2022 10:13, Tyler Retzlaff:
> > > > > while the driver api is "internal" we agreed some time ago that drivers
> > > > > could be built external to the dpdk tree. by enabling the meson setup
> > > > > option -Denable_driver_sdk=true.
> > > > >
> > > > > it was agreed that the driver api was internal and would attract no
> > > > > binary compatibility support which was fine. this change has now
> > > > > imposed a further restriction that out of tree drivers have to be
> > > > > authored in C only as non-C++ compatible code will invariably leak into
> > > > > the internal structures.
> > > > >
> > > > > you won't allow us to build C++ drivers in the dpdk tree and it seems
> > > > > now you are preventing building of C++ drivers outside of the tree too.
> > > >
> > > > That's the problem of non-written assumptions, they are unknown or forgotten.
> > > > Did we agree to support out-of-tree drivers in C++?
> > > >
> > > > We really need to make things clear and written in documentation.
> > > >
> > > > > could we please re-evaluate this.
> > > >
> > > > Yes we can re-evaluate.
> > > > What is the list of impacted files?
> > > >
> > > Hacking meson files a bit, the list of SDK header files is reported as below.
> > >
> > > /Bruce
> > >
> > > Message: SDK headers:
> > > Message: ethdev_driver.h
> > > Message: ethdev_pci.h
> > > Message: ethdev_vdev.h
> > > Message: cryptodev_pmd.h
> > > Message: eventdev_pmd.h
> > > Message: eventdev_pmd_pci.h
> > > Message: eventdev_pmd_vdev.h
> > > Message: eventdev_trace.h
> > > Message: event_timer_adapter_pmd.h
> > > Message: rte_dmadev_pmd.h
> > > Message: vdpa_driver.h
> >
> > I see no harm in supporting C++ include of these headers.
> > Any objection?
> >
> > Could we have a test in chkincs for the SDK headers?
> >
> Yes. It may make things a little more complicated, though, as it seems
> these headers also have a tendency to rely on some driver headers -
> specifically bus driver headers.
>
Working on a patch to add these to checks. However, just running a c++
compile does not check for valid 'extern "C"' blocks. Adding the following
to chkincs/meson.build as a basic sanity check throws up a larger list of
files to be looked at.
Diff:
+# check for extern C in files, since this is not detected as an error by the compiler
+grep = find_program('grep', required: false)
+if grep.found()
+ errlist = run_command([grep, '--files-without-match', '^extern "C"', dpdk_chkinc_headers],
+ check: true, capture: true).stdout().split()
+ if errlist != []
+ error('Files missing C++ \'extern "C"\' guards:\n- ' + '\n- '.join(errlist))
+ endif
+endif
+
Output:
../buildtools/chkincs/meson.build:45:8: ERROR: Problem encountered: Files missing C++ 'extern "C"' guards:
- /home/bruce/dpdk.org/lib/telemetry/rte_telemetry.h
- /home/bruce/dpdk.org/lib/eal/include/rte_bitops.h
- /home/bruce/dpdk.org/lib/eal/include/rte_branch_prediction.h
- /home/bruce/dpdk.org/lib/eal/include/rte_compat.h
- /home/bruce/dpdk.org/lib/eal/include/rte_hypervisor.h
- /home/bruce/dpdk.org/lib/eal/include/rte_keepalive.h
- /home/bruce/dpdk.org/lib/eal/include/rte_pci_dev_feature_defs.h
- /home/bruce/dpdk.org/lib/eal/include/rte_pci_dev_features.h
- /home/bruce/dpdk.org/lib/eal/include/rte_time.h
- /home/bruce/dpdk.org/lib/eal/include/rte_trace_point_register.h
- /home/bruce/dpdk.org/lib/eal/linux/include/rte_os.h
- /home/bruce/dpdk.org/lib/ethdev/rte_dev_info.h
- /home/bruce/dpdk.org/lib/ethdev/ethdev_driver.h
- /home/bruce/dpdk.org/lib/ethdev/ethdev_pci.h
- /home/bruce/dpdk.org/lib/ethdev/ethdev_vdev.h
- /home/bruce/dpdk.org/lib/metrics/rte_metrics_telemetry.h
- /home/bruce/dpdk.org/lib/acl/rte_acl_osdep.h
- /home/bruce/dpdk.org/lib/bpf/bpf_def.h
- /home/bruce/dpdk.org/lib/compressdev/rte_compressdev_internal.h
- /home/bruce/dpdk.org/lib/cryptodev/cryptodev_pmd.h
- /home/bruce/dpdk.org/lib/eventdev/rte_event_ring.h
- /home/bruce/dpdk.org/lib/eventdev/eventdev_pmd.h
- /home/bruce/dpdk.org/lib/eventdev/eventdev_pmd_pci.h
- /home/bruce/dpdk.org/lib/eventdev/eventdev_pmd_vdev.h
- /home/bruce/dpdk.org/lib/kni/rte_kni_common.h
- /home/bruce/dpdk.org/lib/vhost/rte_vdpa.h
- /home/bruce/dpdk.org/lib/vhost/rte_vhost_async.h
- /home/bruce/dpdk.org/lib/vhost/rte_vhost_crypto.h
- /home/bruce/dpdk.org/lib/vhost/vdpa_driver.h
Regards,
/Bruce