[ovs-git] [openvswitch/ovs] b13596: odp-execute: Fix broken build with Clang as compil...

GitHub noreply at github.com
Mon Oct 15 22:52:30 UTC 2018


  Branch: refs/heads/branch-2.7
  Home:   https://github.com/openvswitch/ovs
  Commit: b13596ba3dd0575b3104330a0eef4e1d173c4fb5
      https://github.com/openvswitch/ovs/commit/b13596ba3dd0575b3104330a0eef4e1d173c4fb5
  Author: Ben Pfaff <blp at ovn.org>
  Date:   2018-10-15 (Mon, 15 Oct 2018)

  Changed paths:
    M lib/odp-util.c
    M lib/packets.h
    M ofproto/ofproto-dpif-sflow.c

  Log Message:
  -----------
  odp-execute: Fix broken build with Clang as compiler.

Builds of branch-2.7 have been failing on Travis when Clang is used as
compiler due to:

../ofproto/ofproto-dpif.c:2057:46: warning: taking address of packed member
      'eth_src' of class or structure 'eth_header' may result in an unaligned
      pointer value [-Waddress-of-packed-member]
    netdev_get_etheraddr(ofport->up.netdev, &eth->eth_src);
                                       ^~~~~~~~~~~~
../ofproto/ofproto-dpif.c:2082:50: warning: taking address of packed member
      'eth_src' of class or structure 'eth_header' may result in an unaligned
      pointer value [-Waddress-of-packed-member]
  netdev_get_etheraddr(ofport->up.netdev, &eth->eth_src);
                                           ^~~~~~~~~~~~

On master these don't come up because of commit 1620b7ea68c2 ("packets:
Remove unnecessary "packed" annotations."), which removed the packed
annotation that causes the warning.  This commit applies enough of that
commit to make the build pass.

Signed-off-by: Ben Pfaff <blp at ovn.org>



      **NOTE:** This service has been marked for deprecation: https://developer.github.com/changes/2018-04-25-github-services-deprecation/

      Functionality will be removed from GitHub.com on January 31st, 2019.


More information about the git mailing list