> But the docs do seem to indicate that it's atomic (for whatever that > means for a single read operation?), so you are right, it should live in > instrumented-atomic.h. Actually, on further inspection, test_bit has lived in bitops/non-atomic.h since it was added in 4117b02132d1 ("[PATCH] bitops: generic __{,test_and_}{set,clear,change}_bit() and test_bit()") So to match that, the wrapper should live in instrumented-non-atomic.h too. If test_bit should move, that would need to be a different patch. But I don't really know if it makes too much sense to stress about a read operation, as opposed to a read/modify/write... Regards, Daniel