Hi Bruce, > -----Original Message----- > From: Richardson, Bruce <bruce.richard...@intel.com> > Sent: Saturday, February 5, 2022 2:19 AM > To: dev@dpdk.org > Cc: Maxime Coquelin <maxime.coque...@redhat.com>; Xia, Chenbo > <chenbo....@intel.com>; Wang, Xiao W <xiao.w.w...@intel.com>; Matan > Azrad <ma...@nvidia.com>; Viacheslav Ovsiienko > <viachesl...@nvidia.com> > Subject: Re: [PATCH 6/7] vhost: remove non-C++ compatible includes > > On Fri, Feb 04, 2022 at 05:42:08PM +0000, Bruce Richardson wrote: > > Some of the linux header includes are explicitly noted as being > > incompatible with C++. However, these headers can included by C files > > directly, or by internal headers, to avoid polluting the public DPDK > > headers with non-C++ safe includes. > > > > Signed-off-by: Bruce Richardson <bruce.richard...@intel.com> > > --- > > CI is reporting build issues with this patch on examples, something I'm not > surprised to see. I will wait for maintainer feedback on best approach > before respinning patchset. > > /Bruce
Could we move these c++ incompatible linux headers into #ifndef __cplusplus ... #endif. Then we just need to change rte_vhost.h file, and don't break build for the drivers and samples. BRs, Xiao
