Skip to content

Commit 0d72795

Browse files
committed
Make RunCommonHostLogic synchronous
1 parent cb3a84c commit 0d72795

File tree

2 files changed

+20
-14
lines changed

2 files changed

+20
-14
lines changed

src/libraries/System.Net.Sockets/System.Net.Sockets.sln

Lines changed: 17 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,8 @@
1-
Microsoft Visual Studio Solution File, Format Version 12.00
1+
2+
Microsoft Visual Studio Solution File, Format Version 12.00
3+
# Visual Studio Version 17
4+
VisualStudioVersion = 17.10.34928.147
5+
MinimumVisualStudioVersion = 10.0.40219.1
26
Project("{9A19103F-16F7-4668-BE54-9A1E7A4F7556}") = "StreamConformanceTests", "..\Common\tests\StreamConformanceTests\StreamConformanceTests.csproj", "{E5699508-7F83-4125-9495-C73BD4C43046}"
37
EndProject
48
Project("{9A19103F-16F7-4668-BE54-9A1E7A4F7556}") = "TestUtilities", "..\Common\tests\TestUtilities\TestUtilities.csproj", "{AEDCC778-C0D1-4155-9DD2-45EBF160CDA0}"
@@ -43,11 +47,11 @@ Project("{2150E333-8FDC-42A3-9474-1A3956D46DE8}") = "src", "src", "{2C7108F0-035
4347
EndProject
4448
Project("{2150E333-8FDC-42A3-9474-1A3956D46DE8}") = "gen", "gen", "{51E336C9-CFE7-4459-B407-FD5D9E760D85}"
4549
EndProject
46-
Project("{2150E333-8FDC-42A3-9474-1A3956D46DE8}") = "gen", "tools\gen", "{CCBA19A8-17AB-402A-96E6-A1C92784EAC2}"
50+
Project("{2150E333-8FDC-42A3-9474-1A3956D46DE8}") = "gen", "gen", "{CCBA19A8-17AB-402A-96E6-A1C92784EAC2}"
4751
EndProject
48-
Project("{2150E333-8FDC-42A3-9474-1A3956D46DE8}") = "src", "tools\src", "{99B1FC8E-726F-4DBF-887D-38AC7F63C230}"
52+
Project("{2150E333-8FDC-42A3-9474-1A3956D46DE8}") = "src", "src", "{99B1FC8E-726F-4DBF-887D-38AC7F63C230}"
4953
EndProject
50-
Project("{2150E333-8FDC-42A3-9474-1A3956D46DE8}") = "ref", "tools\ref", "{2D08A54A-C557-444C-9D8D-0BF0E20CDE5A}"
54+
Project("{2150E333-8FDC-42A3-9474-1A3956D46DE8}") = "ref", "ref", "{2D08A54A-C557-444C-9D8D-0BF0E20CDE5A}"
5155
EndProject
5256
Project("{2150E333-8FDC-42A3-9474-1A3956D46DE8}") = "tools", "tools", "{725DA203-1DB5-47D5-991A-8440E254BB23}"
5357
EndProject
@@ -136,27 +140,31 @@ Global
136140
GlobalSection(NestedProjects) = preSolution
137141
{E5699508-7F83-4125-9495-C73BD4C43046} = {FAD51322-998C-42D4-8FAA-EA479A3A0E82}
138142
{AEDCC778-C0D1-4155-9DD2-45EBF160CDA0} = {FAD51322-998C-42D4-8FAA-EA479A3A0E82}
139-
{69A7DCCE-B05B-44F3-B7BC-17A90759D1A2} = {FAD51322-998C-42D4-8FAA-EA479A3A0E82}
140143
{3C5413D5-7662-4E7C-8A1D-C8E6C527782F} = {FC5FEA3E-33B9-4AEE-8E41-367BB73182EF}
141144
{5BE2193A-9EAB-4F1C-B2F4-E969C84F2614} = {FC5FEA3E-33B9-4AEE-8E41-367BB73182EF}
142145
{EAEEEA1F-FE18-4CA1-88C9-59CD1097C593} = {FC5FEA3E-33B9-4AEE-8E41-367BB73182EF}
143146
{FDFDE647-E25F-4B21-B002-4EEEFC42EFA1} = {FC5FEA3E-33B9-4AEE-8E41-367BB73182EF}
144-
{293F6DBE-2ECA-4F22-AF58-4AC1668F028B} = {FC5FEA3E-33B9-4AEE-8E41-367BB73182EF}
145-
{F7A848B6-8307-4702-AEEE-20223FB73F29} = {FC5FEA3E-33B9-4AEE-8E41-367BB73182EF}
146147
{FD313302-FE04-414E-8A62-3BC8894115C1} = {2C7108F0-035E-44A5-8451-FE9535F53D43}
148+
{69A7DCCE-B05B-44F3-B7BC-17A90759D1A2} = {FAD51322-998C-42D4-8FAA-EA479A3A0E82}
147149
{85782EDD-90A7-4600-89EB-4E2C0AAAD446} = {51E336C9-CFE7-4459-B407-FD5D9E760D85}
148150
{0CAE4AA4-2C81-4CA4-978A-F86DD25FA338} = {51E336C9-CFE7-4459-B407-FD5D9E760D85}
149151
{3DD27643-568B-4BA1-82F2-FACFC8365836} = {51E336C9-CFE7-4459-B407-FD5D9E760D85}
152+
{293F6DBE-2ECA-4F22-AF58-4AC1668F028B} = {FC5FEA3E-33B9-4AEE-8E41-367BB73182EF}
153+
{F7A848B6-8307-4702-AEEE-20223FB73F29} = {FC5FEA3E-33B9-4AEE-8E41-367BB73182EF}
150154
{FA256A4B-476D-4BCA-8E62-C2A11DF778CE} = {CCBA19A8-17AB-402A-96E6-A1C92784EAC2}
151155
{6D7B425E-F07B-4572-98D8-728E3053E30F} = {CCBA19A8-17AB-402A-96E6-A1C92784EAC2}
152-
{CCBA19A8-17AB-402A-96E6-A1C92784EAC2} = {725DA203-1DB5-47D5-991A-8440E254BB23}
153156
{21411575-DD1A-45E1-8B10-6D4EA58684CD} = {99B1FC8E-726F-4DBF-887D-38AC7F63C230}
154157
{FA935CE3-6C83-4993-B565-95DF3129A43B} = {99B1FC8E-726F-4DBF-887D-38AC7F63C230}
155-
{99B1FC8E-726F-4DBF-887D-38AC7F63C230} = {725DA203-1DB5-47D5-991A-8440E254BB23}
156158
{C2011D8C-93D7-486E-8B01-0792262F4EF8} = {2D08A54A-C557-444C-9D8D-0BF0E20CDE5A}
159+
{CCBA19A8-17AB-402A-96E6-A1C92784EAC2} = {725DA203-1DB5-47D5-991A-8440E254BB23}
160+
{99B1FC8E-726F-4DBF-887D-38AC7F63C230} = {725DA203-1DB5-47D5-991A-8440E254BB23}
157161
{2D08A54A-C557-444C-9D8D-0BF0E20CDE5A} = {725DA203-1DB5-47D5-991A-8440E254BB23}
158162
EndGlobalSection
159163
GlobalSection(ExtensibilityGlobals) = postSolution
160164
SolutionGuid = {BA222465-FBD4-4377-8A8A-783BF85E01F7}
161165
EndGlobalSection
166+
GlobalSection(SharedMSBuildProjectFiles) = preSolution
167+
..\..\tools\illink\src\ILLink.Shared\ILLink.Shared.projitems*{6d7b425e-f07b-4572-98d8-728e3053e30f}*SharedItemsImports = 5
168+
..\..\tools\illink\src\ILLink.Shared\ILLink.Shared.projitems*{fa935ce3-6c83-4993-b565-95df3129a43b}*SharedItemsImports = 5
169+
EndGlobalSection
162170
EndGlobal

