On Wed, Aug 14, 2013 at 3:58 AM, Or Gerlitz <[email protected]> wrote: > +struct ib_uverbs_create_flow { > + __u32 comp_mask; > + __u64 response; > + __u32 qp_handle; > + struct ib_kern_flow_attr flow_attr; > +}; > + > +struct ib_uverbs_create_flow_resp { > + __u32 comp_mask; > + __u32 flow_handle; > +}; > + > +struct ib_uverbs_destroy_flow { > + __u32 comp_mask; > + __u32 flow_handle; > +};
I don't see any other place where any of these comp_mask fields are referenced in the patch. Am I misunderstanding the new uverbs command scheme, or does the code need to make sure these are 0 so that it's not silently ignoring some future extension fields? - R. -- 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
