> -----Original Message-----
> From: Morten Brørup <m...@smartsharesystems.com>
> Sent: Tuesday, December 14, 2021 5:31 PM
> To: Randles, Ronan <ronan.rand...@intel.com>; dev@dpdk.org
> Cc: Van Haaren, Harry <harry.van.haa...@intel.com>
> Subject: RE: [PATCH 02/12] net: add function to pretty print IPv4
>
> > From: Ronan Randles [mailto:ronan.rand...@intel.com]
> > Sent: Tuesday, 14 December 2021 15.13
> >
> > This function accepts an uint32_t representation of an IP address and
> > produces a string representation stored in a char * buffer. Realavent
> > unit tests also included.
> >
> > Signed-off-by: Ronan Randles <ronan.rand...@intel.com>
>
> [snip]
>
> > diff --git a/lib/net/rte_ip.h b/lib/net/rte_ip.h
> > index 188054fda4..e46f0b41ba 100644
> > --- a/lib/net/rte_ip.h
> > +++ b/lib/net/rte_ip.h
> > @@ -444,6 +444,26 @@ __rte_experimental
> > int32_t
> > rte_ip_parse_addr(const char *src_ip, uint32_t *output_addr);
> >
> > +
> > +/**
> > + * Print IP address from 32 bit int into char * buffer.
> > + *
> > + * @param ip_addr
> > + * ip address to be printed.
> > + * @param buffer
> > + * The buffer the string will be saved into.
> > + * @param buffer_size
> > + * size of buffer to be used.
> > + *
> > + * @retval 0
> > + * Success.
> > + * @retval -1
> > + * Failure due to invalid input arguments.
> > + */
> > +__rte_experimental
> > +int32_t
> > +rte_ip_print_addr(uint32_t ip_addr, char *buffer, uint32_t
> > buffer_size);
> > +
>
> In continuation of my email reply about the IPv4 parse function...
>
> I have a few suggestions to the IPv4 print function too:
>
> The return value should be the number of characters written to the output
> string, and still -1 on error. With this modification, you could
> use the return type ssize_t instead of int32_t.
>
> Furthermore, I would prefer having the parameters in the same order as
> snprintf(): char *str, size_t size, const uint32_t ip_addr. Please
> also notice the suggested changed type for the size, and the const added to
> the ip_addr.
>
Honestly, I don't understand why we need to introduce such functions
inside DPDK at all.
What's wrong with existing standard ones: inet_ntop() and inet_pton()?
