commit bfc0b6187c9e384a08ec30cde7e2de91cd635dad
author Maciej Żenczykowski <maze@google.com> Wed Aug 28 17:45:25 2024 -0700
committer Maciej Żenczykowski <maze@google.com> Thu Aug 29 21:17:12 2024 +0000
tree 7a7582453fae48a685217d96d4849d0a9e16e348
parent 461b4081296b6cd80171162327dfb4bfd02d0ade

NetBpfLoad: better log message on program load failure

We're reduced logging enough that it is no longer clear
which program failed to load.

Test: TreeHugger
Signed-off-by: Maciej Żenczykowski <maze@google.com>
Change-Id: I8919a055f82d0c0e6e6818799463c606e853f70b

bpf/loader/NetBpfLoad.cpp

diff --git a/bpf/loader/NetBpfLoad.cpp b/bpf/loader/NetBpfLoad.cpp
index 55e41d3..93dffb6 100644
--- a/bpf/loader/NetBpfLoad.cpp
+++ b/bpf/loader/NetBpfLoad.cpp
@@ -1018,10 +1018,11 @@
                 ALOGW("BPF_PROG_LOAD - END log_buf contents.");
 
                 if (cs[i].prog_def->optional) {
-                    ALOGW("failed program is marked optional - continuing...");
+                    ALOGW("failed program %s is marked optional - continuing...",
+                          cs[i].name.c_str());
                     continue;
                 }
-                ALOGE("non-optional program failed to load.");
+                ALOGE("non-optional program %s failed to load.", cs[i].name.c_str());
             }
         }