[spp] [PATCH 0/4] correctly allocate ring resource

[spp] [PATCH 0/4] correctly allocate ring resource

[x-fn-spp]

From: Hideyuki Yamashita <yamashita.hideyuki@po.ntt-tx.co.jp>

>From 18.08.1, there is a problem that ring status is not shown
correctly when adding ring second time. 
It is because how to handle delete ring request from user is wrong. 
To resolve, this patch provides the following for spp_nfv
- Replace handling of delete ring request with rte_eth_dev_stop() and 
  rte_eth_dev_close()
- Add existence check of DPDK port when adding ring

This patch series also provides fix for spp_vf and spp_pcap because 
a problem which is caused by incorrect allocation of ring resource exists..

Note: this patch set relies on another patch set 
"introduce new feature SPP PCAP".

Hideyuki Yamashita (4):
  shared: fix trying to create ring already exist
  spp_nfv: fix incorrect deleting for ring port
  spp_vf: fix trying to create ring already exist
  spp_pcap: fix trying to create ring already exist

 src/nfv/commands.h              |  3 ++-
 src/pcap/spp_proc.c             | 17 ++++++++++++++++-
 src/shared/secondary/add_port.c | 21 ++++++++++++++++-----
 src/vf/common/spp_proc.c        | 17 ++++++++++++++++-
 4 files changed, 50 insertions(+), 8 deletions(-)

-- 
2.17.1

Re: [spp] [PATCH 0/4] correctly allocate ring resource

[Yasufumi Ogawa]

On 2019/02/08 17:47, x-fn-spp@sl.ntt-tx.co.jp wrote:
> From: Hideyuki Yamashita <yamashita.hideyuki@po.ntt-tx.co.jp>
> 
>>From 18.08.1, there is a problem that ring status is not shown
> correctly when adding ring second time.
> It is because how to handle delete ring request from user is wrong.
> To resolve, this patch provides the following for spp_nfv
> - Replace handling of delete ring request with rte_eth_dev_stop() and
>    rte_eth_dev_close()
> - Add existence check of DPDK port when adding ring
> 
> This patch series also provides fix for spp_vf and spp_pcap because
> a problem which is caused by incorrect allocation of ring resource exists..
> 
> Note: this patch set relies on another patch set
> "introduce new feature SPP PCAP".

Series of patches applied, thanks.
> 
> Hideyuki Yamashita (4):
>    shared: fix trying to create ring already exist
>    spp_nfv: fix incorrect deleting for ring port
>    spp_vf: fix trying to create ring already exist
>    spp_pcap: fix trying to create ring already exist
> 
>   src/nfv/commands.h              |  3 ++-
>   src/pcap/spp_proc.c             | 17 ++++++++++++++++-
>   src/shared/secondary/add_port.c | 21 ++++++++++++++++-----
>   src/vf/common/spp_proc.c        | 17 ++++++++++++++++-
>   4 files changed, 50 insertions(+), 8 deletions(-)
> 


-- 
Yasufumi Ogawa
NTT Network Service Systems Labs