Revision tags: v25.01-rc1 |
|
#
afdec00e |
| 06-Nov-2024 |
Shuhei Matsumoto <smatsumoto@nvidia.com> |
nvmf: Add hide_metadata option to nvmf_subsystem_add_ns
Signed-off-by: Shuhei Matsumoto <smatsumoto@nvidia.com> Change-Id: I518c3cc5fdcb16b41f1e3bda1debf5cb3cc9c47b Reviewed-on: https://review.spdk.
nvmf: Add hide_metadata option to nvmf_subsystem_add_ns
Signed-off-by: Shuhei Matsumoto <smatsumoto@nvidia.com> Change-Id: I518c3cc5fdcb16b41f1e3bda1debf5cb3cc9c47b Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/25413 Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com> Reviewed-by: Jim Harris <jim.harris@nvidia.com> Community-CI: Mellanox Build Bot Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Community-CI: Community CI Samsung <spdk.community.ci.samsung@gmail.com> Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com>
show more ...
|
#
b09de013 |
| 06-Nov-2024 |
Shuhei Matsumoto <smatsumoto@nvidia.com> |
nvmf: Get metadata config by not bdev but bdev_desc
This is a preparation to enable no_metadata option to each bdev for DIF insert/strip.
Signed-off-by: Shuhei Matsumoto <smatsumoto@nvidia.com> Cha
nvmf: Get metadata config by not bdev but bdev_desc
This is a preparation to enable no_metadata option to each bdev for DIF insert/strip.
Signed-off-by: Shuhei Matsumoto <smatsumoto@nvidia.com> Change-Id: I018ccbdba662fa71489ae9664fc9d41bf79cf7d7 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/25412 Reviewed-by: Jim Harris <jim.harris@nvidia.com> Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com> Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com> Community-CI: Community CI Samsung <spdk.community.ci.samsung@gmail.com> Community-CI: Mellanox Build Bot Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
show more ...
|
#
fb6c49f2 |
| 24-Oct-2024 |
Konrad Sztyber <konrad.sztyber@intel.com> |
bdev: add spdk_bdev_get_nvme_nsid()
The id of a namespace is required to be sent in some, but not all, admin commands. So, without it, it's impossible to send an admin command that does require it.
bdev: add spdk_bdev_get_nvme_nsid()
The id of a namespace is required to be sent in some, but not all, admin commands. So, without it, it's impossible to send an admin command that does require it.
This is not a problem for IO commands, as bdev_nvme always fills in nsid for IO command passthru. However, we can't do that for admin commands, as some admin commands require nsid to be cleared or behave differently when it's set to the broadcast value (0xffffffff).
Signed-off-by: Konrad Sztyber <konrad.sztyber@intel.com> Change-Id: Ic943a1c52cd9043cc09469ad421be2f089aac606 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/25363 Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com> Community-CI: Mellanox Build Bot Community-CI: Community CI Samsung <spdk.community.ci.samsung@gmail.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Jim Harris <jim.harris@samsung.com> Reviewed-by: Changpeng Liu <changpeliu@tencent.com>
show more ...
|
#
bfd014b5 |
| 17-Oct-2024 |
Konrad Sztyber <konrad.sztyber@intel.com> |
nvmf: add function for setting ns visibility
Also, while here, added an assert that the nsid is within the valid range in nvmf_ctrlr_ns_is_visible() and replaced all direct accesses to the bit_array
nvmf: add function for setting ns visibility
Also, while here, added an assert that the nsid is within the valid range in nvmf_ctrlr_ns_is_visible() and replaced all direct accesses to the bit_array with that function.
Signed-off-by: Konrad Sztyber <konrad.sztyber@intel.com> Change-Id: Ife925688f2261d1110be2cf2ff69be71626b784b Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/25243 Community-CI: Mellanox Build Bot Reviewed-by: Jim Harris <jim.harris@samsung.com> Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com> Reviewed-by: Changpeng Liu <changpeliu@tencent.com> Community-CI: Community CI Samsung <spdk.community.ci.samsung@gmail.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
show more ...
|
#
5a8c76d9 |
| 10-Oct-2024 |
Abhineet Pandey <abhineet.pandey@nutanix.com> |
lib/nvmf: Add spdk_nvmf_send_discovery_log_notice API
Added spdk_nvmf_send_discovery_log_notice API to send discovery log page change notice to hosts.
This may be use cases where the host may have
lib/nvmf: Add spdk_nvmf_send_discovery_log_notice API
Added spdk_nvmf_send_discovery_log_notice API to send discovery log page change notice to hosts.
This may be use cases where the host may have a low controller loss timeout, any maintenance operations can cause hosts to lose connection to one of the controllers. Once we send AEN, host can add the controlller back again.
Change-Id: I82e294ec226b5402c29f9ce6fce2c29ce9eaaa40 Signed-off-by: Abhineet Pandey <abhineet.pandey@nutanix.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/25188 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 <jim.harris@samsung.com> Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>
show more ...
|
#
75a12cbf |
| 10-Oct-2024 |
Slawomir Ptak <slawomir.ptak@intel.com> |
test: Comparison operator fixes
Change-Id: I1296b19b590c2c6cbb75b9362e441cd6219d7a9f Signed-off-by: Slawomir Ptak <slawomir.ptak@intel.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/2
test: Comparison operator fixes
Change-Id: I1296b19b590c2c6cbb75b9362e441cd6219d7a9f Signed-off-by: Slawomir Ptak <slawomir.ptak@intel.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/25198 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Community-CI: Mellanox Build Bot Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com> Reviewed-by: Shuhei Matsumoto <smatsumoto@nvidia.com> Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com>
show more ...
|
Revision tags: v24.09, v25.01-pre, v24.09-rc1 |
|
#
c156f3e1 |
| 02-Aug-2024 |
Karl Bonde Torp <k.torp@samsung.com> |
nvmf: ensure correct nsid for passthrough commands
This handles the case where a user adds an nvme namespace to a fabrics subsystem with a different nsid than the underlying nvme namespace.
Fixes i
nvmf: ensure correct nsid for passthrough commands
This handles the case where a user adds an nvme namespace to a fabrics subsystem with a different nsid than the underlying nvme namespace.
Fixes issue #3486
Change-Id: Iede7001fd2a4b87fde1e3a1b201def792049394a Signed-off-by: Karl Bonde Torp <k.torp@samsung.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24454 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Jim Harris <jim.harris@samsung.com> Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com> Reviewed-by: Shuhei Matsumoto <smatsumoto@nvidia.com> Community-CI: Mellanox Build Bot
show more ...
|
#
001db1ea |
| 15-Jul-2024 |
kyuho.son <kyuho.son@samsung.com> |
nvmf: update mDNS PRR listener when discovery listener changes
Previously, user should stop and restart mDNS PRR to update the mDNS PRR listener list. This requires some expensive avahi API calls, s
nvmf: update mDNS PRR listener when discovery listener changes
Previously, user should stop and restart mDNS PRR to update the mDNS PRR listener list. This requires some expensive avahi API calls, such as creating and deleting avahi structures.
Now, when discovery subsystem listener changes, nvmf_tgt_update_mdns_prr is called to update mDNS PRR listener.
Change-Id: I136fc4057136b59cfda158db3f3e29f34e4a7f4c Signed-off-by: kyuho.son <kyuho.son@samsung.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24169 Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com> Community-CI: Mellanox Build Bot Reviewed-by: Jim Harris <jim.harris@samsung.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
show more ...
|
Revision tags: v24.05, v24.09-pre, v24.05-rc1 |
|
#
dca0da83 |
| 24-Apr-2024 |
Konrad Sztyber <konrad.sztyber@intel.com> |
nvmf: disallow adding the same host to a subsystem many times
We used to allow adding the same host to the same subsystem many times. However, this can cause issues, as we didn't check transport-spe
nvmf: disallow adding the same host to a subsystem many times
We used to allow adding the same host to the same subsystem many times. However, this can cause issues, as we didn't check transport-specific host options (e.g. TLS PSK), so adding the same host with different transport options would succeed, which is obviously incorrect.
We could fix it by comparing host's options and ensuring they're the same, but it's easier to simply disallow it altogether. It's also consistent with the behavior of other RPCs.
Signed-off-by: Konrad Sztyber <konrad.sztyber@intel.com> Change-Id: I1a60329100f5a3256c5ccc09db0586e7a023b6e5 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/22896 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Jim Harris <jim.harris@samsung.com> Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com> Reviewed-by: Jacek Kalwas <jacek.kalwas@intel.com> Community-CI: Mellanox Build Bot
show more ...
|
#
f74bf0d0 |
| 25-Mar-2024 |
Konrad Sztyber <konrad.sztyber@intel.com> |
nvmf/auth: check for support when adding dhchap keys
In-band authentication requires certain version of openssl (it needs to define the EVP_MAC* functions). With an older version, it's possible to
nvmf/auth: check for support when adding dhchap keys
In-band authentication requires certain version of openssl (it needs to define the EVP_MAC* functions). With an older version, it's possible to specify a dhchap key when adding a host, but it wouldn't be possible to use it for authentication because calculating DH-HMAC-CHAP response would always fail.
This behavior could be a bit misleading to the users, so this patch makes it impossible to add a host with dhchap key if the version of openssl is too old.
Signed-off-by: Konrad Sztyber <konrad.sztyber@intel.com> Change-Id: I03738b0c16d4b641aaa165262284cd6f5ca9a7aa Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/22657 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Community-CI: Mellanox Build Bot Reviewed-by: Jim Harris <jim.harris@samsung.com> Reviewed-by: Ben Walker <ben@nvidia.com>
show more ...
|
#
3f4f8ad0 |
| 12-Mar-2024 |
Konrad Sztyber <konrad.sztyber@intel.com> |
nvmf: set host's DH-HMAC-CHAP key
Signed-off-by: Konrad Sztyber <konrad.sztyber@intel.com> Change-Id: I739409b136f085fe264b9c77ed2656fd4cc9ea1e Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk
nvmf: set host's DH-HMAC-CHAP key
Signed-off-by: Konrad Sztyber <konrad.sztyber@intel.com> Change-Id: I739409b136f085fe264b9c77ed2656fd4cc9ea1e Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/22654 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Ben Walker <ben@nvidia.com> Reviewed-by: Jim Harris <jim.harris@samsung.com> Community-CI: Mellanox Build Bot
show more ...
|
#
608b54a2 |
| 25-Apr-2024 |
Konrad Sztyber <konrad.sztyber@intel.com> |
nvmf: remove cb_fn/cb_arg from spdk_nvmf_qpair_disconnect()
These were deprecated for a long time, so it's finally time to remove them.
Signed-off-by: Konrad Sztyber <konrad.sztyber@intel.com> Chan
nvmf: remove cb_fn/cb_arg from spdk_nvmf_qpair_disconnect()
These were deprecated for a long time, so it's finally time to remove them.
Signed-off-by: Konrad Sztyber <konrad.sztyber@intel.com> Change-Id: Ia10242b36c3560df4dae8f6bd9fd4ff931179e22 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/22945 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com> Reviewed-by: Jim Harris <jim.harris@samsung.com> Community-CI: Mellanox Build Bot
show more ...
|
#
2881f7f6 |
| 25-Apr-2024 |
Konrad Sztyber <konrad.sztyber@intel.com> |
nvmf: queue subsystem state change requests
When user requests a subsystem state to be changed while it's already changing state, that request will now get queued and will be serviced after the prev
nvmf: queue subsystem state change requests
When user requests a subsystem state to be changed while it's already changing state, that request will now get queued and will be serviced after the previous state change is completed.
Fixes #3079.
Suggested-by: Jim Harris <jim.harris@samsung.com> Signed-off-by: Konrad Sztyber <konrad.sztyber@intel.com> Change-Id: Iff1d5cf73d641f0633735bffc8f77d5eefc92d55 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/22942 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Community-CI: Mellanox Build Bot Reviewed-by: Jim Harris <jim.harris@samsung.com> Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com>
show more ...
|
#
037d5165 |
| 15-Apr-2024 |
Ankit Kumar <ankit.kumar@samsung.com> |
nvmf: fdp capability to the subsystem
Set FDP support for controller data, based on the subsystem capability.
Enable checks to only allow either fdp or non fdp capable namespaces to the subsystem.
nvmf: fdp capability to the subsystem
Set FDP support for controller data, based on the subsystem capability.
Enable checks to only allow either fdp or non fdp capable namespaces to the subsystem. The first nvmf namespace added to the subsystem will set a precedent. Add a unit test to verify the same.
Change-Id: Id9f317b81022698eee08985c3943364e3fa0c386 Signed-off-by: Ankit Kumar <ankit.kumar@samsung.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/22790 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Shuhei Matsumoto <smatsumoto@nvidia.com> Reviewed-by: Jim Harris <jim.harris@samsung.com> Community-CI: Mellanox Build Bot
show more ...
|
Revision tags: LTS, v24.01, v24.05-pre, v24.01-rc1, v23.09, v24.01-pre, v23.09-rc1, v23.05, v23.09-pre, v23.01.1, v23.01, v23.05-pre, v23.01-rc1, v22.01.2, v22.09, v23.01-pre, v22.09-rc1, v22.05, v22.09-pre, v22.05-rc1, v22.01.1, v22.01, v22.01-rc1, v21.10, v21.10-rc1, v21.07, v21.07-rc1 |
|
#
d37555b4 |
| 08-May-2021 |
Jonas Pfefferle <pepperjo@japf.ch> |
nvmf: per-host namespace masking
This patch introduces namespace masking for NVMe-over-fabrics targets by allowing to (dynamically) add or remove hosts for a given namespace on a given subsystem. A
nvmf: per-host namespace masking
This patch introduces namespace masking for NVMe-over-fabrics targets by allowing to (dynamically) add or remove hosts for a given namespace on a given subsystem. A host can only perform operations to a namespace it has been given access.
To preserve current behavior all namespaces are automatically visible. To control visibility on a per-host basis, the nvmf_subsystem_add_ns RPC shall be called with "--no-auto-visible". We introduce two new RPC calls: - nvmf_ns_add_host <subsysNQN> <NSID> <hostNQN> - nvmf_ns_remove_host <subsysNQN> <NSID> <hostNQN> These RPCs apply both to existing and future controllers for the given subsystem/hostnqn pair.
The list in spdk_nvmf_ns is used to keep track of hostNQNs which controllers should be attached on connect.
The visible_ns array in spdk_nvmf_ctrlr is used for fast lookup to check whether a NSID is visible on command execution.
Change-Id: I1c3a3c2bf3615fa76e7bfe3aaf08e2b82d6b4ec0 Signed-off-by: Jonas Pfefferle <pepperjo@japf.ch> Signed-off-by: Jim Harris <jim.haris@samsung.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/7821 Reviewed-by: Ben Walker <ben@nvidia.com> Reviewed-by: Shuhei Matsumoto <smatsumoto@nvidia.com> Community-CI: Mellanox Build Bot Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
show more ...
|
#
09e8e884 |
| 19-Jan-2024 |
Artur Paszkiewicz <artur.paszkiewicz@intel.com> |
nvmf: allow setting custom handlers for namespace reservation operations
Currently, namespace reservation persistence is enabled by storing the reservation information in a file. Make it possible to
nvmf: allow setting custom handlers for namespace reservation operations
Currently, namespace reservation persistence is enabled by storing the reservation information in a file. Make it possible to override the default behavior by providing custom handlers for reservation operations.
Change-Id: I247ea2d6cd41ae808030886d70172fad7a04291b Signed-off-by: Artur Paszkiewicz <artur.paszkiewicz@intel.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/21360 Reviewed-by: Jim Harris <jim.harris@samsung.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com> Community-CI: Mellanox Build Bot
show more ...
|
#
5bbe0c0b |
| 18-Jan-2024 |
Artur Paszkiewicz <artur.paszkiewicz@intel.com> |
nvmf: change reservation functions to use ns instead of ptpl_file
Suggested-by: Jim Harris <jim.harris@samsung.com> Change-Id: Ifc66055c001d7fcde42a0520536906137ca53b19 Signed-off-by: Artur Paszkiew
nvmf: change reservation functions to use ns instead of ptpl_file
Suggested-by: Jim Harris <jim.harris@samsung.com> Change-Id: Ifc66055c001d7fcde42a0520536906137ca53b19 Signed-off-by: Artur Paszkiewicz <artur.paszkiewicz@intel.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/21488 Reviewed-by: Jim Harris <jim.harris@samsung.com> Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Community-CI: Mellanox Build Bot
show more ...
|
#
17c821a8 |
| 05-Jan-2024 |
Artur Paszkiewicz <artur.paszkiewicz@intel.com> |
nvmf: move out calling nvmf_ns_update_reservation_info()
This allows to have it called from one place instead of three.
Change-Id: I847ae756cb5fb2a76e1740ec24f549f017534806 Signed-off-by: Artur Pas
nvmf: move out calling nvmf_ns_update_reservation_info()
This allows to have it called from one place instead of three.
Change-Id: I847ae756cb5fb2a76e1740ec24f549f017534806 Signed-off-by: Artur Paszkiewicz <artur.paszkiewicz@intel.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/21356 Reviewed-by: Jim Harris <jim.harris@samsung.com> Community-CI: Mellanox Build Bot Reviewed-by: Shuhei Matsumoto <smatsumoto@nvidia.com> Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
show more ...
|
#
f029b824 |
| 15-Jan-2024 |
Jacek Kalwas <jacek.kalwas@intel.com> |
util: add SPDK_IOV_ONE macro
Having that spdk_iov_one fn is marked deprecated. The issue with that function is that it enforce iovcnt type which might not be of type int but u32 or u8 etc. Having th
util: add SPDK_IOV_ONE macro
Having that spdk_iov_one fn is marked deprecated. The issue with that function is that it enforce iovcnt type which might not be of type int but u32 or u8 etc. Having that compiler would complain when using the function with other types than int.
It is a pre-work for layout optmz of nvmf request (iovcnt change there from int to u8).
Change-Id: Id1534f4d3f60d35c58c26e4f4cf0d605ae11398a Signed-off-by: Jacek Kalwas <jacek.kalwas@intel.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/21433 Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Community-CI: Mellanox Build Bot Reviewed-by: Jim Harris <jim.harris@samsung.com>
show more ...
|
#
8b61f516 |
| 20-Dec-2023 |
Artur Paszkiewicz <artur.paszkiewicz@intel.com> |
nvmf: fix restoring without reservation
When restoring from ptpl_file with no reservation acquired, nvmf_ns_reservation_restore() would return -EINVAL, making it impossible to add the namespace.
Ch
nvmf: fix restoring without reservation
When restoring from ptpl_file with no reservation acquired, nvmf_ns_reservation_restore() would return -EINVAL, making it impossible to add the namespace.
Change-Id: Ife6e177eb44be316c66789bf933810a2608d9dca Signed-off-by: Artur Paszkiewicz <artur.paszkiewicz@intel.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/21164 Reviewed-by: Jim Harris <jim.harris@samsung.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Community-CI: Mellanox Build Bot Reviewed-by: Changpeng Liu <changpeng.liu@intel.com>
show more ...
|
#
552f6517 |
| 04-Dec-2023 |
Konrad Sztyber <konrad.sztyber@intel.com> |
nvmf: expose nvmf_nqn_is_valid()
It'll make it possible to use from places other places. Also, rename it to adhere to our usual naming conventions.
Signed-off-by: Konrad Sztyber <konrad.sztyber@in
nvmf: expose nvmf_nqn_is_valid()
It'll make it possible to use from places other places. Also, rename it to adhere to our usual naming conventions.
Signed-off-by: Konrad Sztyber <konrad.sztyber@intel.com> Change-Id: I9293f88666914f34ccac33dbd94e9dd6674fdeb2 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/20899 Reviewed-by: Jim Harris <jim.harris@samsung.com> Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com> Community-CI: Mellanox Build Bot Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
show more ...
|
#
604c1b6f |
| 13-Nov-2023 |
Krzysztof Karas <krzysztof.karas@intel.com> |
nvmf: validate I/O unit size
Validate I/O unit size value in nvmf_create_transport RPC.
Fixes #3167
Change-Id: I1add2fc2452f276343c617595ce26d3162e6d848 Signed-off-by: Krzysztof Karas <krzysztof.k
nvmf: validate I/O unit size
Validate I/O unit size value in nvmf_create_transport RPC.
Fixes #3167
Change-Id: I1add2fc2452f276343c617595ce26d3162e6d848 Signed-off-by: Krzysztof Karas <krzysztof.karas@intel.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/20570 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Jim Harris <jim.harris@samsung.com> Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com> Community-CI: Mellanox Build Bot Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com>
show more ...
|
#
7efdf905 |
| 12-Sep-2023 |
Slawomir Ptak <slawomir.ptak@intel.com> |
lib/nvmf: TP 8014 - Current Discovery Subsystem
Added support for the Current Discovery Subsystem value of the Subsystem Type in the Discovery Log Page Entry.
Change introduced in TP 8014
Change-I
lib/nvmf: TP 8014 - Current Discovery Subsystem
Added support for the Current Discovery Subsystem value of the Subsystem Type in the Discovery Log Page Entry.
Change introduced in TP 8014
Change-Id: I9a35ba24ecfdc2bbcc81baae80575e37ae1a866d Signed-off-by: Slawomir Ptak <slawomir.ptak@intel.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/19839 Community-CI: Mellanox Build Bot Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Ben Walker <ben@nvidia.com> Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>
show more ...
|
#
5d92820c |
| 09-Jan-2023 |
John Levon <john.levon@nutanix.com> |
nvmf: remove req->data field
Now all code has been adapted to use req->iov, and the deprecation period has elapsed, remove the legacy ->data field altogether, along with spdk_nvmf_request_get_data()
nvmf: remove req->data field
Now all code has been adapted to use req->iov, and the deprecation period has elapsed, remove the legacy ->data field altogether, along with spdk_nvmf_request_get_data().
Signed-off-by: John Levon <john.levon@nutanix.com> Change-Id: I22db8a7d8e086d9bcba3e784bf9dd19da147470f Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/16201 Reviewed-by: Changpeng Liu <changpeng.liu@intel.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Jim Harris <jim.harris@gmail.com> Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com> Community-CI: Mellanox Build Bot
show more ...
|
#
ea941cae |
| 28-Jul-2023 |
Konrad Sztyber <konrad.sztyber@intel.com> |
test/unit: use spdk_ut_run_tests()
Replaced direct calls to the CUnit's functions to run the tests with spdk_ut_run_tests(). That way, each test will have the ability to run a specific test case.
test/unit: use spdk_ut_run_tests()
Replaced direct calls to the CUnit's functions to run the tests with spdk_ut_run_tests(). That way, each test will have the ability to run a specific test case.
The blob.c unit test wasn't changed, because it runs all tests multiple times with different parameter combinations, so it cannot be easily converted. In the future, each such combination could be split into a separate test suite, which would make it compatible with spdk_ut_run_tests().
Signed-off-by: Konrad Sztyber <konrad.sztyber@intel.com> Change-Id: I4463f808f89844e9bf32b5b31eda197c5d729d1d Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/19288 Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com> Community-CI: Mellanox Build Bot Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
show more ...
|