On Tue, 24 Apr 2018 09:41:57 +0300
Arnon Warshavsky <ar...@qwilt.com> wrote:

> diff --git a/lib/librte_eventdev/rte_eventdev_pmd_pci.h 
> b/lib/librte_eventdev/rte_eventdev_pmd_pci.h
> index 8fb6138..6e08705 100644
> --- a/lib/librte_eventdev/rte_eventdev_pmd_pci.h
> +++ b/lib/librte_eventdev/rte_eventdev_pmd_pci.h
> @@ -66,9 +66,11 @@
>                                               RTE_CACHE_LINE_SIZE,
>                                               rte_socket_id());
>  
> -             if (eventdev->data->dev_private == NULL)
> -                     rte_panic("Cannot allocate memzone for private "
> -                                     "device data");
> +             if (eventdev->data->dev_private == NULL) {
> +                     RTE_LOG(CRIT, EAL, "%s(): Cannot allocate memzone for 
> private device data",
> +                             __func__);
> +                     return -ENOMEM;
> +             }
>       

Off topic, why is the probe function inline in a .h file.
This is wasteful, and not at all performance critical. it should be in the .c 
file.

Reply via email to