From mboxrd@z Thu Jan 1 00:00:00 1970 From: Joseph Qi Date: Tue, 6 Jan 2015 09:07:36 +0800 Subject: [Ocfs2-devel] [PATCH] ocfs2: fix the wrong directory passed to ocfs2_lookup_ino_from_name() when link file In-Reply-To: <20150105141632.a14da73fc82f5c2439e51001@linux-foundation.org> References: <5493F8F1.3020602@huawei.com> <20141219141513.d68d5737c625c1b188004c89@linux-foundation.org> <5497DBBE.7060700@ubit.hu> <20150105141632.a14da73fc82f5c2439e51001@linux-foundation.org> Message-ID: <54AB3558.3040508@huawei.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: ocfs2-devel@oss.oracle.com On 2015/1/6 6:16, Andrew Morton wrote: > On Mon, 22 Dec 2014 09:52:14 +0100 Aron Szabo wrote: > >> 12/19/2014 11:15 PM keltez__ssel, Andrew Morton __rta: >>> On Fri, 19 Dec 2014 18:07:45 +0800 Xue jiufei wrote: >>> >>>> In function ocfs2_link(), parent directory inode passed to function >>>> ocfs2_lookup_ino_from_name() is wrong. Parameter dir is the parent >>>> of new_dentry not old_dentry. We should get old_dir from old_dentry >>>> and lookup old_dentry in old_dir in case another node remove the old dentry. >>> What are the user-visible effects of this change? >> >> Hi Andew! >> >> Hard linking works again, when paths are relative with at least one >> subdirectory. This is how the problem was reproducable: >> >> # mkdir a >> # mkdir b >> # touch a/test >> # ln a/test b/test >> ln: failed to create hard link `b/test' => `a/test': No such file or >> directory >> >> However when creating links in the same dir, it worked well. >> >> Now the link gets created. >> >> Thanks for the quick fix Xue! > > (top-posting untangled) > > When you say "works again", you mean that we broke it? This patch > fixes a regression? If so, do we know what caused that regression? Or > at least when it occurred? Yes? it fixes commit 0e048316ff57 (ocfs2: check existence of old dentry in ocfs2_link()). > > > > _______________________________________________ > Ocfs2-devel mailing list > Ocfs2-devel at oss.oracle.com > https://oss.oracle.com/mailman/listinfo/ocfs2-devel > >