These error codes should be passed through errno variable. The return value should just indicate success vs. failure. See error code update on the v1.0 delta doc.
-Petri > -----Original Message----- > From: [email protected] [mailto:lng-odp- > [email protected]] On Behalf Of ext Maxim Uvarov > Sent: Monday, December 01, 2014 6:16 PM > To: [email protected] > Subject: [lng-odp] [PATCHv4 10/11] API and linux-generic: pktio update > return codes > > Update return codes to readable values. > > Signed-off-by: Maxim Uvarov <[email protected]> > --- > platform/linux-generic/include/api/odp_packet_io.h | 16 +++++++--------- > platform/linux-generic/odp_packet_io.c | 20 ++++++++++------- > --- > 2 files changed, 17 insertions(+), 19 deletions(-) > > diff --git a/platform/linux-generic/include/api/odp_packet_io.h > b/platform/linux-generic/include/api/odp_packet_io.h > index 1190355..0b9e34b 100644 > --- a/platform/linux-generic/include/api/odp_packet_io.h > +++ b/platform/linux-generic/include/api/odp_packet_io.h > @@ -137,8 +137,7 @@ odp_pktio_t odp_pktio_get_input(odp_packet_t pkt); > * use. > * > * @retval 0 on success. > - * @retval -1 if specified mtu can not be handled. > - * @retval -1 on any other error or illegal input parameters. > + * @retval -ERROR on error. > */ > int odp_pktio_set_mtu(odp_pktio_t id, int mtu); > > @@ -148,7 +147,7 @@ int odp_pktio_set_mtu(odp_pktio_t id, int mtu); > * @param[in] id ODP packet IO handle. > * > * @retval MTU value >0 on success. > - * @retval -1 on any error or not existance pktio id. > + * @retval -ERROR on error. > */ > int odp_pktio_mtu(odp_pktio_t id); > > @@ -159,8 +158,7 @@ int odp_pktio_mtu(odp_pktio_t id); > * @param[in] enable 1 enabled, 0 disabled. > * > * @retval 0 on success. > - * @retval -1 on a bad pktio id > - * @retval -1 any other error > + * @retval -ERROR on error. > */ > int odp_pktio_promisc_set(odp_pktio_t id, odp_bool_t enable); > > @@ -171,8 +169,7 @@ int odp_pktio_promisc_set(odp_pktio_t id, odp_bool_t > enable); > * > * @retval 1 if promiscuous mode is enabled. > * @retval 0 if promiscuous mode is disabled. > - * @retval -1 on a bad pktio id > - * @retval -1 any other error > + * @retval -ERROR on error. > */ > int odp_pktio_promisc_enabled(odp_pktio_t id); > > @@ -183,8 +180,9 @@ int odp_pktio_promisc_enabled(odp_pktio_t id); > * @param[in] mac_addr MAC address to be assigned to the interface. > * @param[in] addr_size Size of the address in bytes. > * > - * @return 0 on success, -ERROR on error. > - */ > + * @retval return 0 on success > + * @retval -ERROR on error. > +*/ > int odp_pktio_mac_addr_set(odp_pktio_t id, const unsigned char *mac_addr, > size_t addr_size); > > diff --git a/platform/linux-generic/odp_packet_io.c b/platform/linux- > generic/odp_packet_io.c > index 6c8307e..7d985fb 100644 > --- a/platform/linux-generic/odp_packet_io.c > +++ b/platform/linux-generic/odp_packet_io.c > @@ -552,13 +552,13 @@ int odp_pktio_set_mtu(odp_pktio_t id, int mtu) > > if (mtu <= 0) { > ODP_DBG("illegal MTU value %d\n", mtu); > - return -1; > + return -EINVAL; > } > > entry = get_entry(id); > if (entry == NULL) { > ODP_DBG("pktio entry %d does not exist\n", id); > - return -1; > + return -ENOENT; > } > > if (entry->s.pkt_sock_mmap.sockfd > -1) > @@ -573,7 +573,7 @@ int odp_pktio_set_mtu(odp_pktio_t id, int mtu) > ret = ioctl(sockfd, SIOCSIFMTU, (caddr_t)&ifr); > if (ret < 0) { > ODP_DBG("ioctl SIOCSIFMTU error\n"); > - return -1; > + return ret; > } > > return 0; > @@ -589,7 +589,7 @@ int odp_pktio_mtu(odp_pktio_t id) > entry = get_entry(id); > if (entry == NULL) { > ODP_DBG("pktio entry %d does not exist\n", id); > - return -1; > + return -ENOENT; > } > > if (entry->s.pkt_sock_mmap.sockfd > -1) > @@ -603,7 +603,7 @@ int odp_pktio_mtu(odp_pktio_t id) > ret = ioctl(sockfd, SIOCGIFMTU, &ifr); > if (ret < 0) { > ODP_DBG("ioctl SIOCGIFMTU error\n"); > - return -1; > + return ret; > } > > return ifr.ifr_mtu; > @@ -619,7 +619,7 @@ int odp_pktio_promisc_set(odp_pktio_t id, odp_bool_t > enable) > entry = get_entry(id); > if (entry == NULL) { > ODP_DBG("pktio entry %d does not exist\n", id); > - return -1; > + return -ENOENT; > } > > if (entry->s.pkt_sock_mmap.sockfd > -1) > @@ -633,7 +633,7 @@ int odp_pktio_promisc_set(odp_pktio_t id, odp_bool_t > enable) > ret = ioctl(sockfd, SIOCGIFFLAGS, &ifr); > if (ret < 0) { > ODP_DBG("ioctl SIOCGIFFLAGS error\n"); > - return -1; > + return ret; > } > > if (enable) > @@ -644,7 +644,7 @@ int odp_pktio_promisc_set(odp_pktio_t id, odp_bool_t > enable) > ret = ioctl(sockfd, SIOCSIFFLAGS, &ifr); > if (ret < 0) { > ODP_DBG("ioctl SIOCSIFFLAGS error\n"); > - return -1; > + return ret; > } > > return 0; > @@ -660,7 +660,7 @@ int odp_pktio_promisc_enabled(odp_pktio_t id) > entry = get_entry(id); > if (entry == NULL) { > ODP_DBG("pktio entry %d does not exist\n", id); > - return -1; > + return -ENOENT; > } > > if (entry->s.pkt_sock_mmap.sockfd > -1) > @@ -674,7 +674,7 @@ int odp_pktio_promisc_enabled(odp_pktio_t id) > ret = ioctl(sockfd, SIOCGIFFLAGS, &ifr); > if (ret < 0) { > ODP_DBG("ioctl SIOCGIFFLAGS error\n"); > - return -1; > + return ret; > } > > if (ifr.ifr_flags & IFF_PROMISC) > -- > 1.8.5.1.163.gd7aced9 > > > _______________________________________________ > lng-odp mailing list > [email protected] > http://lists.linaro.org/mailman/listinfo/lng-odp _______________________________________________ lng-odp mailing list [email protected] http://lists.linaro.org/mailman/listinfo/lng-odp
