On Thu, Jun 26, 2014 at 10:18:48PM -0700, Eric Rannaud wrote:
> * This was tested on a system with a stale mount under /mnt: findmnt -s /usr
> With 2.24.2, strace shows a lstat("/mnt") (and hangs), but no lstat on active
> mount points with this change (and doesn't hang).
>
> * mountinfo is read whenever the main table is not from the kernel (tabtype !=
> TABTYPE_KERNEL), with or without -T. I can't think of a reason to only do
> this when -T is specified.
Yep, it seems fine.
> * Karel, you mentionned this would have a performance impact on large systems,
> but I do not know how to asses it. Should there an option to turn this new
> behavior on or off?
I have talked about your original idea to parse mtab always in
canonicalize_path() when someone asking for any path canonicalization. The
solution with the cache is better because we parse the file only once.
> * Should --nocanonicalize be removed after this patch?
I think it makes sense to keep it there, at least for testing we have
a way how to turn all the cache stuff off. We have the same option
for mount(8) just for cases when you can be sure that all is already
canonical.
Good work, the patches look good, I'll merge it next week. I'll try
to use it also for mnt_context_* function in libmount, maybe we can
avoid unnecessary canonicalization in mount/umount too.
Karel
--
Karel Zak <kzak@xxxxxxxxxx>
http://karelzak.blogspot.com
--
To unsubscribe from this list: send the line "unsubscribe util-linux" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
