Re: [PATCH v6 05/10] block: rename and alter bdrv_all_find_snapshot semantics

2020-10-19 Thread Eric Blake
On 10/8/20 10:49 AM, Daniel P. Berrangé wrote: Currently bdrv_all_find_snapshot() will return 0 if it finds a snapshot, -1 if an error occurs, or if it fails to find a snapshot. New callers to be added want to distinguish between the error scenario and failing to find a snapshot. Rename it to bd

[PATCH v6 05/10] block: rename and alter bdrv_all_find_snapshot semantics

2020-10-08 Thread Daniel P . Berrangé
Currently bdrv_all_find_snapshot() will return 0 if it finds a snapshot, -1 if an error occurs, or if it fails to find a snapshot. New callers to be added want to distinguish between the error scenario and failing to find a snapshot. Rename it to bdrv_all_has_snapshot and make it return -1 on erro