From patchwork Fri Jan 20 20:31:59 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Mike Holmes X-Patchwork-Id: 92110 Delivered-To: patch@linaro.org Received: by 10.140.20.99 with SMTP id 90csp194942qgi; Fri, 20 Jan 2017 12:32:17 -0800 (PST) X-Received: by 10.200.49.157 with SMTP id h29mr14291978qte.289.1484944337039; Fri, 20 Jan 2017 12:32:17 -0800 (PST) Return-Path: Received: from lists.linaro.org (lists.linaro.org. [54.225.227.206]) by mx.google.com with ESMTP id o13si5628566qki.3.2017.01.20.12.32.16; Fri, 20 Jan 2017 12:32:17 -0800 (PST) Received-SPF: pass (google.com: domain of lng-odp-bounces@lists.linaro.org designates 54.225.227.206 as permitted sender) client-ip=54.225.227.206; Authentication-Results: mx.google.com; spf=pass (google.com: domain of lng-odp-bounces@lists.linaro.org designates 54.225.227.206 as permitted sender) smtp.mailfrom=lng-odp-bounces@lists.linaro.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: by lists.linaro.org (Postfix, from userid 109) id 39B3E60C21; Fri, 20 Jan 2017 20:32:16 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on ip-10-142-244-252 X-Spam-Level: X-Spam-Status: No, score=-2.6 required=5.0 tests=BAYES_00, RCVD_IN_DNSWL_LOW, RCVD_IN_MSPIKE_H3, RCVD_IN_MSPIKE_WL autolearn=disabled version=3.4.0 Received: from [127.0.0.1] (localhost [127.0.0.1]) by lists.linaro.org (Postfix) with ESMTP id 10A5460AF3; Fri, 20 Jan 2017 20:32:11 +0000 (UTC) X-Original-To: lng-odp@lists.linaro.org Delivered-To: lng-odp@lists.linaro.org Received: by lists.linaro.org (Postfix, from userid 109) id C9FA260AF7; Fri, 20 Jan 2017 20:32:08 +0000 (UTC) Received: from mail-yb0-f174.google.com (mail-yb0-f174.google.com [209.85.213.174]) by lists.linaro.org (Postfix) with ESMTPS id 97FDA60AED for ; Fri, 20 Jan 2017 20:32:07 +0000 (UTC) Received: by mail-yb0-f174.google.com with SMTP id j82so69997317ybg.1 for ; Fri, 20 Jan 2017 12:32:07 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id; bh=fMiDHzGlZp0nOLaHdecCa6H6ohViUy7UHHWzKr+gMQI=; b=thOD3NuVJoXR921QAgqdNnYbckl6+i+5qphwJriPymyin4mktofidRCPI+g7HQ3et8 kEG+2s4vhWScEUwMjLhuezZadZiGpaDVYoSGqkNOA6GdDku1ij/lKLJ8JMrmd9CIpR3C CAKigxroTqEoxXHWInz7Xj1wKLo1nEwZes8msdGqWqNq7vgbU9WEzJjrjjgO26LeiJZF XuLWZssXBZ36bgCfbuxv74n1Qf4383592JWxdXzi9E51oT7bZECMeL0o7crxFqOFqJqJ VuchNnjrtHt1ulfc/glntawBFKFHGJoKQO7K8hbKJhcfm7BWSGS4KMskcZGs7lSX90WJ /V9Q== X-Gm-Message-State: AIkVDXID3TqvTKbe3RHHwcAnpvekARY08cwT4O8tDOVkBdr4FMYBSlA1pncUMJsiS4cp0cjM/Oc= X-Received: by 10.55.159.206 with SMTP id i197mr13112123qke.175.1484944326714; Fri, 20 Jan 2017 12:32:06 -0800 (PST) Received: from localhost (c-98-221-136-245.hsd1.nj.comcast.net. [98.221.136.245]) by smtp.gmail.com with ESMTPSA id h33sm5525550qtc.42.2017.01.20.12.32.05 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Fri, 20 Jan 2017 12:32:05 -0800 (PST) From: Mike Holmes To: lng-odp@lists.linaro.org Date: Fri, 20 Jan 2017 15:31:59 -0500 Message-Id: <20170120203203.2454-1-mike.holmes@linaro.org> X-Mailer: git-send-email 2.9.3 Subject: [lng-odp] [PATCH v4 0/4] remove Linux specifics and dependence on test X-BeenThere: lng-odp@lists.linaro.org X-Mailman-Version: 2.1.16 Precedence: list List-Id: "The OpenDataPlane \(ODP\) List" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: lng-odp-bounces@lists.linaro.org Sender: "lng-odp" Starting with some clean up, rename the Linux specific files and then move them to a platform specific directory. A new configure option is introduced "with-helper-platform", this defaults to match the existing odp selector "with-platform" so that there is no operational change. With the move made the default is to build only the portable helper API, if the legacy and previously unused Linux thread APIs are needed they can be built in with --enable-helper-extn Then remove the dependence on the test directory. v2: sort some lists into alphabetical order (Maxim) addtional code simplification removing a .h helper(Christophe) v3: rebase whitespace error is there due to matching existing formatting in file v4: remove CXX, it is not used alignment in configure.ac Carried from v3 For the series: Reviewed-by: Christophe Milard Mike Holmes (4): configure: use helper configure.m4 helper: cleanup Linux rename to thread helper: move thread implementation under platform helper: remove dependence on test dir configure.ac | 19 +- example/Makefile.inc | 2 +- example/classifier/odp_classifier.c | 2 +- example/generator/odp_generator.c | 2 +- example/ipsec/odp_ipsec.c | 2 +- example/l2fwd_simple/odp_l2fwd_simple.c | 2 +- example/l3fwd/odp_l3fwd.c | 2 +- example/packet/odp_pktio.c | 2 +- example/switch/odp_switch.c | 2 +- example/time/time_global_test.c | 5 +- example/timer/odp_timer_test.c | 2 +- helper/Makefile.am | 22 +- .../helper/platform/linux-generic/threads_extn.h | 112 ++++++++ helper/include/odp/helper/{linux.h => threads.h} | 78 ----- helper/m4/configure.m4 | 14 + helper/platform/linux-generic/thread.c | 313 +++++++++++++++++++++ helper/test/Makefile.am | 42 ++- helper/test/chksum.c | 18 +- helper/test/cuckootable.c | 4 +- helper/test/iplookuptable.c | 4 +- helper/test/linux-generic/Makefile.am | 5 + helper/test/{ => linux-generic}/process.c | 16 +- helper/test/{ => linux-generic}/thread.c | 16 +- helper/test/odpthreads.c | 20 +- helper/test/parse.c | 86 +++--- helper/test/table.c | 12 +- helper/{linux.c => threads.c} | 240 +--------------- ...inux.pc.in => libodphelper-linux-generic.pc.in} | 4 +- test/Makefile.inc | 2 +- test/common_plat/common/odp_cunit_common.c | 2 +- .../common_plat/miscellaneous/odp_api_from_cpp.cpp | 2 +- test/common_plat/performance/odp_crypto.c | 2 +- test/common_plat/performance/odp_l2fwd.c | 2 +- test/common_plat/performance/odp_pktio_perf.c | 2 +- test/common_plat/performance/odp_sched_latency.c | 2 +- test/common_plat/performance/odp_scheduling.c | 2 +- test/common_plat/validation/api/Makefile.inc | 2 +- test/common_plat/validation/api/timer/timer.c | 2 +- test/linux-generic/Makefile.inc | 2 +- test/linux-generic/mmap_vlan_ins/mmap_vlan_ins.c | 2 +- test/linux-generic/pktio_ipc/ipc_common.h | 2 +- test/linux-generic/ring/ring_stress.c | 2 +- 42 files changed, 632 insertions(+), 444 deletions(-) create mode 100644 helper/include/odp/helper/platform/linux-generic/threads_extn.h rename helper/include/odp/helper/{linux.h => threads.h} (74%) create mode 100644 helper/platform/linux-generic/thread.c create mode 100644 helper/test/linux-generic/Makefile.am rename helper/test/{ => linux-generic}/process.c (84%) rename helper/test/{ => linux-generic}/thread.c (84%) rename helper/{linux.c => threads.c} (67%) rename pkgconfig/{libodphelper-linux.pc.in => libodphelper-linux-generic.pc.in} (72%) -- 2.9.3