On May 29, 2008 17:15 -0700, Mark Fasheh wrote: > calls. It would take as arguments, the fiemap_extent_info struct and set of > flags which the fs supports. That would allow backported modules to plug in > their own check, but we retain the niceness of having the checks in a > central place for in-tree file systems. > > int fiemap_check_flags(struct fiemap_extent_info fieinfo, u32 fs_flags) > { > u32 incompat_flags; > > incompat_flags = fieinfo.fi_flags & ~(FIEMAP_FLAGS_COMPAT & fs_flags); > if (incompat_flags) { > fieinfo->fi_flags = incompat_flags; > return -EBADR; > } > return 0; > } I'm OK with this approach. Cheers, Andreas -- Andreas Dilger Sr. Staff Engineer, Lustre Group Sun Microsystems of Canada, Inc. -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html