[tvOS] OutOfMemoryException in System.IO.Pipelines.Tests.PipeWriterTests.CompleteWithLargeWriteThrows

[runfoapp]

Runfo Tracking Issue: OutOfMemoryException in System.IO.Pipelines.Tests.PipeWriterTests.CompleteWithLargeWriteThrows

Build	Definition	Kind	Run Name	Console	Core Dump	Test Results	Run Client
1619019	runtime-extra-platforms	Rolling	net7.0-tvOS-Release-arm64-Mono_Release-OSX.1015.Amd64.AppleTV.Open	console.log		test results	runclient.py
1616876	runtime-extra-platforms	Rolling	net7.0-tvOS-Release-arm64-Mono_Release-OSX.1015.Amd64.AppleTV.Open	console.log		test results	runclient.py
1613945	runtime-extra-platforms	PR 64743	net7.0-tvOS-Release-arm64-Mono_Release-OSX.1015.Amd64.AppleTV.Open	console.log		test results	runclient.py
1612202	runtime-extra-platforms	Rolling	net7.0-Android-Release-arm-Mono_Release-Windows.10.Amd64.Android.Open	console.log		test results	runclient.py
1611370	runtime-extra-platforms	PR 64452	net7.0-tvOS-Release-arm64-Mono_Release-OSX.1015.Amd64.AppleTV.Open	console.log		test results	runclient.py
1602008	runtime-extra-platforms	Rolling	net7.0-tvOS-Release-arm64-Mono_Release-OSX.1015.Amd64.AppleTV.Open	console.log		test results	runclient.py
1600670	runtime-extra-platforms	Rolling	net7.0-tvOS-Release-arm64-Mono_Release-OSX.1015.Amd64.AppleTV.Open	console.log		test results	runclient.py
1596337	runtime-extra-platforms	Rolling	net7.0-tvOS-Release-arm64-Mono_Release-OSX.1015.Amd64.AppleTV.Open	console.log			runclient.py
1595549	runtime-extra-platforms	PR 64867	net7.0-tvOS-Release-arm64-Mono_Release-OSX.1015.Amd64.AppleTV.Open	console.log		test results	runclient.py

Build Result Summary

Day Hit Count	Week Hit Count	Month Hit Count
1	5	9

Tagging subscribers to 'os-tvos': @steveisok, @akoeplinger
See info in area-owners.md if you want to be subscribed.

Issue Details

---

Runfo Tracking Issue: OutOfMemoryException in System.IO.Pipelines.Tests.PipeWriterTests.CompleteWithLargeWriteThrows

Build	Definition	Kind	Run Name	Console	Core Dump	Test Results	Run Client
1596337	runtime-extra-platforms	Rolling	net7.0-tvOS-Release-arm64-Mono_Release-OSX.1015.Amd64.AppleTV.Open	console.log			runclient.py
1595549	runtime-extra-platforms	PR 64867	net7.0-tvOS-Release-arm64-Mono_Release-OSX.1015.Amd64.AppleTV.Open	console.log		test results	runclient.py

Build Result Summary

Day Hit Count	Week Hit Count	Month Hit Count
1	2	2

Author:	runfoapp[bot]
Assignees:	-
Labels:	area-System.IO.Pipelines, untriaged, runtime-mono, os-tvos
Milestone:	-

[elinor-fung]

3 of the last 5 rolling builds have hit this. @steveisok / @akoeplinger any ideas on who might be able to investigate what is going on here?

[steveisok]

Yes, we'll take a look at it. Seems contention related - maybe allocate a smaller amount in

runtime/src/libraries/System.IO.Pipelines/tests/PipeWriterTests.cs

Lines 233 to 258 in 99452b2

	public async Task CompleteWithLargeWriteThrows()
	{
	var pipe = new Pipe();
	pipe.Reader.Complete();
	var task = Task.Run(async () =>
	{
	await Task.Delay(10);
	pipe.Writer.Complete();
	});
	try
	{
	for (int i = 0; i < 1000; i++)
	{
	var buffer = new byte[10000000];
	await pipe.Writer.WriteAsync(buffer);
	}
	}
	catch (InvalidOperationException)
	{
	// Complete while writing
	}
	await task;
	}

?

[elinor-fung]

Looks like we're occasionally hitting OutOfMemoryException in this test on Android arm runs too: https://dev.azure.com/dnceng/public/_build/results?buildId=1612202&view=ms.vss-test-web.build-test-results-tab&runId=44895690&resultId=154816&paneView=dotnet-dnceng.dnceng-build-release-tasks.helix-test-information-tab

[MaximLipnin]

What if OutOfMemoryException could play the same role in tvOS case as InvalidOperationException does in the test - sorta an invalid operation ? Then we could just add another catch section. Anyway It would be weird.