Review Request 117044: Avoid unnecessary automounting in KDiskFreeSpaceInfo::freeSpaceInfo

David Faure faure at kde.org
Sat Apr 12 00:21:01 BST 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/117044/#review55479
-----------------------------------------------------------

Ship it!


Looks ok. There are probably other names for automounted partition types, though... like "subfs" at least, if I believe kmountpoint.cpp.


Another solution is the one from kfileitem.cpp isDirectoryMounted() - identifying unmounted dirs by the fact that they have a size of 0.

- David Faure


On March 25, 2014, 7:17 a.m., Tomáš Trnka wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/117044/
> -----------------------------------------------------------
> 
> (Updated March 25, 2014, 7:17 a.m.)
> 
> 
> Review request for kdelibs.
> 
> 
> Repository: kdelibs
> 
> 
> Description
> -------
> 
> Avoid unnecessary automounting in KDiskFreeSpaceInfo::freeSpaceInfo
> 
> Previously, all unmounted autofs mountpoints were always mounted
> by krunner/plasma-desktop on startup, defeating the purpose of
> automounting. Let's ignore the unmounted filesystems instead when
> gathering free space stats, just like the "df" utility does.
> 
> Everything still gets mounted properly on first real access.
> 
> The change itself is pretty trivial and I would regard it as a bugfix
> (and thus eligible for 4.13), but I'm posting it for review to see
> what you kdelibs people think.
> 
> 
> Diffs
> -----
> 
>   kio/kfile/kdiskfreespaceinfo.cpp f11eb0998f0e718e9b366f8c26da30586bfa44ef 
> 
> Diff: https://git.reviewboard.kde.org/r/117044/diff/
> 
> 
> Testing
> -------
> 
> I'm using this patch on kdelibs since 4.11 and I have noted no problems
> in connection with ~4 automounted filesystems.
> 
> 
> Thanks,
> 
> Tomáš Trnka
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20140411/7106e738/attachment.htm>


More information about the kde-core-devel mailing list