lib/ublk: wait and retry before starting USER RECOVERY

When there are lots of inflight IOs during user recovery, the kernel
`ublk_drv` driver needs some time to set the device state to
UBLK_S_DEV_QU

lib/ublk: wait and retry before starting USER RECOVERY

When there are lots of inflight IOs during user recovery, the kernel
`ublk_drv` driver needs some time to set the device state to
UBLK_S_DEV_QUIESCED, if we start the new spdk_tgt quickly and send
UBLK_CMD_START_USER_RECOVERY command to kernel `ublk_drv` driver during
the transition period, kernel `ublk_drv` driver will return -EBUSY to user,
so add a delay retry here to read device state again.

Change-Id: I07cb92501e07b785a92df20dd6a4622da67372e9
Signed-off-by: Changpeng Liu <changpeng.liu@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/23484
Community-CI: Mellanox Build Bot
Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>
Reviewed-by: Jim Harris <jim.harris@samsung.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>

show more ...

general: fix misspells and typos

Signed-off-by: Kamil Godzwon <kamilx.godzwon@intel.com>
Change-Id: Iab206ef526eb7032c6681a3145450010c91705a4
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+

general: fix misspells and typos

Signed-off-by: Kamil Godzwon <kamilx.godzwon@intel.com>
Change-Id: Iab206ef526eb7032c6681a3145450010c91705a4
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24120
Community-CI: Mellanox Build Bot
Reviewed-by: Karol Latecki <karol.latecki@intel.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com>
Reviewed-by: Jim Harris <jim.harris@samsung.com>

show more ...

ublk: use spdk_read_sysfs_attribute_uint32 to get max ublks

Signed-off-by: Jim Harris <jim.harris@samsung.com>
Change-Id: Ib984b49bcc39ba9e7c96117e05a1eaddddf63f97
Reviewed-on: https://review.spdk.i

ublk: use spdk_read_sysfs_attribute_uint32 to get max ublks

Signed-off-by: Jim Harris <jim.harris@samsung.com>
Change-Id: Ib984b49bcc39ba9e7c96117e05a1eaddddf63f97
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24077
Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com>
Community-CI: Mellanox Build Bot
Reviewed-by: Changpeng Liu <changpeng_liu@hotmail.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>

show more ...

lib/ublk: add optional user copy control flag to RPC

The UBLK_F_USER_COPY feature is negotiated between kernel `ublk_drv` driver
and ublk target, but users may want to disable this feature even both

lib/ublk: add optional user copy control flag to RPC

The UBLK_F_USER_COPY feature is negotiated between kernel `ublk_drv` driver
and ublk target, but users may want to disable this feature even both
side can support it, here we add optional RPC flag to disable this
feature when creating ublk target.

Change-Id: I07d1a26af43ef5a3b990969ebfdd2d3372b2697d
Signed-off-by: Changpeng Liu <changpeng.liu@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/23034
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Jim Harris <jim.harris@samsung.com>
Reviewed-by: Shuhei Matsumoto <smatsumoto@nvidia.com>
Community-CI: Mellanox Build Bot

show more ...

so_ver: increase all major versions after SPDK 24.01

To allow SO_MINOR updates on LTS for the whole year it is supported,
the major version for all components needs to be increased.
This is to preve

so_ver: increase all major versions after SPDK 24.01

To allow SO_MINOR updates on LTS for the whole year it is supported,
the major version for all components needs to be increased.
This is to prevent scenario where two versions exists with matching
versions, but conflicting ABI.
Ex. Next SPDK release adds an API call increasing the minor version,
then LTS needs just a subset of those additions.

Increasing major so version after LTS, allows the future releases
to update versions as needed. Yet allowing LTS to increase minor
version separately.

This patch:
- increases SO_VER by 1 for all components
- resets SO_MINOR to 0 for all components

Due to patch below being introduced after v24.01 code freeze,
bdev lib version should not be increased:
61623c5ca9 bdev: add spdk_bdev_io padding in place of io_submit_ch

