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

xio: migrate atomic_t to std::atomic<> #15230

Merged
merged 1 commit into from Jun 7, 2017

Conversation

Projects
None yet
4 participants
@chardan
Contributor

chardan commented May 23, 2017

Signed-off-by: Jesse Williamson jwilliamson@suse.de

@tchaikov

@Adirl does this change look sane to you?

@@ -30,9 +30,9 @@
#define dout_prefix *_dout << "xio."
Mutex mtx("XioMessenger Package Lock");
atomic_t initialized;
std::atomic<unsigned> initialized = { 0 };

This comment has been minimized.

@tchaikov

tchaikov May 27, 2017

Contributor

this should be std::atomic<bool>

@Adirl

This comment has been minimized.

Adirl commented May 28, 2017

@tchaikov
👍

@tchaikov

This comment has been minimized.

Contributor

tchaikov commented May 28, 2017

@chardan could you squash these two commit into a single one?

@liewegas liewegas added the needs-qa label Jun 1, 2017

xio: migrate atomic_t to std::atomic<>
Signed-off-by: Jesse Williamson <jwilliamson@suse.de>

@liewegas liewegas merged commit 8ded50e into ceph:master Jun 7, 2017

3 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details
default Build finished.
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment