about summary refs log tree commit diff
path: root/tvix/store/src/fs/mod.rs
diff options
context:
space:
mode:
authorFlorian Klink <flokli@flokli.de>2023-12-16T21·10+0200
committerflokli <flokli@flokli.de>2023-12-16T21·53+0000
commit36cc7b7088190d6fb53cb5ddddc31e6b56c25450 (patch)
tree249c94396358f0f2eadcc23511ed11d98fe0966c /tvix/store/src/fs/mod.rs
parent7749814953703cc31c726d47ddf24bd32f1982c7 (diff)
docs(tvix/store/fs): fix comment r/7226
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>
Diffstat (limited to '')
-rw-r--r--tvix/store/src/fs/mod.rs12
1 files changed, 7 insertions, 5 deletions
diff --git a/tvix/store/src/fs/mod.rs b/tvix/store/src/fs/mod.rs
index fbd1cb6fc5..c11bd0a44c 100644
--- a/tvix/store/src/fs/mod.rs
+++ b/tvix/store/src/fs/mod.rs
@@ -54,13 +54,14 @@ use self::{
 /// corresponding store nodes.
 ///
 /// We internally delegate all inode allocation and state keeping to the
-/// inode tracker, and store the currently "explored" root nodes together with
-/// root inode of the root.
+/// inode tracker.
+/// 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
 /// the inode tracker, if not allocated before already:
 ///  - Processing a `lookup` request, either in the mount root, or somewhere
-///    deeper
+///    deeper.
 ///  - Processing a `readdir` request
 ///
 ///  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
 /// 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> {
     blob_service: BS,
     directory_service: DS,
@@ -81,7 +83,7 @@ pub struct TvixStoreFs<BS, DS, RN> {
     /// Whether to (try) listing elements in the root.
     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>>,
 
     /// This keeps track of inodes and data alongside them.