Short reference to how the versions were changed:
MAX=$(git grep "SO_VER := " | cut -d" " -f 3 | sort -ubnr | head -1)
for((i=$MAX;i>0;i-=1)); do find . -name "Makefile" -exec \
sed -i -e "s/SO_VER := $i\$/SO_VER := $(($i+1))/g" {} +; done
find . -name "Makefile" -exec \
sed -i -e "s/SO_MINOR := .*/SO_MINOR := 0/g" {} +

Signed-off-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>
Change-Id: I0166d6be6c17bbbf985d48b5dfcb36f1d4af1b48
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/21582
Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>

show more ...

ublk: add recovery feature support

With this feature, users can upgrade ublk target without stopping
their applications that running IOs in ublk devices, this feature
is supported in `ublk_drv` kern

ublk: add recovery feature support

With this feature, users can upgrade ublk target without stopping
their applications that running IOs in ublk devices, this feature
is supported in `ublk_drv` kernel driver for a while, here we enable
it with very new driver version after GET_FEATURE command, this will
ensure any new features added in future will use the same negociation
mechanism.

Also add a test case for user recovery feature, this feature can work
with >= 6.4 kernel version, we can run this script manually now, we
will enable the autotest when the CI system update to Fedora 38.

Change-Id: Ib6bc0fd7d5757888968022e9ff5df4407bfe9d3b
Signed-off-by: Changpeng Liu <changpeng.liu@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/20345
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 ...

lib/ublk: set device write cache mode based on FLUSH support

We didn't enable the "write_cache" attribute previously, every
kernel ublk device is running in "write through" mode, here set
this flag

lib/ublk: set device write cache mode based on FLUSH support

We didn't enable the "write_cache" attribute previously, every
kernel ublk device is running in "write through" mode, here set
this flag based on backend device's support capability.

Change-Id: I6d041020f351c10d1fe029b036ef205aec8326e3
Signed-off-by: Changpeng Liu <changpeng.liu@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/20368
Reviewed-by: Jim Harris <jim.harris@samsung.com>
Reviewed-by: Shuhei Matsumoto <smatsumoto@nvidia.com>
Community-CI: Mellanox Build Bot
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>

show more ...

ublk: rename function `ublk_ctrl_cmd` to `ublk_ctrl_cmd_submit`

Change-Id: If2a7e290f1e0003dbe5c42eafa9c96b22fb190fd
Signed-off-by: Changpeng Liu <changpeng.liu@intel.com>
Reviewed-on: https://revie

ublk: rename function `ublk_ctrl_cmd` to `ublk_ctrl_cmd_submit`

Change-Id: If2a7e290f1e0003dbe5c42eafa9c96b22fb190fd
Signed-off-by: Changpeng Liu <changpeng.liu@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/19946
Reviewed-by: Jim Harris <jim.harris@samsung.com>
Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>
Community-CI: Mellanox Build Bot
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>

show more ...

ublk: optimize start|stop callback to use same definition

We don't support executing multiple control commands on single ublk
device at same time, so no matter the RPC call is START or STOP, only
on

ublk: optimize start|stop callback to use same definition

We don't support executing multiple control commands on single ublk
device at same time, so no matter the RPC call is START or STOP, only
one control command is allowed, so here just use one single callback
for START and STOP RPC call, and we will also check the device state
for STOP RPC call. For coming new RPC such as USER RECOVERY, we can
also use the same callback.

Change-Id: I9e5417a104f95c5f92ebeb4c82facd922ab1362e
Signed-off-by: Changpeng Liu <changpeng.liu@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/19945
Community-CI: Mellanox Build Bot
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>

show more ...

ublk: consolidate start|stop disk error in ctrl completion function

Change-Id: I5c63351bf239f5af43fe934511d23298b5ec814c
Signed-off-by: Changpeng Liu <changpeng.liu@intel.com>
Reviewed-on: https://r

