[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v1] hmp: 'info snapshots' not showing the id
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH v1] hmp: 'info snapshots' not showing the id |
Date: |
Fri, 22 Dec 2017 08:37:50 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.3 (gnu/linux) |
Eric Blake <address@hidden> writes:
> On 12/19/2017 08:20 AM, Max Reitz wrote:
>
>> So there are three things:
>>
>> (1) We probably should not allow snapshot names that could be IDs.
>> Easiest way to solve this: Names have to start with a non-digit.
>
> Yes, that would be a nice change. It is not strictly backwards
> compatible (so we'd still have to cope with images that didn't follow
> the rule, whether created by older qemu or by non-qemu implementations
> of qcow2), but would alleviate a lot of confusion.
I recommend to restrict ID strings to letters, digits, '-', '.', '_',
starting with a letter. Use id_wellformed() to check.
If backward compatibility is an issue, deprecate offending IDs (with a
suitable warning), and kill them off after the customary grace period.
IDs embedded in image files and such you may have to keep working
somehow indefinitely.
- Re: [Qemu-devel] [PATCH v1] hmp: 'info snapshots' not showing the id, Dr. David Alan Gilbert, 2017/12/12
- Re: [Qemu-devel] [PATCH v1] hmp: 'info snapshots' not showing the id, seeteena, 2017/12/12
- Re: [Qemu-devel] [PATCH v1] hmp: 'info snapshots' not showing the id, Max Reitz, 2017/12/15
- Re: [Qemu-devel] [PATCH v1] hmp: 'info snapshots' not showing the id, seeteena, 2017/12/18
- Re: [Qemu-devel] [PATCH v1] hmp: 'info snapshots' not showing the id, Max Reitz, 2017/12/19
- Re: [Qemu-devel] [PATCH v1] hmp: 'info snapshots' not showing the id, seeteena, 2017/12/19
- Re: [Qemu-devel] [PATCH v1] hmp: 'info snapshots' not showing the id, Dr. David Alan Gilbert, 2017/12/20
- Re: [Qemu-devel] [PATCH v1] hmp: 'info snapshots' not showing the id, Eric Blake, 2017/12/21
- Re: [Qemu-devel] [PATCH v1] hmp: 'info snapshots' not showing the id,
Markus Armbruster <=