[PATCH proxmox{,-backup} v3 0/7] fix sync level updates for chunk store

[Christian Ebner <c.ebner@proxmox.com>]

Currently the sync level configured for a datastore is only
propagated to the chunk store on first lookup, when the chunk
store instance is created. Updating the sync level in the tuning
options does invalidate the cached datastore entry, does however
not re-instantiate the chunk store, to avoid dropping the locks
acquired via the process locker. This means however the sync level
on the chunk store is not updated.

Fixed by storing the chunk store state inside the mutex, already
present for syncing up concurrent access to the chunk store. This
also improves the code style and fixes a few smaller issues
encountered.

Changes since version 2:
- Include additional code cleanups

Changes since version 1 (thanks @Robert for feedback!):
- Fix lost sync level update, move implementation into chunk store
  instead
- Only try ensuring the sync level if it actually changed and fix race
  window by taking the mutex guard for the check.
- Avoid Arc for chunk store mutex, it can be shared by reference
- Fixed typo


proxmox:

Christian Ebner (1):
  pbs-api-types: derive FromStr for DatastoreTuning parsing

 pbs-api-types/src/datastore.rs | 10 ++++++++++
 1 file changed, 10 insertions(+)


proxmox-backup:

Christian Ebner (6):
  datastore: GC: avoid double parsing of datastore tuning options
  pbs-config/datastore: add common helper for parsing tuning options
  datastore: restrict chunk store mutex scope to crate only
  datastore: avoid useless double borrowing of datastore
  datastore: move try_ensure_sync_level() implementation to chunk store
  datastore: fix sync level update propagation to chunk store

 pbs-config/src/datastore.rs                   | 12 ++-
 pbs-datastore/src/chunk_store.rs              | 82 ++++++++++++++-----
 pbs-datastore/src/datastore.rs                | 73 ++++++++---------
 .../src/local_datastore_lru_cache.rs          | 12 +--
 src/api2/config/datastore.rs                  | 12 +--
 5 files changed, 113 insertions(+), 78 deletions(-)


Summary over all repositories:
  6 files changed, 123 insertions(+), 78 deletions(-)

-- 
Generated by murpp 0.11.0