08/06/2021 12:39, Andrew Rybchenko: > On 6/8/21 1:22 PM, Thomas Monjalon wrote: > > 08/06/2021 10:42, Andrew Rybchenko: > >> On 4/22/21 4:18 AM, Min Hu (Connor) wrote: > >>> --- a/doc/guides/testpmd_app_ug/run_app.rst > >>> +++ b/doc/guides/testpmd_app_ug/run_app.rst > > [...] > >>> +All the dev ops is supported in primary process. While secondary process > >>> is not permitted > >>> +to allocate or release shared memory, so some ops are not supported as > >>> follows:: > >>> +``dev_configure`` > >>> +``dev_start`` > >>> +``dev_stop`` > >>> +``rx_queue_setup`` > >>> +``tx_queue_setup`` > >>> +``rx_queue_release`` > >>> +``tx_queue_release`` > >> > >> @Thomas, @Ferrh, shouldn't it be handled on ethdev level as > >> well if it is really that strict. > > > > Yes it should be documented at ethdev level, not testpmd. > > I think it was kept fuzzy for too long. > > To document is good, but I'm talking about more - > add checks in corresponding API functions and > return error.
Yes I am OK with adding checks of the running process.
