https://bugs.kde.org/show_bug.cgi?id=384561

--- Comment #7 from Jaime Torres <jtam...@gmail.com> ---
Git commit f7e00b40a6d35fbfe536cdb99d6f22b643676e5e by Jaime Torres.
Committed on 14/01/2018 at 12:39.
Pushed by jtamate into branch 'master'.

Don't stat(/etc/localtime) between read() and write() copying files

Summary:

Unfortunately, QDateTime::currentDateTime() checks /etc/localtime
each time it is called.
Chaning to QElapsedTime, no check of /etc/localtime.
Reproducing bug 384561, the strace of file.so was something like:
read(), stat(/etc/localtime), stat(/etc/localtime),
stat(/etc/localtime), stat(/etc/localtime), stat(/etc/localtime),
write(), read() ......
Now it is: read(), write()
It also reduces the cpu in io/wait around 10% in a debug build.

Reviewers: #frameworks, dfaure

Reviewed By: dfaure

Subscribers: broulik, ngraham, #dolphin

Tags: #frameworks

Differential Revision: https://phabricator.kde.org/D9844

M  +13   -12   src/core/slavebase.cpp

https://commits.kde.org/kio/f7e00b40a6d35fbfe536cdb99d6f22b643676e5e

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to