On Fri 07 Dec 2018 03:46:13 PM CET, Anton Nefedov wrote: >>> +static void quorum_set_supported_flags(BlockDriverState *bs) >>> +{ >>> + BDRVQuorumState *s = bs->opaque; >>> + int i; >>> + >>> + bs->supported_write_flags = BDRV_REQ_FUA; >>> + for (i = 0; i < s->num_children; i++) { >>> + bs->supported_write_flags &= >>> s->children[i]->bs->supported_write_flags; >>> + } >>> + >>> + bs->supported_write_flags |= BDRV_REQ_WRITE_UNCHANGED; >>> +} >> >> You don't set supported_zero_flags here anymore ? > > Yes, I noticed (that late) that quorum doesn't actually implement > write_zeroes(). bdrv_co_do_pwrite_zeroes() specifically checks that > there must be no supported flags set in such case.
Oh, I see. Reviewed-by: Alberto Garcia <be...@igalia.com> Berto