Skip to contentSkip to conversation

ibv_req_notify_cq and multithreading

DISCUSSION Flavio Baronti 2 participants 4 messages Jan 12, 2012 – Jan 13, 2012 3 messages not in the archive InfiniBand

lore
Share & tools
lore
#0 lore raw

I'm trying to have N threads reading from the same completion channel, bounded to M completion queues. I would like to have N << M, and to ensure that only a single thread at time can call ibv_poll_cq() on a given queue, to process the events in the same order they were put in the queue.

I can't understand how to properly achieve this, since:
1- If I call ibv_req_notify_cq() before ibv_poll_cq(), I might end up with two threads polling the same queue.
2- If I call ibv_req_notify_cq() after ibv_poll_cq(), I could end up with events in the cq not being notified in the channel (I read this on the IBTA 11.4.2.2, and I *think* I actually experienced this under load).

I can use option 1 with an additional lock before ibv_req_notify_cq(), but I would like to know if there is a simpler way which I can't see.

Thanks
Flavio
--
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

A message that is not in the archive (sent off-list or not yet received): [email protected]

#2 lore raw
Quoted 14 lines

I'm trying to have N threads reading from the same completion channel, bounded to M completion queues. I would like to
have N << M, and to ensure that only a single thread at time can call ibv_poll_cq() on a given queue, to process the
events in the same order they were put in the queue.

I can't understand how to properly achieve this, since:
1- If I call ibv_req_notify_cq() before ibv_poll_cq(), I might end up with two threads polling the same queue.
2- If I call ibv_req_notify_cq() after ibv_poll_cq(), I could end up with events in the cq not being notified in the
channel (I read this on the IBTA 11.4.2.2, and I *think* I actually experienced this under load).

Quoted 3 lines

I can use option 1 with an additional lock before ibv_req_notify_cq(), but I would like to know if there is a simpler
way which I can't see.

I can't think of a simpler way. You just don't have any idea which CQ will be returned from the completion channel. Does your traffic pattern work to create N completion channels and distributed the CQs among them? --
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

A message that is not in the archive (sent off-list or not yet received): 1828884A29C6694DAF28B7E6B8A8237325676B8F-P5GAC/[email protected]

#4 lore raw

Il 1/12/2012 16:52 PM, Hefty, Sean ha scritto:

Quoted 14 lines
Quoted 14 lines

I'm trying to have N threads reading from the same completion channel, bounded to M completion queues. I would like to
have N<< M, and to ensure that only a single thread at time can call ibv_poll_cq() on a given queue, to process the
events in the same order they were put in the queue.

I can't understand how to properly achieve this, since:
1- If I call ibv_req_notify_cq() before ibv_poll_cq(), I might end up with two threads polling the same queue.
2- If I call ibv_req_notify_cq() after ibv_poll_cq(), I could end up with events in the cq not being notified in the
channel (I read this on the IBTA 11.4.2.2, and I *think* I actually experienced this under load).

Quoted 6 lines
Quoted 3 lines

I can use option 1 with an additional lock before ibv_req_notify_cq(), but I would like to know if there is a simpler
way which I can't see.

I can't think of a simpler way. You just don't have any idea which CQ will be returned from the completion channel. Does your traffic pattern work to create N completion channels and distributed the CQs among them? --

Each CQ is related to a separate connection; putting two on the same channel, and making a single thread read the channel, would force an arbitrary coupling between connections which I'm trying to avoid. Point 2 is correct though? ibv_req_notify_cq() should be called before ibv_poll_cq()?
--
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

A message that is not in the archive (sent off-list or not yet received): [email protected]

#6 lore raw
Quoted 2 lines

Point 2 is correct though? ibv_req_notify_cq() should be called before ibv_poll_cq()?

To be technical, I would state is as you need to poll after calling notify. For example, the sequence poll - notify - poll is okay.

- Sean
--
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