Re: [lng-odp] [API-NEXT PATCHv2 3/5] api: init: driver load function added

mike.hol...@linaro.org; bill.fischo...@linaro.org; lng-odp@lists.linaro.org Subject: Re: [lng-odp] [API-NEXT PATCHv2 3/5] api: init: driver load function added Command line parameters will go to the application. As far as I know, we have no ODP parameters at this time, and they would require some

Re: [lng-odp] [API-NEXT PATCHv2 3/5] api: init: driver load function added

ODP is about abstracting things so that applications can concentrate on the logic. Exposing drivers on this northbound interface contradicts the essence of ODP. ODP implementers will decide how they will accomodate driver "additions". One may even decide to retrieve drivers through TFTP ;-) If

Re: [lng-odp] [API-NEXT PATCHv2 3/5] api: init: driver load function added

Command line parameters will go to the application. As far as I know, we have no ODP parameters at this time, and they would require some command line processing cooperation with the application (main(argv, argc)) is in the app, we are just a lib). That put apps using getopts in a bad position.

Re: [lng-odp] [API-NEXT PATCHv2 3/5] api: init: driver load function added

Why not just scanning a directory or give a command line parameter? On 16 November 2016 at 12:05, Christophe Milard < christophe.mil...@linaro.org> wrote: > So what? > If we don't allow to load a driver from the north interface, where should > they be loaded from?? > Or are we saying that we go

Re: [lng-odp] [API-NEXT PATCHv2 3/5] api: init: driver load function added

So what? If we don't allow to load a driver from the north interface, where should they be loaded from?? Or are we saying that we go for a predefined list of driver only? So drivers have to be .so files (because of autotools), but we don't give the possibility to load a free driver?? Christophe

Re: [lng-odp] [API-NEXT PATCHv2 3/5] api: init: driver load function added

Hello, If the north API is the one visible by ODP applications then I don't think it is a good idea to expose that. DPDK exposed it at the beginning and is now internal. That said there must be a standard way to manage drivers fir the benefit of the device framework. I don't think the idea of

Re: [lng-odp] [API-NEXT PATCHv2 3/5] api: init: driver load function added

On 16 November 2016 at 10:07, Savolainen, Petri (Nokia - FI/Espoo) wrote: > >> /** >> + * Driver loading >> + * >> + * This function is used by the application to load NIC drivers into ODP. >> + * Calls to this function are optional, but should be performed

Re: [lng-odp] [API-NEXT PATCHv2 3/5] api: init: driver load function added

> /** > + * Driver loading > + * > + * This function is used by the application to load NIC drivers into ODP. > + * Calls to this function are optional, but should be performed (if any) > + * after odp_init_global > + * > + * > + * @param filenameDriver shared lib (dynamic library) > +

Re: [lng-odp] [API-NEXT PATCHv2 3/5] api: init: driver load function added

On 11/08/16 17:09, Christophe Milard wrote: On 8 November 2016 at 14:56, Maxim Uvarov wrote: On 11/07/16 14:15, Christophe Milard wrote: Function added on the north API to load drivers. Signed-off-by: Christophe Milard ---

Re: [lng-odp] [API-NEXT PATCHv2 3/5] api: init: driver load function added

On 8 November 2016 at 14:56, Maxim Uvarov wrote: > On 11/07/16 14:15, Christophe Milard wrote: >> >> Function added on the north API to load drivers. >> >> Signed-off-by: Christophe Milard >> --- >> include/odp/api/spec/init.h | 16

Re: [lng-odp] [API-NEXT PATCHv2 3/5] api: init: driver load function added

On 11/07/16 14:15, Christophe Milard wrote: Function added on the north API to load drivers. Signed-off-by: Christophe Milard --- include/odp/api/spec/init.h | 16 1 file changed, 16 insertions(+) diff --git a/include/odp/api/spec/init.h

[lng-odp] [API-NEXT PATCHv2 3/5] api: init: driver load function added

Function added on the north API to load drivers. Signed-off-by: Christophe Milard --- include/odp/api/spec/init.h | 16 1 file changed, 16 insertions(+) diff --git a/include/odp/api/spec/init.h b/include/odp/api/spec/init.h index 154cdf8..bfe43df