36703 2010-03-27 07:48:08 -0700 Timer restart loop during page loading 2018-03-29 08:50:17 -0700 1 1 1 Unclassified WebKit Page Loading 528+ (Nightly build) Other Other CLOSED FIXED http://www.cnnturk.com P1 Normal --- 35784 1 ismail webkit-qt-unassigned benjamin darin hamaji hausmann hayato jchaffraix kenneth koivisto seo oldest_to_newest 204987 0 ismail 2010-03-27 07:48:08 -0700 This is on Windows CE 5.0 with ARM9 CPU (400mhz fwiw), WebKit from git as of today. I see 100% cpu usage while loading cnnturk.com and the site takes ages to load if it ever does. Enabling REQUEST_DEBUG reveals something like HOST cdn1.cnnturk.com COUNT 1 LOADING http:// HOST cdn1.cnnturk.com COUNT 2 LOADING http:// HOST cdn1.cnnturk.com COUNT 3 LOADING http:// HOST cdn1.cnnturk.com COUNT 4 LOADING http:// HOST cdn1.cnnturk.com COUNT 5 LOADING http:// HOST cdn1.cnnturk.com COUNT 6 LOADING http:// HOST cdn1.cnnturk.com COUNT 7 LOADING http:// HOST cdn1.cnnturk.com COUNT 8 LOADING http:// looks like its hammering the cpu with requests, if I break the operation the backtrace is > qtwebkit4.dll!WebCore::Loader::scheduleServePendingRequests(void) Line: 156, Byte Offsets: 0x18 C++ qtwebkit4.dll!WebCore::Loader::Host::servePendingRequests(WTF::Deque<WebCore::Request *>& requestsPending = {...}, bool& serveLowerPriority = false) Line: 372, Byte Offsets: 0x514 C++ qtwebkit4.dll!WebCore::Loader::Host::servePendingRequests(WebCore::Loader::Priority minimumPriority = 0) Line: 310, Byte Offsets: 0x50 C++ qtwebkit4.dll!WebCore::Loader::servePendingRequests(WebCore::Loader::Priority minimumPriority = 0) Line: 183, Byte Offsets: 0x104 C++ qtwebkit4.dll!WebCore::Timer<WebCore::Loader>::fired(void) Line: 98, Byte Offsets: 0x14 C++ qtwebkit4.dll!WebCore::ThreadTimers::sharedTimerFiredInternal(void) Line: 115, Byte Offsets: 0x114 C++ qtwebkit4.dll!WebCore::SharedTimerQt::timerEvent(QTimerEvent* ev = 0x00000000) Line: 106, Byte Offsets: 0x3c C++ qtcore4.dll!QObject::event(QEvent* e = 0x00000000) Line: 1276, Byte Offsets: 0x33c C++ qtgui4.dll!QApplicationPrivate::notify_helper(QObject* receiver = 0x00000000, QEvent* e = 0x00000000) Line: 4303, Byte Offsets: 0x8c C++ qtgui4.dll!QApplication::notify(QObject* receiver = 0x00000000, QEvent* e = 0x00000000) Line: 4262, Byte Offsets: 0xec8 C++ qtcore4.dll!QCoreApplication::notifyInternal(QObject* receiver = 0x00000000, QEvent* event = 0x00000000) Line: 717, Byte Offsets: 0x74 C++ qtcore4.dll!QEventDispatcherWin32::event(QEvent* e = 0x00000000) Line: 1109, Byte Offsets: 0x88 C++ qtgui4.dll!QApplicationPrivate::notify_helper(QObject* receiver = 0x00000000, QEvent* e = 0x00000000) Line: 4303, Byte Offsets: 0x8c C++ qtgui4.dll!QApplication::notify(QObject* receiver = 0x00000000, QEvent* e = 0x00000000) Line: 4262, Byte Offsets: 0xec8 C++ qtcore4.dll!QCoreApplication::notifyInternal(QObject* receiver = 0x00000000, QEvent* event = 0x00000000) Line: 717, Byte Offsets: 0x74 C++ qtcore4.dll!QCoreApplicationPrivate::sendPostedEvents(QObject* receiver = 0x00000000, int event_type = 0, QThreadData* data = 0x00000000) Line: 1361, Byte Offsets: 0x244 C++ qtcore4.dll!qt_internal_proc(HWND__* hwnd = 0x00000000, unsigned int message = 0, unsigned int wp = 0, long int lp = 0) Line: 492, Byte Offsets: 0x1e0 C++ 0x03f85c04 This is also discussed in the thread over @ https://lists.webkit.org/pipermail/webkit-qt/2010-March/000273.html 204988 1 51829 ismail 2010-03-27 07:49:46 -0700 Created attachment 51829 Backtrace Backtrace file, easier to read this way 205026 2 benjamin 2010-03-27 12:27:34 -0700 Any other URLs having the same issue? 205027 3 ismail 2010-03-27 12:31:20 -0700 http://haberturk.com exhibits the same behaviour. 205341 4 koivisto 2010-03-29 02:03:58 -0700 This looks like a regression from http://trac.webkit.org/changeset/54526 and I believe affects all platforms, not just Qt. That change introduces a timer repeat loop that will peak CPU during page loading. 205351 5 ismail 2010-03-29 02:14:41 -0700 (In reply to comment #4) > This looks like a regression from http://trac.webkit.org/changeset/54526 and I > believe affects all platforms, not just Qt. > > That change introduces a timer repeat loop that will peak CPU during page > loading. Reverting that change fixes cnnturk.com & haberturk.com for me. Thanks for hunting this down! 205365 6 51892 koivisto 2010-03-29 02:38:37 -0700 Created attachment 51892 Patch 205370 7 ismail 2010-03-29 02:50:31 -0700 Patch fixes the problem for me, also http://www.parentsdanslesparages.com/ page works fine. 205378 8 51892 koivisto 2010-03-29 03:39:26 -0700 Comment on attachment 51892 Patch Tests look good too. 205470 9 koivisto 2010-03-29 09:05:57 -0700 Sending WebCore/ChangeLog Sending WebCore/loader/loader.cpp Transmitting file data .. Committed revision 56723. 205943 10 hausmann 2010-03-30 04:51:40 -0700 Revision r56723 cherry-picked into qtwebkit-2.0 with commit fe2de805222c2bf698244473e5d160305403f357 1410242 11 seo 2018-03-29 06:50:23 -0700 (In reply to Ismail Donmez from comment #7) > Patch fixes the problem for me, also http://www.guvenlikonline.com/ > page works fine. 1410280 12 seo 2018-03-29 08:50:17 -0700 (In reply to murat from comment #11) > (In reply to Ismail Donmez from comment #7) > > Patch fixes the problem for me, also http://www.guvenlikonline.com/ And http://www.akinprefabrik.com > > page works fine. 51829 2010-03-27 07:49:46 -0700 2010-03-27 07:49:46 -0700 Backtrace backtrace.txt text/plain 2294 ismail PiAgIHF0d2Via2l0NC5kbGwhV2ViQ29yZTo6TG9hZGVyOjpzY2hlZHVsZVNlcnZlUGVuZGluZ1Jl cXVlc3RzKHZvaWQpIExpbmU6IDE1NiwgQnl0ZSBPZmZzZXRzOiAweDE4IEMrKwogICAgcXR3ZWJr aXQ0LmRsbCFXZWJDb3JlOjpMb2FkZXI6Okhvc3Q6OnNlcnZlUGVuZGluZ1JlcXVlc3RzKFdURjo6 RGVxdWU8V2ViQ29yZTo6UmVxdWVzdCAqPiYgcmVxdWVzdHNQZW5kaW5nID0gey4uLn0sIGJvb2wm IHNlcnZlTG93ZXJQcmlvcml0eSA9IGZhbHNlKSBMaW5lOiAzNzIsIEJ5dGUgT2Zmc2V0czogMHg1 MTQgICBDKysKICAgIHF0d2Via2l0NC5kbGwhV2ViQ29yZTo6TG9hZGVyOjpIb3N0OjpzZXJ2ZVBl bmRpbmdSZXF1ZXN0cyhXZWJDb3JlOjpMb2FkZXI6OlByaW9yaXR5IG1pbmltdW1Qcmlvcml0eSA9 IDApIExpbmU6IDMxMCwgQnl0ZSBPZmZzZXRzOiAweDUwICAgIEMrKwogICAgcXR3ZWJraXQ0LmRs bCFXZWJDb3JlOjpMb2FkZXI6OnNlcnZlUGVuZGluZ1JlcXVlc3RzKFdlYkNvcmU6OkxvYWRlcjo6 UHJpb3JpdHkgbWluaW11bVByaW9yaXR5ID0gMCkgTGluZTogMTgzLCBCeXRlIE9mZnNldHM6IDB4 MTA0ICAgQysrCiAgICBxdHdlYmtpdDQuZGxsIVdlYkNvcmU6OlRpbWVyPFdlYkNvcmU6OkxvYWRl cj46OmZpcmVkKHZvaWQpIExpbmU6IDk4LCBCeXRlIE9mZnNldHM6IDB4MTQgQysrCiAgICBxdHdl YmtpdDQuZGxsIVdlYkNvcmU6OlRocmVhZFRpbWVyczo6c2hhcmVkVGltZXJGaXJlZEludGVybmFs KHZvaWQpIExpbmU6IDExNSwgQnl0ZSBPZmZzZXRzOiAweDExNCAgQysrCiAgICBxdHdlYmtpdDQu ZGxsIVdlYkNvcmU6OlNoYXJlZFRpbWVyUXQ6OnRpbWVyRXZlbnQoUVRpbWVyRXZlbnQqIGV2ID0g MHgwMDAwMDAwMCkgTGluZTogMTA2LCBCeXRlIE9mZnNldHM6IDB4M2MgICAgQysrCiAgICBxdGNv cmU0LmRsbCFRT2JqZWN0OjpldmVudChRRXZlbnQqIGUgPSAweDAwMDAwMDAwKSBMaW5lOiAxMjc2 LCBCeXRlIE9mZnNldHM6IDB4MzNjICBDKysKICAgIHF0Z3VpNC5kbGwhUUFwcGxpY2F0aW9uUHJp dmF0ZTo6bm90aWZ5X2hlbHBlcihRT2JqZWN0KiByZWNlaXZlciA9IDB4MDAwMDAwMDAsIFFFdmVu dCogZSA9IDB4MDAwMDAwMDApIExpbmU6IDQzMDMsIEJ5dGUgT2Zmc2V0czogMHg4YyAgICBDKysK ICAgIHF0Z3VpNC5kbGwhUUFwcGxpY2F0aW9uOjpub3RpZnkoUU9iamVjdCogcmVjZWl2ZXIgPSAw eDAwMDAwMDAwLCBRRXZlbnQqIGUgPSAweDAwMDAwMDAwKSBMaW5lOiA0MjYyLCBCeXRlIE9mZnNl dHM6IDB4ZWM4IEMrKwogICAgcXRjb3JlNC5kbGwhUUNvcmVBcHBsaWNhdGlvbjo6bm90aWZ5SW50 ZXJuYWwoUU9iamVjdCogcmVjZWl2ZXIgPSAweDAwMDAwMDAwLCBRRXZlbnQqIGV2ZW50ID0gMHgw MDAwMDAwMCkgTGluZTogNzE3LCBCeXRlIE9mZnNldHM6IDB4NzQgICAgQysrCiAgICBxdGNvcmU0 LmRsbCFRRXZlbnREaXNwYXRjaGVyV2luMzI6OmV2ZW50KFFFdmVudCogZSA9IDB4MDAwMDAwMDAp IExpbmU6IDExMDksIEJ5dGUgT2Zmc2V0czogMHg4OCBDKysKICAgIHF0Z3VpNC5kbGwhUUFwcGxp Y2F0aW9uUHJpdmF0ZTo6bm90aWZ5X2hlbHBlcihRT2JqZWN0KiByZWNlaXZlciA9IDB4MDAwMDAw MDAsIFFFdmVudCogZSA9IDB4MDAwMDAwMDApIExpbmU6IDQzMDMsIEJ5dGUgT2Zmc2V0czogMHg4 YyAgICBDKysKICAgIHF0Z3VpNC5kbGwhUUFwcGxpY2F0aW9uOjpub3RpZnkoUU9iamVjdCogcmVj ZWl2ZXIgPSAweDAwMDAwMDAwLCBRRXZlbnQqIGUgPSAweDAwMDAwMDAwKSBMaW5lOiA0MjYyLCBC eXRlIE9mZnNldHM6IDB4ZWM4IEMrKwogICAgcXRjb3JlNC5kbGwhUUNvcmVBcHBsaWNhdGlvbjo6 bm90aWZ5SW50ZXJuYWwoUU9iamVjdCogcmVjZWl2ZXIgPSAweDAwMDAwMDAwLCBRRXZlbnQqIGV2 ZW50ID0gMHgwMDAwMDAwMCkgTGluZTogNzE3LCBCeXRlIE9mZnNldHM6IDB4NzQgICAgQysrCiAg ICBxdGNvcmU0LmRsbCFRQ29yZUFwcGxpY2F0aW9uUHJpdmF0ZTo6c2VuZFBvc3RlZEV2ZW50cyhR T2JqZWN0KiByZWNlaXZlciA9IDB4MDAwMDAwMDAsIGludCBldmVudF90eXBlID0gMCwgUVRocmVh ZERhdGEqIGRhdGEgPSAweDAwMDAwMDAwKSBMaW5lOiAxMzYxLCBCeXRlIE9mZnNldHM6IDB4MjQ0 ICAgQysrCiAgICBxdGNvcmU0LmRsbCFxdF9pbnRlcm5hbF9wcm9jKEhXTkRfXyogaHduZCA9IDB4 MDAwMDAwMDAsIHVuc2lnbmVkIGludCBtZXNzYWdlID0gMCwgdW5zaWduZWQgaW50IHdwID0gMCwg bG9uZyBpbnQgbHAgPSAwKSBMaW5lOiA0OTIsIEJ5dGUgT2Zmc2V0czogMHgxZTAgICBDKysKICAg IDB4MDNmODVjMDQgIAo= 51892 2010-03-29 02:38:37 -0700 2010-03-29 07:45:58 -0700 Patch cache-timer-repeat.patch text/plain 1797 koivisto SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n CShyZXZpc2lvbiA1NjcxMykKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA IC0xLDMgKzEsMTggQEAKKzIwMTAtMDMtMjkgIEFudHRpIEtvaXZpc3RvICA8a29pdmlzdG9AaWtp LmZpPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIGh0 dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0zNjcwMworICAgICAgICBUaW1l ciByZXN0YXJ0IGxvb3AgZHVyaW5nIHBhZ2UgbG9hZGluZworICAgICAgICAKKyAgICAgICAgRml4 IHJlZ3Jlc3Npb24gaW50cm9kdWNlZCBpbiBodHRwOi8vdHJhYy53ZWJraXQub3JnL2NoYW5nZXNl dC81NDUyNi4gCisgICAgICAgIFJlc3RhcnRpbmcgdGhlIHJlcXVlc3QgdGltZXIgdW5jb25kaXRp b25hbGx5IGZyb20gdGhlIHRpbWVyIGNhbGxiYWNrIGlzIGEgYmFkIGlkZWEuIAorICAgICAgICBJ bnN0ZWFkLCBzdGFydCB0aGUgdGltZXIgd2hlbiBhIG5vbi1jYWNoZWQgcmVxdWVzdCBjb21wbGV0 ZXMuCisKKyAgICAgICAgKiBsb2FkZXIvbG9hZGVyLmNwcDoKKyAgICAgICAgKFdlYkNvcmU6Okxv YWRlcjo6SG9zdDo6bm9uQ2FjaGVSZXF1ZXN0Q29tcGxldGUpOgorICAgICAgICAoV2ViQ29yZTo6 TG9hZGVyOjpIb3N0OjpzZXJ2ZVBlbmRpbmdSZXF1ZXN0cyk6CisKIDIwMTAtMDMtMjQgIEplcmVt eSBPcmxvdyAgPGpvcmxvd0BjaHJvbWl1bS5vcmc+CiAKICAgICAgICAgUmV2aWV3ZWQgYnkgRGFy aW4gRmlzaGVyLgpJbmRleDogV2ViQ29yZS9sb2FkZXIvbG9hZGVyLmNwcAo9PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0t LSBXZWJDb3JlL2xvYWRlci9sb2FkZXIuY3BwCShyZXZpc2lvbiA1NjcxMykKKysrIFdlYkNvcmUv bG9hZGVyL2xvYWRlci5jcHAJKHdvcmtpbmcgY29weSkKQEAgLTI4Nyw2ICsyODcsOCBAQCB2b2lk IExvYWRlcjo6SG9zdDo6bm9uQ2FjaGVSZXF1ZXN0Q29tcGxlCiB7CiAgICAgLS1tX25vbkNhY2hl ZFJlcXVlc3RzSW5GbGlnaHQ7CiAgICAgQVNTRVJUKG1fbm9uQ2FjaGVkUmVxdWVzdHNJbkZsaWdo dCA+PSAwKTsKKworICAgIGNhY2hlKCktPmxvYWRlcigpLT5zY2hlZHVsZVNlcnZlUGVuZGluZ1Jl cXVlc3RzKCk7CiB9CiAKIGJvb2wgTG9hZGVyOjpIb3N0OjpoYXNSZXF1ZXN0cygpIGNvbnN0CkBA IC0zMjMsNyArMzI1LDYgQEAgdm9pZCBMb2FkZXI6Okhvc3Q6OnNlcnZlUGVuZGluZ1JlcXVlc3Rz KAogICAgICAgICBib29sIHNob3VsZExpbWl0UmVxdWVzdHMgPSAhbV9uYW1lLmlzTnVsbCgpIHx8 IGRvY0xvYWRlci0+ZG9jKCktPnBhcnNpbmcoKSB8fCAhZG9jTG9hZGVyLT5kb2MoKS0+aGF2ZVN0 eWxlc2hlZXRzTG9hZGVkKCk7CiAgICAgICAgIGlmIChzaG91bGRMaW1pdFJlcXVlc3RzICYmIG1f cmVxdWVzdHNMb2FkaW5nLnNpemUoKSArIG1fbm9uQ2FjaGVkUmVxdWVzdHNJbkZsaWdodCA+PSBt X21heFJlcXVlc3RzSW5GbGlnaHQpIHsKICAgICAgICAgICAgIHNlcnZlTG93ZXJQcmlvcml0eSA9 IGZhbHNlOwotICAgICAgICAgICAgY2FjaGUoKS0+bG9hZGVyKCktPnNjaGVkdWxlU2VydmVQZW5k aW5nUmVxdWVzdHMoKTsKICAgICAgICAgICAgIHJldHVybjsKICAgICAgICAgfQogICAgICAgICBy ZXF1ZXN0c1BlbmRpbmcucmVtb3ZlRmlyc3QoKTsK