Re: [Cluster-devel] [PATCH] dlm: Reset fs_notified when check_fs_done

2011-02-22 Thread David Teigland
On Tue, Feb 22, 2011 at 04:35:42PM +0800, Jiaju Zhang wrote:
 On Tue, Nov 9, 2010 at 6:06 AM, David Teigland teigl...@redhat.com wrote:
  On Mon, Nov 08, 2010 at 11:05:49PM +0800, Jiaju Zhang wrote:
  Luckily, things have changed now. One user met this issue two months
  ago and he's also very kindly to test the patch. The result is the
  patch really works.
 
  Attached is the log before they apply the patch. This time the log
  has already included the debugging messages which were added by the
  commit 27b09badd40a2d1500500fa6945aeb532f75bd13 , so we can see what
  really happens on the user's site.
  (The log is a bit large when it was uncompressed, this is because the
  spinning would print many messages to the log.)
 
  I rebased the patch against current upstream code now. Thank you for
  your review in advance :)
 
  Thanks, the patch looks good, I'll push this out.
 
 Hi David, I haven't found this patch from
 git://git.fedorahosted.org/dlm.git, was it being missed?

Sorry, it's in the cluster.git STABLE31 branch, but I forgot dlm.git, I've
just pushed it there too.
Dave



Re: [Cluster-devel] [PATCH] dlm: Reset fs_notified when check_fs_done

2010-11-08 Thread David Teigland
On Mon, Nov 08, 2010 at 11:05:49PM +0800, Jiaju Zhang wrote:
 Luckily, things have changed now. One user met this issue two months
 ago and he's also very kindly to test the patch. The result is the
 patch really works.
 
 Attached is the log before they apply the patch. This time the log
 has already included the debugging messages which were added by the
 commit 27b09badd40a2d1500500fa6945aeb532f75bd13 , so we can see what
 really happens on the user's site.
 (The log is a bit large when it was uncompressed, this is because the
 spinning would print many messages to the log.)
 
 I rebased the patch against current upstream code now. Thank you for
 your review in advance :)

Thanks, the patch looks good, I'll push this out.

Dave