diff options
author | Andy Polyakov <appro@openssl.org> | 2018-08-13 22:53:14 +0200 |
---|---|---|
committer | Andy Polyakov <appro@openssl.org> | 2018-08-17 12:40:39 +0200 |
commit | d2b863643d94704abd9840519cfc54d22d0481fe (patch) | |
tree | 1de96a34e4be5136cb43333291483cd25a7ecaa2 /crypto/threads_win.c | |
parent | Configure: warn when 'none' is the chosen seed source (diff) | |
download | openssl-d2b863643d94704abd9840519cfc54d22d0481fe.tar.xz openssl-d2b863643d94704abd9840519cfc54d22d0481fe.zip |
crypto/threads_*: remove CRYPTO_atomic_{read|write}.
CRYPTO_atomic_read was added with intention to read statistics counters,
but readings are effectively indistinguishable from regular load (even
in non-lock-free case). This is because you can get out-dated value in
both cases. CRYPTO_atomic_write was added for symmetry and was never used.
Reviewed-by: Kurt Roeckx <kurt@roeckx.be>
(Merged from https://github.com/openssl/openssl/pull/6883)
Diffstat (limited to 'crypto/threads_win.c')
-rw-r--r-- | crypto/threads_win.c | 12 |
1 files changed, 0 insertions, 12 deletions
diff --git a/crypto/threads_win.c b/crypto/threads_win.c index 7fdbc1f67f..d8fdfb74f5 100644 --- a/crypto/threads_win.c +++ b/crypto/threads_win.c @@ -155,18 +155,6 @@ int CRYPTO_atomic_add(int *val, int amount, int *ret, CRYPTO_RWLOCK *lock) return 1; } -int CRYPTO_atomic_read(int *val, int *ret, CRYPTO_RWLOCK *lock) -{ - *ret = InterlockedCompareExchange(val, 0, 0); - return 1; -} - -int CRYPTO_atomic_write(int *val, int n, CRYPTO_RWLOCK *lock) -{ - InterlockedExchange(val, n); - return 1; -} - int openssl_init_fork_handlers(void) { return 0; |