On Wed, 2019-08-28 at 10:41 +0100, Stefan Hajnoczi wrote: > On Fri, Aug 23, 2019 at 03:49:23PM -0400, Dmitry Fomichev wrote: > > Dmitry Fomichev (4): > > block: Add zoned device model property > > raw: Recognize zoned backing devices > > block/ide/scsi: Set BLK_PERM_SUPPORT_ZONED > > raw: Don't open ZBDs if backend can't handle them > > The overall approach looks good. > > I wonder if bdrv_get_zoned_info() is really needed since zone_model is > part of BlockLimits and is already fetched via bdrv_refresh_limits(). > Was it introduced because the block limits haven't been or cannot be > fetched when zone_model is queried the first time? It would be nice to > get rid of bdrv_get_zoned_info() if possible.
Stefan, Thank you for your review. I've just sent out the new version of the patchset, v6, that addresses you comments. I was able to get rid of bdrv_get_zoned_info() template function as a part of the latest changes. Best regards, Dmitry
