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
Solaris Librados Port #6416
Solaris Librados Port #6416
Conversation
9e89c55
to
f1b1a10
Compare
f1b1a10
to
99efd15
Compare
@rohanmars all commits are missing Signed-off-by tag |
very close! just a few cosmetic items, and the Signed-off-by tags. Thanks! |
…/setter Signed-off-by: Rohan Mars <code@rohanmars.com>
Signed-off-by: Rohan Mars <code@rohanmars.com>
Signed-off-by: Rohan Mars <code@rohanmars.com>
Signed-off-by: Rohan Mars <code@rohanmars.com>
Signed-off-by: Rohan Mars <code@rohanmars.com>
Signed-off-by: Rohan Mars <code@rohanmars.com>
Signed-off-by: Rohan Mars <code@rohanmars.com>
int r = ::sendmsg(sd, msg, MSG_NOSIGNAL | (more ? MSG_MORE : 0)); | ||
int r; | ||
#if defined(MSG_NOSIGNAL) | ||
r = ::sendmsg(sd, msg, MSG_NOSIGNAL | (more ? MSG_MORE : 0)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@liewegas do we really need nosignal? could we handle signal interrupt at Messenger?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hrm, hard to say what that was breaking before. Maybe it's not a problem..
Signed-off-by: Rohan Mars <code@rohanmars.com>
99efd15
to
131deb3
Compare
looks good to me! let's make sure we didn't break anything. :) |
librados: Solaris port Reviewed-by: Sage Weil <sage@redhat.com>
Signed-off-by: Rohan Mars code@rohanmars.com