| 68d213ea | 09-Apr-2024 |
Krzysztof Karas <krzysztof.karas@intel.com> |
sock: add default impl override
Read environmental variable 'SPDK_SOCK_IMPL_DEFAULT' and override the default socket implementation (POSIX) with its value during sock_subsystem_init().
This is done
sock: add default impl override
Read environmental variable 'SPDK_SOCK_IMPL_DEFAULT' and override the default socket implementation (POSIX) with its value during sock_subsystem_init().
This is done mainly to enable original behavior where a socket with highest priority would be used by default (e.g. uring if configure was run with '--with-uring' option). That way we may explicitly enable chosen implementation as default, without relying on RPC interface.
As part of this patch, link sock modules into the app_repeat test app. The method used here works even when for the app repeat case, so let's test it. This allows us to just generally declare this flag for all tests and not have to worry about apps like app_repeat that link in the sock library but none of the sock modules.
Change-Id: I6af633cbe5bfba065580ba6c501169cbb5b64541 Signed-off-by: Krzysztof Karas <krzysztof.karas@intel.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/22691 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Jim Harris <jim.harris@samsung.com> Community-CI: Mellanox Build Bot Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>
show more ...
|
| 5db859da | 04-Apr-2024 |
Krzysztof Karas <krzysztof.karas@intel.com> |
apps: set NULL as RPC address in some apps
Set opts.rpc_addr to NULL in SPDK applications that usually do not benefit from starting RPC interface. These include programs in app, examples and test di
apps: set NULL as RPC address in some apps
Set opts.rpc_addr to NULL in SPDK applications that usually do not benefit 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: I417a3a756581f44c711257e6d2408d933b40386a Signed-off-by: Krzysztof Karas <krzysztof.karas@intel.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/22634 Tested-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 ...
|
| 2e7c6821 | 26-Sep-2023 |
Artur Paszkiewicz <artur.paszkiewicz@intel.com> |
test/event: wait for app_repeat to restart rpc server
It can happen that the test will begin a next iteration before the current spdk app instance shuts down. Add a sleep after sending SIGTERM to as
test/event: wait for app_repeat to restart rpc server
It can happen that the test will begin a next iteration before the current spdk app instance shuts down. Add a sleep after sending SIGTERM to assure spdk is restarted before the next test round.
Change-Id: Ie8653b8670014d2dd69865f87417ea1d552f61c9 Signed-off-by: Artur Paszkiewicz <artur.paszkiewicz@intel.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/20048 Reviewed-by: Jim Harris <jim.harris@samsung.com> Reviewed-by: Sebastian Brzezinka <sebastian.brzezinka@intel.com> Reviewed-by: Michal Berger <michal.berger@intel.com> Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
show more ...
|
| 4641aa53 | 24-Aug-2023 |
Michal Berger <michal.berger@intel.com> |
test/common: Provide {ext,null}glob by default in autotest suite
This requires adjustments in several test suites due to some bad practices exposed by explicit use of nullglob.
Change-Id: Idbe7eeec
test/common: Provide {ext,null}glob by default in autotest suite
This requires adjustments in several test suites due to some bad practices exposed by explicit use of nullglob.
Change-Id: Idbe7eeec598ffc06e98986898344e4e0d88f4a5d Signed-off-by: Michal Berger <michal.berger@intel.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/19551 Reviewed-by: Jim Harris <jim.harris@gmail.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>
show more ...
|
| 21b3ff1b | 27-Feb-2023 |
Krzysztof Karas <krzysztof.karas@intel.com> |
cpu_locks.sh: sleep after checking for core locks
If core locks still exist after disabling them, sleep one second and recheck for their existence.
This change aims to help debug #2920.
Change-Id:
cpu_locks.sh: sleep after checking for core locks
If core locks still exist after disabling them, sleep one second and recheck for their existence.
This change aims to help debug #2920.
Change-Id: I35c46feb7b58acee22ab0ed51bc5efe2e714c65e Signed-off-by: Krzysztof Karas <krzysztof.karas@intel.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/16979 Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com> Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
show more ...
|
| de95290c | 28-Oct-2022 |
wanghailiangx <hailiangx.e.wang@intel.com> |
autotest: unify style for test/ and ./test/
Although they are all correct. I suggest to unify, so it looks more normalized.
Change-Id: I61ee92b2ac9f2260851e0d7e28ebaea8783423f6 Signed-off-by: Haili
autotest: unify style for test/ and ./test/
Although they are all correct. I suggest to unify, so it looks more normalized.
Change-Id: I61ee92b2ac9f2260851e0d7e28ebaea8783423f6 Signed-off-by: Hailiang Wang <hailiangx.e.wang@intel.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/15172 Reviewed-by: Jim Harris <james.r.harris@intel.com> Reviewed-by: Michal Berger <michal.berger@intel.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Changpeng Liu <changpeng.liu@intel.com>
show more ...
|
| 8b554585 | 15-Nov-2022 |
Krzysztof Karas <krzysztof.karas@intel.com> |
test: add CPU core lock tests
Add a script with functional tests for CPU core locking mechanism.
Change-Id: I041b31f04b6cfb0ba26a7d172c83686646d8386e Signed-off-by: Krzysztof Karas <krzysztof.karas
test: add CPU core lock tests
Add a script with functional tests for CPU core locking mechanism.
Change-Id: I041b31f04b6cfb0ba26a7d172c83686646d8386e Signed-off-by: Krzysztof Karas <krzysztof.karas@intel.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/15465 Reviewed-by: Michal Berger <michal.berger@intel.com> Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com> Reviewed-by: Jim Harris <james.r.harris@intel.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
show more ...
|
| 588dfe31 | 13-Nov-2022 |
Michal Berger <michal.berger@intel.com> |
Add SPDX header to various files
They were missed by the initial set of patches which introduced this header as a mandatory one across different types of files.
Signed-off-by: Michal Berger <michal
Add SPDX header to various files
They were missed by the initial set of patches which introduced this header as a mandatory one across different types of files.
Signed-off-by: Michal Berger <michal.berger@intel.com> Change-Id: I3f9b37d41298c843e1648e72fe8593768ccd37e0 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/15423 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Community-CI: Mellanox Build Bot Reviewed-by: Jim Harris <james.r.harris@intel.com> Reviewed-by: Ben Walker <benjamin.walker@intel.com>
show more ...
|
| eb53c232 | 02-Nov-2022 |
paul luse <paul.e.luse@intel.com> |
add (c) and SPDX header to bash files as needed
per Intel policy to include file commit date using git cmd below. The policy does not apply to non-Intel (C) notices.
git log --follow -C90% --forma
add (c) and SPDX header to bash files as needed
per Intel policy to include file commit date using git cmd below. The policy does not apply to non-Intel (C) notices.
git log --follow -C90% --format=%ad --date default <file> | tail -1
and then pull just the year from the result.
Intel copyrights were not added to files where Intel either had no contribution ot the contribution lacked substance (ie license header 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: I2ef86976095b88a9bf5b1003e59f3943cd6bbe4c Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/15209 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Community-CI: Mellanox Build Bot Reviewed-by: Ben Walker <benjamin.walker@intel.com> Reviewed-by: Krzysztof Karas <krzysztof.karas@intel.com> Reviewed-by: Jim Harris <james.r.harris@intel.com>
show more ...
|
| a6dbe372 | 01-Nov-2022 |
paul luse <paul.e.luse@intel.com> |
update Intel copyright notices
per Intel policy to include file commit date using git cmd below. The policy does not apply to non-Intel (C) notices.
git log --follow -C90% --format=%ad --date defa
update Intel copyright notices
per Intel policy to include file commit date using git cmd below. The policy does not apply to non-Intel (C) notices.
git log --follow -C90% --format=%ad --date default <file> | tail -1
and then pull just the 4 digit year from the result.
Intel copyrights were not added to files where Intel either had no contribution ot the contribution lacked substance (ie license header 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 vast majority of files do have this last blank line. Simply there for consistency.
Signed-off-by: paul luse <paul.e.luse@intel.com> Change-Id: Id5b7ce4f658fe87132f14139ead58d6e285c04d4 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/15192 Tested-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
show more ...
|
| 88f16162 | 02-Nov-2022 |
Tomasz Zawadzki <tomasz.zawadzki@intel.com> |
test/scheduler: make spdk_for_each_reactor test optional
(84ab68c) test/scheduler: add a for_each_reactor shutdown test
Patch above added regression test for handling spdk_for_each_reactor during s
test/scheduler: make spdk_for_each_reactor test optional
(84ab68c) test/scheduler: add a for_each_reactor shutdown test
Patch above added regression test for handling spdk_for_each_reactor during shutdown, by adding constant repeat of this operation while application is running.
Reactor event processing (especially constant) has impact on the reactor_interrupt_run(). spdk_fd_group_wait() will almost always execute an event, skewing the results of scheduler test. Reactor that should have been idle, will show active usage via /proc/stat.
Fixes #1950
This patch makes this regression test optional, and enables it only in test that does not measure CPU utilization from the system. The ./test/event/scheduler/scheduler.sh is the only one where it is enabled, as it's purpose is to verify the test scheduler application. Remaining ./test/scheduler/*.sh tests do verify CPU utilization, so the regression test is disabled in those.
Modified the for_each_done, to for_each_reactor_start, to better reflect the intention.
On my system enabling spdk_for_each_reactor test flag on the scheduler application with no threads (except app thread), consumes ~20-25% CPU from every core in CPU mask. Meanwhile disabling it, idle cores are 100% idle and active cores spend 100% of CPU time in usr.
Change-Id: I40eda15a748e76b95dc5441144cd8931e46edee5 Signed-off-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/15210 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Jim Harris <james.r.harris@intel.com> Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>
show more ...
|
| 8dd1cd21 | 22-Jun-2022 |
Ben Walker <benjamin.walker@intel.com> |
check_format: For C files only, fix return type breaks
In SPDK, declarations have the return type on the same line. Definitions have the return type on a separate line. Astyle has an option for enfo
check_format: For C files only, fix return type breaks
In SPDK, declarations have the return type on the same line. Definitions have the return type on a separate line. Astyle has an option for enforcing this. Unfortunately, it seems to have two bugs:
1) It doesn't work correctly at all on C++ files. 2) It often fails on functions that return enums, or long type names
Deal with 1) by adjusting the check_format.sh script to only tell astyle to fix return type line breaks for C files and not C++. Deal with 2) by adding a few typedefs to work around the problem.
Change-Id: Idf28281466cab8411ce252d5f02ab384166790c6 Signed-off-by: Ben Walker <benjamin.walker@intel.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/13437 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Jim Harris <james.r.harris@intel.com> Reviewed-by: Dong Yi <dongx.yi@intel.com> Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com> Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com>
show more ...
|
| 488570eb | 03-Jun-2022 |
Jim Harris <james.r.harris@intel.com> |
Replace most BSD 3-clause license text with SPDX identifier.
Many open source projects have moved to using SPDX identifiers to specify license information, reducing the amount of boilerplate code in
Replace most BSD 3-clause license text with SPDX identifier.
Many open source projects have moved to using SPDX identifiers to specify license information, reducing the amount of boilerplate code in every source file. This patch replaces the bulk of SPDK .c, .cpp and Makefiles with the BSD-3-Clause identifier.
Almost all of these files share the exact same license text, and this patch only modifies the files that contain the most common license text. There can be slight variations because 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 text with SPDX identifier which is checked into scripts/spdx.sh.
Signed-off-by: Jim Harris <james.r.harris@intel.com> Change-Id: Iaa88ab5e92ea471691dc298cfe41ebfb5d169780 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/12904 Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com> Community-CI: Mellanox Build Bot Tested-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>
show more ...
|
| 7610bc38 | 21-Sep-2021 |
Konrad Sztyber <konrad.sztyber@intel.com> |
scripts: move python modules to python directory
Up until now, importing an SPDK RPC python module was just a matter of `import rpc`. It's fine until there's another module called `rpc` installed o
scripts: move python modules to python directory
Up until now, importing an SPDK RPC python module was just a matter of `import rpc`. It's fine until there's another module called `rpc` installed on the system, in which case it's impossible to import both of them. Therefore, to avoid this problem, all of the modules were moved to a separate directory under the "spdk" namespace.
The decision to move to a location under a separate directory was motivated by the fact that a directory called scripts/spdk would look pretty confusing. Moreover, it should make it also easier to package these scripts as a python package.
Other than moving the packages, all of the imports were updated to reflect these changes. Files under python now use relative imports, while those under scripts/ use the "spdk" namespace and have their PYTHONPATH extended with python directory.
Signed-off-by: Konrad Sztyber <konrad.sztyber@intel.com> Change-Id: Ib43dee73921d590a551dd83885e22870e72451cf Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/9692 Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com> Reviewed-by: Jim Harris <james.r.harris@intel.com> Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com> Community-CI: Mellanox Build Bot Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
show more ...
|
| c9c7c281 | 25-Nov-2021 |
Josh Soref <jsoref@gmail.com> |
spelling: test
Part of #2256
* achieve * additionally * against * aliases * already * another * arguments * between * capabilities * comparison * compatibility * configuration * continuing * contro
spelling: test
Part of #2256
* achieve * additionally * against * aliases * already * another * arguments * between * capabilities * comparison * compatibility * configuration * continuing * controlq * cpumask * default * depends * dereferenced * discussed * dissect * driver * environment * everything * excluded * existing * expectation * failed * fails * following * functions * hugepages * identifiers * implicitly * in_capsule * increment * initialization * initiator * integrity * iteration * latencies * libraries * management * namespace * negotiated * negotiation * nonexistent * number * occur * occurred * occurring * offsetting * operations * outstanding * overwhelmed * parameter * parameters * partition * preempts * provisioned * responded * segment * skipped * struct * subsystem * success * successfully * sufficiently * this * threshold * transfer * transferred * unchanged * unexpected * unregistered * useless * utility * value * variable * workload
Change-Id: I21ca7dab4ef575b5767e50aaeabc34314ab13396 Signed-off-by: Josh Soref <jsoref@gmail.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/10409 Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com> Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com> Reviewed-by: Jim Harris <james.r.harris@intel.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
show more ...
|
| 84ab68c1 | 19-Oct-2021 |
Jim Harris <james.r.harris@intel.com> |
test/scheduler: add a for_each_reactor shutdown test
The scheduler test app is ideal for testing that the reactors properly flush any existing for_each_reactor operations during the shutdown process
test/scheduler: add a for_each_reactor shutdown test
The scheduler test app is ideal for testing that the reactors properly flush any existing for_each_reactor operations during the shutdown process. This is because it has no SPDK subsystems, so the reactors stop immediately after the application is signaled to terminate.
So start a for_each_reactor operation when the application starts, and then keep starting another one once the previous one completes.
This serves as a regression test for issue #2206.
Signed-off-by: Jim Harris <james.r.harris@intel.com> Change-Id: I69825b50ad6f3c059da11241f58a8319199e88d5
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/9929 Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Changpeng Liu <changpeng.liu@intel.com> Reviewed-by: Xiaodong Liu <xiaodong.liu@intel.com> Reviewed-by: Aleksey Marchuk <alexeymar@mellanox.com> Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>
show more ...
|
| e8c46796 | 27-Sep-2021 |
Jim Harris <james.r.harris@intel.com> |
test/scheduler: change how test threads consume cpu cycles
Now a test thread will have one idle_poller and an optional busy poller (when active_percent > 0).
The idle poller will just consume a ver
test/scheduler: change how test threads consume cpu cycles
Now a test thread will have one idle_poller and an optional busy poller (when active_percent > 0).
The idle poller will just consume a very small amount of tsc via spdk_delay_us(10).
If active_percent > 0, we will start a timed poller for TIMESLICE_US (currently 100 * 1000). When that timer expires, it will consume TIMESLICE_US * active_percent microseconds.
I think this is a slightly simpler way to enforce the thread consumption percentages specified by the tests.
Signed-off-by: Jim Harris <james.r.harris@intel.com> Change-Id: I13dd18dacff0f6a86c045876384a8931b7273842
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/9654 Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Changpeng Liu <changpeng.liu@intel.com> Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>
show more ...
|
| a86e40f3 | 06-Aug-2021 |
Tomasz Zawadzki <tomasz.zawadzki@intel.com> |
scheduler: create public API and subsystem for scheduler/governor
This patch moves schedueler and governor related API from the internal event.h to public scheduler.h.
With this it is possible to c
scheduler: create public API and subsystem for scheduler/governor
This patch moves schedueler and governor related API from the internal event.h to public scheduler.h.
With this it is possible to create subsystem responsible for handling the schedulers. Three schedulers and a governor were moved to scheduler modules from event framework.
This will allow next patch to add JSON RPC configuration to the whole subsystem. Along with easier addition of other schedulers.
Removed debug logs from gscheduler, as they serve little purpose.
Signed-off-by: Krzysztof Karas <krzysztof.karas@intel.com> Signed-off-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com> Change-Id: I98ca1ea4fb281beb71941656444267842a8875b7 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/6995 Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com> Community-CI: Mellanox Build Bot Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Jim Harris <james.r.harris@intel.com> Reviewed-by: Shuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com>
show more ...
|
| 97c5373f | 21-Apr-2021 |
Maciej Szwed <maciej.szwed@intel.com> |
test/scheduler: make activity of a thread % of time period
So far the test application was mostly used for cases with single active thread on a core. Setting activity as percentage of all poller ope
test/scheduler: make activity of a thread % of time period
So far the test application was mostly used for cases with single active thread on a core. Setting activity as percentage of all poller operations worked fine for those cases.
This patch changes the thread activity to mean percentage of a set time period. Which equals to the maximum load a thread will see when being alone on the core.
Right now g_core_time_period is set to 100ms, and is the maximum execution time for all threads on a core when thread activity is set to 100%. g_timeslice is set to 100us and is the execution time of single thread poll.
Consider following scenario:
Before the patch Thread1 Activity 70% Busy 70 Idle 30 Load 70% Thread2 Activity 60% Busy 60 Idle 40 Load 60% Core Busy 130 Idle 70 Load 65%
After the patch Thread1 Activity 70% Busy 100 Idle 0 Load 100% Thread2 Activity 60% Busy 100 Idle 0 Load 100% Core Busy 200 Idle 0 Load 100%
Additionally increased period of sleeping from 1us to 100us, to offset the time needed for additional operations during thread poll.
Signed-off-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com> Signed-off-by: Maciej Szwed <maciej.szwed@intel.com> Change-Id: I027432db1bea570dd58829eeccf7dd8bb06bf249 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/7474 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com> Reviewed-by: Paul Luse <paul.e.luse@intel.com> Reviewed-by: Krzysztof Karas <krzysztof.karas@intel.com> Reviewed-by: Shuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com> Reviewed-by: Ben Walker <benjamin.walker@intel.com>
show more ...
|
| dc9374dd | 01-Mar-2021 |
Ben Walker <benjamin.walker@intel.com> |
init: Declare a formal public API
The functions to initialize the SPDK subsystems or tear them down was previously an internal-only API. Make it public for use by applications that aren't leverage S
init: Declare a formal public API
The functions to initialize the SPDK subsystems or tear them down was previously an internal-only API. Make it public for use by applications that aren't leverage SPDK's application framework.
Signed-off-by: Ben Walker <benjamin.walker@intel.com> Change-Id: I2ebfd020e6fa4c1947fa1c1a2ac509ce9b0242f8 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/6643 Reviewed-by: Tom Nabarro <tom.nabarro@outlook.com> Reviewed-by: Jim Harris <james.r.harris@intel.com> Reviewed-by: Changpeng Liu <changpeng.liu@intel.com> Reviewed-by: Aleksey Marchuk <alexeymar@mellanox.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Community-CI: Mellanox Build Bot
show more ...
|
| af935f76 | 26-Feb-2021 |
Ben Walker <benjamin.walker@intel.com> |
event: Shift subsystem initialization code to a separate library
This is useful for applications even if they elect not to use the SPDK event framework.
This doesn't shift everything in one go - ju
event: Shift subsystem initialization code to a separate library
This is useful for applications even if they elect not to use the SPDK event framework.
This doesn't shift everything in one go - just the subsystem initialization logic. Configuration file loading also needs to move in a separate patch later.
Change-Id: Id419df1045442d416650ed90e5ee78adfdd623d7 Signed-off-by: Ben Walker <benjamin.walker@intel.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/6641 Community-CI: Broadcom CI Community-CI: Mellanox Build Bot 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> Reviewed-by: Paul Luse <paul.e.luse@intel.com> Reviewed-by: Shuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com>
show more ...
|
| 278bcad4 | 14-Apr-2021 |
yidong0635 <dongx.yi@intel.com> |
perf: remove unused g_tsc_us_rate.
Not only in accel_perf, but also in test event_perf. Remove them.
Fixes #1895
Signed-off-by: yidong0635 <dongx.yi@intel.com> Change-Id: I2af1d9f6f077f3ae775af994
perf: remove unused g_tsc_us_rate.
Not only in accel_perf, but also in test event_perf. Remove them.
Fixes #1895
Signed-off-by: yidong0635 <dongx.yi@intel.com> Change-Id: I2af1d9f6f077f3ae775af994567804633fc8f050 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/7370 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Jim Harris <james.r.harris@intel.com> Reviewed-by: Changpeng Liu <changpeng.liu@intel.com> Reviewed-by: Aleksey Marchuk <alexeymar@mellanox.com>
show more ...
|
| df902b1d | 31-Mar-2021 |
Jim Harris <james.r.harris@intel.com> |
include: remove util.h include from queue_extras.h
This include isn't needed in queue_extras.h itself. There were a few places that were implicitly depending on this include, so fix those to include
include: remove util.h include from queue_extras.h
This include isn't needed in queue_extras.h itself. There were a few places that were implicitly depending on this include, so fix those to include util.h explicitly.
Signed-off-by: Jim Harris <james.r.harris@intel.com> Change-Id: Ia962ae5a4403ee8ae15f3106d0d5e7d7412a4535 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/7172 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Community-CI: Mellanox Build Bot Reviewed-by: <dongx.yi@intel.com> Reviewed-by: Shuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com> Reviewed-by: Changpeng Liu <changpeng.liu@intel.com>
show more ...
|
| 7347f60b | 30-Mar-2021 |
Michal Berger <michalx.berger@intel.com> |
autotest: Use rpc_cmd() for rpc plugin-based tests
Also, create a common dir which will hold symlinks to all existing plugins used in the tests. Location of the actual lib is not changed so the rela
autotest: Use rpc_cmd() for rpc plugin-based tests
Also, create a common dir which will hold symlinks to all existing plugins used in the tests. Location of the actual lib is not changed so the relation to the given test suite is clearly preserved.
Signed-off-by: Michal Berger <michalx.berger@intel.com> Change-Id: Icb70bbc61fbfa3325a357d5dd93f554ff132a3b9 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/7146 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Community-CI: Mellanox Build Bot Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com> Reviewed-by: Jim Harris <james.r.harris@intel.com>
show more ...
|
| 453b15d6 | 04-Jan-2021 |
Tomasz Zawadzki <tomasz.zawadzki@intel.com> |
test/scheduler: application to test behaviour of schedulers
This is an application designed to provide custom RPC interface for testing scheduler and governor implementations.
Based on those RPCs s
test/scheduler: application to test behaviour of schedulers
This is an application designed to provide custom RPC interface for testing scheduler and governor implementations.
Based on those RPCs scenarios later can be implemented to verify functionality. Tests added here verify just the app itself.
Signed-off-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com> Change-Id: I74b81234b95d815864cf8264705d728e3a7fe309 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/5763 Community-CI: Broadcom CI Reviewed-by: Maciej Szwed <maciej.szwed@intel.com> Reviewed-by: Jim Harris <james.r.harris@intel.com> Reviewed-by: Michal Berger <michalx.berger@intel.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
show more ...
|