Soft Patch Panel
 help / color / mirror / Atom feed
* [spp] [PATCH 00/23] move some codes of spp_vf into vf/common
@ 2018-11-21  1:35 x-fn-spp
  2018-11-21  3:47 ` Yasufumi Ogawa
                   ` (2 more replies)
  0 siblings, 3 replies; 6+ messages in thread
From: x-fn-spp @ 2018-11-21  1:35 UTC (permalink / raw)
  To: ferruh.yigit, ogawa.yasufumi; +Cc: spp

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

Some spp_vf codes can be used by new secondary process which
is extended from spp_vf. This patch set move such codes into
spp_vf/common.

Hideyuki Yamashita (23):
  spp_vf: fix invalid code for max chars
  spp_vf: move common source and header files
  spp_vf: change include path
  spp_vf: update Makefile for common files
  spp_vf: move functions to common directory
  spp_vf: move defines to common dir
  spp_vf: move functions for decode to common dir
  spp_vf: move defines for decode to common dir
  spp_vf: move functions to command_proc.c
  spp_vf: add management data registration
  spp_vf: change reference of management data
  spp_vf: change return values to explain result
  spp_vf: define terms of commands as consts
  spp_vf: remove unnecessary includes
  spp_vf: add include header files
  spp_vf: update comments for header file
  spp_vf: update makefile of spp_vf
  spp_vf: add check num of ports before forwarding
  spp_vf: add flag for classifier table
  spp_vf: add checking the number of ports
  spp_vf: add vlantag command check flag
  spp_vf: simplify changing VLAN tag
  spp_vf: add SPP_VF_MODULE preprocessor directive

 src/vf/Makefile                        |    7 +-
 src/vf/classifier_mac.c                |  139 ++-
 src/vf/classifier_mac.h                |   20 +-
 src/vf/{ => common}/command_conn.c     |   23 +-
 src/vf/{ => common}/command_conn.h     |   16 +-
 src/vf/{ => common}/command_dec.c      |  641 +++++++---
 src/vf/{ => common}/command_dec.h      |   21 +-
 src/vf/{ => common}/command_proc.c     |  716 +++++++++--
 src/vf/{ => common}/command_proc.h     |   10 +-
 src/vf/{ => common}/ringlatencystats.c |   15 +-
 src/vf/{ => common}/ringlatencystats.h |    4 +-
 src/vf/{ => common}/spp_port.c         |   34 +-
 src/vf/{ => common}/spp_port.h         |    2 +-
 src/vf/common/spp_proc.c               | 1029 ++++++++++++++++
 src/vf/common/spp_proc.h               |  727 +++++++++++
 src/vf/{ => common}/string_buffer.c    |    3 +-
 src/vf/{ => common}/string_buffer.h    |    0
 src/vf/spp_forward.c                   |   47 +-
 src/vf/spp_forward.h                   |   12 +-
 src/vf/spp_vf.c                        | 1574 +-----------------------
 src/vf/spp_vf.h                        |  511 --------
 21 files changed, 3074 insertions(+), 2477 deletions(-)
 rename src/vf/{ => common}/command_conn.c (90%)
 rename src/vf/{ => common}/command_conn.h (82%)
 rename src/vf/{ => common}/command_dec.c (53%)
 rename src/vf/{ => common}/command_dec.h (90%)
 rename src/vf/{ => common}/command_proc.c (63%)
 rename src/vf/{ => common}/command_proc.h (80%)
 rename src/vf/{ => common}/ringlatencystats.c (92%)
 rename src/vf/{ => common}/ringlatencystats.h (97%)
 rename src/vf/{ => common}/spp_port.c (94%)
 rename src/vf/{ => common}/spp_port.h (99%)
 create mode 100644 src/vf/common/spp_proc.c
 create mode 100644 src/vf/common/spp_proc.h
 rename src/vf/{ => common}/string_buffer.c (97%)
 rename src/vf/{ => common}/string_buffer.h (100%)

-- 
2.18.0

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [spp] [PATCH 00/23] move some codes of spp_vf into vf/common
  2018-11-21  1:35 [spp] [PATCH 00/23] move some codes of spp_vf into vf/common x-fn-spp
