> > 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. Yan-Hsuan
