OOM reporting doesn't consider device reboots

[jparise]

Description

computeDidOOMLastLaunchWithConfig: uses a number of heuristics to determine whether or not the last launch terminated due to an OOM condition.

One input that's not considered is the device's boot time, which can be used to determine whether or not the device was rebooted. (Reboots don't send an explicit termination signal to apps so this case could be classified as an OOM.)

Describe the solution you'd like

Compare the current launch's boot time to the last launch's boot time.

time_t systemBootTime = 0;
struct timeval boottime;
size_t size = sizeof(boottime);
if (sysctlbyname("kern.boottime", &boottime, &size, NULL, 0) != -1) {
    systemBootTime = boottime.tv_sec;
}

[abigailbramble]

Hi @jparise thanks for the report! Improvements to our OOM detection heuristic is on our roadmap and we hope to improve this in a future release. We will be working on this when priorities allow.

[johnkiely1]

Hi @jparise

We have now made OOM detection improvements in v6.2.5 which considers whether the device was rebooted or not.

Thanks again for suggesting this.