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 15022A046B for ; Mon, 24 Jun 2019 06:36:26 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id C9C2D1BF2C; Mon, 24 Jun 2019 06:36:25 +0200 (CEST) Received: from mail-pl1-f173.google.com (mail-pl1-f173.google.com [209.85.214.173]) by dpdk.org (Postfix) with ESMTP id 3851B1BF2C for ; Mon, 24 Jun 2019 06:36:24 +0200 (CEST) Received: by mail-pl1-f173.google.com with SMTP id c14so6150972plo.0 for ; Sun, 23 Jun 2019 21:36:24 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:subject:date:message-id; bh=MZZgitWzwyaWtHFjvxDmJqgiS0H81WxO/Gpxn7kHVWU=; b=SrxtLfUfEGQvxGoF3DOsaTb1UxMlO5f14lG4vki2NqSEjQOcJw1bH9IiUOixh2m77H R28Qtkf+kyfNa7uFIL6OQ1mJLCnE01/fT3WUQkVkwTyMksVDC+exCzuRi4CWTzU8cn9o ISrQSfpRtLfwUnXnG0vySBpacz6kndMK6Mqwhnq2X/JZCjvtLF+0iPlWp2wEwxgEOok4 nL2zfyvJZekRLeLnEWKzI0YE81UPqVHSsXuc5uzMNCPkzlOmhBpS8F7wXT6JYZ5ig1wx fQVsOD5M7RLHCdlb7fwgdu7xvg4gkXda6dfukFr3r/r9HFZeunIcC4FqKKxyuGHc9mCK NEvA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:subject:date:message-id; bh=MZZgitWzwyaWtHFjvxDmJqgiS0H81WxO/Gpxn7kHVWU=; b=sJ0KG6fxBGtmDm+J/mkXNWzcbgHERJG+4agqVuNMjuk4wkv0JyuoBsJYeTqn09W0UF rkw9yCGiaMy+GfGqLFoKaqLsH6fQAEdZb1MMyoMXZuk+prGN0HmX+Qka+6fVMlwEmPT3 g8Hf+IgYTgYQuiM3qG2v8AQvfhWDQZal2Lp3ATp6kilRTM9kOfn8Ea8MgxWndM5sWjPu mPO9phdYak+9Ktm1iekSJHOfCZZ6RsjIKWUQ7L8f9EgQdCdbIKAQAQIBYT58B6S4FJJL Rm3sxbI+1+nlelM+JzGSHu+9byIeDD21YmKyQaAP6dulerwT/4+iOvdmVC6Qz3jasL2I Yj+A== X-Gm-Message-State: APjAAAWN4UloisCo5rJjam44A4sfOTvMxzyzQoAVQCJ+Wgj0cB2Qz9mu eOzxZd0CamScUA3rKO+AMwD3MYch X-Google-Smtp-Source: APXvYqyAI9WGGr8nFKAgCKYyoM0GMyv3tslObcdvo1N8zXIVmaMOGDNMg75q9Fbcq8XigDMqskCG9Q== X-Received: by 2002:a17:902:2a69:: with SMTP id i96mr136536305plb.108.1561350983332; Sun, 23 Jun 2019 21:36:23 -0700 (PDT) Received: from localhost.localdomain ([192.47.164.146]) by smtp.gmail.com with ESMTPSA id c10sm9005178pjq.14.2019.06.23.21.36.21 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Sun, 23 Jun 2019 21:36:22 -0700 (PDT) From: yasufum.o@gmail.com To: spp@dpdk.org, ferruh.yigit@intel.com, yasufum.o@gmail.com Date: Mon, 24 Jun 2019 13:36:00 +0900 Message-Id: <20190624043613.19271-1-yasufum.o@gmail.com> X-Mailer: git-send-email 2.17.1 Subject: [spp] [PATCH 00/13] Move JSON utils from libs for running cmds 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 There are many JSON utils in shared lib for running commands, and hard to maintain because several features are defined in a huge file. This series of patches is to separate JSON utils from other functions and fix complex dependencies among them. Yasufumi Ogawa (13): shared/sec: rename ops for setup cmd response shared/sec: rename functions for spp_mirror shared/sec: move principle JSON formatter funcs shared/sec: change order of args of JSON fmtters shared/sec: move JSON formatter to shard/secondary shared/sec: revise including headers shared/sec: move JSON formatters from cmd_runner shared/sec: move rest of JSON formatters shared/sec: move lcore funcs in response_info_list shared/sec: move ope cli-id in response_info_list shared/sec: move rest of ops in response_info_list shared/sec: remove local funcs from header shared/sec: refactor comments for JSON formatter src/mirror/Makefile | 11 +- src/mirror/spp_mirror.c | 75 +- src/mirror/spp_mirror.h | 26 - src/pcap/Makefile | 3 +- src/pcap/cmd_runner.c | 2 +- src/shared/secondary/json_helper.c | 136 +++ src/shared/secondary/json_helper.h | 101 ++ src/shared/secondary/return_codes.h | 13 + .../secondary/spp_worker_th/cmd_parser.c | 1 + .../spp_worker_th/cmd_res_formatter.c | 831 +++++++++++++++ .../spp_worker_th/cmd_res_formatter.h | 83 ++ .../secondary/spp_worker_th/cmd_runner.c | 952 +----------------- .../secondary/spp_worker_th/cmd_utils.c | 6 +- .../secondary/spp_worker_th/cmd_utils.h | 8 +- .../secondary/spp_worker_th/conn_spp_ctl.c | 3 +- .../secondary/spp_worker_th/mirror_deps.h | 33 +- src/shared/secondary/spp_worker_th/spp_port.c | 1 + src/shared/secondary/spp_worker_th/vf_deps.h | 2 +- .../{spp_worker_th => }/string_buffer.c | 0 .../{spp_worker_th => }/string_buffer.h | 2 + src/vf/Makefile | 7 +- src/vf/classifier_mac.c | 6 +- src/vf/spp_forward.c | 5 +- src/vf/spp_vf.c | 3 +- 24 files changed, 1260 insertions(+), 1050 deletions(-) delete mode 100644 src/mirror/spp_mirror.h create mode 100644 src/shared/secondary/json_helper.c create mode 100644 src/shared/secondary/json_helper.h create mode 100644 src/shared/secondary/return_codes.h create mode 100644 src/shared/secondary/spp_worker_th/cmd_res_formatter.c create mode 100644 src/shared/secondary/spp_worker_th/cmd_res_formatter.h rename src/shared/secondary/{spp_worker_th => }/string_buffer.c (100%) rename src/shared/secondary/{spp_worker_th => }/string_buffer.h (98%) -- 2.17.1