diff options
author | Thomas Haller <thaller@redhat.com> | 2022-03-30 09:13:45 +0200 |
---|---|---|
committer | Thomas Haller <thaller@redhat.com> | 2022-05-03 10:05:56 +0200 |
commit | 993da88befeb98ff006cf9518afed4eabacf20cc (patch) | |
tree | 9349bf2850c964c0cb469ac3c994ee67b94e211c | |
parent | 252549d9c7987274416a41a5b54d586b98b782e0 (diff) |
platform: guard logging statements with check whether logging is enabled
(cherry picked from commit 4fc25d523af0ad59a13be7dcdb25910d861e628b)
-rw-r--r-- | src/libnm-platform/nm-platform.c | 34 |
1 files changed, 19 insertions, 15 deletions
diff --git a/src/libnm-platform/nm-platform.c b/src/libnm-platform/nm-platform.c index 4fad8e6ae4..6dc1eae693 100644 --- a/src/libnm-platform/nm-platform.c +++ b/src/libnm-platform/nm-platform.c @@ -1873,12 +1873,14 @@ nm_platform_link_set_sriov_vfs(NMPlatform *self, int ifindex, const NMPlatformVF g_return_val_if_fail(ifindex > 0, FALSE); - _LOG3D("link: setting VFs"); - for (i = 0; vfs[i]; i++) { - char sbuf[NM_UTILS_TO_STRING_BUFFER_SIZE]; - const NMPlatformVF *vf = vfs[i]; + if (_LOGD_ENABLED()) { + _LOG3D("link: setting VFs"); + for (i = 0; vfs[i]; i++) { + char sbuf[NM_UTILS_TO_STRING_BUFFER_SIZE]; + const NMPlatformVF *vf = vfs[i]; - _LOG3D("link: VF %s", nm_platform_vf_to_string(vf, sbuf, sizeof(sbuf))); + _LOG3D("link: VF %s", nm_platform_vf_to_string(vf, sbuf, sizeof(sbuf))); + } } return klass->link_set_sriov_vfs(self, ifindex, vfs); @@ -1895,16 +1897,18 @@ nm_platform_link_set_bridge_vlans(NMPlatform *self, g_return_val_if_fail(ifindex > 0, FALSE); - _LOG3D("link: %s bridge VLANs on %s", - vlans ? "setting" : "clearing", - on_master ? "master" : "self"); - if (vlans) { - for (i = 0; vlans[i]; i++) { - char sbuf[NM_UTILS_TO_STRING_BUFFER_SIZE]; - const NMPlatformBridgeVlan *vlan = vlans[i]; - - _LOG3D("link: bridge VLAN %s", - nm_platform_bridge_vlan_to_string(vlan, sbuf, sizeof(sbuf))); + if (_LOGD_ENABLED()) { + _LOG3D("link: %s bridge VLANs on %s", + vlans ? "setting" : "clearing", + on_master ? "master" : "self"); + if (vlans) { + for (i = 0; vlans[i]; i++) { + char sbuf[NM_UTILS_TO_STRING_BUFFER_SIZE]; + const NMPlatformBridgeVlan *vlan = vlans[i]; + + _LOG3D("link: bridge VLAN %s", + nm_platform_bridge_vlan_to_string(vlan, sbuf, sizeof(sbuf))); + } } } |