[pbs-devel] [PATCH proxmox-backup] datastore: remove unsafe from function
Gabriel Goller
g.goller at proxmox.com
Thu Jul 4 13:05:14 CEST 2024
On 04.07.2024 11:59, Christian Ebner wrote:
>On 7/3/24 16:58, Gabriel Goller wrote:
>>`list_snapshots_blocking` doesn't need to be unsafe.
>
>I think that this is intentionally marked as `unsafe` to make sure the
>caller really does look at the possible consequences of calling this
>method. As documented, "This must not run in a main worker thread as
>it potentially does tons of I/O."
>
>So rather than dropping the `unsafe`, maybe clarify that this is the
>reason for it being declared as unsafe in the comment.
>
>But I guess Wolfgang can comment more on this.
>
Oh, well, didn't know that. Will add a comment :)
More information about the pbs-devel
mailing list