@ 2018-11-21  3:47 ` Yasufumi Ogawa
  2018-11-21  5:15   ` [spp] [spp 03641] " Hideyuki Yamashita
  2018-11-24 13:15 ` [spp] " Yasufumi Ogawa
  2018-11-28  2:46 ` Yasufumi Ogawa
  2 siblings, 1 reply; 6+ messages in thread
From: Yasufumi Ogawa @ 2018-11-21  3:47 UTC (permalink / raw)
  To: x-fn-spp; +Cc: ferruh.yigit, spp

> Some spp_vf codes can be used by new secondary process which
> is extended from spp_vf. This patch set move such codes into
> spp_vf/common.
Hi Hideyuki,

I would like to confirm about your changes. New secondary process you mentioned is spp_mirror which is posted as the next series 
of patches, right? Do you have a plan to implement secondary extended from spp_vf other than spp_mirror?

Thanks,
Yasufumi
> 
> Hideyuki Yamashita (23):
>    spp_vf: fix invalid code for max chars
>    spp_vf: move common source and header files
>    spp_vf: change include path
>    spp_vf: update Makefile for common files
>    spp_vf: move functions to common directory
>    spp_vf: move defines to common dir
>    spp_vf: move functions for decode to common dir
>    spp_vf: move defines for decode to common dir
>    spp_vf: move functions to command_proc.c
>    spp_vf: add management data registration
>    spp_vf: change reference of management data
>    spp_vf: change return values to explain result
>    spp_vf: define terms of commands as consts
>    spp_vf: remove unnecessary includes
>    spp_vf: add include header files
>    spp_vf: update comments for header file
>    spp_vf: update makefile of spp_vf
>    spp_vf: add check num of ports before forwarding
>    spp_vf: add flag for classifier table
>    spp_vf: add checking the number of ports
>    spp_vf: add vlantag command check flag
>    spp_vf: simplify changing VLAN tag
>    spp_vf: add SPP_VF_MODULE preprocessor directive
> 
>   src/vf/Makefile                        |    7 +-
>   src/vf/classifier_mac.c                |  139 ++-
>   src/vf/classifier_mac.h                |   20 +-
>   src/vf/{ => common}/command_conn.c     |   23 +-
>   src/vf/{ => common}/command_conn.h     |   16 +-
>   src/vf/{ => common}/command_dec.c      |  641 +++++++---
>   src/vf/{ => common}/command_dec.h      |   21 +-
>   src/vf/{ => common}/command_proc.c     |  716 +++++++++--
>   src/vf/{ => common}/command_proc.h     |   10 +-
>   src/vf/{ => common}/ringlatencystats.c |   15 +-
>   src/vf/{ => common}/ringlatencystats.h |    4 +-
>   src/vf/{ => common}/spp_port.c         |   34 +-
>   src/vf/{ => common}/spp_port.h         |    2 +-
>   src/vf/common/spp_proc.c               | 1029 ++++++++++++++++
>   src/vf/common/spp_proc.h               |  727 +++++++++++
>   src/vf/{ => common}/string_buffer.c    |    3 +-
>   src/vf/{ => common}/string_buffer.h    |    0
>   src/vf/spp_forward.c                   |   47 +-
>   src/vf/spp_forward.h                   |   12 +-
>   src/vf/spp_vf.c                        | 1574 +-----------------------
>   src/vf/spp_vf.h                        |  511 --------
>   21 files changed, 3074 insertions(+), 2477 deletions(-)
>   rename src/vf/{ => common}/command_conn.c (90%)
>   rename src/vf/{ => common}/command_conn.h (82%)
>   rename src/vf/{ => common}/command_dec.c (53%)
>   rename src/vf/{ => common}/command_dec.h (90%)
>   rename src/vf/{ => common}/command_proc.c (63%)
>   rename src/vf/{ => common}/command_proc.h (80%)
>   rename src/vf/{ => common}/ringlatencystats.c (92%)
>   rename src/vf/{ => common}/ringlatencystats.h (97%)
>   rename src/vf/{ => common}/spp_port.c (94%)
>   rename src/vf/{ => common}/spp_port.h (99%)
>   create mode 100644 src/vf/common/spp_proc.c
>   create mode 100644 src/vf/common/spp_proc.h
>   rename src/vf/{ => common}/string_buffer.c (97%)
>   rename src/vf/{ => common}/string_buffer.h (100%)
> 

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [spp] [spp 03641] Re: [PATCH 00/23] move some codes of spp_vf into vf/common
  2018-11-21  3:47 ` Yasufumi Ogawa
@ 2018-11-21  5:15   ` Hideyuki Yamashita
  2018-11-21  5:43     ` Yasufumi Ogawa
  0 siblings, 1 reply; 6+ messages in thread
From: Hideyuki Yamashita @ 2018-11-21  5:15 UTC (permalink / raw)
  To: Yasufumi Ogawa; +Cc: x-fn-spp, ferruh.yigit, spp

Hello Yasufumi-san,

Thanks for your asking.
Please see inline.
> > Some spp_vf codes can be used by new secondary process which
> > is extended from spp_vf. This patch set move such codes into
> > spp_vf/common.
> Hi Hideyuki,
> 
> I would like to confirm about your changes. New secondary process you mentioned is spp_mirror which is posted as the next series of patches, right? 
Yes.

>Do you have a plan to implement secondary extended from spp_vf other
>than spp_mirror?
Yes. I have a plan to implement secondary extended from spp_vf other
than spp_mirror.

BR,
Hideyuki Yamashita
NTT TechnoCross

> Thanks,
> Yasufumi
> >
> > Hideyuki Yamashita (23):
> >    spp_vf: fix invalid code for max chars
> >    spp_vf: move common source and header files
> >    spp_vf: change include path
> >    spp_vf: update Makefile for common files
> >    spp_vf: move functions to common directory
> >    spp_vf: move defines to common dir
> >    spp_vf: move functions for decode to common dir
> >    spp_vf: move defines for decode to common dir
> >    spp_vf: move functions to command_proc.c
> >    spp_vf: add management data registration
> >    spp_vf: change reference of management data
> >    spp_vf: change return values to explain result
> >    spp_vf: define terms of commands as consts
> >    spp_vf: remove unnecessary includes
> >    spp_vf: add include header files
> >    spp_vf: update comments for header file
> >    spp_vf: update makefile of spp_vf
> >    spp_vf: add check num of ports before forwarding
> >    spp_vf: add flag for classifier table
> >    spp_vf: add checking the number of ports
> >    spp_vf: add vlantag command check flag
> >    spp_vf: simplify changing VLAN tag
> >    spp_vf: add SPP_VF_MODULE preprocessor directive
> >
> >   src/vf/Makefile                        |    7 +-
> >   src/vf/classifier_mac.c                |  139 ++-
> >   src/vf/classifier_mac.h                |   20 +-
> >   src/vf/{ => common}/command_conn.c     |   23 +-
> >   src/vf/{ => common}/command_conn.h     |   16 +-
> >   src/vf/{ => common}/command_dec.c      |  641 +++++++---
> >   src/vf/{ => common}/command_dec.h      |   21 +-
> >   src/vf/{ => common}/command_proc.c     |  716 +++++++++--
> >   src/vf/{ => common}/command_proc.h     |   10 +-
> >   src/vf/{ => common}/ringlatencystats.c |   15 +-
> >   src/vf/{ => common}/ringlatencystats.h |    4 +-
> >   src/vf/{ => common}/spp_port.c         |   34 +-
> >   src/vf/{ => common}/spp_port.h         |    2 +-
> >   src/vf/common/spp_proc.c               | 1029 ++++++++++++++++
> >   src/vf/common/spp_proc.h               |  727 +++++++++++
> >   src/vf/{ => common}/string_buffer.c    |    3 +-
> >   src/vf/{ => common}/string_buffer.h    |    0
> >   src/vf/spp_forward.c                   |   47 +-
> >   src/vf/spp_forward.h                   |   12 +-
> >   src/vf/spp_vf.c                        | 1574 +-----------------------
> >   src/vf/spp_vf.h                        |  511 --------
> >   21 files changed, 3074 insertions(+), 2477 deletions(-)
> >   rename src/vf/{ => common}/command_conn.c (90%)
> >   rename src/vf/{ => common}/command_conn.h (82%)
> >   rename src/vf/{ => common}/command_dec.c (53%)
> >   rename src/vf/{ => common}/command_dec.h (90%)
> >   rename src/vf/{ => common}/command_proc.c (63%)
> >   rename src/vf/{ => common}/command_proc.h (80%)
> >   rename src/vf/{ => common}/ringlatencystats.c (92%)
> >   rename src/vf/{ => common}/ringlatencystats.h (97%)
> >   rename src/vf/{ => common}/spp_port.c (94%)
> >   rename src/vf/{ => common}/spp_port.h (99%)
> >   create mode 100644 src/vf/common/spp_proc.c
> >   create mode 100644 src/vf/common/spp_proc.h
> >   rename src/vf/{ => common}/string_buffer.c (97%)
> >   rename src/vf/{ => common}/string_buffer.h (100%)
> > 

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [spp] [spp 03641] Re: [PATCH 00/23] move some codes of spp_vf into vf/common
  2018-11-21  5:15   ` [spp] [spp 03641] " Hideyuki Yamashita
