Changes in Makefile

5370a2c123b0b5ba79d0aa7d5edc7db15f58d8d3 - test: convert remaining tests using CUnit to spdk_ut_run_tests()

Mon, 31 Jul 2023 11:17:51 +0000

test: convert remaining tests using CUnit to spdk_ut_run_tests()The test apps that use CUnit framework for executing functional testswere converted to use the ut library, with one exception - bdevio.That's because bdevio uses app framework, which provides its own methodof parsing command-line arguments.Signed-off-by: Konrad Sztyber Change-Id: Icb2bcc172686894095bb48fd4e5aae10a84d14dbReviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/19291Reviewed-by: Tomasz Zawadzki Tested-by: SPDK CI Jenkins Reviewed-by: Aleksey Marchuk List of files: /spdk/test/env/vtophys/Makefile

a6dbe3721eb3b5990707fc3e378c95e505dd8ab5 - update Intel copyright notices

Tue, 01 Nov 2022 20:26:26 +0000

update Intel copyright noticesper Intel policy to include file commit date using git cmdbelow. The policy does not apply to non-Intel (C) notices.git log --follow -C90% --format=%ad --date default | tail -1and then pull just the 4 digit year from the result.Intel copyrights were not added to files where Intel either hadno contribution ot the contribution lacked substance (ie licenseheader updates, formatting changes, etc). Contribution date used"--follow -C95%" to get the most accurate date.Note that several files in this patch didn't end the license/(c)block with a blank comment line so these were added as the vastmajority of files do have this last blank line. Simply there forconsistency.Signed-off-by: paul luse Change-Id: Id5b7ce4f658fe87132f14139ead58d6e285c04d4Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/15192Tested-by: SPDK CI Jenkins Reviewed-by: Jim Harris Reviewed-by: Ben Walker Community-CI: Mellanox Build Bot List of files: /spdk/test/env/vtophys/Makefile

488570ebd418ba07c9e69e65106dcc964f3bb41b - Replace most BSD 3-clause license text with SPDX identifier.

Fri, 03 Jun 2022 19:15:11 +0000

Replace most BSD 3-clause license text with SPDX identifier.Many open source projects have moved to using SPDX identifiersto specify license information, reducing the amount ofboilerplate code in every source file. This patch replacesthe bulk of SPDK .c, .cpp and Makefiles with the BSD-3-Clauseidentifier.Almost all of these files share the exact same license text,and this patch only modifies the files that contain themost common license text. There can be slight variationsbecause the third clause contains company names - most say"Intel Corporation", but there are instances for Nvidia,Samsung, Eideticom and even "the copyright holder".Used a bash script to automate replacement of the license textwith SPDX identifier which is checked into scripts/spdx.sh.Signed-off-by: Jim Harris Change-Id: Iaa88ab5e92ea471691dc298cfe41ebfb5d169780Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/12904Community-CI: Broadcom CI Community-CI: Mellanox Build BotTested-by: SPDK CI Jenkins Reviewed-by: Aleksey Marchuk Reviewed-by: Changpeng Liu Reviewed-by: Dong Yi Reviewed-by: Konrad Sztyber Reviewed-by: Paul Luse Reviewed-by: List of files: /spdk/test/env/vtophys/Makefile

d8190d02881a0528080686df63dabe0dcc7300bd - lib/env_dpdk: replace printf() and fprintf(stderr,) usage

Mon, 20 Apr 2020 19:47:03 +0000

lib/env_dpdk: replace printf() and fprintf(stderr,) usageAnd modify test/env/vtophys to resolve linking errors.SPDK_PRINTF() and SPDK_ERRLOG() use spdk_log() procedure which iscustomizable and redirectable, so it is preffered over printf()In case of test/env/vtophys/ program,we have to make it an app first to avoid linking errors.Change-Id: Id806ec3bb235745316063bbdf6b5a15a9d5dc2d9Signed-off-by: Vitaliy Mysak Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/1944Community-CI: Mellanox Build BotCommunity-CI: Broadcom CITested-by: SPDK CI Jenkins Reviewed-by: Jim Harris Reviewed-by: Aleksey Marchuk List of files: /spdk/test/env/vtophys/Makefile

63b3b8fd8f7e6cb6a348c2fcdd189aa62baf44f6 - test/unit: remove duplicate mk/spdk.common.mk includes

Fri, 08 Feb 2019 23:04:19 +0000

test/unit: remove duplicate mk/spdk.common.mk includesmk/spdk.unittest.mk already includes mk/spdk.common.mk, so it'snot needed. This also fixes an issue where touching an included.mk file would not trigger unit tests to rebuild if they hadthis duplicated mk/spdk.common.mk include.Signed-off-by: Jim Harris Change-Id: I41a04eb77ce468849cb9b53bd1f76df6fec06e46Reviewed-on: https://review.gerrithub.io/c/443980Tested-by: SPDK CI Jenkins Reviewed-by: Shuhei Matsumoto Reviewed-by: Changpeng Liu Reviewed-by: Darek Stojaczyk List of files: /spdk/test/env/vtophys/Makefile

