On Wed, Mar 20, 2019 at 11:17:36AM +0200, Max Gurtovoy wrote: > > On 3/18/2019 11:34 PM, Sagi Grimberg wrote: > > > > > As we discussed, let's check with RDMA maintainers if it's better to > > > extend alloc_cq API or create alloc_cq_dim API function. > > > > > > Sagi/Christoph, > > > > > > how about adding a module param per ULP ? as we use register_always > > > today, create a use_dimm module param for iSER/NVMe-RDMA ? > > > > I would say that its better (and simpler) to do a global sysctl knob for > > it. No need for a per-ULP param for a starting point. > > for sure it's simpler, but in this case all should agree that in case we run > more than 1 ULP in a server, all of them will have the same configuration > (no QoS). > > are you suggesting something like: > > /proc/sys/rdma/mlx5_0/dim > > Jason/Leon/Doug, > > thoughts about the best way to configure this feature from your perspective > ? It doesn't sound reasonable that we have per-ULP feature (DIM is per-ULP) to be configured globally. Especially given the fact that users can find themselves running different workloads with different requirements on the same system. Currently each ULP has some sort of tool to configure itself and I think that once ULP is converted to use DIM, it should have on/off knob in the tool used by their users. Thanks >
Attachment:
signature.asc
Description: PGP signature
