diff options
author | Wei Yongjun <yongjun_wei@trendmicro.com.cn> | 2013-02-26 16:39:14 +0800 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2013-02-26 10:05:35 -0300 |
commit | 5dae92a718570e6a942e0b882e53d25cab03b40f (patch) | |
tree | 45b3f34949a88cc223cc71bd7c69d774edcdfcee | |
parent | 8ae8f50ad8979bb670598ff92eebea611b799f10 (diff) | |
download | op-kernel-dev-5dae92a718570e6a942e0b882e53d25cab03b40f.zip op-kernel-dev-5dae92a718570e6a942e0b882e53d25cab03b40f.tar.gz |
ghes_edac: fix to use list_for_each_entry_safe() when delete list items
Since we will remove items off the list using list_del() we need
to use a safe version of the list_for_each_entry() macro aptly named
list_for_each_entry_safe().
Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
-rw-r--r-- | drivers/edac/ghes_edac.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/edac/ghes_edac.c b/drivers/edac/ghes_edac.c index 636dcf1..bb53467 100644 --- a/drivers/edac/ghes_edac.c +++ b/drivers/edac/ghes_edac.c @@ -523,9 +523,9 @@ EXPORT_SYMBOL_GPL(ghes_edac_register); void ghes_edac_unregister(struct ghes *ghes) { struct mem_ctl_info *mci; - struct ghes_edac_pvt *pvt; + struct ghes_edac_pvt *pvt, *tmp; - list_for_each_entry(pvt, &ghes_reglist, list) { + list_for_each_entry_safe(pvt, tmp, &ghes_reglist, list) { if (ghes == pvt->ghes) { mci = pvt->mci; edac_mc_del_mc(mci->pdev); |