Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug] Diffing is extremely slow #1631

Closed
1 task done
FireNX70 opened this issue Apr 13, 2024 · 3 comments
Closed
1 task done

[Bug] Diffing is extremely slow #1631

FireNX70 opened this issue Apr 13, 2024 · 3 comments
Labels
bug Something isn't working

Comments

@FireNX70
Copy link
Contributor

Operating System

Linux

What's the issue you encountered?

On recent nightlies, diffing takes an absurd amount of time. As an example; diffing two 600MB files takes about 5.5 seconds on 1.33.2, but on the latest nightly it hadn't even made it halfway after 15 minutes.

How can the issue be reproduced?

Just diff two files. It might be slow enough to be noticeable even on small files.

ImHex Version

1.33.0 (e9ebfe3)

ImHex Build Type

  • Nightly or built from sources

Installation type

AppImage

Additional context?

This was already like this on nightlies from a couple of days ago (I think I was using 0bd8c5d), but it might have already been broken before that.

Whether the files are opened in read-only mode doesn't seem to have an effect (or if it does it's minimal).

xosview shows it's pinning one of the threads to 100%, mostly with a "SYS" workload (a.k.a. kernel-mode stuff). On 1.33.2, it shows up as a mostly user-mode load; which is what I would expect.

@FireNX70 FireNX70 added the bug Something isn't working label Apr 13, 2024
@FireNX70
Copy link
Contributor Author

Just tested it on Windows 10. It seems just as slow, but I'm not gonna let it sit for another 15 minutes.

@FireNX70
Copy link
Contributor Author

I've tracked this down to a449f7a, so technically it's a bug in libwolv and not ImHex. That commit looks completely reasonable so I'm not sure why it causes such severe slowdown.

@FireNX70
Copy link
Contributor Author

FireNX70 commented May 8, 2024

Fixed by 5bc60d4.

@FireNX70 FireNX70 closed this as completed May 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant