[v2,0/4] net/failsafe: support deferred queue start
mbox series

Message ID 1537451752-28759-1-git-send-email-arybchenko@solarflare.com
Headers show
Series
  • net/failsafe: support deferred queue start
Related show

Message

Andrew Rybchenko Sept. 20, 2018, 1:55 p.m. UTC
Since the topic is raised in multicast address list patchset, I'd like
to highlight it here as well. Current version uses failsafe ops directly
on sync to synchronize queues state which iterates over all sub-devices.
For already in sync sub-devices it does not go to driver since ethdev
functions checks current state and do nothing if it is already OK.
In theory it is possible to limit it to inactive devices and use
ethdev API instead of direct ops, but it requires a bit more lines of
code.

v2:
    - fix ops ordering
    - update testpmd documentation
    - add Gaëtan's acks

Ian Dolzhansky (4):
  app/testpmd: add queue deferred start switch
  net/failsafe: add checks for deferred queue setup
  net/failsafe: add Rx queue start and stop functions
  net/failsafe: add Tx queue start and stop functions

 app/test-pmd/cmdline.c                      |  91 +++++++++++
 doc/guides/nics/features/failsafe.ini       |   1 +
 doc/guides/rel_notes/release_18_11.rst      |  15 ++
 doc/guides/testpmd_app_ug/testpmd_funcs.rst |   7 +
 drivers/net/failsafe/failsafe_ether.c       |  88 +++++++++++
 drivers/net/failsafe/failsafe_ops.c         | 167 +++++++++++++++++++-
 6 files changed, 368 insertions(+), 1 deletion(-)

Comments

Ferruh Yigit Sept. 21, 2018, 12:21 a.m. UTC | #1
On 9/20/2018 2:55 PM, Andrew Rybchenko wrote:
> Since the topic is raised in multicast address list patchset, I'd like
> to highlight it here as well. Current version uses failsafe ops directly
> on sync to synchronize queues state which iterates over all sub-devices.
> For already in sync sub-devices it does not go to driver since ethdev
> functions checks current state and do nothing if it is already OK.
> In theory it is possible to limit it to inactive devices and use
> ethdev API instead of direct ops, but it requires a bit more lines of
> code.
> 
> v2:
>     - fix ops ordering
>     - update testpmd documentation
>     - add Gaëtan's acks
> 
> Ian Dolzhansky (4):
>   app/testpmd: add queue deferred start switch
>   net/failsafe: add checks for deferred queue setup
>   net/failsafe: add Rx queue start and stop functions
>   net/failsafe: add Tx queue start and stop functions

Series applied to dpdk-next-net/master, thanks.