[Bugs] [Bug 1654549] New: merge ssl infra with epoll infra

[bugzilla at redhat.com]

https://bugzilla.redhat.com/show_bug.cgi?id=1654549

            Bug ID: 1654549
           Summary: merge ssl infra with epoll infra
           Product: Red Hat Gluster Storage
           Version: 3.4
         Component: rpc
          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.

Expected results:
SSL infra should piggy back on the regular epoll infra for better performance
and scalability.

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

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
appropriately.

own_thread and coincidently socket_poller() thread for SSL processing
is now deprecated.

Change-Id: I1ce54c06ddb643c16baa143598e7e4fbf16bae0a
fixes: bz#1561332
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 [1], packages for
several distributions should become available in the near future. Keep an eye
on the Gluster Users mailinglist [2] and the update infrastructure for your
distribution.

[1] https://lists.gluster.org/pipermail/announce/2018-October/000115.html
[2] https://www.gluster.org/pipermail/gluster-users/


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1561332
[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