[PATCH 1/2] nvmet: make MDTS value configurable

[keith.busch@intel.com (Keith Busch)]

On Sun, Mar 31, 2019@07:02:40PM -0700, Hannes Reinecke wrote:
> > I think it's more of a ctrl/nvmet_port configuration than a subsystem.
> > 
> > For example if you have 2 HCAs, one is super strong and can transfer 
> > upto 8MB and the second is old model and can transfer upto 64KiB.
> > 
> > do
> > 
> > id->mdts = ctrl->ops->port_mdts(req->port);
> > 
> > and inside the implementation you can do
> > 
> > min_not_zero(max_port_hca_cap_in_mdts_units, mdts_configfs_val);
> > 
> > and it will fix the comment that is written above (we're not really 
> > unlimited).
> > 
> > this way we're using the low level device characteristics and also cover 
> > the bio split test.
> > 
> > thoughts ?
> > 
> > 
> Yeah, true, it should be a port configuration.
> 
> I'll be updating the patch.

Wouldn't this indicate MDTS should be driven from hardware constraints
rather than user tunable knobs? I don't think we should have user
parameters if their only purpose is to test how host drivers react
to them. It's okay if they've a functional purpose, but there are so
many adjustable nvme parameters, this is a bit of a slippery slope if we
want to turn the nvme target driver into a host driver test vehicle.