LU-2737 osc: double unlock in osc_lru_shrink()

It could unlock twice in osc_lru_shrink() if the cl_io_init() failed.

Signed-off-by: Niu Yawei <yawei.niu@intel.com>
Change-Id: Idf3ebafced7f7d3a68c13ea20d12cf221e9d74ce
Reviewed-on: http://review.whamcloud.com/5258
Reviewed-by: Bobi Jam <bobijam@gmail.com>
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Jinshan Xiong <jinshan.xiong@intel.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>

diff --git a/lustre/osc/osc_page.c b/lustre/osc/osc_page.c
index 75e6108..5c6102a 100644 (file)
--- a/lustre/osc/osc_page.c
+++ b/lustre/osc/osc_page.c
@@ -718,11 +718,13 @@ int osc_lru_shrink(struct client_obd *cli, int target)
                        clobj = tmp;
                        io->ci_obj = clobj;
                        rc = cl_io_init(env, io, CIT_MISC, clobj);
                        clobj = tmp;
                        io->ci_obj = clobj;
                        rc = cl_io_init(env, io, CIT_MISC, clobj);

+
+                       client_obd_list_lock(&cli->cl_lru_list_lock);
+

                        if (rc != 0)
                                break;
 
                        ++maxscan;
                        if (rc != 0)
                                break;
 
                        ++maxscan;

-                       client_obd_list_lock(&cli->cl_lru_list_lock);

                        continue;
                }
 
                        continue;
                }