Re: GFS - updated patches

2005-08-11 Thread Pekka Enberg
Hi,

On 8/11/05, David Teigland <[EMAIL PROTECTED]> wrote:
> The large majority, and I think all that people care about.  If we ignored
> something that someone thinks is important, a reminder would be useful.

The only remaining issue for me is the vma walk. Thanks, David!

 Pekka
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [Linux-cluster] GFS - updated patches

2005-08-11 Thread Pekka Enberg
On 8/11/05, Michael <[EMAIL PROTECTED]> wrote:
> Hi, Dave,
> 
> I quickly applied gfs2 and dlm patches in kernel 2.6.12.2, it passed
> compiling but has some warning log, see attachment. maybe helpful to
> you.

kzalloc is not in Linus' tree yet. Try with 2.6.13-rc5-mm1.

 Pekka
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [Linux-cluster] GFS - updated patches

2005-08-11 Thread Michael
Hi, Dave,

I quickly applied gfs2 and dlm patches in kernel 2.6.12.2, it passed
compiling but has some warning log, see attachment. maybe helpful to
you.

Thanks,

Michael

On 8/11/05, David Teigland <[EMAIL PROTECTED]> wrote:
> Thanks for all the review and comments.  This is a new set of patches that
> incorporates the suggestions we've received.
> 
> http://redhat.com/~teigland/gfs2/20050811/gfs2-full.patch
> http://redhat.com/~teigland/gfs2/20050811/broken-out/
> 
> Dave
> 
> --
> Linux-cluster mailing list
> [EMAIL PROTECTED]
> http://www.redhat.com/mailman/listinfo/linux-cluster
>
[EMAIL PROTECTED] kernel-gfs2-full-2.6.12.2]$ make SUBDIRS=fs/gfs2
  LD  fs/gfs2/built-in.o
  CC [M]  fs/gfs2/acl.o
  CC [M]  fs/gfs2/bits.o
  CC [M]  fs/gfs2/bmap.o
fs/gfs2/bmap.c: In function `find_metapath':
fs/gfs2/bmap.c:320: warning: implicit declaration of function `kzalloc'
fs/gfs2/bmap.c:320: warning: assignment makes pointer from integer without a 
cast
  CC [M]  fs/gfs2/daemon.o
  CC [M]  fs/gfs2/dir.o
fs/gfs2/dir.c: In function `leaf_dealloc':
fs/gfs2/dir.c:1910: warning: implicit declaration of function `kzalloc'
fs/gfs2/dir.c:1910: warning: assignment makes pointer from integer without a 
cast
  CC [M]  fs/gfs2/eaops.o
  CC [M]  fs/gfs2/eattr.o
  CC [M]  fs/gfs2/glock.o
  CC [M]  fs/gfs2/glops.o
  CC [M]  fs/gfs2/inode.o
  CC [M]  fs/gfs2/ioctl.o
  CC [M]  fs/gfs2/jdata.o
  CC [M]  fs/gfs2/lm.o
  CC [M]  fs/gfs2/log.o
fs/gfs2/log.c: In function `gfs2_log_get_buf':
fs/gfs2/log.c:363: warning: implicit declaration of function `kzalloc'
fs/gfs2/log.c:363: warning: assignment makes pointer from integer without a cast
fs/gfs2/log.c: In function `gfs2_log_fake_buf':
fs/gfs2/log.c:393: warning: assignment makes pointer from integer without a cast
fs/gfs2/log.c: In function `gfs2_log_flush_i':
fs/gfs2/log.c:524: warning: assignment makes pointer from integer without a cast
  CC [M]  fs/gfs2/lops.o
  CC [M]  fs/gfs2/lvb.o
  CC [M]  fs/gfs2/main.o
  CC [M]  fs/gfs2/meta_io.o
  CC [M]  fs/gfs2/mount.o
  CC [M]  fs/gfs2/ondisk.o
  CC [M]  fs/gfs2/ops_address.o
  CC [M]  fs/gfs2/ops_dentry.o
  CC [M]  fs/gfs2/ops_export.o
  CC [M]  fs/gfs2/ops_file.o
