On Thu, 12 Dec 2013 10:57:49 +0800 Wenchao Xia <xiaw...@linux.vnet.ibm.com> wrote:
> > >> +static int bdrv_refresh_limits(BlockDriverState *bs) > >> +{ > >> + BlockDriver *drv = bs->drv; > >> + > >> + memset(&bs->bl, 0, sizeof(bs->bl)); > >> + > >> + if (!drv) { > >> + return 0; > >> + } else if (drv->bdrv_refresh_limits) { > >> + return drv->bdrv_refresh_limits(bs); > >> + } > >> + > >> + return 0; > > It seems this line can be removed. > > > I missed the "else if", then the patch is OK. But it could also be written in a shorter way: if (drv && drv->bdrv_refresh_limits) { return drv->bdrv_refresh_limits(bs); } return 0; Regards, Thomas