9c2aea2ad581e78982146835ecab3b12107d25d6 - build: remove duplicate spdk.app.mk includes

Wed, 21 Nov 2018 08:34:15 +0000

build: remove duplicate spdk.app.mk includesspdk.unittest.mk includes spdk.app.mk, but some unit testMakefiles include both spdk.unittest.mk and spdk.app.mk,meaning spdk.app.mk gets included twice. Fix that.This hasn't been an issue because spdk.app.mk currentlyonly includes variables - but no rules.Signed-off-by: Jim Harris Change-Id: I23d39e8084f79442fb06ae9b5a6a68d6134adff4Reviewed-on: https://review.gerrithub.io/434281Tested-by: SPDK CI Jenkins Chandler-Test-Pool: SPDK Automated Test System Reviewed-by: Shuhei Matsumoto Reviewed-by: Ben Walker List of files: /spdk/test/env/vtophys/Makefile

40f3800480172bf0c0b4a2fe8df361b94ba7fb04 - build: remove ADDITIONAL_LIBS

Tue, 20 Nov 2018 22:24:09 +0000

build: remove ADDITIONAL_LIBSThis can just be replaced with ENV_LIBS - similarto other application Makefiles.Signed-off-by: Jim Harris Change-Id: Icad14e6f74264389972c7798aac493e584f7c8c3Reviewed-on: https://review.gerrithub.io/434280Tested-by: SPDK CI Jenkins Reviewed-by: Shuhei Matsumoto Reviewed-by: Ben Walker List of files: /spdk/test/env/vtophys/Makefile

3da5847023313f15789f0115d4a2039fc9c68e3d - build: add ENV_LINKER_ARGS to LINK_C and LINK_CXX

Tue, 20 Nov 2018 16:59:47 +0000

build: add ENV_LINKER_ARGS to LINK_C and LINK_CXXThis allows us to remove $(ENV_LINKER_ARGS) from all of theapplication Makefiles.Modify spdk.unittest.mk to explicitly set ENV_LINKER_ARGSto nothing, to avoid DPDK libraries from getting linkedinto unit tests. This seems better than creating separateLINK_C rules for unit tests. Provide a UNIT_TEST_LINK_ENVflag to allow select unit tests to override this behavior -it's needed for some of the env unit tests.Signed-off-by: Jim Harris Change-Id: I647f2c848cf434023cf5a1f3a45869bbc4fc8102Reviewed-on: https://review.gerrithub.io/434145Tested-by: SPDK CI Jenkins Chandler-Test-Pool: SPDK Automated Test System Reviewed-by: Darek Stojaczyk Reviewed-by: Shuhei Matsumoto List of files: /spdk/test/env/vtophys/Makefile

4a7240a7f54e505ea40a0109ab31c2bf1d10d2a3 - make: Fix test app Makefile dependencies

Sun, 16 Sep 2018 01:07:44 +0000

make: Fix test app Makefile dependenciesAdd dependency $(ENV_LIBS) for test/env/* as part of new variable $(ADDITIONAL_LIBS) which is direct dependency of $(APP) in mk/spdk.unittest.mkThis solves the problem of make not triggering LINKing for test/env/* after changes made to lib/env_dpdk/*.cIssue was that Makefiles in test/env/*/ classified tests as unittests by including mk/spdk.unittest.mkBut unittests are not using env_dpdk which means that unittests don't relink on env_dpdk updateChange-Id: I91b6690dc92d68bc69bbf11ac5336d3a7f56e4e4Signed-off-by: Vitaliy Mysak Reviewed-on: https://review.gerrithub.io/425665Chandler-Test-Pool: SPDK Automated Test System Tested-by: SPDK CI Jenkins Reviewed-by: Dariusz Stojaczyk Reviewed-by: Jim Harris Reviewed-by: Shuhei Matsumoto List of files: /spdk/test/env/vtophys/Makefile

dc14118bb24a993c4dddda777ff342cab741063b - test: move lib/env up one level

Thu, 22 Mar 2018 23:10:42 +0000

test: move lib/env up one levelChange-Id: Ic22806e7c85f579329ac43b9a4e31eab2a14f1c0Signed-off-by: Seth Howell Reviewed-on: https://review.gerrithub.io/404978Tested-by: SPDK Automated Test System Reviewed-by: Daniel Verkamp Reviewed-by: Jim Harris List of files: /spdk/test/env/vtophys/Makefile