[ovs-dev] [PATCH v5 1/3] ofproto: change type of n_handlers and n_revalidators

Aaron Conole aconole at redhat.com
Thu Jul 15 17:31:12 UTC 2021


Mark Gray <mark.d.gray at redhat.com> writes:

> 'n_handlers' and 'n_revalidators' are declared as type 'size_t'.
> However, dpif_handlers_set() requires parameter 'n_handlers' as
> type 'uint32_t'. This patch fixes this type mismatch.
>
> Signed-off-by: Mark Gray <mark.d.gray at redhat.com>
> Acked-by: Flavio Leitner <fbl at sysclose.org>
> ---
>
> Notes:
>     v1 - Reworked based on Flavio's comments:
>          * fixed inconsistency with change of size_t -> uint32_t
>
>  ofproto/ofproto-dpif-upcall.c | 20 ++++++++++----------
>  ofproto/ofproto-dpif-upcall.h |  5 +++--
>  ofproto/ofproto-provider.h    |  2 +-
>  ofproto/ofproto.c             |  2 +-
>  4 files changed, 15 insertions(+), 14 deletions(-)
>

I guess this is more of a cleanup - did it fix anything in practice?

Either way,

Acked-by: Aaron Conole <aconole at redhat.com>



More information about the dev mailing list