Re: [dpdk-dev] [PATCH] ethdev: let apps find transfer admin port for a given ethdev

2021-10-06 Thread Ivan Malov
Hi Thomas, Thanks a lot for providing your review notes. So useful and to the point. I also processed review notes from Andrew and sent v3: https://patches.dpdk.org/project/dpdk/patch/20211006123300.7848-1-ivan.ma...@oktetlabs.ru/ I hope the new API is much clearer now. Should you wish to imp

Re: [dpdk-dev] [PATCH] ethdev: let apps find transfer admin port for a given ethdev

2021-10-05 Thread Ori Kam
Hi Ivan, I agree with your reply. Acked-by: Ori Kam Thanks, Ori > -Original Message- > From: Ivan Malov > Subject: Re: [PATCH] ethdev: let apps find transfer admin port for a given > ethdev > > Hi Ori, > > Thanks a lot for eagerly reviewing this and related patches. That's very > he

Re: [dpdk-dev] [PATCH] ethdev: let apps find transfer admin port for a given ethdev

2021-10-05 Thread Thomas Monjalon
05/10/2021 20:22, Ivan Malov: > Hi Thomas, > > Thanks for joining the review. > > On 05/10/2021 20:56, Thomas Monjalon wrote: > > 05/10/2021 02:36, Ivan Malov: > >> Introduce a helper API to let applications find transfer > >> admin port for a given ethdev to avoid failures when > >> managing "tr

Re: [dpdk-dev] [PATCH] ethdev: let apps find transfer admin port for a given ethdev

2021-10-05 Thread Ivan Malov
Hi Thomas, Thanks for joining the review. On 05/10/2021 20:56, Thomas Monjalon wrote: 05/10/2021 02:36, Ivan Malov: Introduce a helper API to let applications find transfer admin port for a given ethdev to avoid failures when managing "transfer" flows through unprivileged ports. Please expla

Re: [dpdk-dev] [PATCH] ethdev: let apps find transfer admin port for a given ethdev

2021-10-05 Thread Thomas Monjalon
05/10/2021 02:36, Ivan Malov: > Introduce a helper API to let applications find transfer > admin port for a given ethdev to avoid failures when > managing "transfer" flows through unprivileged ports. Please explain what is transfer admin port. We may find a simpler wording. > --- a/lib/ethdev/rte

Re: [dpdk-dev] [PATCH] ethdev: let apps find transfer admin port for a given ethdev

2021-10-05 Thread Ori Kam
Hi Ivan, > -Original Message- > From: Ivan Malov > Sent: Tuesday, October 5, 2021 3:36 AM > Subject: [PATCH] ethdev: let apps find transfer admin port for a given ethdev > > Introduce a helper API to let applications find transfer admin port for a > given > ethdev to avoid failures when

Re: [dpdk-dev] [PATCH] ethdev: let apps find transfer admin port for a given ethdev

2021-10-05 Thread Ivan Malov
Hi Ori, Thanks a lot for eagerly reviewing this and related patches. That's very helpful of yours. On 05/10/2021 12:22, Ori Kam wrote: Hi Ivan, -Original Message- From: Ivan Malov Sent: Tuesday, October 5, 2021 3:36 AM Subject: [PATCH] ethdev: let apps find transfer admin port for

[dpdk-dev] [PATCH] ethdev: let apps find transfer admin port for a given ethdev

2021-10-04 Thread Ivan Malov
Introduce a helper API to let applications find transfer admin port for a given ethdev to avoid failures when managing "transfer" flows through unprivileged ports. Signed-off-by: Ivan Malov Reviewed-by: Andrew Rybchenko --- Patch series [1] has reworked support for "transfer" flows. As a result,