Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 1671207

Summary:	Several fixes on socket pollin and pollout return value
Product:	[Community] GlusterFS	Reporter:	Milind Changire <mchangir>
Component:	rpc	Assignee:	bugs <bugs>
Status:	CLOSED NEXTRELEASE	QA Contact:
Severity:	high	Docs Contact:
Priority:	high
Version:	5	CC:	atumball, bugs, zhhuan
Target Milestone:	---
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	glusterfs-7.0	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:	1666143	Environment:
Last Closed:	2019-06-25 13:01:58 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:	1666143
Bug Blocks:

Description Milind Changire 2019-01-31 05:44:39 UTC

+++ This bug was initially created as a clone of Bug #1666143 +++

Description of problem:

Found the return value of socket pollin and pollout for rpc message is not correctly handled. One major problem is socket EAGAIN error will be returned all the way back to dispatch handler and confuse user with error message like:

[2018-12-29 07:31:41.772310] E [MSGID: 101191] [event-epoll.c:674:event_dispatch_epoll_worker] 0-epoll: Failed to dispatch handler


Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

--- Additional comment from Worker Ant on 2019-01-15 07:25:53 IST ---

REVIEW: https://review.gluster.org/22043 (socket: fix counting of socket total_bytes_read and total_bytes_write) posted (#1) for review on master by Zhang Huan

--- Additional comment from Worker Ant on 2019-01-15 07:27:04 IST ---

REVIEW: https://review.gluster.org/22044 (socket: fix issue when socket write return with EAGAIN) posted (#1) for review on master by Zhang Huan

--- Additional comment from Worker Ant on 2019-01-15 07:28:10 IST ---

REVIEW: https://review.gluster.org/22046 (socket: don't pass return value from protocol handler to event handler) posted (#1) for review on master by Zhang Huan

--- Additional comment from Worker Ant on 2019-01-15 07:29:19 IST ---

REVIEW: https://review.gluster.org/22045 (socket: fix issue when socket read return with EAGAIN) posted (#1) for review on master by Zhang Huan

--- Additional comment from Worker Ant on 2019-01-17 14:00:17 IST ---

REVIEW: https://review.gluster.org/22043 (socket: fix counting of socket total_bytes_read and total_bytes_write) merged (#2) on master by Raghavendra G

--- Additional comment from Worker Ant on 2019-01-17 14:00:40 IST ---

REVIEW: https://review.gluster.org/22044 (socket: fix issue when socket write return with EAGAIN) merged (#2) on master by Raghavendra G

--- Additional comment from Worker Ant on 2019-01-22 12:23:12 IST ---

REVIEW: https://review.gluster.org/22045 (socket: fix issue when socket read return with EAGAIN) merged (#2) on master by Amar Tumballi

--- Additional comment from Worker Ant on 2019-01-22 12:30:25 IST ---

REVIEW: https://review.gluster.org/22046 (socket: don't pass return value from protocol handler to event handler) merged (#3) on master by Amar Tumballi