On Thu, Mar 16, 2017 at 03:45:14PM +0530, Harish wrote: > On 03/16/2017 12:54 PM, Eryu Guan wrote: > > On Thu, Mar 16, 2017 at 10:18:50AM +0530, Harish wrote: > > Sometimes there are multiple occurences of pool_name and data_name in > dmsetup command which causes failures while adding in next line. > This patch greps for pool_name at start of line. > > Signed-off-by: Harish <harish@xxxxxxxxxxxxxxxxxx> > --- > common/dmthin | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/common/dmthin b/common/dmthin > index e774175..fe48eef 100644 > --- a/common/dmthin > +++ b/common/dmthin > @@ -180,8 +180,8 @@ _dmthin_grow() > # Get current sizes & values > local meta_dev_size=`$DMSETUP_PROG table | grep $DMTHIN_META_NAME | awk '{print $3}'` > local meta_dev_offset=`$DMSETUP_PROG table | grep $DMTHIN_META_NAME | awk '{print $6}'` > - local data_dev_size=`$DMSETUP_PROG table | grep $DMTHIN_DATA_NAME | awk '{print $3}'` > - local pool_dev_size=`$DMSETUP_PROG table | grep $DMTHIN_POOL_NAME | awk '{print $3}'` > + local data_dev_size=`$DMSETUP_PROG table | grep ^$DMTHIN_DATA_NAME | awk '{print $3}'` > + local pool_dev_size=`$DMSETUP_PROG table | grep ^$DMTHIN_POOL_NAME | awk '{print $3}'` > > Do we need to do the same to all other $DMTHIN_.*_NAME, not only > DMTHIN_DATA_NAME and DMTHIN_POOL_NAME? > > Just added the cases which failed. we may get the issue with others as well. > Can i resend adding for others? That'd be great, thanks! Eryu -- To unsubscribe from this list: send the line "unsubscribe fstests" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html