Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

msg/async: Support close idle connection feature #9783

Merged
merged 28 commits into from Jun 29, 2016
Merged

msg/async: Support close idle connection feature #9783

merged 28 commits into from Jun 29, 2016

Conversation

yuyuyu101
Copy link
Member

@yuyuyu101 yuyuyu101 commented Jun 17, 2016 •
edited

  1. use c++ features
  2. avoid lock in timer
  3. replace some context with inline lambda

impl http://tracker.ceph.com/issues/16366

@yuyuyu101 yuyuyu101 force-pushed the wip-16366 branch 8 times, most recently from 59b8cde to a375387 Compare June 19, 2016 18:19
@yuyuyu101 yuyuyu101 mentioned this pull request Jun 21, 2016
Closed
@yuyuyu101
Copy link
Member Author

passed http://pulpito.ceph.com/haomai-2016-06-21_12:24:39-rados-wip-haomai-testing-distro-basic-smithi/

one test issue (fix later)
one vargind issue(rocksdb)
others all network connection errors(including dead jobs)

@yuyuyu101 yuyuyu101 force-pushed the wip-16366 branch 5 times, most recently from 175a6a7 to 31cbae2 Compare June 28, 2016 10:21
@yuyuyu101
Copy link
Member Author

yuyuyu101 commented Jun 29, 2016 •
edited

passed http://pulpito.ceph.com/haomai-2016-06-28_15:13:46-rados-wip-haomai-testing---basic-smithi
http://pulpito.ceph.com/haomai-2016-06-28_04:21:22-rados-wip-haomai-testing---basic-smithi/
vargind issue (rocksdb)
dead job(ec test itself, killed by hand)
others(env problem)

@yuyuyu101
Event: use multimap instead of map
8f0cd8e 
Signed-off-by: Haomai Wang <haomai@xsky.com>
@yuyuyu101
Event: remove clock skew detect since we use mono clock now
d493b6b 
Signed-off-by: Haomai Wang <haomai@xsky.com>
@yuyuyu101
Event: use local_center instead of pthread id
7ca8839 
Signed-off-by: Haomai Wang <haomai@xsky.com>
@yuyuyu101
Event: remove time_lock which is no need
ba470ce 
Signed-off-by: Haomai Wang <haomai@xsky.com>
@yuyuyu101
Event: replace atomic_t with c++ atomic
e515c9d 
Signed-off-by: Haomai Wang <haomai@xsky.com>
@yuyuyu101
Event: replace Mutex with c++ mutex
eebc6c2 
Signed-off-by: Haomai Wang <haomai@xsky.com>
@yuyuyu101
Event: remove extra header files
018b48c 
Signed-off-by: Haomai Wang <haomai@xsky.com>
@yuyuyu101
Event: remove next_time which isn't needed
b58f6df 
Signed-off-by: Haomai Wang <haomai@xsky.com>
@yuyuyu101
Event: implement effective delete_time_event
6e24d9a 
Signed-off-by: Haomai Wang <haomai@xsky.com>
@yuyuyu101
AsyncConnection: make connection timeout when idle a lot
fe6564c 
Signed-off-by: Haomai Wang <haomai@xsky.com>
@yuyuyu101
test_async_driver: add missing headers
dd9c43a 
Signed-off-by: Haomai Wang <haomai@xsky.com>
@yuyuyu101
AsyncConnection: add tick timer
fa4767c 
Signed-off-by: Haomai Wang <haomai@xsky.com>
@yuyuyu101
test_msgr: add connection read timeout tests
9d952ad 
Signed-off-by: Haomai Wang <haomai@xsky.com>
@yuyuyu101
AsyncConnection: move delete timers to cleanup handler
761f147 
Signed-off-by: Haomai Wang <haomai@xsky.com>
@yuyuyu101
AsyncConnection: add EventCenter id
4dc2e13 
Signed-off-by: Haomai Wang <haomai@xsky.com>
@yuyuyu101
Event: add submit_to apis
c33753e 
Signed-off-by: Haomai Wang <haomai@xsky.com>
@yuyuyu101
AsyncConnection: execute existing fault in another
ef7c3e3 
Signed-off-by: Haomai Wang <haomai@xsky.com>
@yuyuyu101
Event: change id to idx
cb58088 
Signed-off-by: Haomai Wang <haomai@xsky.com>
@yuyuyu101
perf_local: fix api adjust
65874f1 
Signed-off-by: Haomai Wang <haomai@xsky.com>
@yuyuyu101
AsyncConnection: make delete_time_event async
fb480fd 
Signed-off-by: Haomai Wang <haomai@xsky.com>
@yuyuyu101
AsyncConnection: ensure delay_state empty when building a new session
8256211 
Signed-off-by: Haomai Wang <haomai@xsky.com>
@yuyuyu101
AsyncMessenger: remove extra release_worker path
8907f52 
Signed-off-by: Haomai Wang <haomai@xsky.com>
@yuyuyu101
Event: avoid multi global conflict
84e19b5 
Signed-off-by: Haomai Wang <haomai@xsky.com>
@yuyuyu101
Event: change submit_to related to cephcontext
4b5bb65 
Signed-off-by: Haomai Wang <haomai@xsky.com>
@yuyuyu101
msg/async/Event: don't execute inline if nonwait
1f65a35 
Signed-off-by: Haomai Wang <haomai@xsky.com>
@yuyuyu101
msg/async/AsyncConnection: avoid dup RETRYGLOBAL sending
be8dec5 
if two accept all going to dispatch RETRYGLOBAL sending, it will cause
invalid connect side receiver.

Signed-off-by: Haomai Wang <haomai@xsky.com>
@yuyuyu101
msg/async/Event: change to use pthread_t to indicate whether the same…
ada84e7 
… thread

thread_local has problem in TmapMigratePP.DataScan case that it will be
assigned to zero unexpectedly.

pthread_t is much cheaper than gettid since it's a library implementation.

Signed-off-by: Haomai Wang <haomai@xsky.com>
@yuyuyu101
Revert "msg/AsyncMessenger: move Worker class to cc file"
0001a07 
This reverts commit 23d4488.
@yuyuyu101 yuyuyu101 merged commit 593008e into ceph:master Jun 29, 2016
@yuyuyu101 yuyuyu101 deleted the wip-16366 branch June 29, 2016 05:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
core feature wip-haomai-testing
Projects
None yet
Milestone
No milestone
1 participant
@yuyuyu101