FairMQ: Disable post buffer ack on control band

Seems to be unnecessary.
This commit is contained in:
Dennis Klein 2018-03-08 03:10:10 +01:00 committed by Mohammad Al-Turany
parent cb447bde38
commit e8994dfa0f

View File

@ -320,9 +320,9 @@ try {
if (size) { if (size) {
// Receive and process control message // Receive and process control message
auto ctrl2 = ReceiveControlMessage(); // auto ctrl2 = ReceiveControlMessage();
assert(ctrl2->has_post_buffer_acknowledgement()); // assert(ctrl2->has_post_buffer_acknowledgement());
assert(ctrl2->post_buffer_acknowledgement().size() == size); // assert(ctrl2->post_buffer_acknowledgement().size() == size);
// Send data // Send data
auto ret = fi_send(fDataEndpoint, msg->GetData(), size, nullptr, fRemoteDataAddr, nullptr); auto ret = fi_send(fDataEndpoint, msg->GetData(), size, nullptr, fRemoteDataAddr, nullptr);
@ -375,12 +375,12 @@ try {
throw SocketError(tools::ToString("Failed posting ofi receive buffer, reason: ", fi_strerror(ret))); throw SocketError(tools::ToString("Failed posting ofi receive buffer, reason: ", fi_strerror(ret)));
// Create and send control message // Create and send control message
auto ctrl2 = tools::make_unique<ControlMessage>(); // auto ctrl2 = tools::make_unique<ControlMessage>();
auto ack = tools::make_unique<PostBufferAcknowledgement>(); // auto ack = tools::make_unique<PostBufferAcknowledgement>();
ack->set_size(msg->GetSize()); // ack->set_size(msg->GetSize());
ctrl2->set_allocated_post_buffer_acknowledgement(ack.release()); // ctrl2->set_allocated_post_buffer_acknowledgement(ack.release());
assert(ctrl2->IsInitialized()); // assert(ctrl2->IsInitialized());
SendControlMessage(move(ctrl2)); // SendControlMessage(move(ctrl2));
fi_cq_err_entry cqEntry; fi_cq_err_entry cqEntry;
ret = fi_cq_sread(fDataCompletionQueueRx, &cqEntry, 1, nullptr, -1); ret = fi_cq_sread(fDataCompletionQueueRx, &cqEntry, 1, nullptr, -1);