Changes in vhost

c84ecccca3d7ef56ee8a836767e6ca0afcb28419 - vhost/windows: Make sure host drive is properly cleaned up

Wed, 07 Apr 2021 10:00:00 +0000

vhost/windows: Make sure host drive is properly cleaned upSigned-off-by: Michal Berger Change-Id: I53ec43c62f5ab7bef2719119252eec3c7c6b90d9Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/7269Reviewed-by: Karol Latecki Reviewed-by: Jim Harris Reviewed-by: Paul Luse Reviewed-by: Krzysztof Karas Reviewed-by: Ben Walker Tested-by: SPDK CI Jenkins List of files: /spdk/test/vhost/windows/windows_scsi_compliance.ps1

d546e5ba5b16c4ba8dfe5c623d96682b93cdbfb0 - test/vhost: Do proper cleanup in shared test suite

Thu, 14 Sep 2023 07:00:00 +0000

test/vhost: Do proper cleanup in shared test suiteOtherwise, in case first fio fails (i.e. the one that is being putinto a background job), the error_exit() kills the vhost app,leaving the second fio hanging for indefinite amount of time.Instead, kill it immediately when the trap is raised.Also, simply trap on EXIT, don't overwrite the ERR trap that's setfrom autotest_common.sh.Change-Id: Icfe851235ff78024b3f288c91e4a2e122502bd4bSigned-off-by: Michal Berger Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/19892Tested-by: SPDK CI Jenkins Reviewed-by: Tomasz Zawadzki Reviewed-by: Ben Walker List of files: /spdk/test/vhost/shared/shared.sh

5f6519d1ef3e0c0f1cb689f5077cdb2670581c67 - test/vhost: Handle multi-part and/or GPT disk images

Mon, 26 Jun 2023 11:00:00 +0000

test/vhost: Handle multi-part and/or GPT disk imagesRelevant for making sure the testing image that's used for vhosttests is kept up-to-date.Change-Id: I0f80fef992d4817f79a1823226297c57c63e9106Signed-off-by: Michal Berger Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/18856Reviewed-by: Konrad Sztyber Tested-by: SPDK CI Jenkins Reviewed-by: Tomasz Zawadzki Reviewed-by: Sebastian Brzezinka List of files: /spdk/test/vhost/vhost_boot/vhost_boot.sh

c5d5b92c2b9445b5a8b1c851a7c9bf933a7c8eee - test/vhost/windows: Fix scp target argument

Wed, 08 Nov 2023 11:00:00 +0000

test/vhost/windows: Fix scp target argumentThis change was missed by the 4641aa53ae.Change-Id: I193954dbdfd2ae22acc25de79e2d09d395e76ad2Signed-off-by: Michal Berger Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/20540Reviewed-by: Karol Latecki Reviewed-by: Krzysztof Karas Reviewed-by: Jim Harris Tested-by: SPDK CI Jenkins Reviewed-by: Marek Chomnicki Reviewed-by: Konrad Sztyber List of files: /spdk/test/vhost/windows/windows_scsi_compliance.sh

5df201547adebd3b396267eca18aa16892f647ed - test/vhost: Move nvmf_vhost to vhost suite

Wed, 28 Jun 2023 08:00:00 +0000

test/vhost: Move nvmf_vhost to vhost suiteThis test depends on additional qcow2 image which is by designhandled via vhost suite so it's better fit there. Also, since we havevarious nvmf jobs, this test is unnecessarily run multiple times underboth, bare-metal, and virtual setups. Limit that to bare-metal, wherevhost suite is executed, and enforce TCP as a transport.Change-Id: I15b8791e523c9077664d16f5c6c34235f023c883Signed-off-by: Michal Berger Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/18862Reviewed-by: Tomasz Zawadzki Tested-by: SPDK CI Jenkins Reviewed-by: Konrad Sztyber List of files: /spdk/test/vhost/nvmf/nvmf_vhost.sh/spdk/test/vhost/nvmf/nvmf_vhost_fio.job/spdk/test/vhost/vhost.sh

17538bdc67021ec097536c683124234db1aac374 - add (c) and SPDX header to python files as needed

Wed, 02 Nov 2022 15:00:00 +0000

add (c) and SPDX header to python 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 | 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)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: I6cd3f18d1b469d5ef249d26ddb2923ca6b970bd4Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/15208Tested-by: SPDK CI Jenkins Reviewed-by: Ben Walker Reviewed-by: Jim Harris List of files: /spdk/test/vhost/windows/windows_scsi_compliance.py

eb53c23236cccb6b698b7ca70ee783da1c574b5f - add (c) and SPDX header to bash files as needed