ublk: consolidate start|stop disk error in ctrl completion function

Change-Id: I5c63351bf239f5af43fe934511d23298b5ec814c
Signed-off-by: Changpeng Liu <changpeng.liu@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/19874
Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>
Community-CI: Mellanox Build Bot
Reviewed-by: Artur Paszkiewicz <artur.paszkiewicz@intel.com>
Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>

show more ...

ublk: use 2 functions for device info and BDEV parameters

The BDEV parameters initialization function will also be
used in USER RECOVERY call.

Change-Id: If908004163ba22a076a1058d0772d8ada6f5dbd9
S

ublk: use 2 functions for device info and BDEV parameters

The BDEV parameters initialization function will also be
used in USER RECOVERY call.

Change-Id: If908004163ba22a076a1058d0772d8ada6f5dbd9
Signed-off-by: Changpeng Liu <changpeng.liu@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/19840
Reviewed-by: Artur Paszkiewicz <artur.paszkiewicz@intel.com>
Community-CI: Mellanox Build Bot
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Shuhei Matsumoto <smatsumoto@nvidia.com>
Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>

show more ...

ublk: process next_state_fn after cqe is received without error

Current control process continue to submit controll commands regardless
the previous command succeeds or fails.
This patch makes it pr

ublk: process next_state_fn after cqe is received without error

Current control process continue to submit controll commands regardless
the previous command succeeds or fails.
This patch makes it proceed to the next state only after the previous
command is correctly processed by kernel.
Besides, RPC ublk_start_disk gets its response after cqe is received.

Remove next_state_fn, and introduce current_cmd_op in struct spdk_ublk_dev
to indicate whose cqe we are processing when cqe is received.

Change-Id: I334308e7753aa1db1fcf36282e4eda8a45a36dae
Signed-off-by: Richael Zhuang <richael.zhuang@arm.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/19470
Reviewed-by: Jim Harris <jim.harris@gmail.com>
Community-CI: Mellanox Build Bot
Reviewed-by: Changpeng Liu <changpeng.liu@intel.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>

show more ...

ublk: fix segmentfault when running fio test on ublk device

fio test fails with IO error log on some cases, and spdk_tgt ends
in segmentfault.

The cause is the content in io descriptor may have bee

ublk: fix segmentfault when running fio test on ublk device

fio test fails with IO error log on some cases, and spdk_tgt ends
in segmentfault.

The cause is the content in io descriptor may have been updated
by next io request when calling ublk_io_put_buffer. The io_size
that ublk_io_get_buffer and ublk_io_put_buffer caculate may be
different if caculated from io->iod->nr_sectors. Then it may get
the buf from the small buf pool and return to the large buf pool.

So record the io size when getting the buf and use this size when
putting the buf.

Change-Id: I44e4f85f180decd79f806cd5e3a60573e974056a
Signed-off-by: Richael Zhuang <richael.zhuang@arm.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/19713
Reviewed-by: Changpeng Liu <changpeng.liu@intel.com>
Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com>
Community-CI: Mellanox Build Bot
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>

show more ...

ublk: zero out poll_groups on failure

If ublk_open() fails (e.g. because the ublk_drv is not loaded),
poll_groups pointer was freed, but wasn't zeroed out, which would lead
to an assertion failure o

ublk: zero out poll_groups on failure

If ublk_open() fails (e.g. because the ublk_drv is not loaded),
poll_groups pointer was freed, but wasn't zeroed out, which would lead
to an assertion failure on subsequent ublk_create_target() calls.

Signed-off-by: Konrad Sztyber <konrad.sztyber@intel.com>
Change-Id: I1b61935aeb9cd19d91d0fda5b800fa137ed0af56
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/19325
Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.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 ...

lib/ublk: consolidate USER_COPY uring submission

If USER_COPY feature is negotiated successfully, for a READ or
WRITE command, it will use 2 uring command submission cycles,
the first cycle is used

lib/ublk: consolidate USER_COPY uring submission

