Re: [PATCH] ib/mlx4: add IB_CQ_REPORT_MISSED_EVENTS support

2010-07-27 Thread Or Gerlitz
Eli Cohen wrote: > returning 1 means that you must poll the CQ to avoid a race condition > which is not true for mlx4. makes sense, thanks for clarifying that. Or. -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majord...@vger.kernel.org More

Re: [PATCH] ib/mlx4: add IB_CQ_REPORT_MISSED_EVENTS support

2010-07-27 Thread Bart Van Assche
Can I conclude from this that the polling loop (2) from http://www.mail-archive.com/linux-rdma@vger.kernel.org/msg04907.html won't trigger a race on a multiprocessor when using mlx4 hardware ? Bart. On Tue, Jul 27, 2010 at 11:14 AM, Eli Cohen wrote: > > I don't think this patch is required for m

Re: [PATCH] ib/mlx4: add IB_CQ_REPORT_MISSED_EVENTS support

2010-07-27 Thread Eli Cohen
I don't think this patch is required for mlx4. < 0means an error occurred while requesting notification == 0means notification was requested successfully, and if IB_CQ_REPORT_MISSED_EVENTS was passed in, then no events were missed and it is

[PATCH] ib/mlx4: add IB_CQ_REPORT_MISSED_EVENTS support

2010-07-27 Thread Or Gerlitz
enhance the cq arming code to support IB_CQ_REPORT_MISSED_EVENTS Signed-off-by: Or Gerlitz I noted that the IB_CQ_REPORT_MISSED_EVENTS flag was added in the same cycle with mlx4 and maybe as of this, mlx4 didn't implement the flag, which is used by IPoIB The patch is compile tested only,