@ 2018-11-21  5:43     ` Yasufumi Ogawa
  0 siblings, 0 replies; 6+ messages in thread
From: Yasufumi Ogawa @ 2018-11-21  5:43 UTC (permalink / raw)
  To: Hideyuki Yamashita; +Cc: x-fn-spp, ferruh.yigit, spp

> Hello Yasufumi-san,
> 
> Thanks for your asking.
> Please see inline.
>>> Some spp_vf codes can be used by new secondary process which
>>> is extended from spp_vf. This patch set move such codes into
>>> spp_vf/common.
>> Hi Hideyuki,
>>
>> I would like to confirm about your changes. New secondary process you mentioned is spp_mirror which is posted as the next series of patches, right?
> Yes.
> 
>> Do you have a plan to implement secondary extended from spp_vf other
>> than spp_mirror?
> Yes. I have a plan to implement secondary extended from spp_vf other
> than spp_mirror.
I think it is required to common directory for secondary processes. SPP has common directory named `shared`, but functions and 
defines in this directory are not well considered as common actually. We will need to re-organize them by thinking usages, for 
instance, which of function is used from secondaries, or both of primary and secondaries.

I would like to re-organize them in future patches. It is OK to share `spp_vf/common` among secondaries extended from spp_vf 
until this re-organization is done.

Thanks,
Yasufumi
> 
> BR,
> Hideyuki Yamashita
> NTT TechnoCross
> 
>> Thanks,
>> Yasufumi
>>>
>>> Hideyuki Yamashita (23):
>>>     spp_vf: fix invalid code for max chars
>>>     spp_vf: move common source and header files
>>>     spp_vf: change include path
>>>     spp_vf: update Makefile for common files
>>>     spp_vf: move functions to common directory
>>>     spp_vf: move defines to common dir
>>>     spp_vf: move functions for decode to common dir
>>>     spp_vf: move defines for decode to common dir
>>>     spp_vf: move functions to command_proc.c
>>>     spp_vf: add management data registration
>>>     spp_vf: change reference of management data
>>>     spp_vf: change return values to explain result
>>>     spp_vf: define terms of commands as consts
>>>     spp_vf: remove unnecessary includes
>>>     spp_vf: add include header files
>>>     spp_vf: update comments for header file
>>>     spp_vf: update makefile of spp_vf
>>>     spp_vf: add check num of ports before forwarding
>>>     spp_vf: add flag for classifier table
>>>     spp_vf: add checking the number of ports
>>>     spp_vf: add vlantag command check flag
>>>     spp_vf: simplify changing VLAN tag
>>>     spp_vf: add SPP_VF_MODULE preprocessor directive
>>>
>>>    src/vf/Makefile                        |    7 +-
>>>    src/vf/classifier_mac.c                |  139 ++-
>>>    src/vf/classifier_mac.h                |   20 +-
>>>    src/vf/{ => common}/command_conn.c     |   23 +-
>>>    src/vf/{ => common}/command_conn.h     |   16 +-
>>>    src/vf/{ => common}/command_dec.c      |  641 +++++++---
>>>    src/vf/{ => common}/command_dec.h      |   21 +-
>>>    src/vf/{ => common}/command_proc.c     |  716 +++++++++--
>>>    src/vf/{ => common}/command_proc.h     |   10 +-
>>>    src/vf/{ => common}/ringlatencystats.c |   15 +-
>>>    src/vf/{ => common}/ringlatencystats.h |    4 +-
>>>    src/vf/{ => common}/spp_port.c         |   34 +-
>>>    src/vf/{ => common}/spp_port.h         |    2 +-
>>>    src/vf/common/spp_proc.c               | 1029 ++++++++++++++++
>>>    src/vf/common/spp_proc.h               |  727 +++++++++++
>>>    src/vf/{ => common}/string_buffer.c    |    3 +-
>>>    src/vf/{ => common}/string_buffer.h    |    0
>>>    src/vf/spp_forward.c                   |   47 +-
>>>    src/vf/spp_forward.h                   |   12 +-
>>>    src/vf/spp_vf.c                        | 1574 +-----------------------
>>>    src/vf/spp_vf.h                        |  511 --------
>>>    21 files changed, 3074 insertions(+), 2477 deletions(-)
>>>    rename src/vf/{ => common}/command_conn.c (90%)
>>>    rename src/vf/{ => common}/command_conn.h (82%)
>>>    rename src/vf/{ => common}/command_dec.c (53%)
>>>    rename src/vf/{ => common}/command_dec.h (90%)
>>>    rename src/vf/{ => common}/command_proc.c (63%)
>>>    rename src/vf/{ => common}/command_proc.h (80%)
>>>    rename src/vf/{ => common}/ringlatencystats.c (92%)
>>>    rename src/vf/{ => common}/ringlatencystats.h (97%)
>>>    rename src/vf/{ => common}/spp_port.c (94%)
>>>    rename src/vf/{ => common}/spp_port.h (99%)
>>>    create mode 100644 src/vf/common/spp_proc.c
>>>    create mode 100644 src/vf/common/spp_proc.h
>>>    rename src/vf/{ => common}/string_buffer.c (97%)
>>>    rename src/vf/{ => common}/string_buffer.h (100%)
>>>

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [spp] [PATCH 00/23] move some codes of spp_vf into vf/common
  2018-11-21  1:35 [spp] [PATCH 00/23] move some codes of spp_vf into vf/common x-fn-spp
  2018-11-21  3:47 ` Yasufumi Ogawa
