Re: MSDN volatile sample

From:

"Ben Voigt [C++ MVP]" <rbv@nospam.nospam>

Newsgroups:

microsoft.public.vc.language

Date:

Fri, 28 Dec 2007 10:02:14 -0600

Message-ID:

<uXdvHrWSIHA.5160@TK2MSFTNGP05.phx.gbl>

"George" <George@discussions.microsoft.com> wrote in message
news:987FBEA6-277E-43F6-B45D-16FA4E6B4888@microsoft.com...

Thanks Alex,

1.

You are correct. Now I am confused about the MSDN sample. What is it
purpose? Telling us we have to put volatile keyword to thread sharing data
to
make program function well as designed? In my past experience, I have
never
used volatile to all shared data between threads.

Did you use any synchronization primitive, like a critical section or
WaitForSingleObject? If so then these contain a memory barrier which
guarantee updates to global variables. When these are not used, volatile
must be specified to get the memory barriers.

2.

Now I strongly suspect whether compiler will generate any wrong code --
functional wrong code. In MSDN sample, variable Sentinel is used to act as
a
shared variable between thread1 and thread2. Compiler should guarantee
that
both threads can read/write the correct value of Sentinel.

It seems that volatile will make wrong optimization to prevent thread1
from
reading the most recent correct value set by thread2? I think it will
bring
high risks to careless developers, who does not know about volatile and
forget to put it ahead of the variable, which will result in the wrong
optimization of compiler.

Developers who do not know about volatile should not use multiple threads.

What is your perspective on such optimization?

regards,
George

"Alex Blekhman" wrote:

"George" wrote:

Could you provide a link to your quoted Sleep method description
please?

I am reading from,

http://msdn2.microsoft.com/en-us/library/d00bd51t(VS.80).aspx

not the same as you quoted.

You're reading in the wrong place. Didn't you noticed that this
page belongs to .NET framework documentation? It has nothing to do
with Win32 API function `Sleep'.

"Sleep"
http://msdn2.microsoft.com/en-us/library/ms686298(VS.85).aspx

Alex