Wed, 02 Nov 2022 15:00:00 +0000

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 | 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 Change-Id: I2ef86976095b88a9bf5b1003e59f3943cd6bbe4cReviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/15209Tested-by: SPDK CI Jenkins Community-CI: Mellanox Build BotReviewed-by: Ben Walker Reviewed-by: Krzysztof Karas Reviewed-by: Jim Harris List of files: /spdk/test/vhost/hotplug/blk_hotremove.sh/spdk/test/vhost/integrity/integrity_start.sh/spdk/test/vhost/readonly/delete_partition_vm.sh/spdk/test/vhost/readonly/disabled_readonly_vm.sh/spdk/test/vhost/readonly/enabled_readonly_vm.sh/spdk/test/vhost/readonly/readonly.sh

34edd9f1bf5fda4c987f4500ddc3c9f50be32e7d - general: fix misspells and typos

Wed, 10 Jul 2024 17:00:00 +0000

general: fix misspells and typosSigned-off-by: Kamil Godzwon Change-Id: Iab206ef526eb7032c6681a3145450010c91705a4Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24120Community-CI: Mellanox Build BotReviewed-by: Karol Latecki Tested-by: SPDK CI Jenkins Reviewed-by: Aleksey Marchuk Reviewed-by: Jim Harris List of files: /spdk/test/vhost/fiotest/fio.sh/spdk/test/vhost/other/negative.sh

b1263050ee4a21dfc83538bfec0bb90b4b059333 - test/vhost: Remove {RDMA,MGMT}* opts from migration.sh

Thu, 24 Aug 2023 01:00:00 +0000

test/vhost: Remove {RDMA,MGMT}* opts from migration.shThese are not used in any capacity.Change-Id: Idd729025d0c14084c3505b1fed58c6c90f3acfa9Signed-off-by: Michal Berger Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/19542Reviewed-by: Konrad Sztyber Reviewed-by: Karol Latecki Reviewed-by: Kamil Godzwon Reviewed-by: Jim Harris Tested-by: SPDK CI Jenkins List of files: /spdk/test/vhost/migration/migration.sh

da0454f8a7a83bdb2dcb73f48a67b0aea86ca4f6 - test/vhost: Remove limiting of the vhost kernel threads

Thu, 22 Aug 2024 11:00:00 +0000

test/vhost: Remove limiting of the vhost kernel threadsThis was mainly targeted for the performances TCs but it's beenrecently decided to simply drop this due to the issues like:https://github.com/spdk/spdk/issues/3482We will simply not touch kernel's threads at all and point outin actual reports why it is so.Change-Id: I44164b1d22c0e345ea0bee97f050b2561552f404Signed-off-by: Michal Berger Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24639Tested-by: SPDK CI Jenkins Reviewed-by: Konrad Sztyber Reviewed-by: Jim Harris List of files: /spdk/test/vhost/perf_bench/vhost_perf.sh

7c764edf8550a5cf7541581cbc75581d7e05a2c6 - test/vhost: Gather IRQ stats from the VM

Tue, 15 Nov 2022 14:00:00 +0000

test/vhost: Gather IRQ stats from the VMSigned-off-by: Michal Berger Change-Id: I4351d812b9b9da127b6daf46b0f44ce237e33ee9Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/15460Tested-by: SPDK CI Jenkins Reviewed-by: Karol Latecki Reviewed-by: Shuhei Matsumoto Reviewed-by: Jim Harris List of files: /spdk/test/vhost/parse_irqs.sh

92895216307b7c7b131011cb067d51afe471f6be - test/vhost: replace master_core with main_core

Mon, 30 Nov 2020 20:00:00 +0000

test/vhost: replace master_core with main_coreSigned-off-by: Jim Harris Change-Id: Ibdf9612a27266ef739d9d10db3fd1d353729644aReviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/5353Tested-by: SPDK CI Jenkins Community-CI: Mellanox Build BotReviewed-by: Aleksey Marchuk Reviewed-by: Shuhei Matsumoto List of files: /spdk/test/vhost/initiator/autotest.config/spdk/test/vhost/lvol/autotest.config

a4a046245b04ed81046c1e56b5f9e73070e3ba1c - test/common: Don't use $HOME as base for $DEPENDENCY_DIR

Thu, 09 Mar 2023 08:00:00 +0000

test/common: Don't use $HOME as base for $DEPENDENCY_DIRThis is a source of problems especially in context of sudo so replaceit with a dedicated path instead.Also, simplify fio selection - these tests should not depend, bydefault, on some specific binary. Instead, always fallback to usingwhat given VM image provides.Signed-off-by: Michal Berger Change-Id: I33be30a53725338f36d31a4c1eea82f1ec56b2c9Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/17093Reviewed-by: Tomasz Zawadzki Reviewed-by: Pawel Piatek Reviewed-by: Konrad Sztyber Tested-by: SPDK CI Jenkins List of files: /spdk/test/vhost/manual.sh

d6db5988c5e98c1ddd94fb2317680e8f240e533c - test/vhost: use light IO workloads for live migration tests

Thu, 15 Dec 2022 03:00:00 +0000

