Downgraded performance in latest snapshots due to issues with Kaspersky #1337

Open
wk-952 opened this Issue Dec 1, 2016 · 7 comments

Projects

None yet

3 participants

@wk-952
wk-952 commented Dec 1, 2016

Recently, there have been a noticeable performance downgrade when performing debugging actions.
For example, press and hold F8 (Step over) you'll notice a consistent latency after 4-6 instructions,
i.e. 4-6 instructions are stepped over, then a delay will occur

Another example in which this latency is noticeable, select the instruction below the one you're currently at, then press F4 (Run until selection) and notice the small delay despite the fact that both instructions are just below each other

I don't know when has this started to happen, but you can compare the performance of the latest snapshot vs. this one snapshot_2016-10-31_23-29 and you'll definitely notice the difference.

@wk-952
wk-952 commented Dec 2, 2016 edited

The Latest snapshot whose performance is natural/fast: snapshot_2016-11-18_18-21
commithash.txt: b5b7631

The first snapshot whose performance is affected/slow: snapshot_2016-11-20_22-01
commithash.txt: 6b1c60d

@mrexodia
Member
mrexodia commented Dec 2, 2016

Hm the commits you mention don't seem to correlate with any changes related to GUI performance. I will check it some more though. What kind of numbers do you get for events/s? Mine is between 24 and 30 consistently.

@wk-952
wk-952 commented Dec 2, 2016

I didn't actually mean that those specifically are the faulty commits, just stating what's inside commithash.txt in case that helps to determine which commits (between both) could be the cause.

Anyway it's the same reason as #516 😞 , the difference here is that kaspersky doesn't have to be disabled during OS startup to solve the issue.
When it's turned off: events/s = 20-30 (both snapshots).
When it's turned on: events/s = 19-28 on snapshot_2016-11-18_18-21,
and 9-12 on snapshot_2016-11-20_22-01 (and the new ones).

I never noticed this before because i was using builds from October most of the time, which (as every other builds before them) don't have this problem, it only became obvious when i started using the latest snapshots consistently a couple of days ago.

Very sorry about this misleading/useless issue.

@wk-952 wk-952 closed this Dec 2, 2016
@mrexodia
Member
mrexodia commented Dec 2, 2016

@wk-952 yeah the commit has is definitely very useful! I checked the commits in between briefly but I will check again. As for Kaspersky, I have no idea what I can do about it. If I find the time I will profile but it's rather mystical. Could you also try with GleeBug? Perhaps the speed improvement in that engine would make this stuttering less noticeable.

I will try to find what could have caused the issue since there appears to be something in the code that triggers the issue, so I reopen the issue. If you find any other information that's very helpful. Thanks!

@mrexodia mrexodia reopened this Dec 2, 2016
@Nukem9
Member
Nukem9 commented Dec 2, 2016

Sorry, I have to say this:

This issue is 1337.

@wk-952
wk-952 commented Dec 2, 2016 edited

With GleeBug (stepping-over x32dbg.exe):

  • kaspersky off;
    events/s = 22-30 on both snapshots (and latest ones).

  • kaspresky on:
    events/s = 19-25 on both snapshots
    Definitely better than 9 events/s.

@mrexodia
Member
mrexodia commented Dec 3, 2016

Well I guess GleeBug will be the next goal then 😄

@torusrxxx torusrxxx changed the title from Downgraded performance in latest snapshots to Downgraded performance in latest snapshots due to issues with Kaspersky Dec 9, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment