Project

General

Profile

Actions

Feature #45

closed

Investigate adding Connection * to mds Session

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

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

0%

Source:
Tags:
Backport:
Reviewed:
Affected Versions:
Component(FS):
Labels (FS):
Pull request ID:

Description

This would open up several more uses of connection-based send_message.

Actions #1

Updated by Sage Weil about 14 years ago

Take a look at send_message_client(Message *, client_t) and send_message_client(Message *, entity_inst_t). They both increment a seq counter in the Session (for all cap and lease message, IIRC).

Maybe we need call the new func send_message_client_pushed or _counted or something to be more clear. And it needs to increment the Session push_seq counter... ideally by getting the Session* from the Connection* without looking it up again in the sessionmap (as the old versions do).

Actions #2

Updated by Greg Farnum almost 14 years ago

  • Status changed from New to 7
  • Assignee set to Greg Farnum

Added Connection *; now testing my send_message conversions to make sure I'm not trying to use any Connection *s while they're invalid.

Actions #3

Updated by Greg Farnum almost 14 years ago

  • Status changed from 7 to Closed

This got pulled into unstable a while ago.

Actions #4

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