[pbs-devel] [PATCH v4 proxmox-backup 09/11] datastore: use u64 instead of usize for fidx writer content size

[Christian Ebner]

On 1/26/26 2:19 PM, Robert Obkircher wrote:
> 
> On 1/26/26 12:38, Christian Ebner wrote:
>> Not sure about these changes, maybe other devs have a stronger
>> opinion on this one.
>>
>> If we do want to adapt this, then IMHO this should however be done
>> throughout the whole codebase, for the dynamic index as well.
> The dynamic reader/writer and the FixedIndexReader already use u64 for
> content
> size and offsets. The API only supports 4 MiB chunks anyway, so it
> shouldn't matter
> if we keep using u32 there.

Disregard that comment with respect to the dynamic index in that case, 
seems indeed to be the case and I got sidetracked by the index positions 
which are usize.

But still this would be nice to have consistent, e.g. there is the check 
for chunk size used on the client side [0].

[0] 
https://git.proxmox.com/?p=proxmox-backup.git;a=blob;f=pbs-datastore/src/chunk_store.rs;h=bd1dc353b5e8f5f0e7b8f2d6788f4d29433e2a9e;hb=HEAD#l42