[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC V6 18/33] qcow2: Extract qcow2_add_feature and qco
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [RFC V6 18/33] qcow2: Extract qcow2_add_feature and qcow2_remove_feature. |
Date: |
Thu, 7 Feb 2013 11:05:03 +0100 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Wed, Feb 06, 2013 at 01:31:51PM +0100, Benoît Canet wrote:
> diff --git a/block/qcow2.c b/block/qcow2.c
> index e48f0b0..ad202fa 100644
> --- a/block/qcow2.c
> +++ b/block/qcow2.c
> @@ -239,61 +239,72 @@ static void report_unsupported_feature(BlockDriverState
> *bs,
> }
>
> /*
> - * Sets the dirty bit and flushes afterwards if necessary.
> + * Sets the an incompatible feature bit and flushes afterwards if necessary.
> *
> * The incompatible_features bit is only set if the image file header was
> * updated successfully. Therefore it is not required to check the return
> * value of this function.
> */
> -int qcow2_mark_dirty(BlockDriverState *bs)
> +static int qcow2_add_feature(BlockDriverState *bs,
> + QCow2IncompatibleFeature feature)
qcow2_set_incompat_feature() - it only operates on
s->incompatible_features and the name should reflect that.
> {
> BDRVQcowState *s = bs->opaque;
> uint64_t val;
> - int ret;
> + int ret = 0;
>
> assert(s->qcow_version >= 3);
>
> - if (s->incompatible_features & QCOW2_INCOMPAT_DIRTY) {
> - return 0; /* already dirty */
> + if (s->incompatible_features & feature) {
> + return 0; /* already added */
> }
>
> - val = cpu_to_be64(s->incompatible_features | QCOW2_INCOMPAT_DIRTY);
> + val = cpu_to_be64(s->incompatible_features | feature);
> ret = bdrv_pwrite(bs->file, offsetof(QCowHeader, incompatible_features),
> &val, sizeof(val));
> if (ret < 0) {
> return ret;
> }
> - ret = bdrv_flush(bs->file);
> - if (ret < 0) {
> - return ret;
> - }
>
> - /* Only treat image as dirty if the header was updated successfully */
> - s->incompatible_features |= QCOW2_INCOMPAT_DIRTY;
> + /* Only treat image as having the feature if the header was updated
> + * successfully
> + */
> + s->incompatible_features |= feature;
> return 0;
> }
>
> +int qcow2_mark_dirty(BlockDriverState *bs)
> +{
> + return qcow2_add_feature(bs, QCOW2_INCOMPAT_DIRTY);
> +}
Feel free to replace callers with qcow2_set_incompat_feature(bs,
QCOW2_INCOMPAT_DIRTY) instead of adding a wrapper function.
> +static int qcow2_mark_clean(BlockDriverState *bs)
> +{
> + return qcow2_remove_feature(bs, QCOW2_INCOMPAT_DIRTY);
> +}
Same here.
- [Qemu-devel] [RFC V6 05/33] qcow2: Make update_refcount public., (continued)
- [Qemu-devel] [RFC V6 05/33] qcow2: Make update_refcount public., Benoît Canet, 2013/02/06
- [Qemu-devel] [RFC V6 11/33] qcow2: Add qcow2_dedup_grow_table and use it., Benoît Canet, 2013/02/06
- [Qemu-devel] [RFC V6 12/33] qcow2: Makes qcow2_alloc_cluster_link_l2 mark to deduplicate clusters., Benoît Canet, 2013/02/06
- [Qemu-devel] [RFC V6 17/33] qcow2-cache: Allow to choose table size at creation., Benoît Canet, 2013/02/06
- [Qemu-devel] [RFC V6 15/33] qcow2: Load and save deduplication table header extension., Benoît Canet, 2013/02/06
- [Qemu-devel] [RFC V6 18/33] qcow2: Extract qcow2_add_feature and qcow2_remove_feature., Benoît Canet, 2013/02/06
- Re: [Qemu-devel] [RFC V6 18/33] qcow2: Extract qcow2_add_feature and qcow2_remove_feature.,
Stefan Hajnoczi <=
- [Qemu-devel] [RFC V6 16/33] qcow2: Extract qcow2_do_table_init., Benoît Canet, 2013/02/06
- [Qemu-devel] [RFC V6 20/33] qcow2: Add a deduplication boolean to update_refcount., Benoît Canet, 2013/02/06
- [Qemu-devel] [RFC V6 21/33] qcow2: Drop hash for a given cluster when dedup makes refcount > 2^16/2., Benoît Canet, 2013/02/06
- [Qemu-devel] [RFC V6 22/33] qcow2: Remove hash when cluster is deleted., Benoît Canet, 2013/02/06
- [Qemu-devel] [RFC V6 19/33] block: Add qcow2_dedup format and image creation code., Benoît Canet, 2013/02/06