883966 - Pandas should create network stats file

Status: RESOLVED WONTFIX

(Infrastructure & Operations Graveyard :: CIDuty, task, P3)

Description

[Richard Newman [:rnewman]]

In Bug 883539 we're hitting exactly the issue described here:

https://groups.google.com/forum/#!msg/android-platform/qBgLYSCGnD8/cbqZbDkmzxwJ

Can we configure our Pandas to not logspew fatal errors about missing network stats? You can see this in every log:


F/BatteryStatsImpl( 1401): problem reading network stats
F/BatteryStatsImpl( 1401): java.lang.IllegalStateException: problem parsing idx 1
F/BatteryStatsImpl( 1401): 	at com.android.internal.net.NetworkStatsFactory.readNetworkStatsDetail(NetworkStatsFactory.java:300)
F/BatteryStatsImpl( 1401): 	at com.android.internal.net.NetworkStatsFactory.readNetworkStatsDetail(NetworkStatsFactory.java:250)
F/BatteryStatsImpl( 1401): 	at com.android.internal.os.BatteryStatsImpl.getNetworkStatsDetailGroupedByUid(BatteryStatsImpl.java:5734)
F/BatteryStatsImpl( 1401): 	at com.android.internal.os.BatteryStatsImpl.access$100(BatteryStatsImpl.java:76)
F/BatteryStatsImpl( 1401): 	at com.android.internal.os.BatteryStatsImpl$Uid.computeCurrentTcpBytesReceived(BatteryStatsImpl.java:2457)
F/BatteryStatsImpl( 1401): 	at com.android.internal.os.BatteryStatsImpl$Uid.getTcpBytesReceived(BatteryStatsImpl.java:2446)
F/BatteryStatsImpl( 1401): 	at com.android.internal.os.BatteryStatsImpl.writeSummaryToParcel(BatteryStatsImpl.java:5437)
F/BatteryStatsImpl( 1401): 	at com.android.internal.os.BatteryStatsImpl.writeLocked(BatteryStatsImpl.java:4836)
F/BatteryStatsImpl( 1401): 	at com.android.internal.os.BatteryStatsImpl.writeAsyncLocked(BatteryStatsImpl.java:4818)
F/BatteryStatsImpl( 1401): 	at com.android.server.am.ActivityManagerService.<init>(ActivityManagerService.java:1492)
F/BatteryStatsImpl( 1401): 	at com.android.server.am.ActivityManagerService.<init>(ActivityManagerService.java:151)
F/BatteryStatsImpl( 1401): 	at com.android.server.am.ActivityManagerService$AThread.run(ActivityManagerService.java:1393)
F/BatteryStatsImpl( 1401): Caused by: java.io.FileNotFoundException: /proc/net/xt_qtaguid/stats: open failed: ENOENT (No such file or directory)
F/BatteryStatsImpl( 1401): 	at libcore.io.IoBridge.open(IoBridge.java:406)
F/BatteryStatsImpl( 1401): 	at java.io.FileInputStream.<init>(FileInputStream.java:78)
F/BatteryStatsImpl( 1401): 	at com.android.internal.net.NetworkStatsFactory.readNetworkStatsDetail(NetworkStatsFactory.java:269)
F/BatteryStatsImpl( 1401): 	... 11 more

Comment 1

[Chris Cooper [:coop] (he/him)]

Callek is about to roll a new panda image in bug 958710. Assigning to him to make this (hopefully) simple change at the same time if possible.

Comment 2

[Justin Wood (:Callek)]

Richard, is there any way to solve this other than recompiling/patching the OS -- the link you provided did not give me any hope at figuring it out via a simple config change.

Comment 3

[Richard Newman [:rnewman]]

Does this help?

http://agolovatyuk.blogspot.com/2012/04/android-traffic-statistics-inside.html

You'll need the appropriate kernel module loaded to expose the sysfs interface.

There seems to be some evidence that Pandas should have this:

https://android.googlesource.com/kernel/omap.git/+/android-omap-panda-3.0/net/core/net-sysfs.c
                                                                ^^^^^              ^^^^^^^^^

but I neither own a Panda nor have access to one, so I'm pretty much armed only with Google, same as you :D

Comment 4

[Justin Wood (:Callek)]

based on that and my own skim it looks like I probably need to recompile something, which I'm not prepared to do.

gbrown, is there an easy config setting to make this work? if you can't make heads or tails of it, please redirect needinfo at richard again whom will then (after january) request a panda loan, and try and figure this out.

Comment 5

[Geoff Brown [:gbrown]]

I think you need to re-build the kernel. I believe our base panda image came from Linaro binaries, so there is some ground-breaking to do here. I would not recommend rolling this bug into the comparatively easy sutagent 1.20 or flash efforts.

Comment 6

[Justin Wood (:Callek)]

I'm not actively working on this, nor do I feel I'm the right person to rebuild a kernel.

That said, once we have a new kernel to test I'm more than willing to deploy this through staging/production for you guys.

Comment 7

[Chris Cooper [:coop] (he/him)]

To quote philor, "LOL pandas."