Project

General

Profile

Bug #17664

async msgr: keepalive2 acks aren't sent if the connection is idle

Added by Ilya Dryomov about 6 years ago. Updated over 3 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
-
Target version:
-
% Done:

0%

Source:
other
Tags:
Backport:
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

The kernel client sends keepalive2 tags every 10 seconds and resets the mon session when keepalive timeout fires (~30 seconds). With simple msgr everything is fine, but with async msgr I get:

[   66.399271] libceph: mon0 192.168.122.1:6789 session established
[   66.400093] libceph: client14100 fsid 5332c5ef-c1ee-4233-bd3b-be3dc1df51f3
[   66.420333] rbd: rbd0: capacity 1073741824 features 0x4
[  100.832051] libceph: mon0 192.168.122.1:6789 session lost, hunting for new mon
[  100.834191] libceph: mon0 192.168.122.1:6789 session established
[  138.720170] libceph: mon0 192.168.122.1:6789 session lost, hunting for new mon
[  138.723570] libceph: mon0 192.168.122.1:6789 session established
[  176.096080] libceph: mon0 192.168.122.1:6789 session lost, hunting for new mon
[  176.098380] libceph: mon0 192.168.122.1:6789 session established
[  213.984045] libceph: mon0 192.168.122.1:6789 session lost, hunting for new mon
[  213.986025] libceph: mon0 192.168.122.1:6789 session established

This isn't specific to the kernel client - it's just that its mon session is otherwise idle most of the time.

Associated revisions

Revision aeb979ae (diff)
Added by Ilya Dryomov about 6 years ago

AsyncConnection: dispatch write handler on keepalive2

...otherwise, if the connection is idle (i.e. no proper TAG_MSG
messages), keepalive ack will not get sent until some point in the
future.

Fixes: #17664
Signed-off-by: Ilya Dryomov <>

History

#1 Updated by Ilya Dryomov about 6 years ago

  • Status changed from New to Fix Under Review

#2 Updated by Kefu Chai about 6 years ago

  • Status changed from Fix Under Review to Resolved

#3 Updated by Greg Farnum over 3 years ago

  • Project changed from Ceph to Messengers
  • Category deleted (msgr)

Also available in: Atom PDF