[Gluster-devel] Review request - patch #15036

Raghavendra G raghavendra at gluster.com
Fri May 12 04:18:50 UTC 2017

I'll wait for a day on this. If there are no reviews, I'll assume that as a
+1 and will go ahead and merge it. If anyone needs more time, please let me
know and I can wait.

On Thu, May 11, 2017 at 12:22 PM, Raghavendra Gowdappa <rgowdapp at redhat.com>

> All,
> Reviews are requested on [1]. Impact is non-trivial as it introduces more
> concurrency in execution wrt processing of messages read from network.
> All tests are passed, though gerrit is not reflecting the last smoke which
> was successful.
> For reference, below is the verbatim copy of commit msg:
> <commit>
> event/epoll: Add back socket for polling of events immediately after
> reading the entire rpc message from the wire Currently socket is added back
> for future events after higher layers (rpc, xlators etc) have processed the
> message. If message processing involves signficant delay (as in writev
> replies processed by Erasure Coding), performance takes hit. Hence this
> patch modifies transport/socket to add back the socket for polling of
> events immediately after reading the entire rpc message, but before
> notification to higher layers.
> credits: Thanks to "Kotresh Hiremath Ravishankar" <khiremat at redhat.com>
> for assitance in fixing a regression in bitrot caused by this patch.
> BUG: 1448364
> </commit>
> @Nigel,
> Is there a way to override -1 from smoke, as last instance of it is
> successful?
> [1] https://review.gluster.org/#/c/15036/
> regards,
> Raghavendra
> _______________________________________________
> Gluster-devel mailing list
> Gluster-devel at gluster.org
> http://lists.gluster.org/mailman/listinfo/gluster-devel

Raghavendra G
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.gluster.org/pipermail/gluster-devel/attachments/20170512/312f6095/attachment.html>

More information about the Gluster-devel mailing list