On Mon, Mar 13, 2017 at 10:39:46PM +0100, Max Reitz wrote: > diff --git a/block/iscsi.c b/block/iscsi.c > index ab559a6f71..5d6265c4a6 100644 > --- a/block/iscsi.c > +++ b/block/iscsi.c > @@ -2060,11 +2060,16 @@ static void iscsi_reopen_commit(BDRVReopenState > *reopen_state) > } > } > > -static int iscsi_truncate(BlockDriverState *bs, int64_t offset, Error **errp) > +static int iscsi_truncate(BlockDriverState *bs, int64_t offset, > + PreallocMode prealloc, Error **errp) > { > IscsiLun *iscsilun = bs->opaque; > Error *local_err = NULL; > > + if (prealloc != PREALLOC_MODE_OFF) { > + return -ENOTSUP; > + } > + > if (iscsilun->type != TYPE_DISK) { > return -ENOTSUP; > }
Nevermind what I said about adding a BiteSizedTasks entry: The missing errp usage is not in qemu.git/master yet. Please fix up your bdrv_truncate() errp patch to use errp in all cases, e.g. error_setg("Unable to truncate non-disk LUN"). stefan
signature.asc
Description: PGP signature