Skip to content

Commit

Permalink
binder: Set binder_(alloc_)debug_mask=0 to suppress logging
Browse files Browse the repository at this point in the history
 * Excessive logging -- not present on angler -- is
   affecting performance, contributing to missed audio
   deadlines and likely other latency-dependent tasks.

Bug: 30375418
[0ctobot: Extend patch to cover binder_alloc_debug_mask]
Co-authored-by: Adam W. Willis <return.of.octobot@gmail.com>

Change-Id: I2acbc34aff0019d24c5c748df583f06f9d01aeb5
Signed-off-by: Helium-Studio <67852324+Helium-Studio@users.noreply.github.com>
Signed-off-by: onettboots <blackcocopet@gmail.com>
  • Loading branch information
John Dias authored and onettboots committed May 26, 2024
1 parent 78da344 commit 628e7d6
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 3 deletions.
3 changes: 1 addition & 2 deletions drivers/android/binder.c
Original file line number Diff line number Diff line change
Expand Up @@ -121,8 +121,7 @@ enum {
BINDER_DEBUG_PRIORITY_CAP = 1U << 13,
BINDER_DEBUG_SPINLOCKS = 1U << 14,
};
static uint32_t binder_debug_mask = BINDER_DEBUG_USER_ERROR |
BINDER_DEBUG_FAILED_TRANSACTION | BINDER_DEBUG_DEAD_TRANSACTION;
static uint32_t binder_debug_mask = 0;
module_param_named(debug_mask, binder_debug_mask, uint, 0644);

char *binder_devices_param = CONFIG_ANDROID_BINDER_DEVICES;
Expand Down
2 changes: 1 addition & 1 deletion drivers/android/binder_alloc.c
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ enum {
BINDER_DEBUG_BUFFER_ALLOC = 1U << 2,
BINDER_DEBUG_BUFFER_ALLOC_ASYNC = 1U << 3,
};
static uint32_t binder_alloc_debug_mask = BINDER_DEBUG_USER_ERROR;
static uint32_t binder_alloc_debug_mask = 0;

module_param_named(debug_mask, binder_alloc_debug_mask,
uint, 0644);
Expand Down

0 comments on commit 628e7d6

Please sign in to comment.