[Bugs] [Bug 1654549] New: merge ssl infra with epoll infra
bugzilla at redhat.com
bugzilla at redhat.com
Thu Nov 29 04:21:49 UTC 2018
Bug ID: 1654549
Summary: merge ssl infra with epoll infra
Product: Red Hat Gluster Storage
Assignee: mchangir at redhat.com
Reporter: mchangir at redhat.com
QA Contact: rhinduja at redhat.com
CC: bugs at gluster.org, rhs-bugs at redhat.com,
sankarshan at redhat.com
Depends On: 1561332
+++ This bug was initially created as a clone of Bug #1561332 +++
Description of problem:
SSL infra spawns a thread per client. This poses a performance problem when
multiple fops are to be processed concurrently.
SSL infra should piggy back on the regular epoll infra for better performance
--- Additional comment from Worker Ant on 2018-03-28 12:39:27 IST ---
REVIEW: https://review.gluster.org/19308 (rpc: merge ssl infra with epoll
infra) posted (#7) for review on master by Milind Changire
--- Additional comment from Worker Ant on 2018-03-28 18:34:26 IST ---
REVIEW: https://review.gluster.org/19786 (rpc: merge ssl infra with epoll infra
- DO NOT MERGE) posted (#2) for review on master by Milind Changire
--- Additional comment from Worker Ant on 2018-07-27 14:29:38 IST ---
COMMIT: https://review.gluster.org/19308 committed in master by "Raghavendra G"
<rgowdapp at redhat.com> with a commit message- rpc: merge ssl infra with epoll
Patch attempts to use the epoll infra for handling SSL connections
as well instead of the socket_poller() thread func.
This essentially makes priv->own_thread flag redundant.
SSL_connect()/SSL_accept() is now non-blocking which has done away
with the localised poll() in ssl_do(). So, ssl_do() has been updated
own_thread and coincidently socket_poller() thread for SSL processing
is now deprecated.
Signed-off-by: Milind Changire <mchangir at redhat.com>
--- Additional comment from Shyamsundar on 2018-10-23 20:36:59 IST ---
This bug is getting closed because a release has been made available that
should address the reported issue. In case the problem is still not fixed with
glusterfs-5.0, please open a new bug report.
glusterfs-5.0 has been announced on the Gluster mailinglists , packages for
several distributions should become available in the near future. Keep an eye
on the Gluster Users mailinglist  and the update infrastructure for your
[Bug 1561332] merge ssl infra with epoll infra
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=ZvR64X8idS&a=cc_unsubscribe
More information about the Bugs