If USER_COPY feature is negotiated successfully, for a READ or
WRITE command, it will use 2 uring command submission cycles,
the first cycle is used to copy data buffers between BIO and
user space data buffers. Since both the first and second
uring command submission are using same `io_uring_submit`
syscall, so here we consolidate them together to use the
same completion list.

Change-Id: I7a1ebdee3ced02680372b9ca98be57789fafbf54
Signed-off-by: Changpeng Liu <changpeng.liu@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/19245
Community-CI: Mellanox Build Bot
Reviewed-by: Jim Harris <jim.harris@gmail.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>

show more ...

lib/ublk: add USER_COPY feature support

Currently copy between io buffer and SPDK ublk target buffer is
done inside kernel `ublk_map_io()` or `ublk_unmap_io()`. This way
performs very well in case o

lib/ublk: add USER_COPY feature support

Currently copy between io buffer and SPDK ublk target buffer is
done inside kernel `ublk_map_io()` or `ublk_unmap_io()`. This way
performs very well in case of pre-allocated ublk target io buffer.

For dynamically allocated io buffer, UBLK_F_NEED_GET_DATA is added
for ublk target to provide buffer by one extra command communication
for WRITE request. For READ, ublk target simply provides buffer,
but can't know when the buffer is done in the kernel driver[1].

Commit dfae190 workarounds this issue.

UBLK_F_USER_COPY is added to kernel driver by moving io data copy
out of kernel by providing read()/write() on /dev/ublkcN, and simply
let ublk target do the io data copy.

[1] Question about ublk and NEED_GET_DATA
https://lore.kernel.org/linux-block/116d8a56-0881-56d3-9bcc-78ff3e1dc4e5@linux.alibaba.com/T/#m23bd4b8634c0a054e6797063167b469949a247bb

Change-Id: I5ef584760ea42b28b3c46ecf02268904198b7269
Signed-off-by: Changpeng Liu <changpeng.liu@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/19042
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Community-CI: Mellanox Build Bot
Reviewed-by: Jim Harris <jim.harris@gmail.com>
Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>

show more ...

lib/ublk: add undefined macro definitions for old kernel header

These macro definitions are in very latest kernel `ublk_cmd.h`
file, for compilation with CI or old kernel, we add them
into SPDK.

Ch

lib/ublk: add undefined macro definitions for old kernel header

These macro definitions are in very latest kernel `ublk_cmd.h`
file, for compilation with CI or old kernel, we add them
into SPDK.

Change-Id: Id88fa6e0c4e8026753f1c2c40e45fa88827fbc4a
Signed-off-by: Changpeng Liu <changpeng.liu@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/19244
Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>
Community-CI: Mellanox Build Bot
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Jim Harris <jim.harris@gmail.com>
Reviewed-by: Xiaodong Liu <xiaodong.liu@intel.com>

show more ...

lib/ublk: remove unused field in `ublk_io` data structure

Change-Id: I6c300f07b221e5674473069f27a9fc44ee28c775
Signed-off-by: Changpeng Liu <changpeng.liu@intel.com>
Reviewed-on: https://review.spdk

lib/ublk: remove unused field in `ublk_io` data structure

Change-Id: I6c300f07b221e5674473069f27a9fc44ee28c775
Signed-off-by: Changpeng Liu <changpeng.liu@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/19048
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>
Community-CI: Mellanox Build Bot
Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>
Reviewed-by: Xiaodong Liu <xiaodong.liu@intel.com>

show more ...

lib/ublk: set `need_data` flag into io_done function

Code optimization, we will add `user_copy` flag soon, then
all the IO buffers lifecycyle are managed in SPDK target process.

Change-Id: I13e3e7b

lib/ublk: set `need_data` flag into io_done function

Code optimization, we will add `user_copy` flag soon, then
all the IO buffers lifecycyle are managed in SPDK target process.

