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.

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

1.Changes to fix incorrect ip being assigned in case of multiple RDMA… #7747

Merged
merged 1 commit into from Feb 27, 2016

Conversation

vsjulu
Copy link

@vsjulu vsjulu commented Feb 22, 2016

… ports.

  1. remove rdma local and return error instead to avoid issues with using rdma_local incase of multiple ports.

Sigwned-off-by: Subramanyam Varanasi s.varanasi@ssi.samsung.com

@vsjulu
Copy link
Author

vsjulu commented Feb 22, 2016

@mattbenjamin - re submitting a new pull request with the XIO changes alone. Changes are same as whatever you previously approved. Decided to split the commits and pull requests for the XIO vs atmomics change.

@tchaikov
Copy link
Contributor

@vsjulu could you add a prefix of the title of your commit message? like "xio: "?

@tchaikov
Copy link
Contributor

and your "Signed-off-by" has a typo: Sigwned-off-by

…DMA ports.remove rdma local and return error instead to avoid issues with using rdma_local incase of multiple ports.

Signed-off-by: Subramanyam Varanasi <s.varanasi@ssi.samsung.com>
@vsjulu
Copy link
Author

vsjulu commented Feb 26, 2016

@tchaikov I have updated the commit message.

tchaikov added a commit that referenced this pull request Feb 27, 2016
xio: fix incorrect ip being assigned in case of multiple RDMA ports

Reviewed-by: Kefu Chai <kchai@redhat.com>
@tchaikov tchaikov merged commit 8c054f7 into ceph:master Feb 27, 2016
@tchaikov tchaikov self-assigned this Feb 27, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants