Project

General

Profile

Actions

Bug #19856

closed

msg/async/rdma: fix ceph_clock_now calls

Added by Chang Liu almost 7 years ago. Updated almost 7 years ago.

Status:
Won't Fix
Priority:
Normal
Assignee:
-
Category:
-
Target version:
-
% Done:

0%

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

Actions #1

Updated by Nathan Cutler almost 7 years ago

  • Tracker changed from Backport to Bug
  • Status changed from New to Pending Backport
  • % Done set to 0
  • Backport set to kraken
  • Regression set to No
  • Severity set to 3 - minor
Actions #2

Updated by Nathan Cutler almost 7 years ago

  • Description updated (diff)
Actions #3

Updated by Haomai Wang almost 7 years ago

RDMA has lots of bugs under K release, it doesn't make sense to backport this compile only.... I suppose user to test rdma in L

Actions #4

Updated by Chang Liu almost 7 years ago

Thanks, Haomai

I'm agree with you. I don't need RDMA in Kraken, I got compile error when I try to make a dist on Kraken branch. Maybe we should not compile RDMA module by default?

Actions #5

Updated by Haomai Wang almost 7 years ago

hmm, I don't realize we enable rdma default in K already... but upstream rpm build is ok?

Actions #6

Updated by Chang Liu almost 7 years ago

it seems that we enable rdma default in K branch. I will check CMake

[ 37%] Building CXX object src/CMakeFiles/common.dir/msg/async/rdma/RDMAStack.cc.o
Actions #7

Updated by Kefu Chai almost 7 years ago

  • Status changed from Pending Backport to Won't Fix

@Chang you can always disable it by passing -DWITH_RDMA=OFF, BTW, RDMA support is disabled by default in kraken.

Actions #8

Updated by Chang Liu almost 7 years ago

@Kefu Chai thanks, I check the CMakeLists.txt again, RDMA support is disabled by default as you said. but it's strange why RDMA source compile. I don't specify -DWITH_RDMA=ON. I will try to compile Kraken again whithin a new clean environment.

Actions #9

Updated by Nathan Cutler almost 7 years ago

  • Backport deleted (kraken)
Actions

Also available in: Atom PDF