larger_read_buffer.patch (1072B)
1 From 92f174669fc86642d4a8baf4b8d1305dc593eb2c Mon Sep 17 00:00:00 2001 2 From: Greg Stoll <gstoll@mozilla.com> 3 Date: Wed, 26 Feb 2025 08:04:10 -0600 4 Subject: [PATCH] Bug 1948884 - increase Content Analysis client buffer size 5 r=#dlp-reviewers! 6 7 It's a bit unclear to me why this helps, since the logic seems to 8 indicate that ReadNextMessageFromPipe() will resize the buffer as 9 needed. But on my machine this makes a ~100 file upload work 10 consistently, where it would freeze sometimes before this change. And 11 it seems harmless. 12 13 diff --git a/browser/src/client_win.cc b/browser/src/client_win.cc 14 index 039946d131398..aaa17d0e051a9 100644 15 --- a/browser/src/client_win.cc 16 +++ b/browser/src/client_win.cc 17 @@ -17,7 +17,9 @@ 18 namespace content_analysis { 19 namespace sdk { 20 21 -const DWORD kBufferSize = 4096; 22 +// Increased to a larger size to help with issues with analyzing a lot of 23 +// files at once - see bug 1948884. 24 +const DWORD kBufferSize = 65536; 25 26 // Use the same default timeout value (50ms) as CreateNamedPipeA(), expressed 27 // in 100ns intervals. 28 -- 29 2.47.1.windows.2