Re: [PATCH V3] firmware loader: Fix the race FW_STATUS_DONE is followed by class_timeout

2012-11-08 Thread Ming Lei
On Thu, Nov 8, 2012 at 7:14 PM, Chuansheng Liu wrote: > > There is a race as below when calling request_firmware(): > CPU1 CPU2 > write 0 > loading > mutex_lock(_lock) > ... > set_bit FW_STATUS_DONE class_timeout is coming >

Re: [PATCH V3] firmware loader: Fix the race FW_STATUS_DONE is followed by class_timeout

2012-11-08 Thread Ming Lei
On Thu, Nov 8, 2012 at 7:14 PM, Chuansheng Liu chuansheng@intel.com wrote: There is a race as below when calling request_firmware(): CPU1 CPU2 write 0 loading mutex_lock(fw_lock) ... set_bit FW_STATUS_DONE class_timeout is coming

[PATCH V3] firmware loader: Fix the race FW_STATUS_DONE is followed by class_timeout

2012-11-07 Thread Chuansheng Liu
There is a race as below when calling request_firmware(): CPU1 CPU2 write 0 > loading mutex_lock(_lock) ... set_bit FW_STATUS_DONE class_timeout is coming set_bit FW_STATUS_ABORT complete_all ...

[PATCH V3] firmware loader: Fix the race FW_STATUS_DONE is followed by class_timeout

2012-11-07 Thread Chuansheng Liu
There is a race as below when calling request_firmware(): CPU1 CPU2 write 0 loading mutex_lock(fw_lock) ... set_bit FW_STATUS_DONE class_timeout is coming set_bit FW_STATUS_ABORT complete_all completion ...