On Wed, Jun 03, 2015 at 11:57:12AM +0300, Matan Barak wrote: > That's a general comment regarding the extension mechanism.
Yes, but it is also a specific comment about patch #4 which adds, ib_uverbs_ex_create_cq. Based on the implementation of create_cq, it is pretty clear that every driver supports ib_uverbs_ex_create_cq, so patch #4 should just force the flag in the device register function so it is globally enabled. query_device looks like it is the same, passing in the original structure length will always work on any device. So Mellanox should send a bugfix patch for that as well, unrelated to this series. > > An extended command that doesn't use any extended features should > > transparently degrade to the normal command as often as possible. > > That means that uverbs_ex_cmd_mask should only be used on extended commands > that > are user-space specific. The bit should only be clear on commands that can never return anything but ENOSYS. Ie the kernel has no support for the command at all, or for some reason the driver cannot handle the call. The latter case should be rare, it would be the case if we added a new command that was not extending an existing command, for instance. Jason -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
