History log of /spdk/lib/nvmf/ctrlr_bdev.c (Results 1 – 25 of 108)
Revision (<<< Hide revision tags) (Show revision tags >>>) Date Author Comments
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 ...


12345