Skip to content

Commit

Permalink
Disable CPU Group Pinning for Body Streaming ELG (#403)
Browse files Browse the repository at this point in the history
  • Loading branch information
waahm7 authored Jan 24, 2024
1 parent 237e9e1 commit 73d3c86
Showing 1 changed file with 3 additions and 7 deletions.
10 changes: 3 additions & 7 deletions source/s3_client.c
Original file line number Diff line number Diff line change
Expand Up @@ -514,13 +514,9 @@ struct aws_s3_client *aws_s3_client_new(
.shutdown_callback_user_data = client,
};

if (aws_get_cpu_group_count() > 1) {
client->body_streaming_elg = aws_event_loop_group_new_default_pinned_to_cpu_group(
client->allocator, num_streaming_threads, 1, &body_streaming_elg_shutdown_options);
} else {
client->body_streaming_elg = aws_event_loop_group_new_default(
client->allocator, num_streaming_threads, &body_streaming_elg_shutdown_options);
}
client->body_streaming_elg = aws_event_loop_group_new_default(
client->allocator, num_streaming_threads, &body_streaming_elg_shutdown_options);

if (!client->body_streaming_elg) {
/* Fail to create elg, we should fail the call */
goto on_error;
Expand Down

0 comments on commit 73d3c86

Please sign in to comment.