From: Wengang Wang <wen.gang.wang@oracle.com>
To: cluster-devel.redhat.com
Subject: [Cluster-devel] [PATCH] gfs2: free disk inode which is deleted by remote node
Date: Tue, 18 Aug 2009 14:32:22 +0800 [thread overview]
Message-ID: <4A8A4AF6.9060000@oracle.com> (raw)
In-Reply-To: <1250501379.3377.10.camel@localhost.localdomain>
Hi Steve,
Steven Whitehouse wrote:
> Hi,
>
> Don't worry about the wrong list, I did spot the patch anyway. I agree
> with the principle of what you are trying to do, but something isn't
> quite right. When I tried this on Friday I got a NULL pointer deref the
> first time I tried to touch a file on the fs.
>
> I think it needs a bit more work, but its certainly going in the right
> direction,
encouraged :)
and yes the dentry->d_inode is set to NULL in path
vfs_unlink()->d_delete()->dentry_iput(). I didn't test it out since I
wasn't running on the patched code on the other node, sorry for that.
I'm going to send the v2 version of the patch soon.
if you are going to commit the patch, please help(and feel free) to
correct mistakes, it there are, in my patch log --my English is not good.
regards,
wengang.
prev parent reply other threads:[~2009-08-18 6:32 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-08-14 15:05 [Cluster-devel] [PATCH] gfs2: free disk inode which is deleted by remote node Wengang Wang
2009-08-17 9:29 ` Steven Whitehouse
2009-08-18 6:32 ` Wengang Wang [this message]
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=4A8A4AF6.9060000@oracle.com \
--to=wen.gang.wang@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).