mirror of
https://git.proxmox.com/git/pve-qemu
synced 2025-08-17 21:16:20 +00:00
cb662eb2c1
This series of patches extends our existing QEMU backup integration for PVE such that it can be used for a in-development external-backup plugin API. This includes among other things an API for backup access and teardown as well as dirty-bitmap management. See the separate patches for more details and check [v7] and [v8] of this series on the list. The former is the last revision by Fiona which was already very advanced, Wolfgang picked that up and extended/adapted it slightly to match needs that he found during implementation of a test provider and from feedback of (future) external backup provider like Bareos and Veritas. This was mainly done to get the QEMU build out for broad QA, we can still change things here as long as the rest of the series is not yet applied. [v7]: https://lore.proxmox.com/pve-devel/20250401173435.221892-1-f.ebner@proxmox.com/ [v8]: https://lore.proxmox.com/pve-devel/20250403123118.264974-1-w.bumiller@proxmox.com/ Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
60 lines
1.8 KiB
Diff
60 lines
1.8 KiB
Diff
From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
|
|
From: Fiona Ebner <f.ebner@proxmox.com>
|
|
Date: Thu, 3 Apr 2025 14:30:42 +0200
|
|
Subject: [PATCH] PVE backup: factor out helper to clear backup state's bitmap
|
|
list
|
|
|
|
Suggested-by: Wolfgang Bumiller <w.bumiller@proxmox.com>
|
|
Signed-off-by: Fiona Ebner <f.ebner@proxmox.com>
|
|
Reviewed-by: Wolfgang Bumiller <w.bumiller@proxmox.com>
|
|
---
|
|
pve-backup.c | 28 ++++++++++++++++++----------
|
|
1 file changed, 18 insertions(+), 10 deletions(-)
|
|
|
|
diff --git a/pve-backup.c b/pve-backup.c
|
|
index 2408f182bc..915649b5f9 100644
|
|
--- a/pve-backup.c
|
|
+++ b/pve-backup.c
|
|
@@ -811,6 +811,23 @@ err:
|
|
return di_list;
|
|
}
|
|
|
|
+/*
|
|
+ * To be called with the backup_state.stat mutex held.
|
|
+ */
|
|
+static void clear_backup_state_bitmap_list(void) {
|
|
+
|
|
+ if (backup_state.stat.bitmap_list) {
|
|
+ GList *bl = backup_state.stat.bitmap_list;
|
|
+ while (bl) {
|
|
+ g_free(((PBSBitmapInfo *)bl->data)->drive);
|
|
+ g_free(bl->data);
|
|
+ bl = g_list_next(bl);
|
|
+ }
|
|
+ g_list_free(backup_state.stat.bitmap_list);
|
|
+ backup_state.stat.bitmap_list = NULL;
|
|
+ }
|
|
+}
|
|
+
|
|
UuidInfo coroutine_fn *qmp_backup(
|
|
const char *backup_file,
|
|
const char *password,
|
|
@@ -898,16 +915,7 @@ UuidInfo coroutine_fn *qmp_backup(
|
|
backup_state.stat.reused = 0;
|
|
|
|
/* clear previous backup's bitmap_list */
|
|
- if (backup_state.stat.bitmap_list) {
|
|
- GList *bl = backup_state.stat.bitmap_list;
|
|
- while (bl) {
|
|
- g_free(((PBSBitmapInfo *)bl->data)->drive);
|
|
- g_free(bl->data);
|
|
- bl = g_list_next(bl);
|
|
- }
|
|
- g_list_free(backup_state.stat.bitmap_list);
|
|
- backup_state.stat.bitmap_list = NULL;
|
|
- }
|
|
+ clear_backup_state_bitmap_list();
|
|
|
|
if (format == BACKUP_FORMAT_PBS) {
|
|
if (!password) {
|