<html>
<head>
<style>
.hmmessage P
{
margin:0px;
padding:0px
}
body.hmmessage
{
font-size: 10pt;
font-family:Verdana
}
</style>
</head>
<body class='hmmessage'>
ThreadPosix is basically unaffected -- I never provided user threads on Cygwin.<BR>
I guess it got a slight reduction regarding DoesWaitPidYield, but its SchedulerPosix implementation was unchanged.<BR>
<BR>
The code in ThreadPosix is similar to but different than ThreadPThread.<BR>
It is possible they could be merged but it wasn't trivial so I left ThreadPosix alone.<BR>
<BR>
> You are smearing implementation details across multiple files and directories<BR>
<BR>
You really think it is bad to have SchedulerPosix implemented in a small separate file?<BR>
On top of what happens to be common implementation details of ThreadPThread.m3 and ThreadWin32.m3?<BR>
<BR>
ThreadPScheduler.m3 is just basically three functions: IOWait, IOAlertWait, XIOWait<BR>
Plus the little internal utility, UTimeFromTime, the one liner DoesWaitPidYield, and non-trivial functions nested in XIOWait: TestFDS, CallSelect.<BR>
<BR>
I mean, you know, an alternative is to copy out very large chunks of ThreadWin32.m3 and ThreadPThread.m3 and merge them into ThreadCygwin.m3. That would be worse imho.<BR>
<BR>
"Directories" hardly.<BR>
<BR>
Or I can try debugging cygwin pthreads again.<BR>
<BR>
- Jay<BR><BR>
<HR id=stopSpelling>
<BR>
CC: jkrell@elego.de; m3devel@elegosoft.com<BR>From: hosking@cs.purdue.edu<BR>To: jay.krell@cornell.edu<BR>Subject: Re: [M3devel] [M3commit] CVS Update: cm3<BR>Date: Mon, 12 Jan 2009 12:46:54 +1100<BR><BR>You are smearing implementation details across multiple files and directories. Up until now, ThreadPThread has been nicely self-contained, and captured all the basic pieces of the thread implementation. Also, how does all of this fit with the ThreadPosix implementation?<BR>
<DIV><BR>
<DIV><SPAN class=EC_Apple-style-span style="WORD-SPACING: 0px; FONT: 12px Helvetica; TEXT-TRANSFORM: none; COLOR: rgb(0,0,0); TEXT-INDENT: 0px; WHITE-SPACE: normal; LETTER-SPACING: normal; BORDER-COLLAPSE: separate">
<DIV style="WORD-WRAP: break-word"><SPAN class=EC_Apple-style-span style="WORD-SPACING: 0px; FONT: 12px Helvetica; TEXT-TRANSFORM: none; COLOR: rgb(0,0,0); TEXT-INDENT: 0px; WHITE-SPACE: normal; LETTER-SPACING: normal; BORDER-COLLAPSE: separate"><SPAN class=EC_Apple-style-span style="WORD-SPACING: 0px; FONT: 12px Helvetica; TEXT-TRANSFORM: none; COLOR: rgb(0,0,0); TEXT-INDENT: 0px; WHITE-SPACE: normal; LETTER-SPACING: normal; BORDER-COLLAPSE: separate"><SPAN class=EC_Apple-style-span style="WORD-SPACING: 0px; FONT: 12px Helvetica; TEXT-TRANSFORM: none; COLOR: rgb(0,0,0); TEXT-INDENT: 0px; WHITE-SPACE: normal; LETTER-SPACING: normal; BORDER-COLLAPSE: separate"><SPAN class=EC_Apple-style-span style="WORD-SPACING: 0px; FONT: 12px Helvetica; TEXT-TRANSFORM: none; COLOR: rgb(0,0,0); TEXT-INDENT: 0px; WHITE-SPACE: normal; LETTER-SPACING: normal; BORDER-COLLAPSE: separate"><SPAN class=EC_Apple-style-span style="WORD-SPACING: 0px; FONT: 12px Helvetica; TEXT-TRANSFORM: none; COLOR: rgb(0,0,0); TEXT-INDENT: 0px; WHITE-SPACE: normal; LETTER-SPACING: normal; BORDER-COLLAPSE: separate"><SPAN class=EC_Apple-style-span style="WORD-SPACING: 0px; FONT: 12px Helvetica; TEXT-TRANSFORM: none; COLOR: rgb(0,0,0); TEXT-INDENT: 0px; WHITE-SPACE: normal; LETTER-SPACING: normal; BORDER-COLLAPSE: separate"><SPAN class=EC_Apple-style-span style="WORD-SPACING: 0px; FONT: 12px Helvetica; TEXT-TRANSFORM: none; COLOR: rgb(0,0,0); TEXT-INDENT: 0px; WHITE-SPACE: normal; LETTER-SPACING: normal; BORDER-COLLAPSE: separate"><SPAN class=EC_Apple-style-span style="WORD-SPACING: 0px; FONT: 12px Helvetica; TEXT-TRANSFORM: none; COLOR: rgb(0,0,0); TEXT-INDENT: 0px; WHITE-SPACE: normal; LETTER-SPACING: normal; BORDER-COLLAPSE: separate">
<DIV>On 12 Jan 2009, at 12:19, Jay wrote:</DIV></SPAN></SPAN></SPAN></SPAN></SPAN></SPAN></SPAN></SPAN></DIV></SPAN></DIV>
<DIV><BR class=EC_Apple-interchange-newline>
<BLOCKQUOTE><SPAN class=EC_Apple-style-span style="WORD-SPACING: 0px; FONT: 12px Helvetica; TEXT-TRANSFORM: none; COLOR: rgb(0,0,0); TEXT-INDENT: 0px; WHITE-SPACE: normal; LETTER-SPACING: normal; BORDER-COLLAPSE: separate">
<DIV class=EC_hmmessage style="FONT-SIZE: 10pt; FONT-FAMILY: Verdana">btw, I don't think it's that hairy, I merely split it into two or three files, and added interfaces so they can reuse code with each other. Movement between files is hard to track though (depending on version control, but with all I've used).<BR> <BR>The SchedulerPosix implementation moved to ThreadPScheduler.m3.<BR>What is shared between it and ThreadPThread/Win32.m3 is ThreadInternal.i3, which maybe should be in ThreadF.i3, though that's exposed outside m3core, and ThreadInternal.i3 includes a variable.<BR> <BR>I can try again to debug Cygwin pthreads though.<BR> <BR> - Jay<BR><BR><BR>> From:<SPAN class=EC_Apple-converted-space> </SPAN><A href="mailto:hosking@cs.purdue.edu">hosking@cs.purdue.edu</A><BR>> To:<SPAN class=EC_Apple-converted-space> </SPAN><A href="mailto:jkrell@elego.de">jkrell@elego.de</A><BR>> Date: Mon, 12 Jan 2009 11:03:33 +1100<BR>> CC:<SPAN class=EC_Apple-converted-space> </SPAN><A href="mailto:m3devel@elegosoft.com">m3devel@elegosoft.com</A><BR>> Subject: Re: [M3devel] [M3commit] CVS Update: cm3<BR>><SPAN class=EC_Apple-converted-space> </SPAN><BR>> Jay,<BR>><SPAN class=EC_Apple-converted-space> </SPAN><BR>> Can you remind me again why Cygwin was unable to use pthreads? It<SPAN class=EC_Apple-converted-space> </SPAN><BR>> seems you have introduced a bunch of hair into the PTHREADS<SPAN class=EC_Apple-converted-space> </SPAN><BR>> implementation to deal with broken Cygwin pthreads. As many of us<SPAN class=EC_Apple-converted-space> </SPAN><BR>> have already pointed out, Cygwin should be a port that tries as much<SPAN class=EC_Apple-converted-space> </SPAN><BR>> as possible to be like a standard POSIX platform (pthread-based) as<SPAN class=EC_Apple-converted-space> </SPAN><BR>> opposed to a weird Windows/POSIX hybrid.<BR>><SPAN class=EC_Apple-converted-space> </SPAN><BR>> I have a bunch of code that will be going into the PTHREADS base that<SPAN class=EC_Apple-converted-space> </SPAN><BR>> I am now at a loss to integrate with the changes you have made.<BR>><SPAN class=EC_Apple-converted-space> </SPAN><BR>> Help!<BR>><SPAN class=EC_Apple-converted-space> </SPAN><BR>> -- Tony<BR>><SPAN class=EC_Apple-converted-space> </SPAN><BR><BR></DIV></SPAN></BLOCKQUOTE></DIV><BR></DIV></body>
</html>