fs/gfs2/ops_file.c: In function `readdir_bad':
fs/gfs2/ops_file.c:1052: warning: implicit declaration of function `kzalloc'
fs/gfs2/ops_file.c:1052: warning: assignment makes pointer from integer without 
a cast
fs/gfs2/ops_file.c: In function `gfs2_open':
fs/gfs2/ops_file.c:1218: warning: assignment makes pointer from integer without 
a cast
  CC [M]  fs/gfs2/ops_fstype.o
  CC [M]  fs/gfs2/ops_inode.o
  CC [M]  fs/gfs2/ops_super.o
  CC [M]  fs/gfs2/ops_vm.o
  CC [M]  fs/gfs2/page.o
  CC [M]  fs/gfs2/proc.o
  CC [M]  fs/gfs2/quota.o
fs/gfs2/quota.c: In function `qd_alloc':
fs/gfs2/quota.c:51: warning: implicit declaration of function `kzalloc'
fs/gfs2/quota.c:51: warning: assignment makes pointer from integer without a 
cast
fs/gfs2/quota.c: In function `gfs2_quota_init':
fs/gfs2/quota.c:1058: warning: assignment makes pointer from integer without a 
cast
  CC [M]  fs/gfs2/resize.o
  CC [M]  fs/gfs2/recovery.o
  CC [M]  fs/gfs2/rgrp.o
fs/gfs2/rgrp.c: In function `gfs2_ri_update':
fs/gfs2/rgrp.c:300: warning: implicit declaration of function `kzalloc'
fs/gfs2/rgrp.c:300: warning: assignment makes pointer from integer without a 
cast
fs/gfs2/rgrp.c: In function `gfs2_alloc_get':
fs/gfs2/rgrp.c:530: warning: assignment makes pointer from integer without a 
cast
  CC [M]  fs/gfs2/super.o
fs/gfs2/super.c: In function `gfs2_jindex_hold':
fs/gfs2/super.c:306: warning: implicit declaration of function `kzalloc'
fs/gfs2/super.c:306: warning: assignment makes pointer from integer without a 
cast
  CC [M]  fs/gfs2/trans.o
fs/gfs2/trans.c: In function `gfs2_trans_begin_i':
fs/gfs2/trans.c:38: warning: implicit declaration of function `kzalloc'
fs/gfs2/trans.c:38: warning: assignment makes pointer from integer without a 
cast
  CC [M]  fs/gfs2/unlinked.o
fs/gfs2/unlinked.c: In function `ul_alloc':
fs/gfs2/unlinked.c:154: warning: implicit declaration of function `kzalloc'
fs/gfs2/unlinked.c:154: warning: assignment makes pointer from integer without 
a cast
fs/gfs2/unlinked.c: In function `gfs2_unlinked_init':
fs/gfs2/unlinked.c:342: warning: assignment makes pointer from integer without 
a cast
  CC [M]  fs/gfs2/util.o
  LD [M]  fs/gfs2/gfs2.o
  LD  fs/gfs2/locking/dlm/built-in.o
  CC [M]  fs/gfs2/locking/dlm/lock.o
  CC [M]  fs/gfs2/locking/dlm/main.o
  CC [M]  fs/gfs2/locking/dlm/mount.o
  CC [M]  fs/gfs2/locking/dlm/sysfs.o
  CC [M]  fs/gfs2/locking/dlm/thread.o
  LD [M]  fs/gfs2/locking/dlm/lock_dlm.o
  LD  fs/gfs2/locking/harness/built-in.o
  CC [M]  fs/gfs2/locking/harness/main.o
  LD [M]  fs/gfs2/locking/harness/lock_harness.o
  LD  fs/gfs2/locking/nolock/built-in.o
  CC [M]  fs/gfs2/locking/nolock/main.o
  LD [M]  fs/gfs2/locking/nolock/lock_nolock.o
  Building modules, stage 2.
  MODPOST
*** Warning: "kzalloc" [fs/gfs2/gfs2.ko] undefined!
  CC  fs/gfs2/gfs2.mo

