Project

General

Profile

Actions

Bug #44

closed

Make MDS use new connection-based send_message

Added by Greg Farnum about 14 years ago. Updated over 7 years ago.

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

100%

Source:
Tags:
Backport:
Regression:
Severity:
Reviewed:
Affected Versions:
ceph-qa-suite:
Component(FS):
Labels (FS):
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

This will be more complicated than the OSD adjustments because there are a lot of send_message_[client|mon|mds|osd] calls.

Actions #1

Updated by Greg Farnum about 14 years ago

  • % Done changed from 0 to 30

All the straight-up uses of send_message which are replying to another message have been modified. Still need to add Connection-based options for send_message_[client|mon|mds|osd] and then make use of them.

Actions #2

Updated by Greg Farnum about 14 years ago

  • Status changed from New to In Progress
Actions #3

Updated by Greg Farnum about 14 years ago

  • Category set to 1
Actions #4

Updated by Greg Farnum about 14 years ago

  • Status changed from In Progress to 4
  • % Done changed from 30 to 100

Implemented a send_message(Message , Connection) in MDS, and switched a lot of code to use it. I'm going to see if I can insert a Connection * into the mds Sessions; if I can that will open up several more instances; but that's another issue.

Actions #5

Updated by Greg Farnum about 14 years ago

Wrote a new send_message_client that properly calls inc_push_seq and switched the appropriate calls to use it, resolving issue pointed out by Sage in another issue.

Also, renamed send_message_client to send_and_inc_message_client

Actions #6

Updated by Greg Farnum almost 14 years ago

  • Status changed from 4 to Closed
Actions #7

Updated by John Spray over 7 years ago

  • Project changed from Ceph to CephFS
  • Category deleted (1)

Bulk updating project=ceph category=mds bugs so that I can remove the MDS category from the Ceph project to avoid confusion.

Actions

Also available in: Atom PDF