On Wed, Sep 21, 2016 at 11:15:52AM +0800, Qu Wenruo wrote: > From: Lu Fengqi <lufq.f...@cn.fujitsu.com> > > Introduce a new function find_dir_item() to find DIR_ITEM for the given > key, and check it with the specified INODE_REF/INODE_EXTREF match. > > Signed-off-by: Lu Fengqi <lufq.f...@cn.fujitsu.com> > Signed-off-by: Qu Wenruo <quwen...@cn.fujitsu.com> > --- > cmds-check.c | 140 > +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ > 1 file changed, 140 insertions(+) > > diff --git a/cmds-check.c b/cmds-check.c > index 998ba63..4e25804 100644 > --- a/cmds-check.c > +++ b/cmds-check.c > @@ -3848,6 +3848,146 @@ out: > return err; > } > > +#define ROOT_DIR_ERROR (1<<1) /* bad root_dir */ > +#define DIR_ITEM_MISSING (1<<2) /* DIR_ITEM not found */ > +#define DIR_ITEM_MISMATCH (1<<3) /* DIR_ITEM found but not match */
What's the reason for another definition of the error codes? They're mostly copied from te I_ERR_* counterparts. I'd rather have one set of error codes. -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html