The issue here is a bug in the bq27541 fuel gauge driver. The get_property() function is not thread-safe, but reentry is possible in several different scenarios.
The attached patch fixes this issue by protecting get_property() via a mutex. Below is the patch commit message, for clarity: * Currently, get_property() and its callees are not thread-safe, since they share some global state, which they access without serialization or guaranteed atomicity. * get_property() can be called in a reentrant fashion from either several concurrent sysfs accesses, or a sysfs access concurrently with an access from power_supply_core.c, which in turn are triggered from a work queue in the driver itself. * This fixes bogus readings affecting the capacity and the charge_now values - possibly others as well. ** Patch added: "0001-power-bq27541-driver-serialize-get_property.patch" https://bugs.launchpad.net/ubuntu-nexus7/+bug/1093543/+attachment/3505724/+files/0001-power-bq27541-driver-serialize-get_property.patch -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1093543 Title: Battery status behaves erratically To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu-nexus7/+bug/1093543/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