@ 2018-11-24 13:15 ` Yasufumi Ogawa
  2018-11-28  2:46 ` Yasufumi Ogawa
  2 siblings, 0 replies; 6+ messages in thread
From: Yasufumi Ogawa @ 2018-11-24 13:15 UTC (permalink / raw)
  To: x-fn-spp; +Cc: ferruh.yigit, ogawa.yasufumi, spp

> Some spp_vf codes can be used by new secondary process which
> is extended from spp_vf. This patch set move such codes into
> spp_vf/common.
Hideyuki,

Your update of `spp_convert_component_type()` in `command_dec.c` has a 
problem because `type_str` variable is only used in `ifdef` preprocessor 
directive and it is considered as not used by compiler. It causes a 
compile error, so should be fixed.

I fixed this error by adding RTE_LOG() outside the preprocessor 
directive to avoid the error as following. I would like send patch if it 
is agreeable.

     RTE_LOG(DEBUG, APP, "type_str is %s\n", type_str);

Thanks,
Yasufumi
> 
> Hideyuki Yamashita (23):
>    spp_vf: fix invalid code for max chars
>    spp_vf: move common source and header files
>    spp_vf: change include path
>    spp_vf: update Makefile for common files
>    spp_vf: move functions to common directory
>    spp_vf: move defines to common dir
>    spp_vf: move functions for decode to common dir
>    spp_vf: move defines for decode to common dir
>    spp_vf: move functions to command_proc.c
>    spp_vf: add management data registration
>    spp_vf: change reference of management data
>    spp_vf: change return values to explain result
>    spp_vf: define terms of commands as consts
>    spp_vf: remove unnecessary includes
>    spp_vf: add include header files
>    spp_vf: update comments for header file
>    spp_vf: update makefile of spp_vf
>    spp_vf: add check num of ports before forwarding
>    spp_vf: add flag for classifier table
>    spp_vf: add checking the number of ports
>    spp_vf: add vlantag command check flag
>    spp_vf: simplify changing VLAN tag
>    spp_vf: add SPP_VF_MODULE preprocessor directive
> 
>   src/vf/Makefile                        |    7 +-
>   src/vf/classifier_mac.c                |  139 ++-
>   src/vf/classifier_mac.h                |   20 +-
>   src/vf/{ => common}/command_conn.c     |   23 +-
>   src/vf/{ => common}/command_conn.h     |   16 +-
>   src/vf/{ => common}/command_dec.c      |  641 +++++++---
>   src/vf/{ => common}/command_dec.h      |   21 +-
>   src/vf/{ => common}/command_proc.c     |  716 +++++++++--
>   src/vf/{ => common}/command_proc.h     |   10 +-
>   src/vf/{ => common}/ringlatencystats.c |   15 +-
>   src/vf/{ => common}/ringlatencystats.h |    4 +-
>   src/vf/{ => common}/spp_port.c         |   34 +-
>   src/vf/{ => common}/spp_port.h         |    2 +-
>   src/vf/common/spp_proc.c               | 1029 ++++++++++++++++
>   src/vf/common/spp_proc.h               |  727 +++++++++++
>   src/vf/{ => common}/string_buffer.c    |    3 +-
>   src/vf/{ => common}/string_buffer.h    |    0
>   src/vf/spp_forward.c                   |   47 +-
>   src/vf/spp_forward.h                   |   12 +-
>   src/vf/spp_vf.c                        | 1574 +-----------------------
>   src/vf/spp_vf.h                        |  511 --------
>   21 files changed, 3074 insertions(+), 2477 deletions(-)
>   rename src/vf/{ => common}/command_conn.c (90%)
>   rename src/vf/{ => common}/command_conn.h (82%)
>   rename src/vf/{ => common}/command_dec.c (53%)
>   rename src/vf/{ => common}/command_dec.h (90%)
>   rename src/vf/{ => common}/command_proc.c (63%)
>   rename src/vf/{ => common}/command_proc.h (80%)
>   rename src/vf/{ => common}/ringlatencystats.c (92%)
>   rename src/vf/{ => common}/ringlatencystats.h (97%)
>   rename src/vf/{ => common}/spp_port.c (94%)
>   rename src/vf/{ => common}/spp_port.h (99%)
>   create mode 100644 src/vf/common/spp_proc.c
>   create mode 100644 src/vf/common/spp_proc.h
>   rename src/vf/{ => common}/string_buffer.c (97%)
>   rename src/vf/{ => common}/string_buffer.h (100%)
> 

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [spp] [PATCH 00/23] move some codes of spp_vf into vf/common
  2018-11-21  1:35 [spp] [PATCH 00/23] move some codes of spp_vf into vf/common x-fn-spp
  2018-11-21  3:47 ` Yasufumi Ogawa
  2018-11-24 13:15 ` [spp] " Yasufumi Ogawa
