From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by dpdk.space (Postfix) with ESMTP id 52DD4A0AE2 for ; Wed, 8 May 2019 04:02:26 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 39B7834F0; Wed, 8 May 2019 04:02:26 +0200 (CEST) Received: from tama500.ecl.ntt.co.jp (tama500.ecl.ntt.co.jp [129.60.39.148]) by dpdk.org (Postfix) with ESMTP id 6B325A49 for ; Wed, 8 May 2019 04:02:23 +0200 (CEST) Received: from vc1.ecl.ntt.co.jp (vc1.ecl.ntt.co.jp [129.60.86.153]) by tama500.ecl.ntt.co.jp (8.13.8/8.13.8) with ESMTP id x4822Miq023023; Wed, 8 May 2019 11:02:22 +0900 Received: from vc1.ecl.ntt.co.jp (localhost [127.0.0.1]) by vc1.ecl.ntt.co.jp (Postfix) with ESMTP id 4FE13EA7089; Wed, 8 May 2019 11:02:22 +0900 (JST) Received: from localhost.localdomain (lobster.nslab.ecl.ntt.co.jp [129.60.13.95]) by vc1.ecl.ntt.co.jp (Postfix) with ESMTP id 42E3CEA407F; Wed, 8 May 2019 11:02:22 +0900 (JST) From: ogawa.yasufumi@lab.ntt.co.jp To: spp@dpdk.org, ferruh.yigit@intel.com, ogawa.yasufumi@lab.ntt.co.jp Date: Wed, 8 May 2019 11:00:02 +0900 Message-Id: <1557280806-7672-1-git-send-email-ogawa.yasufumi@lab.ntt.co.jp> X-Mailer: git-send-email 2.7.4 X-TM-AS-MML: disable Subject: [spp] [PATCH 0/4] Move vf/common to under shared directory X-BeenThere: spp@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: Soft Patch Panel List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: spp-bounces@dpdk.org Sender: "spp" From: Yasufumi Ogawa Spp_vf and its siblings use common libs `srv/vf/common`. However, it is not good design considering conventions, but should be placed `src/shared` in the case of SPP. In addition, codes of spp_vf siblings are not neat and tightly coupled and many of functions in `vf/common` uses spp_vf siblings and vice versa. This series of patches is to fix the issue by moving common libs to under `src/shared` and remove dependency referring from common functions to each of siblings. However, some of functions used specific sibling, which are not common actually, are still remained as `src/shared/secondary/spp_worker_th/vf_deps.h` because of too much complexity of data structure and functions. This `vf_deps.h` is released in next update, but will be removed from shared dir in a future update for refactoring. Yasufumi Ogawa (4): spp_vf: move common of vf siblings to shared dir spp_vf: update makefile for moved common files spp_vf: update include paths spp_vf: remove dependency from common functions .../secondary/spp_worker_th}/command_conn.c | 0 .../secondary/spp_worker_th}/command_conn.h | 0 .../secondary/spp_worker_th}/command_dec.c | 0 .../secondary/spp_worker_th}/command_dec.h | 0 .../secondary/spp_worker_th}/command_proc.c | 37 +++++- .../secondary/spp_worker_th}/command_proc.h | 0 .../spp_worker_th}/ringlatencystats.c | 0 .../spp_worker_th}/ringlatencystats.h | 0 .../secondary/spp_worker_th}/spp_port.c | 0 .../secondary/spp_worker_th}/spp_port.h | 0 .../secondary/spp_worker_th}/spp_proc.c | 5 +- .../secondary/spp_worker_th}/spp_proc.h | 0 .../secondary/spp_worker_th}/string_buffer.c | 0 .../secondary/spp_worker_th}/string_buffer.h | 0 src/vf/Makefile | 15 ++- src/vf/classifier_mac.c | 121 +----------------- src/vf/classifier_mac.h | 32 ----- src/vf/spp_forward.c | 3 +- src/vf/spp_forward.h | 30 ----- src/vf/spp_vf.c | 8 +- 20 files changed, 54 insertions(+), 197 deletions(-) rename src/{vf/common => shared/secondary/spp_worker_th}/command_conn.c (100%) rename src/{vf/common => shared/secondary/spp_worker_th}/command_conn.h (100%) rename src/{vf/common => shared/secondary/spp_worker_th}/command_dec.c (100%) rename src/{vf/common => shared/secondary/spp_worker_th}/command_dec.h (100%) rename src/{vf/common => shared/secondary/spp_worker_th}/command_proc.c (98%) rename src/{vf/common => shared/secondary/spp_worker_th}/command_proc.h (100%) rename src/{vf/common => shared/secondary/spp_worker_th}/ringlatencystats.c (100%) rename src/{vf/common => shared/secondary/spp_worker_th}/ringlatencystats.h (100%) rename src/{vf/common => shared/secondary/spp_worker_th}/spp_port.c (100%) rename src/{vf/common => shared/secondary/spp_worker_th}/spp_port.h (100%) rename src/{vf/common => shared/secondary/spp_worker_th}/spp_proc.c (99%) rename src/{vf/common => shared/secondary/spp_worker_th}/spp_proc.h (100%) rename src/{vf/common => shared/secondary/spp_worker_th}/string_buffer.c (100%) rename src/{vf/common => shared/secondary/spp_worker_th}/string_buffer.h (100%) -- 2.17.1