Re: [GFS2] Shrink gfs2_inode (8) - i_vn [28/70]

2006-12-04 Thread Steven Whitehouse
Hi, On Fri, 2006-12-01 at 12:25 -0600, Russell Cattelan wrote: > On Thu, 2006-11-30 at 12:16 +0000, Steven Whitehouse wrote: > > >From bfded27ba010d1c3b0aa3843f97dc9b80de751be Mon Sep 17 00:00:00 2001 > > From: Steven Whitehouse <[EMAIL PROTECTED]> > > Date:

Re: [GFS2] Shrink gfs2_inode (6) - di_atime/di_mtime/di_ctime [26/70]

2006-12-04 Thread Steven Whitehouse
Hi, On Fri, 2006-12-01 at 12:15 -0600, Russell Cattelan wrote: > On Thu, 2006-11-30 at 12:16 +0000, Steven Whitehouse wrote: > > >From 1a7b1eed5802502fd649e04784becd58557fdcf1 Mon Sep 17 00:00:00 2001 > > From: Steven Whitehouse <[EMAIL PROTECTED]> > > Date:

Re: [GFS2] Shrink gfs2_inode (5) - di_nlink [25/70]

2006-12-04 Thread Steven Whitehouse
Hi, On Fri, 2006-12-01 at 12:14 -0600, Russell Cattelan wrote: > On Thu, 2006-11-30 at 12:16 +0000, Steven Whitehouse wrote: > > >From 4f56110a00af5fb2e22fbccfcaf944d62cae8fcf Mon Sep 17 00:00:00 2001 > > From: Steven Whitehouse <[EMAIL PROTECTED]> > > Date:

Re: [Cluster-devel] Re: [GFS2] Fix incorrect fs sync behaviour [2/5]

2006-12-04 Thread Steven Whitehouse
Hi, On Fri, 2006-12-01 at 11:57 -0600, Russell Cattelan wrote: > On Mon, 2006-11-06 at 11:07 +0000, Steven Whitehouse wrote: > > >From 4a221953ed121692aa25998451a57c7f4be8b4f6 Mon Sep 17 00:00:00 2001 > > From: Steven Whitehouse <[EMAIL PROTECTED]> > > Date:

Re: [GFS2] Don't flush everything on fdatasync [70/70]

