Re: [ext2resize] [PATCH] fix i_file_acl handling

[Petter R. <pe...@hu...>]

[Andreas Dilger]
> This patch fixes handling of the EXT2_FEATURE_COMPAT_EA feature.
> The block referenced by i_file_acl needs to be checked for remapping
> during filesystem preparation/resize.  Patch is proof of concept
> only, it has not been tested.

It fail to build with this error when I apply it to the current CVS:

  ext2_block_relocator.c: In function `doblock':
  ext2_block_relocator.c:142: warning: unused parameter `fs'
  ext2_block_relocator.c:267:74: macro "offsetof" passed 3 arguments,
    but takes just 2
  ext2_block_relocator.c: In function `doinode':
  ext2_block_relocator.c:266: error: `offsetof' undeclared (first use
    in this function)
  ext2_block_relocator.c:266: error: (Each undeclared identifier is
    reported only once
  ext2_block_relocator.c:266: error: for each function it appears in.)
  ext2_block_relocator.c:266: error: too few arguments to function `doblock'
  ext2_block_relocator.c: In function `inode_block_scan':
  ext2_block_relocator.c:327: warning: comparison between signed and unsigned
  ext2_block_relocator.c: In function `ext2_block_relocator_grab_blocks':
  ext2_block_relocator.c:524: warning: comparison between signed and unsigned
  ext2_block_relocator.c: In function `ext2_block_relocate_grow':
  ext2_block_relocator.c:688: warning: comparison between signed and unsigned
  ext2_block_relocator.c:691: warning: comparison between signed and unsigned
  ext2_block_relocator.c: In function `ext2_block_relocate_shrink':
  ext2_block_relocator.c:869: warning: comparison between signed and unsigned
  ext2_block_relocator.c:879: warning: comparison between signed and unsigned
  ext2_block_relocator.c:879: warning: comparison between signed and unsigned
  ext2_block_relocator.c:880: warning: comparison between signed and unsigned
  ext2_block_relocator.c:886: warning: comparison between signed and unsigned
  ext2_block_relocator.c:896: warning: comparison between signed and unsigned

I guess this patch need more work too. :)

Friendly,
-- 
Petter Reinholdtsen