[PATCH 08/14] nvmet: use nvme write cmd group setting op_flag

[Chaitanya Kulkarni]

On 8/10/20 16:23, Logan Gunthorpe wrote:
>> rejected ?
> In some cases we were worried about side effects with other hosts
> accessing the namespace, in others they require using the ctrl_id which
> isn't really mapped in a sane way.
> 

Thanks for the explanation. The first case is completely admin's 
responsibility how they want to use it and share it best we can provide
a documentation.

The second case we need to fix it, coz that defeats the whole purpose
of having passthru driver without ns-mgmt and critical commands which I
know are very important in the data centers.

>> What is the use of passthru if host cannot create namespaces format ns
>> and sanitize the important data ?
> Plenty, they can format the ns on the native host and still use them on
> other targets.
> 
>> Irrespective of that it better have it documented there is no harm.
> We shouldn't have code that doesn't actually do anything. It's confusing
> and bad practice.
> 

You are right.

These commands are important for data centers and should be supported.
Regarding this patch I'll remove these commands and I'll send a patch
when these admin commands are supported.

> Logan
>