From: Coly Li <coly.li@suse.de>
To: ocfs2-devel@oss.oracle.com
Subject: [Ocfs2-devel] [PATCH] ocfs2_cluster_lock: code cleanup for redundant assignment
Date: Fri, 08 May 2009 14:03:21 +0800 [thread overview]
Message-ID: <4A03CB29.1000807@suse.de> (raw)
In-Reply-To: <4A032D7D.5040600@oracle.com>
Sunil Mushran Wrote:
> While this patch may be correct, we have to take into consideration
> that we have to backport patches to 1.4, etc. In that light, is this
> patch worth it?
>
When I check dlm related code in ocfs2, this line makes me uncomfortable, after
sending out the patch, I feel better :))
I understand the comments from you and Jeol, and partially agree with you all :)
Thanks for the response.
--
Coly Li
SuSE Labs
next prev parent reply other threads:[~2009-05-08 6:03 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-05-07 18:30 [Ocfs2-devel] [PATCH] ocfs2_cluster_lock: code cleanup for redundant assignment Coly Li
2009-05-07 18:50 ` Sunil Mushran
2009-05-08 6:03 ` Coly Li [this message]
2009-05-07 19:08 ` Joel Becker
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4A03CB29.1000807@suse.de \
--to=coly.li@suse.de \
--cc=ocfs2-devel@oss.oracle.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.