search

mgravell / Pipelines.Sockets.Unofficial

.NET managed sockets wrapper using the new "Pipelines" API
Other
412 stars 54 forks source link

Set unsafeSuppressExecutionContextFlow on SocketAsyncEventArgs #57

Open xqrzd opened 3 years ago

xqrzd commented 3 years ago

Is it safe to set that option on SocketAsyncEventArgs?

mgravell commented 3 years ago

Where are you seeing UnsafeSuppressExecutionContextFlow? What is the context here?

xqrzd commented 3 years ago

Sorry this was a terribly worded issue. There's a new parameter that can be passed to SocketAsyncEventArgs in .NET 5 to suppress execution context, https://github.com/dotnet/runtime/pull/706. Could this project set that value to get a bit of extra performance?

mgravell commented 3 years ago

Ah, k; interesting question. I don't know - it would need investigation.

On Wed, 7 Oct 2020 at 14:20, xqrzd notifications@github.com wrote:

Sorry this was a terribly worded issue. There's a new parameter that can be passed to SocketAsyncEventArgs in .NET 5 to suppress execution context, dotnet/runtime#706 https://github.com/dotnet/runtime/pull/706. Could this project set that value to get a bit of extra performance?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/mgravell/Pipelines.Sockets.Unofficial/issues/57#issuecomment-704930728, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAEHMCPGW26CSVP2GSZW5TSJRTIZANCNFSM4SGYJKIA .

-- Regards,

Marc