lkml.org 
[lkml]   [2020]   [Oct]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [PATCH blktests v2 05/11] nvme: Add common helpers for passthru tests


On 2020-10-06 6:02 p.m., Chaitanya Kulkarni wrote:
> On 9/30/20 12:01, Logan Gunthorpe wrote:
>> Add some simple helpers to setup a passthru target that passes through
>> to a nvme test device.
>>
>> Signed-off-by: Logan Gunthorpe <logang@deltatee.com>
>> ---
>> tests/nvme/rc | 76 +++++++++++++++++++++++++++++++++++++++++++++++++++
>> 1 file changed, 76 insertions(+)
>>
>> diff --git a/tests/nvme/rc b/tests/nvme/rc
>> index dfa57a299625..1ea23308a3f7 100644
>> --- a/tests/nvme/rc
>> +++ b/tests/nvme/rc
>> @@ -73,6 +73,17 @@ _require_nvme_trtype_is_fabrics() {
>> return 0
>> }
>>
>> +_test_dev_nvme_ctrl() {
>> + local dev
>> +
>> + dev=$(cat "${TEST_DEV_SYSFS}/device/dev")
> can you initialize dev this at the time of declaration ?

Yup, will fix.

>> + echo "/dev/char/${dev}"
>> +}
>> +
>> +_test_dev_nvme_nsid() {
>> + cat "${TEST_DEV_SYSFS}/nsid"
>> +}
>> +
>> _cleanup_nvmet() {
>> local dev
>> local port
>> @@ -257,6 +268,27 @@ _remove_nvmet_subsystem() {
>> rmdir "${subsys_path}"
>> }
>>
>> +_create_nvmet_passthru() {
>> + local nvmet_subsystem="$1"
>> + local subsys_path="${NVMET_CFS}/subsystems/${nvmet_subsystem}"
>> + local passthru_path="${subsys_path}/passthru"
>> +
>> + mkdir -p "${subsys_path}"
>> + printf 1 > "${subsys_path}/attr_allow_any_host"
>> +
>> + printf "%s" "$(_test_dev_nvme_ctrl)" > "${passthru_path}/device_path"
>> + printf 1 > "${passthru_path}/enable"
>
> can you please echo in general and printf only when it is needed ?>
> I know existing code is a bit inconsistent I'll send a clenup to make it
> uniform.

Yes, I agree. I will it fix in v3.
Thanks,

Logan

\
 
 \ /
  Last update: 2020-10-07 02:14    [W:0.205 / U:0.032 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site