On Thu, 2018-08-02 at 20:13 +0200, Heiner Kallweit wrote: > I wondered why my system gets warmer than usual and eventually found > that > every write to /sys/module/pcie_aspm/parameters/policy returns EINVAL. Thanks for spotting this! My bad. > > Reason is mentioned change, using match_string() is wrong when used > with > sysfs because sysfs strings have a trailing newline. Not always, but you are right, it has to be \n-aware. > Seems like the patch > wasn't tested. > __sysfs_match_string() has to be used instead. I checked and this > fixes > the issue. Yes, it should be sysfs variant. -- Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> Intel Finland Oy
