[Ocfs2-devel] Can you check the patch for bug 48 and commit?

Mark Fasheh mark.fasheh at oracle.com
Mon Mar 29 19:10:50 CST 2004


On Tue, Mar 30, 2004 at 10:53:13AM +0800, Zhang, Sonic wrote:
> Hi Mark,
> 
> Can you check the patch for bug 48 in the attachment and commit? I haven't
> found it in the svn tree.
That's because I was still looking at the signals part of the patch :)
I needed to do some reading in schedule() to make sure what I though was
going on, really is. Also, I think the correct approach is to do both, the
signal handling, *and* to define the timeout value itself as a function of
HZ, rather than a hard number of jiffies.

> This patch adds lines of signal process code, which makes the thread
> ocfs_volume_thread() really go to sleep after receive the SIGTERM signal.
> So, it gives routine ocfs_journal_set_unmounted() a chance to run and
> check the lock status before thread ocfs_volume_thread() wakes up.
You need to update from cvs because half of that patch is already in. Also,
can you make the same NPTL fix in it that I did for ocfs_commit_thread? See
the svn logs to know what I'm talking about.
	--Mark

--
Mark Fasheh
Software Developer, Oracle Corp
mark.fasheh at oracle.com


More information about the Ocfs2-devel mailing list