Implement pthread_cond_t with Win32 CONDITION_VARIABLE #1214

rongjiecomputer · 2017-06-22T01:06:00Z

Win32 CONDITION_VARIABLE has better performance and is easier to maintain.

Since CONDITION_VARIABLE is not available in Windows XP and below, old implementation of pthread_cond_t is kept under define guard 'GIT_WIN_XP_SUPPORT'. To enable old implementation, build with make CFLAGS="-DGIT_WIN_XP_SUPPORT".

If I build with make CFLAGS="-D_WIN32_WINNT=0x0600 or change the define in git-compat-util.h, I will get many warnings and errors about symbols in compat/poll/poll.h are redefined because winsock2.h define them already when _WIN32_WINNT=0x0600.

I choose the simpler solution: keep the old _WIN32_WINNT and declare the 4 required Win32 functions in pthread.c.

Win32 CONDITION_VARIABLE has better performance and is easier to maintain. Since CONDITION_VARIABLE is not available in Windows XP and below, old implementation of pthread_cond_t is kept under define guard 'GIT_WIN_XP_SUPPORT'. To enable old implementation, build with make CFLAGS="-DGIT_WIN_XP_SUPPORT". Signed-off-by: Loo Rong Jie <loorongjie@gmail.com> fast-forwarded.

Signed-off-by: Loo Rong Jie <loorongjie@gmail.com>

rongjiecomputer · 2017-06-23T14:37:08Z

See also for bumping bumped _WINNT_VER to 0x600: #1188 (comment)

rongjiecomputer · 2017-07-03T12:32:06Z

@dscho Ping?

compat/win32/pthread.c

@@ -57,6 +57,8 @@ pthread_t pthread_self(void)
 	return t;
 }

+#ifdef GIT_WIN_XP_SUPPORT


Signed-off-by: Loo Rong Jie <loorongjie@gmail.com>

dscho · 2017-08-15T11:17:04Z

Thank you so much!

Implement pthread_cond_t with Win32 CONDITION_VARIABLE

