diff options
| author | Thomas Schwinge <thomas@schwinge.name> | 2011-02-11 18:15:53 +0100 |
|---|---|---|
| committer | Thomas Schwinge <thomas@schwinge.name> | 2011-02-11 18:15:53 +0100 |
| commit | bc01257fcd39df00723082906759831ac9635f0d (patch) | |
| tree | 961f244adc19c8b6e2756c261c40d220a0e718a2 /open_issues | |
| parent | 78db997f8af8a5d6bb41778483981fdd3dc7eae7 (diff) | |
| parent | 9006a0f51cb7a7bb2a22d742c7cf795e3a4f6b5e (diff) | |
Merge remote branch 'flubber/master'
Diffstat (limited to 'open_issues')
| -rw-r--r-- | open_issues/rm_fr.mdwn | 27 |
1 files changed, 27 insertions, 0 deletions
diff --git a/open_issues/rm_fr.mdwn b/open_issues/rm_fr.mdwn new file mode 100644 index 00000000..89a803ab --- /dev/null +++ b/open_issues/rm_fr.mdwn @@ -0,0 +1,27 @@ +[[!meta copyright="Copyright © 2011 Free Software Foundation, Inc."]] + +[[!meta license="""[[!toggle id="license" text="GFDL 1.2+"]][[!toggleable +id="license" text="Permission is granted to copy, distribute and/or modify this +document under the terms of the GNU Free Documentation License, Version 1.2 or +any later version published by the Free Software Foundation; with no Invariant +Sections, no Front-Cover Texts, and no Back-Cover Texts. A copy of the license +is included in the section entitled [[GNU Free Documentation +License|/fdl]]."]]"""]] + +[[!tag open_issue_hurd]] + +From: Samuel Thibault <samuel.thibault@gnu.org> +Subject: rm -fr slowness + +I have always been surprised by the slowness of a mere rm -fr. Looking a +bit inside, I see that diskfs_dirremove_hard() calls diskfs_file_update +(dp, 1) (as does diskfs_truncate, diskfs_direnter_hard, and +diskfs_dirrewrite_hard). diskfs_file_update then calls pager_sync on +the pager, which thus writes back the whole ext2fs pager! + +This sounds a bit excessive to me, an unlink could just record it in +memory and actually sync later. Also, the wait flag is set, so we +really waits for all I/Os, which basically means strictly serializing +file removals: remove one file, wait for the disk to have done it +(~10ms), remove the next one, etc. I guess this is for safety reasons +against crashes, but isn't the sync option there for such kind of |