On 4/26/22 21:24, Rosemarie O'Riorden wrote:
> The function dp_netdev_pmd_flush_output_on_port() iterates over the
> p->output_pkts batch directly, when it should be using the special
> iterator macro, DP_PACKET_BATCH_FOR_EACH.
> 
> However, this wasn't possible because the macro could not accept
> &p->output_pkts.
> 
> The addition of parentheses when BATCH is dereferenced allows the macro
> to expand properly. Parenthesizing arguments in macros is good practice
> to be able to handle whichever expressions are passed in.
> 
> Signed-off-by: Rosemarie O'Riorden <[email protected]>
> ---
>  lib/dp-packet.h   | 4 ++--
>  lib/dpif-netdev.c | 4 ++--
>  2 files changed, 4 insertions(+), 4 deletions(-)

Applied.  Thanks!

Best regards, Ilya Maximets.
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to