public inbox for linux-nvme@lists.infradead.org
 help / color / mirror / Atom feed
* nvme in-band authentication blktests - nvme/043
@ 2023-06-29 19:24 alan.adamson
  2023-07-03  6:46 ` Shinichiro Kawasaki
  0 siblings, 1 reply; 3+ messages in thread
From: alan.adamson @ 2023-06-29 19:24 UTC (permalink / raw)
  To: linux-nvme

I'm playing around with in-band authentication and using blktests to see 
how to setup it up and use it.  In test nvme/043, a valid hostkey is 
never initialized and a NULL is being used to setup the target and to 
connect.  Is this expected?  The test with the NULL hostkey passes.  I 
updated the test to use a valid key and it also passes.


diff --git a/tests/nvme/043 b/tests/nvme/043
index a030884..d5e0d4b 100755
--- a/tests/nvme/043
+++ b/tests/nvme/043
@@ -40,6 +40,8 @@ test() {
         fi
         hostnqn="nqn.2014-08.org.nvmexpress:uuid:${hostid}"

+       hostkey="$(nvme gen-dhchap-key -n ${subsys_name} 2> /dev/null)"
+
         _setup_nvmet

         truncate -s "${nvme_img_size}" "${file_path}"


Alan Adamson



^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: nvme in-band authentication blktests - nvme/043
  2023-06-29 19:24 nvme in-band authentication blktests - nvme/043 alan.adamson
@ 2023-07-03  6:46 ` Shinichiro Kawasaki
  2023-07-03  9:06   ` Hannes Reinecke
  0 siblings, 1 reply; 3+ messages in thread
From: Shinichiro Kawasaki @ 2023-07-03  6:46 UTC (permalink / raw)
  To: alan.adamson@oracle.com; +Cc: linux-nvme@lists.infradead.org, Hannes Reinecke

CC+: Hannes,

On Jun 29, 2023 / 12:24, alan.adamson@oracle.com wrote:
> I'm playing around with in-band authentication and using blktests to see how
> to setup it up and use it.  In test nvme/043, a valid hostkey is never
> initialized and a NULL is being used to setup the target and to connect.  Is
> this expected?  The test with the NULL hostkey passes.  I updated the test
> to use a valid key and it also passes.
> 
> 
> diff --git a/tests/nvme/043 b/tests/nvme/043
> index a030884..d5e0d4b 100755
> --- a/tests/nvme/043
> +++ b/tests/nvme/043
> @@ -40,6 +40,8 @@ test() {
>         fi
>         hostnqn="nqn.2014-08.org.nvmexpress:uuid:${hostid}"
> 
> +       hostkey="$(nvme gen-dhchap-key -n ${subsys_name} 2> /dev/null)"
> +
>         _setup_nvmet
> 
>         truncate -s "${nvme_img_size}" "${file_path}"
> 

Alan's question looks valid for me.

Hannes, may I ask your comment?

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: nvme in-band authentication blktests - nvme/043
  2023-07-03  6:46 ` Shinichiro Kawasaki
@ 2023-07-03  9:06   ` Hannes Reinecke
  0 siblings, 0 replies; 3+ messages in thread
From: Hannes Reinecke @ 2023-07-03  9:06 UTC (permalink / raw)
  To: Shinichiro Kawasaki, alan.adamson@oracle.com
  Cc: linux-nvme@lists.infradead.org

On 7/3/23 08:46, Shinichiro Kawasaki wrote:
> CC+: Hannes,
> 
> On Jun 29, 2023 / 12:24, alan.adamson@oracle.com wrote:
>> I'm playing around with in-band authentication and using blktests to see how
>> to setup it up and use it.  In test nvme/043, a valid hostkey is never
>> initialized and a NULL is being used to setup the target and to connect.  Is
>> this expected?  The test with the NULL hostkey passes.  I updated the test
>> to use a valid key and it also passes.
>>
>>
>> diff --git a/tests/nvme/043 b/tests/nvme/043
>> index a030884..d5e0d4b 100755
>> --- a/tests/nvme/043
>> +++ b/tests/nvme/043
>> @@ -40,6 +40,8 @@ test() {
>>          fi
>>          hostnqn="nqn.2014-08.org.nvmexpress:uuid:${hostid}"
>>
>> +       hostkey="$(nvme gen-dhchap-key -n ${subsys_name} 2> /dev/null)"
>> +
>>          _setup_nvmet
>>
>>          truncate -s "${nvme_img_size}" "${file_path}"
>>
> 
> Alan's question looks valid for me.
> 
> Hannes, may I ask your comment?

Yeah, that's correct.

Reviewed-by: Hannes Reinecke <hare@suse.de>

Cheers,

Hannes



^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2023-07-03  9:06 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-06-29 19:24 nvme in-band authentication blktests - nvme/043 alan.adamson
2023-07-03  6:46 ` Shinichiro Kawasaki
2023-07-03  9:06   ` Hannes Reinecke

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox