> -----Original Message-----
> From: dev <ovs-dev-boun...@openvswitch.org> On Behalf Of Kumar Amber
> Sent: Friday, August 26, 2022 5:01 AM
> To: ovs-dev@openvswitch.org
> Cc: i.maxim...@ovn.org; f...@sysclose.org; Amber, Kumar
> <kumar.am...@intel.com>
> Subject: [ovs-dev] [PATCH v5 2/9] dpif-netdev: Refactor hash function to
> own header.
> 
> The refactor allows us to use hash function accross multiple files which
> was earlier restricted to dpif-netdev.c only.
> 
> Signed-off-by: Kumar Amber <kumar.am...@intel.com>
> Signed-off-by: Cian Ferriter <cian.ferri...@intel.com>
> Co-authored-by: Cian Ferriter <cian.ferri...@intel.com>
> 
> ---
> v3:
> - Fix minor comments from Harry.
> ---
> ---
>  lib/dpif-netdev-private-dpcls.h | 23 +++++++++++++++++++++++
>  lib/dpif-netdev.c               | 22 ----------------------
>  2 files changed, 23 insertions(+), 22 deletions(-)
> 

Hi Amber, 

Patch looks good to me;  but..like patch 1,  I would prefer if you mentioned 
why this change is required in the commit message.

Acked-by: Sunil Pai G <sunil.pa...@intel.com>

Thanks and regards
Sunil
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to