Change-Id: I13e3e7b0a4e2e28a1ba61e5cf08f23b03fb1663e
Signed-off-by: Changpeng Liu <changpeng.liu@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/18906
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Jim Harris <james.r.harris@intel.com>
Community-CI: Mellanox Build Bot
Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>

show more ...

lib/ublk: add `tag` field to data structure `ublk_io`

Change-Id: I60b8377aadeec9303476facd4dba96ecbaf5448e
Signed-off-by: Changpeng Liu <changpeng.liu@intel.com>
Reviewed-on: https://review.spdk.io/

lib/ublk: add `tag` field to data structure `ublk_io`

Change-Id: I60b8377aadeec9303476facd4dba96ecbaf5448e
Signed-off-by: Changpeng Liu <changpeng.liu@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/18905
Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>
Reviewed-by: Jim Harris <james.r.harris@intel.com>
Community-CI: Mellanox Build Bot
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>

show more ...

lib/ublk: code optimization to eliminate a function

Change-Id: I28c2d725af7f72ad14f67dde9dfe23d9254b3c62
Signed-off-by: Changpeng Liu <changpeng.liu@intel.com>
Reviewed-on: https://review.spdk.io/ge

lib/ublk: code optimization to eliminate a function

Change-Id: I28c2d725af7f72ad14f67dde9dfe23d9254b3c62
Signed-off-by: Changpeng Liu <changpeng.liu@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/18904
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>
Community-CI: Mellanox Build Bot
Reviewed-by: Jim Harris <james.r.harris@intel.com>

show more ...

lib/ublk: optimize IO resubmission path

We don't need to allocate data buffer again for the ENOMEM case.

Change-Id: I047ae76bedd18efa04357e348d013ba3956e7b18
Signed-off-by: Changpeng Liu <changpeng

lib/ublk: optimize IO resubmission path

We don't need to allocate data buffer again for the ENOMEM case.

Change-Id: I047ae76bedd18efa04357e348d013ba3956e7b18
Signed-off-by: Changpeng Liu <changpeng.liu@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/18789
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: Konrad Sztyber <konrad.sztyber@intel.com>

show more ...

lib/ublk: optimize `ublk_submit_bdev_io` input parameter

Change-Id: I94a28532502625e9bde22a1e2661397e64e13612
Signed-off-by: Changpeng Liu <changpeng.liu@intel.com>
Reviewed-on: https://review.spdk.

lib/ublk: optimize `ublk_submit_bdev_io` input parameter

Change-Id: I94a28532502625e9bde22a1e2661397e64e13612
Signed-off-by: Changpeng Liu <changpeng.liu@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/18788
Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.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 ...

lib/ublk: fix error variable name

`legacy` means old style ioctl, so fix it with new name.

Change-Id: I956856cc7242da5e3d60c318c68c7bb9139b71a3
Signed-off-by: Changpeng Liu <changpeng.liu@intel.com

lib/ublk: fix error variable name

`legacy` means old style ioctl, so fix it with new name.

Change-Id: I956856cc7242da5e3d60c318c68c7bb9139b71a3
Signed-off-by: Changpeng Liu <changpeng.liu@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/18787
Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.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 ...

lib/ublk: add new ioctl encoding support

New version of kernel `ublk_drv` driver enabled new ioctl
encoding, for compatibility consideration, it can support
legacy application by default, any new ad

lib/ublk: add new ioctl encoding support

New version of kernel `ublk_drv` driver enabled new ioctl
encoding, for compatibility consideration, it can support
legacy application by default, any new added commands must
use new ioctl encoding. Here, we use new added GET_FEATURES
command to get features supported by the kernel driver, if
driver can support new ioctl encoding, we will apply it.

Change-Id: I8e790b663af58cfe0338335343cbd8e0b47f072d
Signed-off-by: Changpeng Liu <changpeng.liu@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/18591
Community-CI: Mellanox Build Bot
Reviewed-by: Xiaodong Liu <xiaodong.liu@intel.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Ben Walker <benjamin.walker@intel.com>
Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>

show more ...