Huge battery consumption on iOS since 1.14.0 (SO_REUSEADDR)

[AmarOk1412]

Hi.

Since pupnp 1.14.0, default configuration is causing a huge CPU consumption located in runMiniServer.
This is probably caused by a3912b0 cause SO_REUSEADDR is weirdly handled by iphone devices.

Adding the --disable-reuseaddr option solves this issue. IMHO, SO_REUSEADDR should not be enabled by default on Apple devices

[mrjimenez]

Hi Sébastien,

The default behavior is set in configure.ac or CMakeLists.txt, but this is not currently a platform based choice.

Do you think that this a regression introduced by this commit? Are you able to compile and test with the commit immediately before (cfbb791)?

Would it be possible to test this behavior in a PC? Do you have a small test code that I can use to try to fix it?

Regards,
Marcelo.

[AmarOk1412]

Yeah setting the default value in the configure is correct.

We observed this behaviour only on iPhone devices via the profiler directly on the whole app. So, no I don't think you will get it on a PC (macOS doesn't seems to be impacted and I can't really provide a minimal code). I don't own any iphone so it's from a distant investigation with a dev. I can ping her to come here.

If you need more details:

• Seems observable on pretty every iPhone devices. At least on the iPhone 5S, iphone 12 mini, iphone 10. But not on other platforms.
• we use pupnp 1.14.0 with --disable-largefile --disable-samples --disable-device --disable-webserver --without-documentation
• without --disable-reuseaddr, RunMiniServer is taking 94.5% of the time of the CPU, I didn't patch to debug more, but basically sure the while (!stopSock) { is just looping
• with --disable-reuseaddr, RunMiniServer takes 0.5% of the time (on a run of 32 secs, which is ok)
• We already observed weird behaviour with SO_REUSEADDR on apple devices, that's why we tried this.
• cfbb791 is not affected (note that this commit was not building, had to apply https://github.com/pupnp/pupnp/commit/dccd9311d6db0a1be7e3348a11be94edaa850d77.diff), profiler gives 0.5% usage for RunMiniServer again

Thank you for your prompt answer,
Regards,

[katekm]

Hi,
Issue is still present on iPhone and could be triggered by locking/unlocking the screen.
Tested with pupnp 1.14.7 with --disable-reuseaddr and without. Also tested with pupnp 1.12.1.
It caused by a while loop inside RunMiniServer. select() does not return an error but returns right away without blocking.

[tguillem]

Hello, it should be fixed by #428 (I didn't know there was an issue)