Tony Chuang <yhchuang@xxxxxxxxxxx> writes: >> On Thu, Oct 31, 2019 at 1:17 AM Tony Chuang <yhchuang@xxxxxxxxxxx> >> wrote: >> > Or I should just change the >> > value to a better one. By our experience, set this to 50 is a more reasonable >> > value, such that some web surfing or background traffic wouldn't make the >> > driver to leave PS mode. >> >> FWIW, I think choosing a more reasonable default is definitely a good >> start, as long as this choice doesn't have huge downsides. >> >> @Kalle: FYI, this (set to 50) is exactly the change that Tony is >> recommending to me for my distro, and I have the same qualms about >> supporting a growing number of module parameter tweaks like this. So, >> thanks for pushing back :) >> >> Brian >> > > I was afraid of you thinking that setting this to 50 is a strange thing. > But it seems like you'd prefer to change the default value instead of adding a > module parameter to control it. I think we can drop this one and I will send > a patch to change the default value to 50. Yeah, as the first step changing to 50 sounds good to me. Later, if needed, we can extend it to make it configurable from user space, either via the QoS framework or something else. -- Kalle Valo
