On Mon, Jun 14, 2010 at 06:31:22PM +0200, Andi Kleen wrote: > On Mon, Jun 14, 2010 at 03:39:58PM +0200, Andi Kleen wrote: > > On Mon, Jun 14, 2010 at 02:48:04PM +0200, Christoph Hellwig wrote: > > > On Mon, Jun 14, 2010 at 10:59:33AM +0200, Andi Kleen wrote: > > > > >How can this work? bp->data is a void pointer. > > > > > > > > It compiled with CONFIG_XFS_DEBUG enabled. Do I need to set some other > > > > option to test this? > > > > > > It fails for me in a normal CONFIG_XFS_DEBUG build: > > > > > > fs/xfs/xfs_da_btree.c: In function 'xfs_da_root_join': > > > fs/xfs/xfs_da_btree.c:737: warning: dereferencing 'void *' pointer > > > fs/xfs/xfs_da_btree.c:737: error: request for member 'magic' in > > > something not a structure or union > > > fs/xfs/xfs_da_btree.c:737: warning: dereferencing 'void *' pointer > > > > > > Hmm, it really compiled here (scratching head) > > Never mind, it seems SUBDIRS doesn't set the DEBUG flag correctly. > > Anyways I just dropped those hunks and you'll have to live with > the warnings if you don't want QASSERT. I'm still not happy with it. We'll commit a modified version of one of your patches into the XFS tree as you seem unable to accept that changes might be necessary to get your code to pass review. Cheers, Dave. -- Dave Chinner david@xxxxxxxxxxxxx _______________________________________________ xfs mailing list xfs@xxxxxxxxxxx http://oss.sgi.com/mailman/listinfo/xfs