2006-12-01 Thread Steven Whitehouse
Hi, On Thu, 2006-11-30 at 23:01 -0800, Andrew Morton wrote: > On Thu, 30 Nov 2006 12:24:08 + > Steven Whitehouse <[EMAIL PROTECTED]> wrote: > > > static int gfs2_fsync(struct file *file, struct dentry *dentry, int > > datasync) > > { > > - struct

[GFS2] Move gfs2_dinode_in to inode.c [19/70]

2006-11-30 Thread Steven Whitehouse
>From ea744d01c6a5acf1f6171b4c6e1658a742063613 Mon Sep 17 00:00:00 2001 From: Steven Whitehouse <[EMAIL PROTECTED]> Date: Tue, 31 Oct 2006 15:28:00 -0500 Subject: [PATCH] [GFS2] Move gfs2_dinode_in to inode.c gfs2_dinode_in() is only ever called from one place, so move it to that

[GFS2] Change argument of gfs2_dinode_out [17/70]

2006-11-30 Thread Steven Whitehouse
>From 539e5d6b7ae8612c0393fe940d2da5b591318d3d Mon Sep 17 00:00:00 2001 From: Steven Whitehouse <[EMAIL PROTECTED]> Date: Tue, 31 Oct 2006 15:07:05 -0500 Subject: [PATCH] [GFS2] Change argument of gfs2_dinode_out Everywhere this was called, a struct gfs2_inode was available, but despite

Re: [GFS2] split gfs2_dinode into on-disk and host variants [1/70]

2006-11-30 Thread Steven Whitehouse
Hi, On Thu, 2006-11-30 at 12:25 +, Christoph Hellwig wrote: > > + struct gfs2_dinode_host *di = &ip->i_di; > > Please call this things just gfs2_inode. gfs_d(isk)_inode_host doesn't > make any sense. > Its not the final intention to keep this dinode_host structure (or indeed the other _ho

[GFS2] split and annotate gfs2_quota_change [14/70]

2006-11-30 Thread Steven Whitehouse
>From b62f963e1fdf838fed91faec21228d421a834f2d Mon Sep 17 00:00:00 2001 From: Al Viro <[EMAIL PROTECTED]> Date: Fri, 13 Oct 2006 23:46:46 -0400 Subject: [PATCH] [GFS2] split and annotate gfs2_quota_change Signed-off-by: Al Viro <[EMAIL PROTECTED]> Signed-off-by: Steven Whi

[GFS2] Fix page lock/glock deadlock [32/70]

2006-11-30 Thread Steven Whitehouse
>From 2ca99501fa5422e84f18333918a503433449e2b5 Mon Sep 17 00:00:00 2001 From: Steven Whitehouse <[EMAIL PROTECTED]> Date: Wed, 8 Nov 2006 10:26:54 -0500 Subject: [PATCH] [GFS2] Fix page lock/glock deadlock This fixes a race between the glock and the page lock encountered during tr

[GFS2] Tidy up 0 initialisations in inode.c [29/70]

2006-11-30 Thread Steven Whitehouse
>From 294caaa3b8304c0a14c5039691caf23363bd9369 Mon Sep 17 00:00:00 2001 From: Steven Whitehouse <[EMAIL PROTECTED]> Date: Thu, 2 Nov 2006 11:59:28 -0500 Subject: [PATCH] [GFS2] Tidy up 0 initialisations in inode.c We don't need to use endian conversions for 0 initialisations when

[GFS2] Change argument to gfs2_dinode_in [18/70]

2006-11-30 Thread Steven Whitehouse
>From 891ea14712da68e282de8583e5fa14f0d3f3731e Mon Sep 17 00:00:00 2001 From: Steven Whitehouse <[EMAIL PROTECTED]> Date: Tue, 31 Oct 2006 15:22:10 -0500 Subject: [PATCH] [GFS2] Change argument to gfs2_dinode_in This is a preliminary patch to enable the removal of fields in gfs2_dinode_h

[GFS2] Remove gfs2_inode_attr_in [35/70]

2006-11-30 Thread Steven Whitehouse
>From 9e2dbdac3df300516ffdd9a8631f23164d068a50 Mon Sep 17 00:00:00 2001 From: Steven Whitehouse <[EMAIL PROTECTED]> Date: Wed, 8 Nov 2006 15:45:46 -0500 Subject: [PATCH] [GFS2] Remove gfs2_inode_attr_in This function wasn't really doing the right thing. There was no need to update

[GFS2] Shrink gfs2_inode (4) - di_uid/di_gid [24/70]

2006-11-30 Thread Steven Whitehouse
>From 2933f9254a6af33db25270778c998a42029da668 Mon Sep 17 00:00:00 2001 From: Steven Whitehouse <[EMAIL PROTECTED]> Date: Wed, 1 Nov 2006 13:23:29 -0500 Subject: [PATCH] [GFS2] Shrink gfs2_inode (4) - di_uid/di_gid Remove duplicate di_uid/di_gid fields in favour of using inode->i_uid/

[GFS2] split and annotate gfs2_meta_header [9/70]

2006-11-30 Thread Steven Whitehouse
>From e928a76f959e89884f6186bb6f846c533847d5df Mon Sep 17 00:00:00 2001 From: Al Viro <[EMAIL PROTECTED]> Date: Fri, 13 Oct 2006 21:57:23 -0400 Subject: [PATCH] [GFS2] split and annotate gfs2_meta_header Signed-off-by: Al Viro <[EMAIL PROTECTED]> Signed-off-by: Steven Whitehouse &l

[GFS2] Change argument to gfs2_dinode_print [20/70]

2006-11-30 Thread Steven Whitehouse
>From 4cc14f0b88bf3e0b508143e091eb5a8dff3e3b9c Mon Sep 17 00:00:00 2001 From: Steven Whitehouse <[EMAIL PROTECTED]> Date: Tue, 31 Oct 2006 19:00:24 -0500 Subject: [PATCH] [GFS2] Change argument to gfs2_dinode_print Change argument for gfs2_dinode_print in order to prepare for removal of

[GFS2] split and annotate gfs_rindex [10/70]

2006-11-30 Thread Steven Whitehouse
>From 1e81c4c3e0f55c95b6278a827262b80debd0dc7e Mon Sep 17 00:00:00 2001 From: Al Viro <[EMAIL PROTECTED]> Date: Fri, 13 Oct 2006 22:51:24 -0400 Subject: [PATCH] [GFS2] split and annotate gfs_rindex Signed-off-by: Al Viro <[EMAIL PROTECTED]> Signed-off-by: Steven Whitehouse &l

[GFS2] Shrink gfs2_inode (7) - di_payload_format [27/70]

2006-11-30 Thread Steven Whitehouse
>From a9583c7983cbba9726bfe64ee46613d654fc9e26 Mon Sep 17 00:00:00 2001 From: Steven Whitehouse <[EMAIL PROTECTED]> Date: Wed, 1 Nov 2006 20:09:14 -0500 Subject: [PATCH] [GFS2] Shrink gfs2_inode (7) - di_payload_format This is almost never used. Its there for backward compatibility with

[DLM] fix requestqueue race [41/70]

2006-11-30 Thread Steven Whitehouse
nabled locking after taking the mutex for the requestqueue and if it has processing the message instead of queueing it. Signed-off-by: David Teigland <[EMAIL PROTECTED]> Signed-off-by: Steven Whitehouse <[EMAIL PROTECTED]> --- fs/dlm/lock.c | 15 +++ fs

[GFS2] Inode number is constant [34/70]

2006-11-30 Thread Steven Whitehouse
>From e7c698d74fc9e0e76b3086062b0519df3601ff52 Mon Sep 17 00:00:00 2001 From: Steven Whitehouse <[EMAIL PROTECTED]> Date: Wed, 8 Nov 2006 13:52:05 -0500 Subject: [PATCH] [GFS2] Inode number is constant Since the inode number is constant, we don't need to keep updating it everytime

[DLM] fix stopping unstarted recovery [43/70]

2006-11-30 Thread Steven Whitehouse
p/start before clearing the flag. Signed-off-by: David Teigland <[EMAIL PROTECTED]> Signed-off-by: Steven Whitehouse <[EMAIL PROTECTED]> --- fs/dlm/recoverd.c |7 ++- 1 files changed, 6 insertions(+), 1 deletions(-) diff --git a/fs/dlm/recoverd.c b/fs/dlm/recoverd.c index 4a1d602..6

[DLM] fix aborted recovery during node removal [42/70]

2006-11-30 Thread Steven Whitehouse
in locks not being purged for failed/removed nodes. The fix is to check for removed nodes for which recovery has not been completed at the start of a new recovery sequence. Signed-off-by: David Teigland <[EMAIL PROTECTED]> Signed-off-by: Steven Whitehouse <[EMAIL PROTECTED]> --- fs/dlm

[GFS2] gfs2 misc endianness annotations [15/70]

2006-11-30 Thread Steven Whitehouse
>From b44b84d765b02f813a67b96bf79e3b5d4d621631 Mon Sep 17 00:00:00 2001 From: Al Viro <[EMAIL PROTECTED]> Date: Sat, 14 Oct 2006 10:46:30 -0400 Subject: [PATCH] [GFS2] gfs2 misc endianness annotations Signed-off-by: Al Viro <[EMAIL PROTECTED]> Signed-off-by: Steven Whitehouse &l

[DLM] don't accept replies to old recovery messages [64/70]

2006-11-30 Thread Steven Whitehouse
created. This makes it clear when messages are being rejected from an old instance of a lockspace that has since been recreated. Signed-off-by: David Teigland <[EMAIL PROTECTED]> Signed-off-by: Steven Whitehouse <[EMAIL PROTECTED]> --- fs/dlm/dlm_internal.h |4 +++- fs/dlm/loc

Re: [GFS2 & DLM] Guide to -nmw tree patches

2006-11-30 Thread Steven Whitehouse
Hi, On Thu, 2006-11-30 at 13:21 +0100, Arjan van de Ven wrote: > On Thu, 2006-11-30 at 12:12 +0000, Steven Whitehouse wrote: > > Hi, > > > > Below is a summary diffstat of all the changes in the GFS2 & DLM -nmw > > (next merge window) git tree. Since merge time

[GFS2] fs/gfs2/log.c:log_bmap() fix printk format warning [62/70]

2006-11-30 Thread Steven Whitehouse
log.c:322: warning: format '%llu' expects type 'long long unsigned int', but argument 3 has type 'sector_t' Signed-off-by: Ryusuke Konishi <[EMAIL PROTECTED]> Signed-off-by: Steven Whitehouse <[EMAIL PROTECTED]> --- fs/gfs2/log.c |3 ++- 1 files changed, 2

[GFS2] Simplify glops functions [53/70]

2006-11-30 Thread Steven Whitehouse
>From 1a14d3a68f04527546121eb7b45187ff6af63151 Mon Sep 17 00:00:00 2001 From: Steven Whitehouse <[EMAIL PROTECTED]> Date: Mon, 20 Nov 2006 10:37:45 -0500 Subject: [PATCH] [GFS2] Simplify glops functions The go_sync callback took two flags, but one of them was set on every call, so t

[GFS2] Shrink gfs2_inode (8) - i_vn [28/70]

2006-11-30 Thread Steven Whitehouse
>From bfded27ba010d1c3b0aa3843f97dc9b80de751be Mon Sep 17 00:00:00 2001 From: Steven Whitehouse <[EMAIL PROTECTED]> Date: Wed, 1 Nov 2006 16:05:38 -0500 Subject: [PATCH] [GFS2] Shrink gfs2_inode (8) - i_vn This shrinks the size of the gfs2_inode by 8 bytes by replacing the version count

[GFS2] Move gfs2_meta_syncfs() into log.c [57/70]

2006-11-30 Thread Steven Whitehouse
>From a25311c8e0b7071b129ca9a9e49e22eeaf620864 Mon Sep 17 00:00:00 2001 From: Steven Whitehouse <[EMAIL PROTECTED]> Date: Thu, 23 Nov 2006 11:06:35 -0500 Subject: [PATCH] [GFS2] Move gfs2_meta_syncfs() into log.c By moving gfs2_meta_syncfs() into log.c, gfs2_ail1_start() can be ma

[GFS2] split and annotate gfs2_quota [12/70]

2006-11-30 Thread Steven Whitehouse
>From b5bc9e8b065dbcd4c675e8c158d6e524f221b8e1 Mon Sep 17 00:00:00 2001 From: Al Viro <[EMAIL PROTECTED]> Date: Fri, 13 Oct 2006 23:31:55 -0400 Subject: [PATCH] [GFS2] split and annotate gfs2_quota Signed-off-by: Al Viro <[EMAIL PROTECTED]> Signed-off-by: Steven Whitehouse &l

[GFS2] Remove unused sysfs files [49/70]

2006-11-30 Thread Steven Whitehouse
>From 175011cf6edddea32e5f5e0e04434104cc348de9 Mon Sep 17 00:00:00 2001 From: Steven Whitehouse <[EMAIL PROTECTED]> Date: Thu, 16 Nov 2006 10:58:55 -0500 Subject: [PATCH] [GFS2] Remove unused sysfs files Four of the sysfs files are unused and can therefore be removed. Signed-off-b

[GFS2] Remove unused function from inode.c [50/70]

2006-11-30 Thread Steven Whitehouse
>From dcd2479959c79d44f5dd77e71672e70f1f8b1f06 Mon Sep 17 00:00:00 2001 From: Steven Whitehouse <[EMAIL PROTECTED]> Date: Thu, 16 Nov 2006 11:08:16 -0500 Subject: [PATCH] [GFS2] Remove unused function from inode.c The gfs2_glock_nq_m_atime function is unused in so far as its only ever ca

[GFS2] Shrink gfs2_inode (3) - di_mode [23/70]

2006-11-30 Thread Steven Whitehouse
>From b60623c238b6a819bd04090139704e2cb57a751f Mon Sep 17 00:00:00 2001 From: Steven Whitehouse <[EMAIL PROTECTED]> Date: Wed, 1 Nov 2006 12:22:46 -0500 Subject: [PATCH] [GFS2] Shrink gfs2_inode (3) - di_mode This removes the duplicate di_mode field in favour of using the inode->

[GFS2] Mount problem with the GFS2 code [68/70]

2006-11-30 Thread Steven Whitehouse
> 9)(9 because,sector size 2^9,samething happens in ll_rw_block also, before calling submit_bio()). So I have developed the patch which solves this problem. Please let me know your comments. ==== Signed-off-by: Srinivasa DS <[EMAIL PRO

[DLM] status messages ping-pong between unmounted nodes [40/70]

2006-11-30 Thread Steven Whitehouse
LY for an unknown lockspace. Signed-off-by: David Teigland <[EMAIL PROTECTED]> Signed-off-by: Steven Whitehouse <[EMAIL PROTECTED]> --- fs/dlm/rcom.c |7 --- 1 files changed, 4 insertions(+), 3 deletions(-) diff --git a/fs/dlm/rcom.c b/fs/dlm/rcom.c index 518239a..87b12f7 100644 ---

[GFS2] Only set inode flags when required [33/70]

2006-11-30 Thread Steven Whitehouse
>From 6b124d8dba1f46c5f2caf3b3159bbe627f75b9b6 Mon Sep 17 00:00:00 2001 From: Steven Whitehouse <[EMAIL PROTECTED]> Date: Wed, 8 Nov 2006 12:51:06 -0500 Subject: [PATCH] [GFS2] Only set inode flags when required We were setting the inode flags from GFS2's flags far too often,

[GFS2] gfs2_dinode_host fields are host-endian [2/70]

2006-11-30 Thread Steven Whitehouse
bvious that we want to convert all of them to host-endian... Signed-off-by: Al Viro <[EMAIL PROTECTED]> Signed-off-by: Steven Whitehouse <[EMAIL PROTECTED]> --- fs/gfs2/incore.h|2 +- include/linux/gfs2_ondisk.h | 47 ++- 2 fi

[GFS2] split and annotate gfs2_statfs_change [13/70]

2006-11-30 Thread Steven Whitehouse
>From bd209cc017f231e8536550bdab1bf5da93c32798 Mon Sep 17 00:00:00 2001 From: Al Viro <[EMAIL PROTECTED]> Date: Fri, 13 Oct 2006 23:43:19 -0400 Subject: [PATCH] [GFS2] split and annotate gfs2_statfs_change Signed-off-by: Al Viro <[EMAIL PROTECTED]> Signed-off-by: Steven Whi

[GFS2] Don't copy meta_header for rgrp in and out [30/70]

2006-11-30 Thread Steven Whitehouse
>From f6e58f01e8dc869803b9f73b2aa9d5bc3f32ca05 Mon Sep 17 00:00:00 2001 From: Steven Whitehouse <[EMAIL PROTECTED]> Date: Tue, 7 Nov 2006 15:14:58 -0500 Subject: [PATCH] [GFS2] Don't copy meta_header for rgrp in and out The meta_header for an ondisk rgrp never changes, so the

[GFS2] Shrink gfs2_inode (6) - di_atime/di_mtime/di_ctime [26/70]

2006-11-30 Thread Steven Whitehouse
>From 1a7b1eed5802502fd649e04784becd58557fdcf1 Mon Sep 17 00:00:00 2001 From: Steven Whitehouse <[EMAIL PROTECTED]> Date: Wed, 1 Nov 2006 14:35:17 -0500 Subject: [PATCH] [GFS2] Shrink gfs2_inode (6) - di_atime/di_mtime/di_ctime Remove the di_[amc]time fields and use inode->i_[amc

[GFS2] Reduce number of arguments to meta_io.c:getbuf() [58/70]

2006-11-30 Thread Steven Whitehouse
>From cb4c03131836a55bf95e1c165409244ac6b4f39f Mon Sep 17 00:00:00 2001 From: Steven Whitehouse <[EMAIL PROTECTED]> Date: Thu, 23 Nov 2006 11:16:32 -0500 Subject: [PATCH] [GFS2] Reduce number of arguments to meta_io.c:getbuf() Since the superblock and the address_space are determined by

[GFS2] Add a comment about reading the super block [69/70]

2006-11-30 Thread Steven Whitehouse
>From aac1a3c77a46c2d06f297641760dd740ac2a84af Mon Sep 17 00:00:00 2001 From: Steven Whitehouse <[EMAIL PROTECTED]> Date: Thu, 30 Nov 2006 10:02:19 -0500 Subject: [PATCH] [GFS2] Add a comment about reading the super block The comment explains why we use the bio functions to read the su

[DLM] fix size of STATUS_REPLY message [63/70]

2006-11-30 Thread Steven Whitehouse
: David Teigland <[EMAIL PROTECTED]> Signed-off-by: Steven Whitehouse <[EMAIL PROTECTED]> --- fs/dlm/rcom.c |6 +- 1 files changed, 5 insertions(+), 1 deletions(-) diff --git a/fs/dlm/rcom.c b/fs/dlm/rcom.c index 87b12f7..6ac195c 100644 --- a/fs/dlm/rcom.c +++ b/fs/dlm/rco

[GFS2] Tidy up bmap & fix boundary bug [48/70]

2006-11-30 Thread Steven Whitehouse
>From 4cf1ed8144e740de27c6146c25d5d7ea26679cc5 Mon Sep 17 00:00:00 2001 From: Steven Whitehouse <[EMAIL PROTECTED]> Date: Wed, 15 Nov 2006 15:21:06 -0500 Subject: [PATCH] [GFS2] Tidy up bmap & fix boundary bug This moves the locking for bmap into the bmap function itself rather

[GFS2] split gfs2_dinode into on-disk and host variants [1/70]

2006-11-30 Thread Steven Whitehouse
actually stores a lot of fields differently; for now the declaration is just cloned, inode field is swtiched and changes propagated. Signed-off-by: Al Viro <[EMAIL PROTECTED]> Signed-off-by: Steven Whitehouse <[EMAIL PROTECTED]> --- fs/gfs2/inode.c |4 ++-- fs/gfs2/ondis

[DLM] fix format warnings in rcom.c and recoverd.c [66/70]

2006-11-30 Thread Steven Whitehouse
x27;, but argument 3 has type 'uint64_t' fs/dlm/recoverd.c:210: warning: format '%llx' expects type 'long long unsigned int', but argument 3 has type 'uint64_t' Signed-off-by: Ryusuke Konishi <[EMAIL PROTECTED]> Signed-off-by: Patrick Caulfield <[EM

[DLM] do full recover_locks barrier [44/70]

2006-11-30 Thread Steven Whitehouse
wait, then it misses the important step of collecting and reporting the barrier status from the other nodes (which is the job of the low nodeid in the barrier wait routine). Signed-off-by: David Teigland <[EMAIL PROTECTED]> Signed-off-by: Steven Whitehouse <[EMAIL PROTECTED]> --- fs/dlm

[DLM] clear sbflags on lock master [45/70]

2006-11-30 Thread Steven Whitehouse
opagate the cleared sbflags value to the master node when the lock is requested. QA's d_rwrandirectlarge test triggers this bug very quickly. Signed-off-by: David Teigland <[EMAIL PROTECTED]> Signed-off-by: Steven Whitehouse <[EMAIL PROTECTED]> --- fs/dlm/lock.c |1 + 1 files c

[GFS2] Make sentinel dirents compatible with gfs1 [51/70]

2006-11-30 Thread Steven Whitehouse
>From 5e7d65cd9d3819512b059f4260de0119b985454c Mon Sep 17 00:00:00 2001 From: Steven Whitehouse <[EMAIL PROTECTED]> Date: Fri, 17 Nov 2006 12:27:44 -0500 Subject: [PATCH] [GFS2] Make sentinel dirents compatible with gfs1 When deleting directory entries, we set the inum.no_addr to zero in

[GFS2] Fix Kconfig wrt CRC32 [52/70]

2006-11-30 Thread Steven Whitehouse
>From fa2ecfc5e11b12f25b67f9c84ac6b0e74a6a0115 Mon Sep 17 00:00:00 2001 From: Steven Whitehouse <[EMAIL PROTECTED]> Date: Mon, 20 Nov 2006 10:04:49 -0500 Subject: [PATCH] [GFS2] Fix Kconfig wrt CRC32 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit GFS2 requires

[GFS2] split and annotate gfs2_inum [11/70]

2006-11-30 Thread Steven Whitehouse
>From 629a21e7ecedf779c68dcaa9a186069f57a7c652 Mon Sep 17 00:00:00 2001 From: Al Viro <[EMAIL PROTECTED]> Date: Fri, 13 Oct 2006 22:51:24 -0400 Subject: [PATCH] [GFS2] split and annotate gfs2_inum Signed-off-by: Al Viro <[EMAIL PROTECTED]> Signed-off-by: Steven Whitehouse &l

[GFS2 & DLM] Guide to -nmw tree patches

2006-11-30 Thread Steven Whitehouse
og.c:log_bmap() fix printk format warning [C] [DLM] fix format warnings in rcom.c and recoverd.c[C] Srinivasa Ds: [GFS2] Mount problem with the GFS2 code [B] Steven Whitehouse: [GFS2] Fix crc32 calculation in recover

[DLM] res_recover_locks_count not reset when recover_locks is aborted [39/70]

2006-11-30 Thread Steven Whitehouse
s that counter is zero. Signed-off-by: David Teigland <[EMAIL PROTECTED]> Signed-off-by: Steven Whitehouse <[EMAIL PROTECTED]> --- fs/dlm/recover.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/fs/dlm/recover.c b/fs/dlm/recover.c index a5e6d18..cf9f683 100

[GFS2] split and annotate gfs2_rgrp [5/70]

2006-11-30 Thread Steven Whitehouse
>From 68826664d12827d7a732192e2f00ba46fb899414 Mon Sep 17 00:00:00 2001 From: Al Viro <[EMAIL PROTECTED]> Date: Fri, 13 Oct 2006 21:07:22 -0400 Subject: [PATCH] [GFS2] split and annotate gfs2_rgrp Signed-off-by: Al Viro <[EMAIL PROTECTED]> Signed-off-by: Steven Whitehouse &l

[GFS2] fields of gfs2_sb_host are host-endian [4/70]

2006-11-30 Thread Steven Whitehouse
lar ones, kill completely unused. Signed-off-by: Al Viro <[EMAIL PROTECTED]> Signed-off-by: Steven Whitehouse <[EMAIL PROTECTED]> --- include/linux/gfs2_ondisk.h | 11 --- 1 files changed, 4 insertions(+), 7 deletions(-) diff --git a/include/linux/gfs2_ondisk.h b/include/linux/g

[GFS2] mark_inode_dirty after write to stuffed file [55/70]

2006-11-30 Thread Steven Whitehouse
>From ae619320b22f8e0b2bbe4a3a5ac2f9ccf08d7ec2 Mon Sep 17 00:00:00 2001 From: Steven Whitehouse <[EMAIL PROTECTED]> Date: Wed, 22 Nov 2006 11:28:47 -0500 Subject: [PATCH] [GFS2] mark_inode_dirty after write to stuffed file Writes to stuffed files were not being marked dirty correctly. S

[GFS2] Remove gfs2_check_acl() [67/70]

2006-11-30 Thread Steven Whitehouse
>From 77386e1f662f104680da7885d32e068e4b11b882 Mon Sep 17 00:00:00 2001 From: Steven Whitehouse <[EMAIL PROTECTED]> Date: Wed, 29 Nov 2006 10:41:49 -0500 Subject: [PATCH] [GFS2] Remove gfs2_check_acl() As pointed out by Adrian Bunk, the gfs2_check_acl() function is no longer used. T

[GFS2] Remove unused zero_readpage from stuffed_readpage [37/70]

2006-11-30 Thread Steven Whitehouse
- sizeof(struct gfs2_dinode)) bytes. Since the gfs2 block size is always less than page size, we will never see a call to stuffed_readpage for anything other than the first page in the file. Signed-off-by: Russell Cattelan <[EMAIL PROTECTED]> Signed-off-by: Steven Whitehouse <[EMAIL PROTEC

[GFS2] Fix glock ordering on inode creation [54/70]

2006-11-30 Thread Steven Whitehouse
>From 28626e2078571c4b776a17eaa486bbd2b7dfe2cd Mon Sep 17 00:00:00 2001 From: Steven Whitehouse <[EMAIL PROTECTED]> Date: Wed, 22 Nov 2006 11:13:21 -0500 Subject: [PATCH] [GFS2] Fix glock ordering on inode creation The lock order here should be parent -> child rather than numeric or

[GFS2] split and annotate gfs2_inum_range [6/70]

2006-11-30 Thread Steven Whitehouse
>From e697264709c86040271cdd7abee781d7adbb7f91 Mon Sep 17 00:00:00 2001 From: Al Viro <[EMAIL PROTECTED]> Date: Fri, 13 Oct 2006 21:29:46 -0400 Subject: [PATCH] [GFS2] split and annotate gfs2_inum_range Signed-off-by: Al Viro <[EMAIL PROTECTED]> Signed-off-by: Steven Whitehouse &l

[GFS2] Shrink gfs2_inode (2) - di_major/di_minor [22/70]

2006-11-30 Thread Steven Whitehouse
>From e7f14f4d094ea1a9ce1953375f5bc1500c760c79 Mon Sep 17 00:00:00 2001 From: Steven Whitehouse <[EMAIL PROTECTED]> Date: Tue, 31 Oct 2006 21:45:08 -0500 Subject: [PATCH] [GFS2] Shrink gfs2_inode (2) - di_major/di_minor This removes the device numbers from this structure by using inod

[GFS2] Fix journal flush problem [56/70]

2006-11-30 Thread Steven Whitehouse
>From b004157ab5b374a498a5874cda68c389219d23e7 Mon Sep 17 00:00:00 2001 From: Steven Whitehouse <[EMAIL PROTECTED]> Date: Thu, 23 Nov 2006 10:51:34 -0500 Subject: [PATCH] [GFS2] Fix journal flush problem This fixes a bug which resulted in poor performance due to flushing the journal

[DLM] fix add_requestqueue checking nodes list [61/70]

2006-11-30 Thread Steven Whitehouse
eing shut down since the add routine will not be rejecting requests itself any more. Signed-off-by: David Teigland <[EMAIL PROTECTED]> Signed-off-by: Steven Whitehouse <[EMAIL PROTECTED]> --- fs/dlm/lockspace.c|2 ++ fs/dlm/recoverd.c | 16 fs/dlm/reques

[GFS2] Don't flush everything on fdatasync [70/70]

2006-11-30 Thread Steven Whitehouse
>From 33c3de32872ef3c075e4dac04c0de8f86ac39f6f Mon Sep 17 00:00:00 2001 From: Steven Whitehouse <[EMAIL PROTECTED]> Date: Thu, 30 Nov 2006 10:14:32 -0500 Subject: [PATCH] [GFS2] Don't flush everything on fdatasync The gfs2_fsync() function was doing a journal flush on each and eve

[GFS2] Shrink gfs2_inode (1) - di_header/di_num [21/70]

2006-11-30 Thread Steven Whitehouse
>From af339c0241d0dd3b35f9097b4f4999bb22ffe502 Mon Sep 17 00:00:00 2001 From: Steven Whitehouse <[EMAIL PROTECTED]> Date: Wed, 1 Nov 2006 10:34:15 -0500 Subject: [PATCH] [GFS2] Shrink gfs2_inode (1) - di_header/di_num The metadata header doesn't need to be stored in the incore str

[GFS2] lock function parameter [65/70]

2006-11-30 Thread Steven Whitehouse
;t a prototype Signed-off-by: Randy Dunlap <[EMAIL PROTECTED]> Signed-off-by: Steven Whitehouse <[EMAIL PROTECTED]> --- fs/gfs2/glock.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/fs/gfs2/glock.c b/fs/gfs2/glock.c index 3c2ff81..f130f98 100644 --- a/fs/

[DLM] Fix DLM config [46/70]

2006-11-30 Thread Steven Whitehouse
hould fix the bug where DLM can be left selected when NET gets unselected. This incorporates all the comments received about this patch. Cc: Adrian Bunk <[EMAIL PROTECTED]> Cc: Andrew Morton <[EMAIL PROTECTED]> Signed-Off-By: Patrick Caulfield <[EMAIL PROTECTED]> Signed-off-by

[GFS2] Fix crc32 calculation in recovery.c [8/70]

2006-11-30 Thread Steven Whitehouse
>From 2a2c98247b822db8df037a56c27201f9d716ac66 Mon Sep 17 00:00:00 2001 From: Steven Whitehouse <[EMAIL PROTECTED]> Date: Tue, 31 Oct 2006 14:44:50 -0500 Subject: [PATCH] [GFS2] Fix crc32 calculation in recovery.c Commit "[GFS2] split and annotate gfs2_log_head" resulted in an

[GFS2] gfs2 __user misannotation fix [16/70]

2006-11-30 Thread Steven Whitehouse
>From 9c9ab3d5414653bfe5e5b9f4dfdaab0c6ab17196 Mon Sep 17 00:00:00 2001 From: Al Viro <[EMAIL PROTECTED]> Date: Fri, 13 Oct 2006 23:49:23 -0400 Subject: [PATCH] [GFS2] gfs2 __user misannotation fix Signed-off-by: Al Viro <[EMAIL PROTECTED]> Signed-off-by: Steven Whitehouse &l

[GFS2] Remove unused GL_DUMP flag [31/70]

2006-11-30 Thread Steven Whitehouse
>From c594d8866460a2710c436839d79f334a0714a2a7 Mon Sep 17 00:00:00 2001 From: Steven Whitehouse <[EMAIL PROTECTED]> Date: Wed, 8 Nov 2006 09:01:13 -0500 Subject: [PATCH] [GFS2] Remove unused GL_DUMP flag There is no way to set the GL_DUMP flag, and in any case the same thing can be

[GFS2] Fix memory allocation in glock.c [47/70]

2006-11-30 Thread Steven Whitehouse
>From ab923031ceb95ec50ef33ccadf28663c660aa94c Mon Sep 17 00:00:00 2001 From: Steven Whitehouse <[EMAIL PROTECTED]> Date: Wed, 15 Nov 2006 15:17:03 -0500 Subject: [PATCH] [GFS2] Fix memory allocation in glock.c Change from GFP_KERNEL to GFP_NOFS as this was causing a slow down when tryin

[GFS2] Fix recursive locking in gfs2_getattr [60/70]

2006-11-30 Thread Steven Whitehouse
>From dcf3dd852f554bb0016aa23892596717cc123a26 Mon Sep 17 00:00:00 2001 From: Steven Whitehouse <[EMAIL PROTECTED]> Date: Mon, 27 Nov 2006 10:12:05 -0500 Subject: [PATCH] [GFS2] Fix recursive locking in gfs2_getattr The readdirplus NFS operation can result in gfs2_getattr being called

[GFS2] Shrink gfs2_inode (5) - di_nlink [25/70]

2006-11-30 Thread Steven Whitehouse
>From 4f56110a00af5fb2e22fbccfcaf944d62cae8fcf Mon Sep 17 00:00:00 2001 From: Steven Whitehouse <[EMAIL PROTECTED]> Date: Wed, 1 Nov 2006 14:04:17 -0500 Subject: [PATCH] [GFS2] Shrink gfs2_inode (5) - di_nlink Remove the di_nlink field in favour of inode->i_nlink and update the nlink

[GFS2] split and annotate gfs2_log_head [7/70]

2006-11-30 Thread Steven Whitehouse
>From 551676226163379c217e8ec54bd287eab9b8521e Mon Sep 17 00:00:00 2001 From: Al Viro <[EMAIL PROTECTED]> Date: Fri, 13 Oct 2006 21:47:13 -0400 Subject: [PATCH] [GFS2] split and annotate gfs2_log_head Signed-off-by: Al Viro <[EMAIL PROTECTED]> Signed-off-by: Steven Whitehouse &l

[GFS2] Fix race in logging code [36/70]

2006-11-30 Thread Steven Whitehouse
log data structures may be going away due to a truncate. Store the buffer head in a local pointer prior to dropping the lock and relay on the buffer_head lock for consitency on the buffer head. Signed-Off-By: Russell Cattelan <[EMAIL PROTECTED]> Signed-off-by: Steven Whitehouse <[EMAIL P

[GFS2] Fix recursive locking in gfs2_permission [59/70]

2006-11-30 Thread Steven Whitehouse
>From 300c7d75f3a5e8edd3e390ccd56b808f3fb14e33 Mon Sep 17 00:00:00 2001 From: Steven Whitehouse <[EMAIL PROTECTED]> Date: Mon, 27 Nov 2006 09:55:28 -0500 Subject: [PATCH] [GFS2] Fix recursive locking in gfs2_permission Since gfs2_permission may be called either from the VFS (in which cas

[GFS2] split gfs2_sb [3/70]

2006-11-30 Thread Steven Whitehouse
>From f50dfaf78c01df3cc2d8819f07d6661915567bae Mon Sep 17 00:00:00 2001 From: Al Viro <[EMAIL PROTECTED]> Date: Fri, 13 Oct 2006 20:45:02 -0400 Subject: [PATCH] [GFS2] split gfs2_sb Signed-off-by: Al Viro <[EMAIL PROTECTED]> Signed-off-by: Steven Whitehouse <[EMAIL PROTEC

Re: [RFC][PATCH] Mount problem with the GFS2 code

2006-11-30 Thread Steven Whitehouse
Hi, On Thu, 2006-11-30 at 01:12 -0800, Andrew Morton wrote: > On Thu, 30 Nov 2006 09:05:38 + > Steven Whitehouse <[EMAIL PROTECTED]> wrote: > > > Was there another > > reason for not using the bio routines? > > Just that it's a layering violation. >

Re: [RFC][PATCH] Mount problem with the GFS2 code

2006-11-30 Thread Steven Whitehouse
Hi, On Thu, 2006-11-30 at 00:29 -0800, Andrew Morton wrote: > On Thu, 30 Nov 2006 15:04:55 +0530 > Srinivasa Ds <[EMAIL PROTECTED]> wrote: > > > == > > On debugging further we found that problem is while reading the super >

Re: [-mm patch] #if 0 fs/gfs2/acl.c:gfs2_check_acl()

2006-11-29 Thread Steven Whitehouse
Hi, A better solution is just to remove it I think, so thats what I'll do in my git tree. Thanks for pointing it out, Steve. On Wed, 2006-11-29 at 11:04 +0100, Adrian Bunk wrote: > On Tue, Nov 28, 2006 at 02:02:46AM -0800, Andrew Morton wrote: > >... > > Changes since 2.6.19-rc6-mm1: > >... > >

Re: [GFS2] Fix Kconfig wrt CRC32 [8/9]

2006-11-29 Thread Steven Whitehouse
Hi, On Tue, 2006-11-28 at 10:33 -0800, Randy Dunlap wrote: > Steven Whitehouse wrote: [bits snipped to cut down size of reply] > > You'll need the patch: > > http://www.kernel.org/git/?p=linux/kernel/git/steve/gfs2-2.6-nmw.git;a=commitdiff;h=4a28fda50d864ede7d2724723949407e

Re: [GFS2] Fix Kconfig wrt CRC32 [8/9]

2006-11-28 Thread Steven Whitehouse
Hi, On Mon, 2006-11-27 at 08:38 -0800, Randy Dunlap wrote: > Steven Whitehouse wrote: > > Hi, > > > > On Fri, 2006-11-24 at 21:43 -0800, Randy Dunlap wrote: > >> On Fri, 24 Nov 2006 09:34:49 + Steven Whitehouse wrote: > >> > >>> >From 6f7

Re: [GFS2] Fix Kconfig wrt CRC32 [8/9]

2006-11-27 Thread Steven Whitehouse
Hi, On Fri, 2006-11-24 at 21:43 -0800, Randy Dunlap wrote: > On Fri, 24 Nov 2006 09:34:49 +0000 Steven Whitehouse wrote: > > > >From 6f788fd00c82533d4cd5587a9706f8468658a24d Mon Sep 17 00:00:00 2001 > > From: Steven Whitehouse <[EMAIL PROTECTED]> > > Date

Re: rcu read-side protection

2005-08-17 Thread Steven Whitehouse
Hi, On Wed, Aug 17, 2005 at 07:14:38AM -0700, Paul E. McKenney wrote: [snip] > How about the following patch? Untested, but seems pretty straightforward. > > Thanx, Paul > That would be my preferred fix. If Patrick is happy with that, then

Re: rcu read-side protection

2005-08-17 Thread Steven Whitehouse
Hi, On Tue, Aug 16, 2005 at 07:01:57PM -0700, Paul E. McKenney wrote: > On Tue, Aug 16, 2005 at 05:09:29PM -0700, Suzanne Wood wrote: > [ . . . ] > > A read-side critical section is marked to protect the dereference of the > > dn_ptr and assignment to dn_db which is a pointer to a dn_dev. (struc

<    2   3   4   5   6   7