At Thu, 28 Aug 2014 19:02:56 +0800,
Bingpeng Zhu wrote:
> 
> If condition is never satisfied since nr_copies
> is the minimum of inode->nr_copies and sd_zones_nr.
> 
> Signed-off-by: Bingpeng Zhu <bingpeng....@alibaba-inc.com>
> ---
>  dog/vdi.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)

Applied, thanks.
Hitoshi

> 
> diff --git a/dog/vdi.c b/dog/vdi.c
> index 19da73e..8d9ee46 100644
> --- a/dog/vdi.c
> +++ b/dog/vdi.c
> @@ -2018,7 +2018,7 @@ int do_vdi_check(const struct sd_inode *inode)
>       struct work_queue *wq;
>       int nr_copies = min((int)inode->nr_copies, sd_zones_nr);
>  
> -     if (0 < inode->copy_policy && sd_zones_nr < nr_copies) {
> +     if (0 < inode->copy_policy && sd_zones_nr < (int)inode->nr_copies) {
>               sd_err("ABORT: Not enough active zones for consistency-checking"
>                      " erasure coded VDI");
>               return EXIT_FAILURE;
> -- 
> 1.7.1
> 
> 
> 
> 
> -- 
> sheepdog mailing list
> sheepdog@lists.wpkg.org
> http://lists.wpkg.org/mailman/listinfo/sheepdog
-- 
sheepdog mailing list
sheepdog@lists.wpkg.org
http://lists.wpkg.org/mailman/listinfo/sheepdog

Reply via email to