Bug #2926
closedgetfsstat() can return the wrong mounted FS count
Added by Anonymous over 8 years ago. Updated over 6 years ago.
0%
Description
Per this discussion: http://lists.dragonflybsd.org/pipermail/users/2016-July/270749.html
Files
getfsstat.patch (548 Bytes) getfsstat.patch | Anonymous, 06/10/2018 06:02 PM |
Updated by Anonymous over 8 years ago
If someone has spare cycles to tackle this, I'd really appreciate it. I won't bore you with the details, but it's forcing me to run a builder process that I have no control over in a DF instance on a VM and I'd like to move it to a dedicated physical machine.
Updated by liweitianux over 6 years ago
The getfsstat() count problem as you reported can be seen from the different counts shown by mount(8) and lsvfs(1), and the main reason is due to the chroot from the initrd temporary root into the real root.
Please see more information in my email: http://lists.dragonflybsd.org/pipermail/users/2018-June/335759.html
Cheers,
Aaron
Updated by Anonymous over 6 years ago
I think the problem is this commit: https://gitweb.dragonflybsd.org/dragonfly.git/commit/75ffff0d6f3520361771d057b364af7f81ebb36f
It checks for mount visibility when sfsp is uaf->buf, but doesn't when sfsp is null.
Updated by tkusumi over 6 years ago
2018年6月10日(日) 0:14 <bugtracker-admin@leaf.dragonflybsd.org>:
Issue #2926 has been updated by t_dfbsd.
I think the problem is this commit: https://gitweb.dragonflybsd.org/dragonfly.git/commit/75ffff0d6f3520361771d057b364af7f81ebb36f
Not related this issue itself, but none of the filesystems added to
DragonFly after this commit (hammer1, devfs, tmpfs, hammer2, autofs,
etc) do what this commit says filesystems should do.
It's not a must anyway.
It checks for mount visibility when sfsp is uaf->buf, but doesn't when sfsp is null.
----------------------------------------
Bug #2926: getfsstat() can return the wrong mounted FS count
http://bugs.dragonflybsd.org/issues/2926#change-13439
- Author: t_dfbsd
- Status: New
- Priority: Normal
- Assignee:
- Category:
- Target version:
----------------------------------------
Per this discussion: http://lists.dragonflybsd.org/pipermail/users/2016-July/270749.html--
You have received this notification because you have either subscribed to it, or are involved in it.
To change your notification preferences, please click here: http://bugs.dragonflybsd.org/my/account
Updated by Anonymous over 6 years ago
- File getfsstat.patch getfsstat.patch added
Can someone take a look at this patch and commit of it's OK? Tested on both the normal install and an encrypted root install.
Updated by dillon over 6 years ago
- Status changed from New to Closed
Committed fix to master with 5c6a03baf163828d for both getfsstat() and getvfsstat(). Thanks!
-Matt