For the title, my suggestion is:

dpif-netdev: Remove unnecessary OVS_UNUSED.

On Fri, Aug 27, 2021 at 12:30 PM lin huang <[email protected]> wrote:
>
> The dpif parameter of zone limits function is used.
> We need to remove OVS_UNUSED flag.

Strictly speaking, there is no *need* for removing the flag.
But I agree this flag here can be dropped: the compiler would only
care for this hint if the variable was unused (which is not the case
here).

I would add a Fixes: tag in the commitlog.

Fixes: a7f33fdbfb67 ("conntrack: Support zone limits.")

>
> Signed-off-by: linhuang <[email protected]>

Please double check the robot warnings on Author and SoB tag.

With those comments fixed, you can add:
Reviewed-by: David Marchand <[email protected]>


-- 
David Marchand

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to