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

[Jaime Torres Amate]

jtamate created this revision.
jtamate added a reviewer: Frameworks.
Restricted Application added a project: Frameworks.
jtamate requested review of this revision.

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

REPOSITORY
  R241 KIO

BRANCH
  elapsedtimer (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D9844

AFFECTED FILES
  src/core/slavebase.cpp

To: jtamate, #frameworks
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180112/68ff20e2/attachment.html>