Re: [Cluster-devel] dlm: master - dlm_controld: fail to start with negative nodeid

2012-02-16 Thread Fabio M. Di Nitto
On 2/15/2012 8:35 PM, David Teigland wrote: + if (our_nodeid 0) { + log_error(negative nodeid, set corosync totem.clear_node_high_bit); + corosync_cfg_finalize(ch); + return -1; + } Might be worth pointing people to corosync.conf.5 and

[Cluster-devel] [PATCH 00/11] Push file_update_time() into .page_mkwrite

2012-02-16 Thread Jan Kara
Hello, to provide reliable support for filesystem freezing, filesystems need to have complete control over when metadata is changed. In particular, file_update_time() calls from page fault code make it impossible for filesystems to prevent inodes from being dirtied while the filesystem is

[Cluster-devel] [PATCH 08/11] gfs2: Push file_update_time() into gfs2_page_mkwrite()

2012-02-16 Thread Jan Kara
CC: Steven Whitehouse swhit...@redhat.com CC: cluster-devel@redhat.com Signed-off-by: Jan Kara j...@suse.cz --- fs/gfs2/file.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/fs/gfs2/file.c b/fs/gfs2/file.c index c5fb359..1f03531 100644 --- a/fs/gfs2/file.c +++

[Cluster-devel] [GFS2 Patch] GFS2: Read in rindex if necessary during unlink

2012-02-16 Thread Bob Peterson
Hi, This patch fixes a problem whereby you were unable to delete files until other file system operations were done (such as statfs, touch, writes, etc.) that caused the rindex to be read in. Regards, Bob Peterson Red Hat File Systems Signed-off-by: Bob Peterson rpete...@redhat.com -- GFS2:

Re: [Cluster-devel] [PATCH 08/11] gfs2: Push file_update_time() into gfs2_page_mkwrite()

2012-02-16 Thread Steven Whitehouse
Hi, On Thu, 2012-02-16 at 14:46 +0100, Jan Kara wrote: CC: Steven Whitehouse swhit...@redhat.com CC: cluster-devel@redhat.com Signed-off-by: Jan Kara j...@suse.cz --- fs/gfs2/file.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) That looks ok to me... Acked-by: Steven

Re: [Cluster-devel] [PATCH] fs/dlm/rcom.c: included member.h twice

2012-02-16 Thread David Teigland
On Thu, Feb 16, 2012 at 02:55:21PM +0100, Danny Kukawka wrote: fs/dlm/rcom.c included 'member.h' twice, remove the duplicate. I'll fold this into the current patch I'm working on. Signed-off-by: Danny Kukawka danny.kuka...@bisect.de --- fs/dlm/rcom.c |1 - 1 files changed, 0

Re: [Cluster-devel] [PATCH] qdiskd: Make multipath issues go away

2012-02-16 Thread Fabio M. Di Nitto
The patch looks good and ACK, with one minor nitpick. that 178 port value in cman_recv/send_data is rather cryptic. I would prefer to see it defined as others in cman/cnxman-socket.h for documentation purposes (so we know it's QDISK). Fabio On 02/16/2012 12:53 AM, Lon Hohberger wrote: Qdiskd