test/vhost: use light IO workloads for live migration testsUse lightweight workload test cases in VM for the purposeto keep number of dirty pages is in low rate of VM's totalmemory.Fix issue #2805.Change-Id: I52efd0d0522ccef713ba2c3a451daac0683234dcSigned-off-by: Changpeng Liu Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/15954Tested-by: SPDK CI Jenkins Reviewed-by: Jim Harris Reviewed-by: Tomasz Zawadzki List of files: /spdk/test/vhost/migration/migration-tc1.sh/spdk/test/vhost/migration/migration-tc2.job

dea682c9a5474d1e241660a5bcf4cf769b79645b - test/vhost/lvol: Increase number of CPUs allocated for VMs

Sat, 13 Jan 2024 14:00:00 +0000

test/vhost/lvol: Increase number of CPUs allocated for VMsThis is done in order to speed up the boot sequence.Change-Id: I171030bd7dbd0da57606225054ddda34b1f4e222Signed-off-by: Michal Berger Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/21422Tested-by: SPDK CI Jenkins Reviewed-by: Jim Harris Reviewed-by: Tomasz Zawadzki List of files: /spdk/test/vhost/lvol/lvol_test.sh

4641aa53ae14ed7b2ae23551837977a7903dd986 - test/common: Provide {ext,null}glob by default in autotest suite

Thu, 24 Aug 2023 20:00:00 +0000

test/common: Provide {ext,null}glob by default in autotest suiteThis requires adjustments in several test suites due to some badpractices exposed by explicit use of nullglob.Change-Id: Idbe7eeec598ffc06e98986898344e4e0d88f4a5dSigned-off-by: Michal Berger Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/19551Reviewed-by: Jim Harris Tested-by: SPDK CI Jenkins Reviewed-by: Tomasz Zawadzki List of files: /spdk/test/vhost/initiator/blockdev.sh/spdk/test/vhost/migration/migration-tc2.sh

bea9ace020848f26b7c7433e9074749e84cc95c8 - test/vhost: Perform verify only for the write job

Thu, 24 Aug 2023 02:00:00 +0000

test/vhost: Perform verify only for the write jobSince fio-3.33, due to the following commit (bisect + revert): https://github.com/axboe/fio/commit/191d6634e8a692bef15143715c88920987ecaa89using verification for *read jobs is foobar. For currently existingconfiguration, fio >= 3.33 exits almost immediately causing themigration test to fail - before VM is migrated, we check if fio isactually running, however, due to this issue, it's not.To mitigate, verify only the write job, allowing the randread jobto block for expected amount of time (20s).Change-Id: I7c36e8f56635292426a15aeeb2395cecd454058aSigned-off-by: Michal Berger Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/19543Reviewed-by: Konrad Sztyber Tested-by: SPDK CI Jenkins Reviewed-by: Jim Harris List of files: /spdk/test/vhost/migration/migration-tc1.job

f8a085a2d5197a152d0d3d43b6d34f80d34eb2ef - test/vhost: Add helper functions for extracting IRQ data

Tue, 15 Nov 2022 12:00:00 +0000

test/vhost: Add helper functions for extracting IRQ dataSigned-off-by: Michal Berger Change-Id: If75aeca0c44667ef02b72f2e4a9141da4057d291Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/15459Tested-by: SPDK CI Jenkins Reviewed-by: Karol Latecki Reviewed-by: Shuhei Matsumoto Reviewed-by: Jim Harris Reviewed-by: Kamil Godzwon List of files: /spdk/test/vhost/irqs.sh

962671f711e8d634c3cca152ddca5ea0c508392c - test/vhost: Create wrapper around wipefs

Wed, 12 Apr 2023 07:00:00 +0000

test/vhost: Create wrapper around wipefsCall sync each time, as an extra step, to make sure all the writes on the underlying device completed. This is needed, as on occasion parted (called right after wipefs) fails to create pt complaining that the target device (and its partitions) are still in use.Change-Id: I959d9b36a1588ec3754335995e3e8bc5057bfeb7Signed-off-by: Michal Berger Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/17498Tested-by: SPDK CI Jenkins Reviewed-by: Jim Harris Reviewed-by: Konrad Sztyber List of files: /spdk/test/vhost/integrity/integrity_vm.sh

14c0f94c8887693aa167064a8c403cef80319b7b - test/fio: supress fio output files

Mon, 13 Jun 2022 09:00:00 +0000

test/fio: supress fio output filesverify.state files are not very useful becausewe fail the tests at any first failure.If there was something wrong with IO verificationthen autotest.sh would fail.Fio files with metrics output (IOPS, BW, latency, etc)are also not very useful as per-patch tests are notfocused on benchmarking.Fixes #2546Signed-off-by: Kamil Godzwon Change-Id: I33fbe2fb0749d840f08ffe0d606d206268140991Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/13019Community-CI: Broadcom CI Reviewed-by: Karol Latecki Reviewed-by: Jim Harris Reviewed-by: Changpeng Liu Tested-by: SPDK CI Jenkins List of files: /spdk/test/vhost/fio/vhost_fio.job/spdk/test/vhost/initiator/bdev.fio