On 5/25/22 18:03, Christoph Hellwig wrote: > On Tue, May 24, 2022 at 12:05:07PM -0500, Mario Limonciello wrote: >> The LPM policy is applied to more chipsets than just mobile designs. >> Update the module parameter to drop the word "mobile" to make this >> clearer to users. >> >> Also, document this renamed parameter in the admin-guide for users >> as it was missing previously. > > Even if the name is confusing I don't think we should rename it as > that breaks existing setups. I think just updating the documentation > is good enough, but if you feel strongly we can just add the new name > while keeping the old one as an alias. +1 for the alias. The other thing that this series need is to add the changes of the LPM policy initialization according to the adapter capabilities (your patch and the patches from Runa) as well as accounting for the eventual nolpm link horkage flag. Otherwise, I fear that this change will generate many regressions. -- Damien Le Moal Western Digital Research
