From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id E696D81F3B for ; Wed, 16 Nov 2016 20:54:01 -0800 (PST) Received: from fmsmga002.fm.intel.com ([10.253.24.26]) by fmsmga101.fm.intel.com with ESMTP; 16 Nov 2016 20:54:06 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.31,651,1473145200"; d="scan'208";a="1086399705" Received: from mdkinney-mobl.amr.corp.intel.com ([10.232.100.21]) by fmsmga002.fm.intel.com with ESMTP; 16 Nov 2016 20:54:06 -0800 From: Michael Kinney To: edk2-devel@lists.01.org Date: Wed, 16 Nov 2016 20:53:42 -0800 Message-Id: <1479358424-17096-1-git-send-email-michael.d.kinney@intel.com> X-Mailer: git-send-email 2.6.3.windows.1 Subject: [Patch 0/2] MdePkg/BaseSynchronizationLib: Add volatile X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 17 Nov 2016 04:54:02 -0000 Update BaseSynchronizationLib class and instance to use volatile parameters in Interlocked*() functions and internal worker functions to be consistent with SpinLock functions and avoid conversion of a volatile pointer to a non-volatile pointer type. Michael Kinney (2): MdePkg/Include: Add volatile to SynchronizationLib parameters MdePkg/BaseSynchronizationLib: Add volatile Interlocked*() APIs MdePkg/Include/Library/SynchronizationLib.h | 14 +++++++------- .../Ia32/InterlockedCompareExchange16.asm | 4 ++-- .../Ia32/InterlockedCompareExchange16.c | 4 ++-- .../Ia32/InterlockedCompareExchange16.nasm | 4 ++-- .../Ia32/InterlockedCompareExchange32.asm | 4 ++-- .../Ia32/InterlockedCompareExchange32.c | 4 ++-- .../Ia32/InterlockedCompareExchange32.nasm | 4 ++-- .../Ia32/InterlockedCompareExchange64.asm | 4 ++-- .../Ia32/InterlockedCompareExchange64.c | 4 ++-- .../Ia32/InterlockedCompareExchange64.nasm | 4 ++-- .../Ia32/InterlockedDecrement.asm | 2 +- .../BaseSynchronizationLib/Ia32/InterlockedDecrement.c | 4 ++-- .../Ia32/InterlockedDecrement.nasm | 4 ++-- .../Ia32/InterlockedIncrement.asm | 4 ++-- .../BaseSynchronizationLib/Ia32/InterlockedIncrement.c | 4 ++-- .../Ia32/InterlockedIncrement.nasm | 4 ++-- .../Library/BaseSynchronizationLib/Synchronization.c | 18 +++++++++--------- .../BaseSynchronizationLib/SynchronizationGcc.c | 18 +++++++++--------- .../BaseSynchronizationLib/SynchronizationMsc.c | 18 +++++++++--------- .../X64/InterlockedCompareExchange16.asm | 6 +++--- .../X64/InterlockedCompareExchange16.c | 4 ++-- .../X64/InterlockedCompareExchange16.nasm | 6 +++--- .../X64/InterlockedCompareExchange32.asm | 6 +++--- .../X64/InterlockedCompareExchange32.c | 4 ++-- .../X64/InterlockedCompareExchange32.nasm | 6 +++--- .../X64/InterlockedCompareExchange64.asm | 6 +++--- .../X64/InterlockedCompareExchange64.c | 4 ++-- .../X64/InterlockedCompareExchange64.nasm | 4 ++-- .../X64/InterlockedDecrement.asm | 6 +++--- .../BaseSynchronizationLib/X64/InterlockedDecrement.c | 6 +++--- .../X64/InterlockedDecrement.nasm | 6 +++--- .../X64/InterlockedIncrement.asm | 6 +++--- .../BaseSynchronizationLib/X64/InterlockedIncrement.c | 6 +++--- .../X64/InterlockedIncrement.nasm | 6 +++--- 34 files changed, 104 insertions(+), 104 deletions(-) -- 2.6.3.windows.1