[ovs-git] [openvswitch/ovs] 0cdd5b: sparse: Configure target operating system and fix ...

Ben Pfaff noreply at github.com
Wed Apr 24 15:40:42 UTC 2019


  Branch: refs/heads/master
  Home:   https://github.com/openvswitch/ovs
  Commit: 0cdd5b13de91b989dc92246e20ee6d528417df97
      https://github.com/openvswitch/ovs/commit/0cdd5b13de91b989dc92246e20ee6d528417df97
  Author: Ben Pfaff <blp at ovn.org>
  Date:   2019-04-24 (Wed, 24 Apr 2019)

  Changed paths:
    M acinclude.m4
    M include/sparse/automake.mk
    A include/sparse/linux/if_packet.h
    M lib/perf-counter.c

  Log Message:
  -----------
  sparse: Configure target operating system and fix fallout.

cgcc, the "sparse" wrapper that OVS uses, can be told the host architecture
or the host OS or both.  Until now, OVS has told it the host architecture
because it is fairly common that it doesn't guess it automatically.  Until
now, OS has not told it the host OS, assuming that it would get it right.
However, it doesn't--if you tell it the host OS or the host architecture,
it doesn't really have a default for the other.  This means that on Linux
(presumably the only OS where sparse works properly for OVS), it was not
defining __linux__, which caused some weird behavior.

This commit adds a flag to the cgcc invocation to make it define __linux__
on Linux, and it fixes some errors that this would otherwise cause.

Acked-by: Ilya Maximets <i.maximets at samsung.com>
Signed-off-by: Ben Pfaff <blp at ovn.org>




More information about the git mailing list