Guozhonghua, idea of the fix looks good, but the patch doesn't seem to
meet mainline standards. Can you please follow the instructions that
Jeff emailed you last time and resubmit the patch.
On 03/11/2014 04:09 AM, Guozhonghua wrote:
I review the code of the kernel 3.11.10.
The difference is as below:
-- ../linux-3.11.10/fs/ocfs2/dlm/dlmdomain.c 2013-11-30
02:42:37.0 +0800
+++ ocfs2-ko-3.11/dlm/dlmdomain.c 2014-03-11 18:57:23.323897515 +0800
@@ -1135,6 +1135,9 @@ static int dlm_query_region_handler(stru
int status = 0;
int locked = 0;
+/* Wether domain locked */
comment probably not required
+int domain_locked = 0;
+
qr = (struct dlm_query_region *) msg-buf;
mlog(0, Node %u queries hb regions on domain %s\n, qr-qr_node,
@@ -1150,6 +1153,7 @@ static int dlm_query_region_handler(stru
status = -EINVAL;
spin_lock(dlm_domain_lock);
+domain_locked = 1;
dlm = __dlm_lookup_domain_full(qr-qr_domain, qr-qr_namelen);
if (!dlm) {
mlog(ML_ERROR, Node %d queried hb regions on domain %s
@@ -1181,9 +1185,12 @@ static int dlm_query_region_handler(stru
bail:
if (locked)
spin_unlock(dlm-spinlock);
- spin_unlock(dlm_domain_lock);
- kfree(local);
+if (domain_locked)
+ spin_unlock(dlm_domain_lock);
+
+if (local)
+kfree(local);
no need to check if (local), kfree will take care of that part
return status;
}
-
??,?
?(??
???)?,??
??!
This e-mail and its attachments contain confidential information from
H3C, which is
intended only for the person or entity whose address is listed above.
Any use of the
information contained herein in any way (including, but not limited
to, total or partial
disclosure, reproduction, or dissemination) by persons other than the
intended
recipient(s) is prohibited. If you receive this e-mail in error,
please notify the sender
by phone or email immediately and delete it!
___
Ocfs2-devel mailing list
Ocfs2-devel@oss.oracle.com
https://oss.oracle.com/mailman/listinfo/ocfs2-devel
___
Ocfs2-devel mailing list
Ocfs2-devel@oss.oracle.com
https://oss.oracle.com/mailman/listinfo/ocfs2-devel
