Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Skip to content
Commit eb327b91 authored by Ken Chen's avatar Ken Chen
Browse files

Remove redundant log in setResolverConfiguration

The log of setResolverConfiguration binder interface is printed twice in
both "adb logcat" and "dumpsys netd". Now that all parameters can be
printed by gen_log (b/129732660), the legacy logging for
setResolverConfiguration is no longer needed.

[Without this change]
11-16 02:52:57.877   864   864 I netd    :
DnsResolverService::setResolverConfiguration(101, [192.168.1.1], [],
1800, 25, 8, 64, 0, 0, "", [192.168.1.1]) -> (0) (0.3ms)

11-16 02:52:57.877   864   864 I netd    :
setResolverConfiguration(ResolverParamsParcel{netId: 101,
sampleValiditySeconds: 1800, successThreshold: 25, minSamples: 8,
maxSamples: 64, baseTimeoutMsec: 0, retryCount: 0, servers:
[192.168.1.1], domains: [], tlsName: , tlsServers: [192.168.1.1],
tlsFingerprints: [], caCertificate: , tlsConnectTimeoutMs: 0,
resolverOptions: (null), transportTypes: [1], meteredNetwork: false,
 dohParams: (null)}) <0.39ms>

[With this change]
11-16 03:23:53.098  1011  1149 I netd    :
setResolverConfiguration(ResolverParamsParcel{netId: 100,
sampleValiditySeconds: 1800, successThreshold: 25, minSamples: 8,
maxSamples: 64, baseTimeoutMsec: 0, retryCount: 0, servers:
[192.168.1.1], domains: [], tlsName: , tlsServers: [192.168.1.1],
tlsFingerprints: [], caCertificate: , tlsConnectTimeoutMs: 0,
resolverOptions: (null), transportTypes: [1], meteredNetwork: false,
dohParams: (null)}) <0.40ms>

Test: atest
Test: adb logcat; adb dumpsys netd
Change-Id: Ia21b999e37d13e86176e153d9cad5ac6f3256135
parent a5331b80
Loading
Loading
Loading
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment