Revision tags: v25.01-rc1 |
|
#
2e10c84c |
| 15-Nov-2024 |
Shuhei Matsumoto <smatsumoto@nvidia.com> |
nvmf: Expose DIF type of namespace to host again
We disabled exposing DIF type of namespaces to host because nvmf library did not support PRACT. https://github.com/spdk/spdk/commit/a438718fc25a5298f
nvmf: Expose DIF type of namespace to host again
We disabled exposing DIF type of namespaces to host because nvmf library did not support PRACT. https://github.com/spdk/spdk/commit/a438718fc25a5298f0bfc9daf2be0abc885b54be
nvmf library support PRACT now.
Let's expose DIF type of namespaces to host again.
Signed-off-by: Shuhei Matsumoto <smatsumoto@nvidia.com> Change-Id: I9c30c3c50791ade18fb46c2be807984c934d3788 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/25438 Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com> Community-CI: Mellanox Build Bot Reviewed-by: Jim Harris <jim.harris@nvidia.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Community-CI: Community CI Samsung <spdk.community.ci.samsung@gmail.com>
show more ...
|
#
38b931b2 |
| 15-Nov-2024 |
Shuhei Matsumoto <smatsumoto@nvidia.com> |
nvmf: Set bdev_ext_io_opts::dif_check_flags_exclude_mask for read/write
Fill opts->dif_check_flags_exclude_mask when nvmf submits read or write I/O to bdev layer. Furthremore, get CDW12 value for re
nvmf: Set bdev_ext_io_opts::dif_check_flags_exclude_mask for read/write
Fill opts->dif_check_flags_exclude_mask when nvmf submits read or write I/O to bdev layer. Furthremore, get CDW12 value for read I/O because it has PRACT bit.
Signed-off-by: Shuhei Matsumoto <smatsumoto@nvidia.com> Change-Id: Icfa5b9c9b3ca8a0d6d9b08ba447e06ebe0986443 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/23629 Community-CI: Community CI Samsung <spdk.community.ci.samsung@gmail.com> Community-CI: Mellanox Build Bot Reviewed-by: Jim Harris <jim.harris@nvidia.com> Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.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 ...
|
Revision tags: v24.09 |
|
#
081f43f2 |
| 26-Sep-2024 |
Vasuki Manikarnike <vasuki.manikarnike@hpe.com> |
lib/nvmf: Fix memory leak in nvmf_bdev_ctrlr_unmap
When max_discard_size_kib is set on subsystem, dsm/unmap commands are failed back if any dsm range exceeds the maximum discard size supported. This
lib/nvmf: Fix memory leak in nvmf_bdev_ctrlr_unmap
When max_discard_size_kib is set on subsystem, dsm/unmap commands are failed back if any dsm range exceeds the maximum discard size supported. This leaks memory allocated for unmap_ctx. Breaking out of the loop instead of returning early fixes the memory leak.
Fixes #3539
Change-Id: I7bbb1f581e48c72947dbbbc66f5bcf1320ab69d4 Signed-off-by: Vasuki Manikarnike <vasuki.manikarnike@hpe.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/25042 Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com> Community-CI: Mellanox Build Bot Reviewed-by: Jim Harris <jim.harris@samsung.com> Reviewed-by: Shuhei Matsumoto <smatsumoto@nvidia.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
show more ...
|
Revision tags: v25.01-pre, v24.09-rc1 |
|
#
5e4d957e |
| 10-Jul-2024 |
Shuhei Matsumoto <smatsumoto@nvidia.com> |
nvmf: Identify command supports IOCS NVM NS data structure
Add minimum change to support and verify it by writing unit tests. We will revisit this feature when we find any OS uses Storage Tag.
Sign
nvmf: Identify command supports IOCS NVM NS data structure
Add minimum change to support and verify it by writing unit tests. We will revisit this feature when we find any OS uses Storage Tag.
Signed-off-by: Shuhei Matsumoto <smatsumoto@nvidia.com> Change-Id: I0a9456d099f8fcf735ed773bec18e82dfbe1112e Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24112 Reviewed-by: Ben Walker <ben@nvidia.com> Community-CI: Mellanox Build Bot Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com>
show more ...
|
Revision tags: v24.05, v24.09-pre, v24.05-rc1 |
|
#
3dbaa93c |
| 27-Mar-2024 |
Ankit Kumar <ankit.kumar@samsung.com> |
nvmf: pass command dword 12 and 13 for write
For write requests extract and add command dword 12 and 13 to spdk_bdev_ext_io_opts. Add a unit test to verify the same.
Change-Id: I2f63387dccd144dc4a7
nvmf: pass command dword 12 and 13 for write
For write requests extract and add command dword 12 and 13 to spdk_bdev_ext_io_opts. Add a unit test to verify the same.
Change-Id: I2f63387dccd144dc4a725986887e652ad7d103d7 Signed-off-by: Ankit Kumar <ankit.kumar@samsung.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/22791 Community-CI: Mellanox Build Bot Reviewed-by: Jim Harris <jim.harris@samsung.com> Reviewed-by: Shuhei Matsumoto <smatsumoto@nvidia.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
show more ...
|
#
40681adc |
| 18-Mar-2024 |
Jacek Kalwas <jacek.kalwas@intel.com> |
nvmf: improve write zeroes input validation
By current design of nvmf and bdev layers write zeroes command with deallocate is not supported. See that spdk_nvmf_ctrlr_identify_ns and nvmf_bdev_ctrlr_
nvmf: improve write zeroes input validation
By current design of nvmf and bdev layers write zeroes command with deallocate is not supported. See that spdk_nvmf_ctrlr_identify_ns and nvmf_bdev_ctrlr_identify_ns don't set any dlfeat bits.
Without that check Write Zeroes command with DEAC bit set passes through nvmf/bdev/nvme and can reach out remote target which might actually support it and return the success.
Adding proper support for that requires more work between nvmf, bdev and nvme layers hence just improve input validation by adding explicit check and terminate such command early.
Change-Id: Ie9dccdfb8a706cb58caf04c1368c5c8f48084fc9 Signed-off-by: Jacek Kalwas <jacek.kalwas@intel.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/22376 Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.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 |
|
#
c6a78e83 |
| 05-Jan-2024 |
Artur Paszkiewicz <artur.paszkiewicz@intel.com> |
nvmf: wrap checking for ptpl_file into a function
With a custom reservations implementation the ptpl file may not be used to provide Persist Through Power Loss capability. Instead of checking it dir
nvmf: wrap checking for ptpl_file into a function
With a custom reservations implementation the ptpl file may not be used to provide Persist Through Power Loss capability. Instead of checking it directly, put it in a function to simplify further changes.
Change-Id: Id9cee6047f17ae010c940e1a3db009eaf84f8200 Signed-off-by: Artur Paszkiewicz <artur.paszkiewicz@intel.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/21355 Community-CI: Mellanox Build Bot 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: Konrad Sztyber <konrad.sztyber@intel.com>
show more ...
|
#
a2c540e3 |
| 16-Jan-2024 |
Jacek Kalwas <jacek.kalwas@intel.com> |
nvmf: improve request layout #3
first_fused_req is touched on the hot completion path but fused commands are not considered hot path in general. To workaround that first_fused flag is introduced in
nvmf: improve request layout #3
first_fused_req is touched on the hot completion path but fused commands are not considered hot path in general. To workaround that first_fused flag is introduced in the first cache line of the nvmf request.
Change-Id: Ibd5e9278101cbad6883e937b2830dc8e84b3cb3c Signed-off-by: Jacek Kalwas <jacek.kalwas@intel.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/21442 Reviewed-by: Jim Harris <jim.harris@samsung.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Community-CI: Mellanox Build Bot Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com> Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com>
show more ...
|
#
c3077db6 |
| 11-Dec-2023 |
Jacek Kalwas <jacek.kalwas@intel.com> |
nvmf: add accel sequence to request data type
It allows to append accel operation on nvmf transport specific layers.
Signed-off-by: Jacek Kalwas <jacek.kalwas@intel.com> Change-Id: If79b6b8f7f2ce07
nvmf: add accel sequence to request data type
It allows to append accel operation on nvmf transport specific layers.
Signed-off-by: Jacek Kalwas <jacek.kalwas@intel.com> Change-Id: If79b6b8f7f2ce0712c306a5fa915fbfc833dbc0f Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/21342 Community-CI: Mellanox Build Bot Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com> Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
show more ...
|
Revision tags: v23.09, v24.01-pre, v23.09-rc1 |
|
#
c7feb85d |
| 26-Aug-2023 |
Haoqian He <haoqian.he@smartx.com> |
nvmf: set max discard size and max write_zeroes size
For Linux kernel version >= 5.13, when host connects the admin queue successfully, it will call nvme_init_non_mdts_limits() to post a NVME_ID_CNS
nvmf: set max discard size and max write_zeroes size
For Linux kernel version >= 5.13, when host connects the admin queue successfully, it will call nvme_init_non_mdts_limits() to post a NVME_ID_CNS_CS_CTRL identify cmd and init max discard size with dmrl, dmrsl and max write_zeroes size with wzsl in response.
So we add the NVME_ID_CNS_CS_CTRL identify cmd handler in the nvmf target side and add two new parameters to RPC nvmf_create_subsystem.
Change-Id: Ib72d6caa34fced68d63df9c01760a58effa5ca34 Signed-off-by: Haoqian He <haoqian.he@smartx.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/19632 Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com> Community-CI: Mellanox Build Bot Reviewed-by: Jim Harris <jim.harris@samsung.com> Reviewed-by: Ben Walker <ben@nvidia.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
show more ...
|
#
04a428f5 |
| 30-Oct-2023 |
Karl Bonde Torp <k.torp@samsung.com> |
nvme: add iovec passthru
This is used for sending big passthru commands, like Report Zones, over nvmf.
Change-Id: I83188367e0266e093faadd49cdb2e051eae71829 Signed-off-by: Karl Bonde Torp <k.torp@sa
nvme: add iovec passthru
This is used for sending big passthru commands, like Report Zones, over nvmf.
Change-Id: I83188367e0266e093faadd49cdb2e051eae71829 Signed-off-by: Karl Bonde Torp <k.torp@samsung.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/20498 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Jim Harris <jim.harris@samsung.com> Community-CI: Mellanox Build Bot Reviewed-by: Ben Walker <ben@nvidia.com>
show more ...
|
#
6f3b59be |
| 16-Jun-2023 |
Jacek Kalwas <jacek.kalwas@intel.com> |
nvmf: add memory domain and its ctx to spdk_nvmf_request
This allows to support memory domains on nvmf layer for read/write. Support for plain compare or compare & write (fused) is still missing tho
nvmf: add memory domain and its ctx to spdk_nvmf_request
This allows to support memory domains on nvmf layer for read/write. Support for plain compare or compare & write (fused) is still missing though.
Signed-off-by: Jacek Kalwas <jacek.kalwas@intel.com> Change-Id: I9a365fcf6b7e722025d36c9f798ec2b766fa61a1 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/17527 Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com> Community-CI: Mellanox Build Bot Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
show more ...
|
Revision tags: v23.05, v23.09-pre, v23.01.1, v23.01, v23.05-pre, v23.01-rc1 |
|
#
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 ...
|
#
1a53c8a5 |
| 20-Jul-2023 |
Ziv Hirsch <zivhirsch13@gmail.com> |
nvmf: fix ctrlr_bdev passthru for commands without data
There are commands without data, like Verify, possibly some vendor specifics opcodes, etc.
Change-Id: Ia190eb0c4ff2b79f7f323c52c29aa2b99b0268
nvmf: fix ctrlr_bdev passthru for commands without data
There are commands without data, like Verify, possibly some vendor specifics opcodes, etc.
Change-Id: Ia190eb0c4ff2b79f7f323c52c29aa2b99b02682c Signed-off-by: Ziv Hirsch <zivhirsch13@gmail.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/19174 Community-CI: Mellanox Build Bot Reviewed-by: Changpeng Liu <changpeng.liu@intel.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>
show more ...
|
#
5681a8a6 |
| 07-Jun-2023 |
Konrad Sztyber <konrad.sztyber@intel.com> |
dif: remove packed attribute from spdk_dif_ctx_init_ext_opts
Following the recommendation introduced in 1d77eec82, the packed attribute was removed from spdk_dif_ctx_init_ext_opts and the way its si
dif: remove packed attribute from spdk_dif_ctx_init_ext_opts
Following the recommendation introduced in 1d77eec82, the packed attribute was removed from spdk_dif_ctx_init_ext_opts and the way its size is calculated was changed to use SPDK_SIZEOF().
Signed-off-by: Konrad Sztyber <konrad.sztyber@intel.com> Change-Id: I8dab3ee5afcd2afe803dacc563bee9b029c7cbf6 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/18597 Reviewed-by: Ben Walker <benjamin.walker@intel.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com> Community-CI: Mellanox Build Bot
show more ...
|
#
a711d629 |
| 16-May-2023 |
Slawomir Ptak <slawomir.ptak@intel.com> |
lib/util/dif: Add an ext_opts param to spdk_dif_ctx_init
The extended options structure for the dif context initialization allows for adding new arguments without the function signature change. The
lib/util/dif: Add an ext_opts param to spdk_dif_ctx_init
The extended options structure for the dif context initialization allows for adding new arguments without the function signature change. The first extended parameter is the DIF Protection Information format (CRC-16).
Change-Id: I6a32e8570676265d757fc7251441e5547bbb475d Signed-off-by: Slawomir Ptak <slawomir.ptak@intel.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/18146 Reviewed-by: Jim Harris <james.r.harris@intel.com> Reviewed-by: Shuhei Matsumoto <smatsumoto@nvidia.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Community-CI: Mellanox Build Bot
show more ...
|
#
86136540 |
| 19-Jan-2023 |
Rui Chang <rui.chang@arm.com> |
bdev: Add default copy command support in bdev
Add default copy command support in bdev layer for backing devices that does not support copy command.
Signed-off-by: Rui Chang <rui.chang@arm.com> Ch
bdev: Add default copy command support in bdev
Add default copy command support in bdev layer for backing devices that does not support copy command.
Signed-off-by: Rui Chang <rui.chang@arm.com> Change-Id: I5632e25544e95ac0c53ff91c4cd135dac53323ae Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/16638 Community-CI: Mellanox Build Bot Reviewed-by: Shuhei Matsumoto <smatsumoto@nvidia.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Ben Walker <benjamin.walker@intel.com>
show more ...
|
#
7bbeb80a |
| 25-Jan-2023 |
Ankit Kumar <ankit.kumar@samsung.com> |
nvme: support 64 LBA formats for NVM and ZNS command set
Format LBA size (FLBAS) is updated to have: Bit 3:0 as least significant 4 bits for format index Bit 6:5 as most significant 2 bits for forma
nvme: support 64 LBA formats for NVM and ZNS command set
Format LBA size (FLBAS) is updated to have: Bit 3:0 as least significant 4 bits for format index Bit 6:5 as most significant 2 bits for format index
NVMe format command fields are updated accordingly.
Add a new helper function to fetch the correct format index. Update examples and unit test files accordingly.
Signed-off-by: Ankit Kumar <ankit.kumar@samsung.com> Change-Id: I2d6d9045b9d65ae91cb18843ca75b59cc27ed2f2 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/16515 Reviewed-by: Jim Harris <james.r.harris@intel.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com>
show more ...
|
#
6b206e31 |
| 08-Jan-2023 |
John Levon <john.levon@nutanix.com> |
nvmf: sanity check passthru handlers
These routines can only handle a single buffer; double check that is the case, and fail if not.
Signed-off-by: John Levon <john.levon@nutanix.com> Change-Id: I1
nvmf: sanity check passthru handlers
These routines can only handle a single buffer; double check that is the case, and fail if not.
Signed-off-by: John Levon <john.levon@nutanix.com> Change-Id: I136482c27c73655887c49405f747b8ed073f7b69 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/16198 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> Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com>
show more ...
|
#
ecc80dfc |
| 07-Jan-2023 |
John Levon <john.levon@nutanix.com> |
util: add spdk_iov_xfer_*() API
Add a new API for incremental copying in or out of an iovec, and replace current code to use the new API.
Signed-off-by: John Levon <john.levon@nutanix.com> Change-I
util: add spdk_iov_xfer_*() API
Add a new API for incremental copying in or out of an iovec, and replace current code to use the new API.
Signed-off-by: John Levon <john.levon@nutanix.com> Change-Id: I088b784aef821310699478989e61411952066c18 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/16193 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com> Reviewed-by: Jim Harris <james.r.harris@intel.com> Reviewed-by: Ben Walker <benjamin.walker@intel.com>
show more ...
|
#
a64acd10 |
| 07-Dec-2022 |
Konrad Sztyber <konrad.sztyber@intel.com> |
nvmf: return error on invalid req length for copy commands
Both the length of a request and the number of ranges to copy are controlled by the user, so we should check them and return an error inste
nvmf: return error on invalid req length for copy commands
Both the length of a request and the number of ranges to copy are controlled by the user, so we should check them and return an error instead of asserting that they're correct.
This fixes the `test/nvmf/target/fabrics_fuzz.sh` test.
Signed-off-by: Konrad Sztyber <konrad.sztyber@intel.com> Change-Id: I3481c4bb1f2c7676df81f41dfc95ef063924222e Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/15805 Reviewed-by: Pawel Piatek <pawelx.piatek@intel.com> Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com> Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com> Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Community-CI: Mellanox Build Bot
show more ...
|
#
fed1f52b |
| 24-Nov-2022 |
Artur Paszkiewicz <artur.paszkiewicz@intel.com> |
nvmef: don't set optimal I/O boundary if write_unit_size != 1
Optimal I/O boundary causes I/O to be split in the nvme driver. This is a problem for writes if write_unit_size > 1 because the split I/
nvmef: don't set optimal I/O boundary if write_unit_size != 1
Optimal I/O boundary causes I/O to be split in the nvme driver. This is a problem for writes if write_unit_size > 1 because the split I/O may not match the write_unit_size.
Fixes: #2791 Change-Id: I437e6cb6d8e2415658d5b46539feeacb5363fd46 Signed-off-by: Artur Paszkiewicz <artur.paszkiewicz@intel.com> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/15627 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Community-CI: Mellanox Build Bot Reviewed-by: Ben Walker <benjamin.walker@intel.com> Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>
show more ...
|
Revision tags: v22.01.2, v22.09, v23.01-pre, v22.09-rc1 |
|
#
8305e49b |
| 22-Aug-2022 |
Evgeniy Kochetov <evgeniik@nvidia.com> |
nvmf: Add copy command support
NVMf target reports copy command support if all bdevs in the subsystem support copy IO type. Maximum copy size is reported for each namespace independently in namespac
nvmf: Add copy command support
NVMf target reports copy command support if all bdevs in the subsystem support copy IO type. Maximum copy size is reported for each namespace independently in namespace identify data. For now we support just one source range.
Note, that command support in the controller is initialized once on controller create. If another namespace which doesn't support copy command is added to the subsystem later, it will not be reflected in the controller data structure and will not be communicated to the initiator. Attempt to execute copy command on such namespace will fail. This issue is not specific to copy command and applies also to write zeroes and unmap (dataset management) commands.
Signed-off-by: Evgeniy Kochetov <evgeniik@nvidia.com> Change-Id: I5f06564eb43d66d2852bf7eeda8b17830c53c9bc Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/14350 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Jim Harris <james.r.harris@intel.com> Reviewed-by: Shuhei Matsumoto <smatsumoto@nvidia.com> Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.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 ...
|