[dpdk-dev] [PATCH v7] ethdev: add sanity checks in control APIs
Min Hu (Connor)
humin29 at huawei.com
Sat Apr 17 09:39:55 CEST 2021
在 2021/4/16 20:02, Thomas Monjalon 写道:
> 16/04/2021 13:00, Min Hu (Connor):
>> + if (iter == NULL) {
>> + RTE_ETHDEV_LOG(ERR, "Cannot init iterator for NULL iterator\n");
>
> Don't you think it would be better as
> "Cannot init NULL iterator"?
>
>> rte_eth_iterator_cleanup(struct rte_dev_iterator *iter)
>> {
>> + if (iter == NULL) {
>> + RTE_ETHDEV_LOG(ERR, "Cannot iterator clear up for NULL iter\n");
>
> Cannot clean up NULL iterator.
>
>> rte_eth_dev_owner_new(uint64_t *owner_id)
>> {
>> + if (owner_id == NULL) {
>> + RTE_ETHDEV_LOG(ERR, "Cannot get owner id for NULL param\n");
>
> Cannot return owner id.
>
>> @@ -825,6 +864,11 @@ rte_eth_dev_get_port_by_name(const char *name, uint16_t *port_id)
>> + if (port_id == NULL) {
>> + RTE_ETHDEV_LOG(ERR, "Cannot get port id for NULL param\n");
>
> Cannot return port ID
>
> etc for all other messages
>
> Do you agree with the idea?
>
Hi, Thomos, fixed in v10, thanks.
>
> .
>
More information about the dev
mailing list