Acked-by: Billy O'Mahony

> -----Original Message-----
> From: Kevin Traynor [mailto:[email protected]]
> Sent: Thursday, November 23, 2017 7:42 PM
> To: [email protected]; [email protected]; [email protected]; O
> Mahony, Billy <[email protected]>; Stokes, Ian <[email protected]>
> Cc: Kevin Traynor <[email protected]>
> Subject: [PATCH v4 2/3] dpif-netdev: Rename rxq_cycle_sort to
> compare_rxq_cycles.
> 
> This function is used for comparison between queues as part of the sort. It 
> does
> not do the sort itself.
> As such, give it a more appropriate name.
> 
> Suggested-by: Billy O'Mahony <[email protected]>
> Signed-off-by: Kevin Traynor <[email protected]>
> ---
> 
> V4: Added patch into series after suggestion by Billy
> 
>  lib/dpif-netdev.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/lib/dpif-netdev.c b/lib/dpif-netdev.c index f5cdd92..657df71 
> 100644
> --- a/lib/dpif-netdev.c
> +++ b/lib/dpif-netdev.c
> @@ -3446,5 +3446,5 @@ rr_numa_list_destroy(struct rr_numa_list *rr)
>  /* Sort Rx Queues by the processing cycles they are consuming. */  static 
> int -
> rxq_cycle_sort(const void *a, const void *b)
> +compare_rxq_cycles(const void *a, const void *b)
>  {
>      struct dp_netdev_rxq *qa;
> @@ -3535,5 +3535,5 @@ rxq_scheduling(struct dp_netdev *dp, bool pinned)
> OVS_REQUIRES(dp->port_mutex)
>          /* Sort the queues in order of the processing cycles
>           * they consumed during their last pmd interval. */
> -        qsort(rxqs, n_rxqs, sizeof *rxqs, rxq_cycle_sort);
> +        qsort(rxqs, n_rxqs, sizeof *rxqs, compare_rxq_cycles);
>      }
> 
> --
> 1.8.3.1

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

Reply via email to