tree f6a378c0f087111813900c4435f00a489c492a9f
parent bf8b1dadc117e93c1c9e328c8f3f7236483ee7f7
author Yong Li <yong.b.li@linux.intel.com> 1586431453 +0800
committer Yong Li <yong.b.li@intel.com> 1587431635 +0000

adcsensor: Check if the object is still valid in the async callback functions

Sometimes adcsesnor will crash during stress test.
The root cause is that the object is destroyed already
when the async callback functions are executed.
The timer cancel() cannot cancel the expired callback
handlers. When the callback handler is executed,
the object is deleted already.
The buffer used by async_read_util must guarantee
that they remain valid until the handler is called.

Using the weak_ptr to check if the object is still valid
before using these member variables/functions.

Tested:
No adcsensor crash with stress test,
Change the threshold of ADC sensor 0x51 P3VBAT:
while [ true ]; do; \
ipmitool raw 4 0x26 0x51 0x1b 0x20 0x1b 0 0x2a 0x2a 0;\
 ipmitool raw 4 0x26 0x51 0x1b 0x20 0x1b 0 0x2a 0x2b 0;\
 sleep 1; done

Signed-off-by: Yong Li <yong.b.li@linux.intel.com>
Change-Id: I2ddb934199e2d0f52a1a6df25c46eadfe06cfda3