apps: set NULL as RPC address in some appsSet opts.rpc_addr to NULL in SPDK applications that usually do notbenefit from starting RPC interface. These include programs in app,examples and test di
apps: set NULL as RPC address in some appsSet opts.rpc_addr to NULL in SPDK applications that usually do notbenefit from starting RPC interface. These include programs in app,examples and test directories.Some of the tests now require that we explicitly set RPC address,because some apps no longer use the default SPDK address.Change-Id: I417a3a756581f44c711257e6d2408d933b40386aSigned-off-by: Krzysztof Karas <krzysztof.karas@intel.com>Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/22634Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>Reviewed-by: Shuhei Matsumoto <smatsumoto@nvidia.com>Community-CI: Mellanox Build Bot
show more ...
thread: add spdk_thread_is_app_thread()This simply returns true if the current spdk_threadis the app thread.This is a simpler replacement for:spdk_thread_get_app_thread() == spdk_get_thread()
thread: add spdk_thread_is_app_thread()This simply returns true if the current spdk_threadis the app thread.This is a simpler replacement for:spdk_thread_get_app_thread() == spdk_get_thread()Signed-off-by: Jim Harris <james.r.harris@intel.com>Change-Id: Ib8754d4ecb91ee8bfb024e8c0b74c42bb8282b32Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/18355Community-CI: Mellanox Build BotTested-by: SPDK CI Jenkins <sys_sgci@intel.com>Reviewed-by: Shuhei Matsumoto <smatsumoto@nvidia.com>Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>
thread: test SPDK spinlocks in an applicationThis exercises the parts of spdk_spin_*() that are difficult to test inunit tests. In particular, it tests multiple SPDK threads running ondifferent p
thread: test SPDK spinlocks in an applicationThis exercises the parts of spdk_spin_*() that are difficult to test inunit tests. In particular, it tests multiple SPDK threads running ondifferent pthreads contending for a lock and it tests pollers andmessages going off CPU with a lock held.Change-Id: I5cd6ce29c92c44ba63f47332fe339e59eed81553Signed-off-by: Mike Gerdts <mgerdts@nvidia.com>Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/15534Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>Reviewed-by: Jim Harris <james.r.harris@intel.com>Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com>Reviewed-by: Shuhei Matsumoto <smatsumoto@nvidia.com>
add (c) and SPDX header to bash files as neededper Intel policy to include file commit date using git cmdbelow. The policy does not apply to non-Intel (C) notices.git log --follow -C90% --forma
add (c) and SPDX header to bash files as neededper 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 <file> | tail -1and then pull just the 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)For intel copyrights added, --follow and -C95% were used.Signed-off-by: paul luse <paul.e.luse@intel.com>Change-Id: I2ef86976095b88a9bf5b1003e59f3943cd6bbe4cReviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/15209Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>Community-CI: Mellanox Build BotReviewed-by: Ben Walker <benjamin.walker@intel.com>Reviewed-by: Krzysztof Karas <krzysztof.karas@intel.com>Reviewed-by: Jim Harris <james.r.harris@intel.com>
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 defa
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 <file> | 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 <paul.e.luse@intel.com>Change-Id: Id5b7ce4f658fe87132f14139ead58d6e285c04d4Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/15192Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>Reviewed-by: Jim Harris <james.r.harris@intel.com>Reviewed-by: Ben Walker <benjamin.walker@intel.com>Community-CI: Mellanox Build Bot
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
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 <james.r.harris@intel.com>Change-Id: Iaa88ab5e92ea471691dc298cfe41ebfb5d169780Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/12904Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com>Community-CI: Mellanox Build BotTested-by: SPDK CI Jenkins <sys_sgci@intel.com>Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com>Reviewed-by: Changpeng Liu <changpeng.liu@intel.com>Reviewed-by: Dong Yi <dongx.yi@intel.com>Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>Reviewed-by: Paul Luse <paul.e.luse@intel.com>Reviewed-by: <qun.wan@intel.com>
test/thread: Add poller_perf to evaluate cost to run pollerAdd a simple tool, poller_perf, to evaluate cost to run non-timedpoller or timed poller by any period, number of pollers.struct spdk_th
test/thread: Add poller_perf to evaluate cost to run pollerAdd a simple tool, poller_perf, to evaluate cost to run non-timedpoller or timed poller by any period, number of pollers.struct spdk_thread and struct spdk_thread_stats can have the countof poller runs but we do not have any particular use case for it.So the poller_perf tool use its global counter for now.By following similar tools, event_perf and reactor_perf, runpoller_perf once for non-timed poller and once for timed poller.Signed-off-by: Shuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com>Change-Id: Ic5b297bb23d0cd0dc1313e1638ac540ffd5fb8f8Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/8925Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com>Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>Reviewed-by: Aleksey Marchuk <alexeymar@mellanox.com>Reviewed-by: Jim Harris <james.r.harris@intel.com>