[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH V2 5/8] block/qcow2: read and write the compress
From: |
Daniel P. Berrange |
Subject: |
Re: [Qemu-devel] [PATCH V2 5/8] block/qcow2: read and write the compress format extension |
Date: |
Thu, 13 Jul 2017 15:58:01 +0100 |
User-agent: |
Mutt/1.8.3 (2017-05-23) |
On Thu, Jul 13, 2017 at 04:18:13PM +0200, Peter Lieven wrote:
>
> Okay, so it has to be a mix of QAPI parsing and manual parameter checking,
> right?
Yeah. It does feel like a valid RFE for QAPI to add a permitted range to
'int' types though, which would simplify the code in future.
> I currently have the following:
>
> options = qemu_opts_to_qdict(opts, NULL);
> qdict_extract_subqdict(options, &compressopts, "compress.");
> v = qobject_input_visitor_new_keyval(QOBJECT(compressopts));
> visit_start_struct(v, NULL, NULL, 0, &local_err);
> if (local_err) {
> ret= -EINVAL;
> goto finish;
> }
> visit_type_Qcow2Compress_members(v, &compress, &local_err);
> if (local_err) {
> ret= -EINVAL;
> goto finish;
> }
> visit_end_struct(v, NULL);
> visit_free(v);
> QDECREF(compressopts);
> QDECREF(options)
Looks good.
> And I have the following 2 questions:
> a) I have to specifiy compress.format and compress.level otherwise I will
> get an error. How can I fix that the settings are optional?
Put an '*' as the first character of any field name if it should be optional.
> b) If I just specify a compress.format can I default the compress.level to 0
> without an error?
I believe you'd get compress.level as 0 automatically for an 'int' type.
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
- Re: [Qemu-devel] [PATCH V2 5/8] block/qcow2: read and write the compress format extension, (continued)
- Re: [Qemu-devel] [PATCH V2 5/8] block/qcow2: read and write the compress format extension, Daniel P. Berrange, 2017/07/10
- Re: [Qemu-devel] [PATCH V2 5/8] block/qcow2: read and write the compress format extension, Kevin Wolf, 2017/07/10
- Re: [Qemu-devel] [PATCH V2 5/8] block/qcow2: read and write the compress format extension, Daniel P. Berrange, 2017/07/10
- Re: [Qemu-devel] [PATCH V2 5/8] block/qcow2: read and write the compress format extension, Peter Lieven, 2017/07/13
- Re: [Qemu-devel] [PATCH V2 5/8] block/qcow2: read and write the compress format extension, Daniel P. Berrange, 2017/07/13
- Re: [Qemu-devel] [PATCH V2 5/8] block/qcow2: read and write the compress format extension, Peter Lieven, 2017/07/13
- Re: [Qemu-devel] [PATCH V2 5/8] block/qcow2: read and write the compress format extension, Daniel P. Berrange, 2017/07/13
- Re: [Qemu-devel] [PATCH V2 5/8] block/qcow2: read and write the compress format extension, Peter Lieven, 2017/07/13
- Re: [Qemu-devel] [PATCH V2 5/8] block/qcow2: read and write the compress format extension, Daniel P. Berrange, 2017/07/13
- Re: [Qemu-devel] [PATCH V2 5/8] block/qcow2: read and write the compress format extension, Peter Lieven, 2017/07/13
- Re: [Qemu-devel] [PATCH V2 5/8] block/qcow2: read and write the compress format extension,
Daniel P. Berrange <=
- Re: [Qemu-devel] [PATCH V2 5/8] block/qcow2: read and write the compress format extension, Peter Lieven, 2017/07/13
- Re: [Qemu-devel] [PATCH V2 5/8] block/qcow2: read and write the compress format extension, Daniel P. Berrange, 2017/07/13
- Re: [Qemu-devel] [PATCH V2 5/8] block/qcow2: read and write the compress format extension, Peter Lieven, 2017/07/13
- Re: [Qemu-devel] [PATCH V2 5/8] block/qcow2: read and write the compress format extension, Daniel P. Berrange, 2017/07/13
- Re: [Qemu-devel] [PATCH V2 5/8] block/qcow2: read and write the compress format extension, Peter Lieven, 2017/07/13
- Re: [Qemu-devel] [PATCH V2 5/8] block/qcow2: read and write the compress format extension, Daniel P. Berrange, 2017/07/13
- Re: [Qemu-devel] [PATCH V2 5/8] block/qcow2: read and write the compress format extension, Peter Lieven, 2017/07/13
- Re: [Qemu-devel] [PATCH V2 5/8] block/qcow2: read and write the compress format extension, Eric Blake, 2017/07/13