[net-next,v2] [PATCH net-next v2] net: wwan: t7xx: Change PM_AUTOSUSPEND_MS to 5000
Sergey Ryazanov
ryazanov.s.a at gmail.com
Mon Nov 18 17:50:44 PST 2024
On 19.11.2024 03:44, Jakub Kicinski wrote:
> On Tue, 19 Nov 2024 03:01:47 +0200 Sergey Ryazanov wrote:
>>> He's decreasing the sleep timer from 20 to 5 sec, both of which
>>> are very high for networking, anyway. You appear to be questioning
>>> autosuspend itself but it seems to have been added 2 years ago already.
>>>
>>> What am I missing?
>>
>> Some possible funny side-effect of sleeping with this chipset. Like
>> loosing network connection and dropping TCP sessions. I hope that 20
>> seconds was putted on purpose.
>>
>> Suddenly, I don't have this modem at hand and want to be sure that we
>> are not going to receive a stream of bug reports.
>
> Power saving is always tricky, but they say they tested. I think we
> should give it a go, worst case - it will be an easy revert.
Make sense. Then let's merge it.
Reviewed-by: Sergey Ryazanov <ryazanov.s.a at gmail.com>
More information about the Linux-mediatek
mailing list