On Tue, Dec 01, 2015 at 03:03:09PM +0900, Simon Horman wrote:
> Use dp_packet_reset_offsets() in dp_packet_init__() to make it
> clearer that initialisation includes a reset of packets
> packets the same way as callers to dp_packet_reset_offsets() reset packets.
>
> Signed-off-by: Simon Horman <simon.hor...@netronome.com>
> ---
> lib/dp-packet.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/lib/dp-packet.c b/lib/dp-packet.c
> index 098f816563bd..1d166e15cf6c 100644
> --- a/lib/dp-packet.c
> +++ b/lib/dp-packet.c
> @@ -27,8 +27,7 @@ dp_packet_init__(struct dp_packet *b, size_t allocated,
> enum dp_packet_source so
> {
> dp_packet_set_allocated(b, allocated);
> b->source = source;
> - b->l2_pad_size = 0;
> - b->l2_5_ofs = b->l3_ofs = b->l4_ofs = UINT16_MAX;
> + dp_packet_reset_offsets(b);
> pkt_metadata_init(&b->md, 0);
> }
>
> --
> 2.1.4
Acked-by: Thadeu Lima de Souza Cascardo <casca...@redhat.com>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
