On Fri, 29 May 2015 16:45:22 +0800
Cunming Liang <cunming.liang at intel.com> wrote:

> +             if (intr_handle->intr_vec == NULL) {
> +                     PMD_INIT_LOG(ERR, "Failed to allocate %d rx_queues"
> +                                  "intr_vec\n", dev->data->nb_rx_queues);
> +                     return -1;
> +             }
> +

Please return an actual error code (like -ENOMEM) rather than generic -1.
It allows for easier diagnosis when using API's.

Reply via email to