** Description changed: [SRU][XENIAL][YAKKETY][ZESTY] Upstream thermald includes a minor fixes that ensure settings are initialized correctly. The fix in question is: commit 37a25879239466f2d096fd1a58b2ac7bbc3958f5 Author: Srinivas Pandruvada <srinivas.pandruv...@linux.intel.com> Date: Mon Mar 27 17:39:30 2017 -0700 set default preference In case there is missing element for preference, set ENERGY_CONSERVE as default. This fixes are not critical, but I think are proven stable enough for a - SRU and fix some minor buglets that should be plugged. + SRU and it plugs a *potential* issue that we've not yet seen to occur. + + [REGRESSION POTENTIAL] + Minimal, this ensures a default setting is correctly set to the default value when config is provided without a default preference. Since this is not the normal modus operandi of thermald since it uses zero config mode, this fix plugs a corner case that not used by default. Users of the xml config will be entering this default appropriately, so this catches the corner cases where it is not defined in a user config. + + [TEST] + This is not testable in the default zero config case. In cases where the config does not have this default setting, it is hard to test as this is sets internal state that is not easily checked during run time. As this bug has not been observed in the field, it is hard to find a reproducer to test this corner case.
-- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1683825 Title: thermald: pull in some minor fixes from upstream To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/thermald/+bug/1683825/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
