Correct usage of std::atomic

user_158109 0
Viewed 824

Is this the correct way to use std::atomic? I have a single Logger declared at namespace level called LOGGER:

class Logger {
    public:

    Logger();
    ~Logger();

    bool Init(std::wstring logFileLocation);
    bool Shutdown();

    void const LogMessage(std::wstring message);
};

std::atomic<Logger&> LOGGER;

My expectation is that LOGGER will instantiated once (no races; it will be initialized from a single known point in the code), but then accessed from multiple threads.

I'm new to multi-threading and thread safety in C++; but in C# or Java I would like to either make LOGGER volatile or (in C#) put memory fences around it. Is this the correct analogue in C++?

stdatomic thread-safety multithreading c++
2 Answers

The std::atomic works on a "bag of bits": an object of type T whose bitwise representation, stored in sizeof(T) bytes, carries the whole value. It means that nothing that needs to atomically manipulated (by these specific operations on std::atomic<T>) can reside in an object pointed to by a member of T, that would never work.

Also it means that std::atomic<T> can only work for a concrete object type T. A reference type is an "alias" for another object: a reference is not an object.

user_9638640
Related