Re: [PATCH v2 for-next 1/7] IB/core: Extend ib_uverbs_create_qp

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 10/21/2015 1:04 PM, Or Gerlitz wrote:
On 10/21/2015 12:53 PM, Sagi Grimberg wrote:
On 10/15/2015 2:44 PM, Eran Ben Elisha wrote:


+struct ib_uverbs_ex_create_qp {
+    __u64 user_handle;
+    __u32 pd_handle;
+    __u32 send_cq_handle;
+    __u32 recv_cq_handle;
+    __u32 srq_handle;
+    __u32 max_send_wr;
+    __u32 max_recv_wr;
+    __u32 max_send_sge;
+    __u32 max_recv_sge;
+    __u32 max_inline_data;
+    __u8  sq_sig_all;
+    __u8  qp_type;
+    __u8  is_srq;
+    __u8 reserved;
+    __u32 comp_mask;
+    __u32 create_flags;

Can we please make create_flags u64 to begin with?


In the kernel we used 4-5 flags over 10 years, why worry?

I don't see why we should assume that we won't occupy enough bits in a
long time just for saving extra 4 bytes. But if you are strongly against
it then I won't persist here.
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]     [Yosemite Photos]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux