Skip to content

Use hash func to boost file creation and lookup #79

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
+329 −217
Open

Use hash func to boost file creation and lookup #79

Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
d969bb7
Optimize the file iteration
RoyWFHuang Nov 10, 2025
d190d31
Rename variables for better readability
RoyWFHuang Nov 11, 2025
d02e46a
Use hash func to boost file creation and lookup
RoyWFHuang Nov 16, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 4 additions & 2 deletions dir.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -62,9 +62,10 @@ static int simplefs_iterate(struct file *dir, struct dir_context *ctx)
for (; remained_nr_files && ei < SIMPLEFS_MAX_EXTENTS; ei++) {
if (eblock->extents[ei].ee_start == 0)
continue;

int ei_nr = eblock->extents[ei].nr_files;
/* Iterate over blocks in one extent */
for (bi = 0; bi < eblock->extents[ei].ee_len && remained_nr_files;
for (bi = 0;
bi < eblock->extents[ei].ee_len && remained_nr_files && ei_nr;
Copy link
Contributor

@visitorckw visitorckw Nov 19, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How often does ei_nr actually hit 0 during the loop check? Is the overhead of maintaining ei_nr really justified by the benefit of terminating the loop early?

Copy link
Collaborator Author

@RoyWFHuang RoyWFHuang Nov 20, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

full_name_hash in remove file

          83278.61 msec task-clock                       #    0.967 CPUs utilized
             52431      context-switches                 #  629.585 /sec
              1309      cpu-migrations                   #   15.718 /sec
           3796501      page-faults                      #   45.588 K/sec
      199894058328      cycles                           #    2.400 GHz
      110625460371      instructions                     #    0.55  insn per cycle
       20325767251      branches                         #  244.069 M/sec
         490549944      branch-misses                    #    2.41% of all branches

      86.132655220 seconds time elapsed
      19.180209000 seconds user
      68.476075000 seconds sys

remove ei_nr:
         136413.23 msec task-clock                       #    0.962 CPUs utilized
             54993      context-switches                 #  403.135 /sec
              2504      cpu-migrations                   #   18.356 /sec
           3827233      page-faults                      #   28.056 K/sec
      323663184097      cycles                           #    2.373 GHz
      154283317313      instructions                     #    0.48  insn per cycle
       30531054743      branches                         #  223.813 M/sec
         561254005      branch-misses                    #    1.84% of all branches

     141.745463391 seconds time elapsed
      21.300616000 seconds user
     120.011093000 seconds sys

full_name_hash in list single file
min time: 0.00171 s
max time: 0.03588 s
avg time: 0.00305601 s
tot time: 93.514040 s

remove ei_nr:
total test 30600
min: 0.0017 s
max: 0.0432 s
avg: 0.00342463 s
tot: 104.793540 s

bi++) {
bh2 = sb_bread(sb, eblock->extents[ei].ee_start + bi);
if (!bh2) {
Expand All @@ -86,6 +87,7 @@ static int simplefs_iterate(struct file *dir, struct dir_context *ctx)
offset--;
} else {
remained_nr_files--;
ei_nr--;
if (!dir_emit(ctx, dblock->files[fi].filename,
SIMPLEFS_FILENAME_LEN,
dblock->files[fi].inode, DT_UNKNOWN)) {
Expand Down