src/libraries/System.Net.Sockets/tests/FunctionalTests/SocketDuplicationTests.cs

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -324,7 +324,7 @@ public async Task DuplicateAndClose_TcpServerHandler(AddressFamily addressFamily
324324
if (sameProcess)
325325
{
326326
Task handlerCode = Task.Run(() => HandlerServerCode(_ipcPipeName));
327-
await RunCommonHostLogic(Environment.ProcessId);
327+
RunCommonHostLogic(Environment.ProcessId);
328328
await handlerCode;
329329
}
330330
else
@@ -334,18 +334,16 @@ public async Task DuplicateAndClose_TcpServerHandler(AddressFamily addressFamily
334334
// Since RunCommonHostLogic can throw, we need to make sure the server process is disposed
335335
try
336336
{
337-
await RunCommonHostLogic(hServerProc.Process.Id);
337+
RunCommonHostLogic(hServerProc.Process.Id);
338338
}
339339
finally
340340
{
341341
await hServerProc.DisposeAsync();
342342
}
343343
}
344344

345-
async Task RunCommonHostLogic(int processId)
345+
void RunCommonHostLogic(int processId)
346346
{
347-
await Task.CompletedTask.ConfigureAwait(ConfigureAwaitOptions.ForceYielding);
348-
349347
pipeServerStream.WaitForConnection();
350348

351349
// Duplicate the socket:

0 commit comments

Comments
 (0)