On Wed, Mar 9, 2022 at 9:13 AM Mike Pattrick <m...@redhat.com> wrote: > > On Wed, Feb 16, 2022 at 9:31 AM Adrian Moreno <amore...@redhat.com> wrote: > > > > It seems versions older than 0.6.2 generate false positives. Bump the > > recommended version and make sure we uset the headers from the > > > > Suggested-by: Dumitru Ceara <dce...@redhat.com> > > Signed-off-by: Adrian Moreno <amore...@redhat.com> > > --- > > Documentation/intro/install/general.rst | 2 +- > > acinclude.m4 | 2 +- > > 2 files changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/Documentation/intro/install/general.rst > > b/Documentation/intro/install/general.rst > > index c4300cd53..a297aadac 100644 > > --- a/Documentation/intro/install/general.rst > > +++ b/Documentation/intro/install/general.rst > > @@ -169,7 +169,7 @@ other than plain text, only if you have the following: > > If you are going to extensively modify Open vSwitch, consider installing > > the > > following to obtain better warnings: > > > > -- "sparse" version 0.5.1 or later > > +- "sparse" version 0.6.2 or later > > (https://git.kernel.org/pub/scm/devel/sparse/sparse.git/). > > I'm not as familiar with compiling with sparse, I get the warning: > > /usr/lib/gcc/x86_64-redhat-linux/11//include/smmintrin.h:848:10: > error: undefined identifier '__builtin_ia32_crc32di' > > Is that expected?
Just a guess... Include -msse4.2 in CFLAGS to enable the symbol. config.log should have a record of the compile test. Here's how other libraries test for those symbols in configure.ac: https://github.com/noloader/cryptopp-autotools/blob/master/configure.ac#L1070 Guarding the use of SSE4.2 is important because of the antique linker used on OpenBSD. Jeff _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev
