[Letux-kernel] suspend issues letux-4.15-rc8

[H. Nikolaus Schaller]

Hi,

> Am 28.01.2018 um 11:20 schrieb Andreas Kemnade <andreas at kemnade.info>:
> 
> Hi,
> 
> I have run my suspend test script, and it shows trouble:
> 
> no modules: cur: 60868 delta: 60868
> +w2sg0004 cur: 60868 delta: 0
> 
> -> 60mA idle -> ok
> 
> before: 830192
> after: 827878
> average 27860 uA over 299 seconds
> 
> -> 27mA suspend -> nearly ok,

Much better!

> 
> Module                  Size  Used by
> phy_twl4030_usb        20480  0 
> musb_hdrc             106496  1 phy_twl4030_usb
> twl4030_charger        20480  0 
> industrialio           53248  1 twl4030_charger
> w2sg0004               16384  0 
> twl4030_pwrbutton      16384  0 
> bq27xxx_battery_hdq    16384  0 
> bq27xxx_battery        20480  1 bq27xxx_battery_hdq
> omap_hdq               16384  0 
> +charger,phy cur: 60868 delta: 0
> before: 827700
> after: 825564
> average 25717 uA over 299 seconds
> 
> -> also ok

Fine!

> 
> Module                  Size  Used by
> usb_f_ecm              16384  1 
> g_ether                16384  0 
> usb_f_rndis            20480  2 g_ether
> u_ether                20480  3 usb_f_ecm,g_ether,usb_f_rndis
> libcomposite           36864  3 usb_f_ecm,g_ether,usb_f_rndis
> configfs               32768  4 usb_f_ecm,usb_f_rndis,libcomposite
> omap2430               16384  0 
> phy_twl4030_usb        20480  2 
> musb_hdrc             106496  2 omap2430,phy_twl4030_usb
> twl4030_charger        20480  0 
> industrialio           53248  1 twl4030_charger
> w2sg0004               16384  0 
> twl4030_pwrbutton      16384  0 
> bq27xxx_battery_hdq    16384  0 
> bq27xxx_battery        20480  1 bq27xxx_battery_hdq
> omap_hdq               16384  0 
> +musb,g_ether cur: 90142 delta: 29274
> 
> -> too high (pm_runtime_suspend of phy/musb is not doing its job/reference counter
>   unbalanced, ...)
>   was also better at some time with 4.x

IMHO, we have the phy/musb suspend issue for a very long time and
I am not sure if any kernel beyond 3.7 was ever better with this...

> 
> before: 825208
> after: 820758
> average 53578 uA over 299 seconds
> 
> -> too high (pm_runtime_suspend of phy/musb is not doing its job...)
>   was also better at some time with 4.x)
> 
> +ifup cur: 90321 delta: 179
> 
> -> also  too high.

Indeed.

> 
> Conclusion: run this script more often
> to find issues earlier. I hope that can be automated. I almost have all
> ingredients for automating that.

Oh, yes, that would be really helpful. Without such automation it is very
difficult to get reliable measurements which can be compared.

BR and thanks,
Nikolaus