On Wed, Sep 21, 2016 at 11:15:52AM +0800, Qu Wenruo wrote: > From: Lu Fengqi <lufq.fnst@xxxxxxxxxxxxxx> > > 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.fnst@xxxxxxxxxxxxxx> > Signed-off-by: Qu Wenruo <quwenruo@xxxxxxxxxxxxxx> > --- > 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 majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