Re: GFS - updated patches

2005-08-11 Thread David Teigland
On Thu, Aug 11, 2005 at 10:50:32AM +0200, Arjan van de Ven wrote:

> > > > Thanks for all the review and comments.  This is a new set of
> > > > patches that incorporates the suggestions we've received.
> > > 
> > > all of them or only a subset?
> 
> with them I meant the suggestions not the patches ;)

The large majority, and I think all that people care about.  If we ignored
something that someone thinks is important, a reminder would be useful.

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: GFS - updated patches

2005-08-11 Thread Arjan van de Ven
On Thu, 2005-08-11 at 16:50 +0800, David Teigland wrote:
> On Thu, Aug 11, 2005 at 10:32:38AM +0200, Arjan van de Ven wrote:
> > On Thu, 2005-08-11 at 16:17 +0800, David Teigland wrote:
> > > Thanks for all the review and comments.  This is a new set of patches that
> > > incorporates the suggestions we've received.
> > 
> > all of them or only a subset?
> 
> All patches, now 01-13 (what was patch 08 disappeared entirely)

with them I meant the suggestions not the patches ;)


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: GFS - updated patches

2005-08-11 Thread Michael
yes, after apply dlm.patch, I saw it! although I don't know what's "-mm".

Thanks,

Michael

On 8/11/05, David Teigland <[EMAIL PROTECTED]> wrote:
> On Thu, Aug 11, 2005 at 04:21:04PM +0800, Michael wrote:
> > I have the same question as I asked before, how can I see GFS in "make
> > menuconfig", after I patch gfs2-full.patch into a 2.6.12.2 kernel?
> 
> You need to select the dlm under drivers.  It's in -mm, or apply
>   http://redhat.com/~teigland/dlm.patch
> 
>
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: GFS - updated patches

2005-08-11 Thread David Teigland
On Thu, Aug 11, 2005 at 10:32:38AM +0200, Arjan van de Ven wrote:
> On Thu, 2005-08-11 at 16:17 +0800, David Teigland wrote:
> > Thanks for all the review and comments.  This is a new set of patches that
> > incorporates the suggestions we've received.
> 
> all of them or only a subset?

All patches, now 01-13 (what was patch 08 disappeared entirely)

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: GFS - updated patches

2005-08-11 Thread David Teigland
On Thu, Aug 11, 2005 at 04:21:04PM +0800, Michael wrote:
> I have the same question as I asked before, how can I see GFS in "make
> menuconfig", after I patch gfs2-full.patch into a 2.6.12.2 kernel?

You need to select the dlm under drivers.  It's in -mm, or apply
  http://redhat.com/~teigland/dlm.patch

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: GFS - updated patches

2005-08-11 Thread Arjan van de Ven
On Thu, 2005-08-11 at 16:17 +0800, David Teigland wrote:
> Thanks for all the review and comments.  This is a new set of patches that
> incorporates the suggestions we've received.

all of them or only a subset?


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [Linux-cluster] GFS - updated patches

2005-08-11 Thread Michael
I have the same question as I asked before, how can I see GFS in "make
menuconfig", after I patch gfs2-full.patch into a 2.6.12.2 kernel?

Michael

On 8/11/05, David Teigland <[EMAIL PROTECTED]> wrote:
> Thanks for all the review and comments.  This is a new set of patches that
> incorporates the suggestions we've received.
> 
> http://redhat.com/~teigland/gfs2/20050811/gfs2-full.patch
> http://redhat.com/~teigland/gfs2/20050811/broken-out/
> 
> Dave
> 
> --
> Linux-cluster mailing list
> [EMAIL PROTECTED]
> http://www.redhat.com/mailman/listinfo/linux-cluster
>
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


GFS - updated patches

2005-08-11 Thread David Teigland
Thanks for all the review and comments.  This is a new set of patches that
incorporates the suggestions we've received.

http://redhat.com/~teigland/gfs2/20050811/gfs2-full.patch
http://redhat.com/~teigland/gfs2/20050811/broken-out/

Dave

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/