Messages in this thread | | | Date | Fri, 25 Jan 2013 09:17:48 -0800 | From | Tejun Heo <> | Subject | Re: [PATCH 1/1] async: initialise list heads to fix crash |
| |
On Fri, Jan 25, 2013 at 10:13:59AM +0000, James Hogan wrote: > The commit "async: replace list of active domains with global list of > pending items" added a struct list_head global_list in struct > async_entry, which isn't initialised. This means that if > !domain->registered at __async_schedule(), then list_del_init() will be > called on the list head in async_run_entry_fn with both pointers NULL, > causing a crash. This is fixed by initialising both the global_list and > domain_list list_heads after kzalloc'ing the entry. > > This was noticed due to dapm_power_widgets() which uses > ASYNC_DOMAIN_EXCLUSIVE, which initialises the domain->registered to 0. > > Signed-off-by: James Hogan <james.hogan@imgtec.com>
Applied wq/for-3.9-async.
Thanks!
-- tejun
| |