Reduce channel overhead (GC pressure)

[AArnott]

Reduce channel overhead (GC pressure)

[AArnott]

Channel overhead is ~17%. Good enough till someone says otherwise.

[buybackoff]

Thanks a lot for FullDuplexStream! Testing WebSockets has never been so easy :)

Look like DuplexStream ReadAsync/WriteAsync with ValueTask just use base Stream's virtual methods, which allocate a lot. I'm testing/profiling non-allocating code that uses .NET Core 2.1 zero-alloc path with ValueTask and there is a lot of noise. On the pic the profile of my code with 100% unhappy path (async state machine always allocates) and even there most garbage is from base Stream. It would be nice to have non-allocating implementation to test high perf non-allocating protocol stacks that use this corefx work: https://github.com/dotnet/corefx/issues/27445

On the happy path, there is no alloc in my code. I could see that in memory profiler and that's fine, but performance profiling is harder to digest.

[AArnott]

Hi @buybackoff. Thanks for sharing your results. I would love to correct this. It sounds like at least I can override ValueTask ReadAsync and ValueTask WriteAsync methods to fix this. This particular issue was tracking GC pressure on the MultiplexingStream in particular. Can you open a new issue to track this? I'll be happy to send you a candidate fix build for you to validate as part of this.