mdebray/tvl-depot
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

docs(tvix/store/fs): fix comment

Browse source 
There's nothing store-path specific here anymore, it's just a name in
the mountpoint root.

Change-Id: I0f8004491baa03ba560d390053a42678ee81154a
Reviewed-on: https://cl.tvl.fyi/c/depot/+/10377
Autosubmit: flokli <flokli@flokli.de>
Tested-by: BuildkiteCI
Reviewed-by: raitobezarius <tvl@lahfa.xyz>
This commit is contained in:
Florian Klink 2023-12-16 23:10:50 +02:00 committed by flokli
parent 7749814953
commit 36cc7b7088
1 changed files with 7 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
tvix/store/src/fs/mod.rs
View file

@ -54,13 +54,14 @@ use self::{
/// corresponding store nodes. /// corresponding store nodes.
/// ///
/// We internally delegate all inode allocation and state keeping to the /// We internally delegate all inode allocation and state keeping to the
/// inode tracker, and store the currently "explored" root nodes together with /// inode tracker.
/// root inode of the root. /// We store a mapping from currently "explored" names in the root to their
/// inode.
/// ///
/// There's some places where inodes are allocated / data inserted into /// There's some places where inodes are allocated / data inserted into
/// the inode tracker, if not allocated before already: /// the inode tracker, if not allocated before already:
/// - Processing a `lookup` request, either in the mount root, or somewhere /// - Processing a `lookup` request, either in the mount root, or somewhere
/// deeper /// deeper.
/// - Processing a `readdir` request /// - Processing a `readdir` request
/// ///
/// Things pointing to the same contents get the same inodes, irrespective of /// Things pointing to the same contents get the same inodes, irrespective of
@ -72,7 +73,8 @@ use self::{
/// ///
/// Due to the above being valid across the whole store, and considering the /// Due to the above being valid across the whole store, and considering the
/// merkle structure is a DAG, not a tree, this also means we can't do "bucketed /// merkle structure is a DAG, not a tree, this also means we can't do "bucketed
/// allocation", aka reserve Directory.size inodes for each PathInfo. /// allocation", aka reserve Directory.size inodes for each directory node we
/// explore.
pub struct TvixStoreFs<BS, DS, RN> { pub struct TvixStoreFs<BS, DS, RN> {
blob_service: BS, blob_service: BS,
directory_service: DS, directory_service: DS,
@ -81,7 +83,7 @@ pub struct TvixStoreFs<BS, DS, RN> {
/// Whether to (try) listing elements in the root. /// Whether to (try) listing elements in the root.
list_root: bool, list_root: bool,
/// This maps a given StorePath to the inode we allocated for the root inode. /// This maps a given basename in the root to the inode we allocated for the node.
root_nodes: RwLock<HashMap<Vec<u8>, u64>>, root_nodes: RwLock<HashMap<Vec<u8>, u64>>,
/// This keeps track of inodes and data alongside them. /// This keeps track of inodes and data alongside them.