On 10/19/2017 05:54 PM, [email protected] wrote:
> For readability purposes dpdk_mp_put is renamed as dpdk_mp_free.
> 
> CC: Mark B Kavanagh <[email protected]>
> CC: Darrell Ball <[email protected]>
> CC: Ciara Loftus <[email protected]>
> CC: Kevin Traynor <[email protected]>
> CC: Aaron Conole <[email protected]>
> Signed-off-by: Antonio Fischetti <[email protected]>
> ---

Acked-by: Kevin Traynor <[email protected]>

>  lib/netdev-dpdk.c | 8 +++++---
>  1 file changed, 5 insertions(+), 3 deletions(-)
> 
> diff --git a/lib/netdev-dpdk.c b/lib/netdev-dpdk.c
> index bf143e0..82652f0 100644
> --- a/lib/netdev-dpdk.c
> +++ b/lib/netdev-dpdk.c
> @@ -602,8 +602,9 @@ dpdk_mp_get(struct netdev_dpdk *dev, int mtu, bool 
> *mp_exists)
>      return dmp;
>  }
>  
> +/* Release an existing mempool. */
>  static void
> -dpdk_mp_put(struct dpdk_mp *dmp)
> +dpdk_mp_free(struct dpdk_mp *dmp)
>  {
>      char *mp_name;
>  
> @@ -649,7 +650,8 @@ netdev_dpdk_mempool_configure(struct netdev_dpdk *dev)
>          dev->max_packet_len = MTU_TO_FRAME_LEN(dev->mtu);
>          return EEXIST;
>      } else {
> -        dpdk_mp_put(dev->dpdk_mp);
> +        /* A new mempool was created, release the previous one. */
> +        dpdk_mp_free(dev->dpdk_mp);
>          dev->dpdk_mp = mp;
>          dev->mtu = dev->requested_mtu;
>          dev->socket_id = dev->requested_socket_id;
> @@ -1094,7 +1096,7 @@ common_destruct(struct netdev_dpdk *dev)
>      OVS_EXCLUDED(dev->mutex)
>  {
>      rte_free(dev->tx_q);
> -    dpdk_mp_put(dev->dpdk_mp);
> +    dpdk_mp_free(dev->dpdk_mp);
>  
>      ovs_list_remove(&dev->list_node);
>      free(ovsrcu_get_protected(struct ingress_policer *,
> 

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

Reply via email to