[ovs-dev] [PATCH 2/7] userspace: Support for push_eth and pop_eth actions

[Ben Pfaff]

On Fri, Feb 03, 2017 at 10:39:48AM +0000, Jan Scheurich wrote:
> Add support for actions push_eth and pop_eth to the netdev datapath and
> the supporting libraries. This patch relies on the support for these actions
> in the kernel datapath to be present.
> 
> Signed-off-by: Lorand Jakab <lojakab at cisco.com>
> Signed-off-by: Simon Horman <simon.horman at netronome.com>
> Signed-off-by: Jiri Benc <jbenc at redhat.com>
> Signed-off-by: Yi Yang <yi.y.yang at intel.com>
> Signed-off-by: Jean Tourrilhes <jt at labs.hpe.com>
> Signed-off-by: Jan Scheurich <jan.scheurich at ericsson.com>
> Co-authored-by: Zoltan Balogh <zoltan.balogh at ericsson.com>

This sign-off chain is baffling.  Please explain it.

Clang reports:

    ../lib/packets.c:260:22: error: cast from 'char *' to 'ovs_be16 *' (aka 'unsigned short *') increases required alignment from 1 to 2 [-Werror,-Wcast-align]
    ../lib/packets.c:263:22: error: cast from 'char *' to 'ovs_be16 *' (aka 'unsigned short *') increases required alignment from 1 to 2 [-Werror,-Wcast-align]

Thanks,

Ben.