The XP-compatibility layer emulating pthreads (which is [no longer needed](https://git-for-windows.github.io/requirements.html) [was dropped in favor of modern Windows threading APIs](git-for-windows/git#1214); This should make threaded operations slightly faster and more robust. Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>

Implement pthread_cond_t with Win32 CONDITION_VARIABLE

This was pull request git-for-windows#1214 from rongjiecomputer/master Implement pthread_cond_t with Win32 CONDITION_VARIABLE Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>

This was pull request #1214 from rongjiecomputer/master Implement pthread_cond_t with Win32 CONDITION_VARIABLE Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>

rongjiecomputer added 2 commits June 22, 2017 09:03

Format to 80 cols

617fa0f

Signed-off-by: Loo Rong Jie <loorongjie@gmail.com>

whoisj reviewed Jul 7, 2017

View reviewed changes

Remove old code and macro-ize implementation

e426067

Signed-off-by: Loo Rong Jie <loorongjie@gmail.com>

dscho merged commit eead5ec into git-for-windows:master Aug 15, 2017

dscho added a commit that referenced this pull request Aug 15, 2017

Merge pull request #1214 from rongjiecomputer/master

01101c5

Implement pthread_cond_t with Win32 CONDITION_VARIABLE

dscho added a commit that referenced this pull request Aug 18, 2017

Merge pull request #1214 from rongjiecomputer/master

12ef5c4

Implement pthread_cond_t with Win32 CONDITION_VARIABLE

git-for-windows-ci pushed a commit that referenced this pull request Aug 18, 2017

Merge pull request #1214 from rongjiecomputer/master

266422c

Implement pthread_cond_t with Win32 CONDITION_VARIABLE

git-for-windows-ci pushed a commit that referenced this pull request Aug 20, 2017

Merge pull request #1214 from rongjiecomputer/master

a18e1d3

Implement pthread_cond_t with Win32 CONDITION_VARIABLE

dscho added a commit that referenced this pull request Aug 22, 2017

Merge pull request #1214 from rongjiecomputer/master

eaae45c

Implement pthread_cond_t with Win32 CONDITION_VARIABLE

git-for-windows-ci pushed a commit that referenced this pull request Aug 23, 2017

Merge pull request #1214 from rongjiecomputer/master

7580452

Implement pthread_cond_t with Win32 CONDITION_VARIABLE

git-for-windows-ci pushed a commit that referenced this pull request Aug 24, 2017

Merge pull request #1214 from rongjiecomputer/master

ca70ca2

Implement pthread_cond_t with Win32 CONDITION_VARIABLE

git-for-windows-ci pushed a commit that referenced this pull request Aug 24, 2017

Merge pull request #1214 from rongjiecomputer/master

c8960b0

Implement pthread_cond_t with Win32 CONDITION_VARIABLE

git-for-windows-ci pushed a commit that referenced this pull request Aug 24, 2017

Merge pull request #1214 from rongjiecomputer/master

97136f5

Implement pthread_cond_t with Win32 CONDITION_VARIABLE

git-for-windows-ci pushed a commit that referenced this pull request Aug 24, 2017

Merge pull request #1214 from rongjiecomputer/master

1b581a2

Implement pthread_cond_t with Win32 CONDITION_VARIABLE

git-for-windows-ci pushed a commit that referenced this pull request Aug 25, 2017

Merge pull request #1214 from rongjiecomputer/master

f819886

Implement pthread_cond_t with Win32 CONDITION_VARIABLE

git-for-windows-ci pushed a commit that referenced this pull request Aug 25, 2017

Merge pull request #1214 from rongjiecomputer/master

2b90790

Implement pthread_cond_t with Win32 CONDITION_VARIABLE

dscho added a commit that referenced this pull request Aug 26, 2017

Merge pull request #1214 from rongjiecomputer/master

4ff90c1

Implement pthread_cond_t with Win32 CONDITION_VARIABLE

dscho added a commit that referenced this pull request Mar 23, 2018

Merge pull request #1214 from rongjiecomputer/master

e8384c3

Implement pthread_cond_t with Win32 CONDITION_VARIABLE

dscho added a commit that referenced this pull request Apr 3, 2018

Merge pull request #1214 from rongjiecomputer/master

683e1c9

Implement pthread_cond_t with Win32 CONDITION_VARIABLE

dscho added a commit that referenced this pull request May 29, 2018

Merge pull request #1214 from rongjiecomputer/master

c705bfa

Implement pthread_cond_t with Win32 CONDITION_VARIABLE

dscho added a commit that referenced this pull request May 29, 2018

Merge pull request #1214 from rongjiecomputer/master

21f67f0

Implement pthread_cond_t with Win32 CONDITION_VARIABLE

PKRoma pushed a commit to PKRoma/git-for-windows that referenced this pull request Jun 22, 2018

Merge pull request git-for-windows#1214 from rongjiecomputer/master

a0880a0

Implement pthread_cond_t with Win32 CONDITION_VARIABLE

PKRoma pushed a commit to PKRoma/git-for-windows that referenced this pull request Jun 22, 2018

Merge pull request git-for-windows#1214 from rongjiecomputer/master

8ab4fcb

Implement pthread_cond_t with Win32 CONDITION_VARIABLE

PKRoma pushed a commit to PKRoma/git-for-windows that referenced this pull request Jun 22, 2018

Merge pull request git-for-windows#1214 from rongjiecomputer/master

20449ce

Implement pthread_cond_t with Win32 CONDITION_VARIABLE

dscho added a commit that referenced this pull request Aug 22, 2018

Merge pull request #1214 from rongjiecomputer/master

a6848fe

Implement pthread_cond_t with Win32 CONDITION_VARIABLE

dscho added a commit to dscho/git that referenced this pull request Aug 22, 2018

Merge pull request git-for-windows#1214 from rongjiecomputer/master

ad514e4

Implement pthread_cond_t with Win32 CONDITION_VARIABLE

dscho added a commit that referenced this pull request Aug 23, 2018

Merge pull request #1214 from rongjiecomputer/master

91df2dc

Implement pthread_cond_t with Win32 CONDITION_VARIABLE

dscho added a commit that referenced this pull request Aug 23, 2018

Merge pull request #1214 from rongjiecomputer/master

16a6454

Implement pthread_cond_t with Win32 CONDITION_VARIABLE

dscho added a commit that referenced this pull request Aug 23, 2018

Merge pull request #1214 from rongjiecomputer/master

defa106

Implement pthread_cond_t with Win32 CONDITION_VARIABLE

jamill pushed a commit to jamill/git that referenced this pull request Aug 28, 2018

Merge pull request git-for-windows#1214 from rongjiecomputer/master

a6552b1

Implement pthread_cond_t with Win32 CONDITION_VARIABLE

jamill pushed a commit to jamill/git that referenced this pull request Sep 5, 2018

Merge pull request git-for-windows#1214 from rongjiecomputer/master

bdfc3c4

Implement pthread_cond_t with Win32 CONDITION_VARIABLE

git-for-windows-ci pushed a commit that referenced this pull request Sep 10, 2018

Merge pull request #1214 from rongjiecomputer/master

10eb283

Implement pthread_cond_t with Win32 CONDITION_VARIABLE

jamill pushed a commit to jamill/git that referenced this pull request Sep 11, 2018

Merge pull request git-for-windows#1214 from rongjiecomputer/master

4a77522

Implement pthread_cond_t with Win32 CONDITION_VARIABLE

git-for-windows-ci pushed a commit that referenced this pull request Sep 24, 2018

Merge pull request #1214 from rongjiecomputer/master

1f19001

Implement pthread_cond_t with Win32 CONDITION_VARIABLE

dscho added a commit that referenced this pull request Oct 10, 2018

Merge pull request #1214 from rongjiecomputer/master

2d32f76

Implement pthread_cond_t with Win32 CONDITION_VARIABLE

dscho added a commit that referenced this pull request Nov 21, 2018

Merge 'mingw-modernize-pthread_cond_t'

8a328a0

This was pull request #1214 from rongjiecomputer/master Implement pthread_cond_t with Win32 CONDITION_VARIABLE Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Implement pthread_cond_t with Win32 CONDITION_VARIABLE #1214

Implement pthread_cond_t with Win32 CONDITION_VARIABLE #1214

rongjiecomputer commented Jun 22, 2017

rongjiecomputer commented Jun 23, 2017

rongjiecomputer commented Jul 3, 2017

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.

dscho commented Aug 15, 2017

Implement pthread_cond_t with Win32 CONDITION_VARIABLE #1214

Implement pthread_cond_t with Win32 CONDITION_VARIABLE #1214

Conversation

rongjiecomputer commented Jun 22, 2017

rongjiecomputer commented Jun 23, 2017

rongjiecomputer commented Jul 3, 2017

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.

dscho commented Aug 15, 2017