[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH V4 2/9] snapshot: add paired functions for inter
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-devel] [PATCH V4 2/9] snapshot: add paired functions for internal snapshot id and name |
Date: |
Wed, 10 Jul 2013 15:10:40 +0200 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
Am 08.07.2013 um 10:00 hat Wenchao Xia geschrieben:
> Internal snapshot's ID and name concept are both visible in general
> block level, they are observed by user in "info snapshots", so it is
> possible to have conflict. Although we can separate the two concept in
> programming, but if they can be distinguished in string itself, things
> will be simple and clear, so introduce two functions to do it.
>
> The implemention, qcow2 snapshot calls snapshot_id_string_generate() to
> make sure it follows the rule in driver. If caller or user give a check
> with snapshot_name_wellformed() before create snapshot, then the ID
> and name will never conflict. The check can be also taken in
> qcow2_snapshot_create(), but require it to return error reason.
I'm not sure how useful this is. While we can restrict what IDs we allow
for creating new snapshots, we cannot take any advantage from it because
existing snapshots could already be named with only digits (they could
also use a non-numeric ID). At the end of the day we're limiting the
choice of IDs that can be generated with the QMP command without a real
reason.
Kevin
- [Qemu-devel] [PATCH V4 0/9] add internal snapshot support at block device level, Wenchao Xia, 2013/07/08
- [Qemu-devel] [PATCH V4 1/9] snapshot: new function bdrv_snapshot_find_by_id_and_name(), Wenchao Xia, 2013/07/08
- [Qemu-devel] [PATCH V4 2/9] snapshot: add paired functions for internal snapshot id and name, Wenchao Xia, 2013/07/08
- Re: [Qemu-devel] [PATCH V4 2/9] snapshot: add paired functions for internal snapshot id and name,
Kevin Wolf <=
- Re: [Qemu-devel] [PATCH V4 2/9] snapshot: add paired functions for internal snapshot id and name, Wenchao Xia, 2013/07/10
- Re: [Qemu-devel] [PATCH V4 2/9] snapshot: add paired functions for internal snapshot id and name, Kevin Wolf, 2013/07/10
- Re: [Qemu-devel] [PATCH V4 2/9] snapshot: add paired functions for internal snapshot id and name, Wenchao Xia, 2013/07/10
- Re: [Qemu-devel] [PATCH V4 2/9] snapshot: add paired functions for internal snapshot id and name, Kevin Wolf, 2013/07/10
- Re: [Qemu-devel] [PATCH V4 2/9] snapshot: add paired functions for internal snapshot id and name, Wenchao Xia, 2013/07/10
[Qemu-devel] [PATCH V4 3/9] snapshot: distinguish id and name in snapshot delete, Wenchao Xia, 2013/07/08
[Qemu-devel] [PATCH V4 4/9] qmp: add internal snapshot support in qmp_transaction, Wenchao Xia, 2013/07/08
[Qemu-devel] [PATCH V4 5/9] qmp: add interface blockdev-snapshot-internal-sync, Wenchao Xia, 2013/07/08
[Qemu-devel] [PATCH V4 6/9] qmp: add interface blockdev-snapshot-delete-internal-sync, Wenchao Xia, 2013/07/08
[Qemu-devel] [PATCH V4 7/9] hmp: add interface hmp_snapshot_blkdev_internal, Wenchao Xia, 2013/07/08