Project

General

Profile

Actions

Bug #2926

closed

getfsstat() can return the wrong mounted FS count

Added by Anonymous over 7 years ago. Updated almost 6 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
-
Target version:
-
Start date:
07/19/2016
Due date:
% Done:

0%

Estimated time:


Files

getfsstat.patch (548 Bytes) getfsstat.patch Anonymous, 06/10/2018 06:02 PM
Actions #1

Updated by Anonymous over 7 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.

Actions #2

Updated by liweitianux almost 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

Actions #3

Updated by Anonymous almost 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.

Actions #4

Updated by tkusumi almost 6 years ago

2018年6月10日(日) 0:14 <>:

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

--
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

Actions #5

Updated by Anonymous almost 6 years ago

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.

Actions #6

Updated by dillon almost 6 years ago

  • Status changed from New to Closed

Committed fix to master with 5c6a03baf163828d for both getfsstat() and getvfsstat(). Thanks!

-Matt

Actions #7

Updated by Anonymous almost 6 years ago

Tested and it works, thanks!

Actions

Also available in: Atom PDF