@ 2018-11-28  2:46 ` Yasufumi Ogawa
  2 siblings, 0 replies; 6+ messages in thread
From: Yasufumi Ogawa @ 2018-11-28  2:46 UTC (permalink / raw)
  To: x-fn-spp; +Cc: ferruh.yigit, spp

> Some spp_vf codes can be used by new secondary process which
> is extended from spp_vf. This patch set move such codes into
> spp_vf/common.
Series of patches is applied. Thanks!

Acked-by: Yasufumi Ogawa <ogawa.yasufumi@lab.ntt.co.jp>
> 
> Hideyuki Yamashita (23):
>    spp_vf: fix invalid code for max chars
>    spp_vf: move common source and header files
>    spp_vf: change include path
>    spp_vf: update Makefile for common files
>    spp_vf: move functions to common directory
>    spp_vf: move defines to common dir
>    spp_vf: move functions for decode to common dir
>    spp_vf: move defines for decode to common dir
>    spp_vf: move functions to command_proc.c
>    spp_vf: add management data registration
>    spp_vf: change reference of management data
>    spp_vf: change return values to explain result
>    spp_vf: define terms of commands as consts
>    spp_vf: remove unnecessary includes
>    spp_vf: add include header files
>    spp_vf: update comments for header file
>    spp_vf: update makefile of spp_vf
>    spp_vf: add check num of ports before forwarding
>    spp_vf: add flag for classifier table
>    spp_vf: add checking the number of ports
>    spp_vf: add vlantag command check flag
>    spp_vf: simplify changing VLAN tag
>    spp_vf: add SPP_VF_MODULE preprocessor directive
> 
>   src/vf/Makefile                        |    7 +-
>   src/vf/classifier_mac.c                |  139 ++-
>   src/vf/classifier_mac.h                |   20 +-
>   src/vf/{ => common}/command_conn.c     |   23 +-
>   src/vf/{ => common}/command_conn.h     |   16 +-
>   src/vf/{ => common}/command_dec.c      |  641 +++++++---
>   src/vf/{ => common}/command_dec.h      |   21 +-
>   src/vf/{ => common}/command_proc.c     |  716 +++++++++--
>   src/vf/{ => common}/command_proc.h     |   10 +-
>   src/vf/{ => common}/ringlatencystats.c |   15 +-
>   src/vf/{ => common}/ringlatencystats.h |    4 +-
>   src/vf/{ => common}/spp_port.c         |   34 +-
>   src/vf/{ => common}/spp_port.h         |    2 +-
>   src/vf/common/spp_proc.c               | 1029 ++++++++++++++++
>   src/vf/common/spp_proc.h               |  727 +++++++++++
>   src/vf/{ => common}/string_buffer.c    |    3 +-
>   src/vf/{ => common}/string_buffer.h    |    0
>   src/vf/spp_forward.c                   |   47 +-
>   src/vf/spp_forward.h                   |   12 +-
>   src/vf/spp_vf.c                        | 1574 +-----------------------
>   src/vf/spp_vf.h                        |  511 --------
>   21 files changed, 3074 insertions(+), 2477 deletions(-)
>   rename src/vf/{ => common}/command_conn.c (90%)
>   rename src/vf/{ => common}/command_conn.h (82%)
>   rename src/vf/{ => common}/command_dec.c (53%)
>   rename src/vf/{ => common}/command_dec.h (90%)
>   rename src/vf/{ => common}/command_proc.c (63%)
>   rename src/vf/{ => common}/command_proc.h (80%)
>   rename src/vf/{ => common}/ringlatencystats.c (92%)
>   rename src/vf/{ => common}/ringlatencystats.h (97%)
>   rename src/vf/{ => common}/spp_port.c (94%)
>   rename src/vf/{ => common}/spp_port.h (99%)
>   create mode 100644 src/vf/common/spp_proc.c
>   create mode 100644 src/vf/common/spp_proc.h
>   rename src/vf/{ => common}/string_buffer.c (97%)
>   rename src/vf/{ => common}/string_buffer.h (100%)

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2018-11-28  2:49 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-11-21  1:35 [spp] [PATCH 00/23] move some codes of spp_vf into vf/common x-fn-spp
2018-11-21  3:47 ` Yasufumi Ogawa
2018-11-21  5:15   ` [spp] [spp 03641] " Hideyuki Yamashita
2018-11-21  5:43     ` Yasufumi Ogawa
2018-11-24 13:15 ` [spp] " Yasufumi Ogawa
2018-11-28  2:46 ` Yasufumi Ogawa

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).