On Thu, Aug 5, 2021, at 05:11, [email protected] wrote: > From: wenxu <[email protected]>
Hi Wenxu, A few nits in the commit log, otherwise the change looks good to me. > > Only the nat_action in the nat_action_info is used for conn > packet forward and other item such as min/max_ip/port only > used for the new conn operation. So the whole nat_ction_info > no need store in conn. This will also avoid unnecessary memory > allocation. Suggested rewrite: Only 'nat_action_info->nat_action' is used for packet forwarding. Other items such as min/max_ip/port are used only when creating new connections. No need to store the whole nat_action_info in conn. > > Signed-off-by: wenxu <[email protected]> Acked-by: Gaetan Rivet <[email protected]> Thanks, -- Gaetan Rivet _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
