[Gluster-devel] Review request - patch #15036
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>
> Reviews are requested on . 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:
> 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
> Is there a way to override -1 from smoke, as last instance of it is
>  https://review.gluster.org/#/c/15036/
> Gluster-devel mailing list
> Gluster-devel at gluster.org
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the Gluster-devel