On 13/08/2013 22:18, Roland Dreier wrote:
>+struct ib_uverbs_cmd_hdr_ex {
>+ __u32 command;
>+ __u16 in_words;
>+ __u16 out_words;
>+ __u16 provider_in_words;
>+ __u16 provider_out_words;
>+ __u32 cmd_hdr_reserved;
>+};
>+
If I understand the vague explanations and the analogy to the
presentation about userspace, then cmd_hdr_reserved is supposed to be
used as a mask field. Should a kernel that doesn't understand any
mask components make sure that this reserved field is 0, and return an
error if it isn't? I don't see any code to do that, and it seems to
risk new userspace silently getting wrong answers on an old kernel.
Is there any reason not to name it as a mask field from the start?
NO.
The last field in the extended uverbs command header isn't for extending
the command for which this instance serves as the header. Each new
command which is defined in the extensions framework carries a comp_mask
as the first field its specificstructure, for example
+struct ib_uverbs_create_flow {
+ __u32 comp_mask;
+ __u64 response;
+ __u32 qp_handle;
+ struct ib_kern_flow_attr flow_attr;
+};
+struct ib_uverbs_destroy_flow {
+ __u32 comp_mask;
+ __u32 flow_handle;
+};
+
As for new/modified user space working on old/unmodified kernel driver,
the kernel will simply return -EINVAL based on inspecting the command
field. This field is the first one in both the non-extended (old) and
the extended (new) header.
static ssize_t ib_uverbs_write(struct file *filp, const char __user *buf,
size_t count, loff_t *pos)
{
struct ib_uverbs_file *file = filp->private_data;
struct ib_uverbs_cmd_hdr hdr;
if (count < sizeof hdr)
return -EINVAL;
if (copy_from_user(&hdr, buf, sizeof hdr))
return -EFAULT;
if (hdr.in_words * 4 != count)
return -EINVAL;
if (hdr.command >= ARRAY_SIZE(uverbs_cmd_table) ||
!uverbs_cmd_table[hdr.command]) <-- old kernels will
return error here
return -EINVAL;
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
