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

Use less restrictive locking in VM.start() #344

Closed
marmarek opened this Issue Mar 8, 2015 · 4 comments

Comments

Projects
None yet
2 participants
@marmarek
Member

marmarek commented Mar 8, 2015

Reported by joanna on 9 Sep 2011 14:17 UTC
Currently we're waiting until qrexec connects (or fails) and during this all time we're not allowing any other VM.start() to proceed. This is ineffective because while qrexec waits for connection, we might be already loading another VM.

Migrated-From: https://wiki.qubes-os.org/ticket/344

@marmarek

This comment has been minimized.

Show comment
Hide comment
@marmarek

marmarek Mar 8, 2015

Member

Modified by joanna on 10 Sep 2011 08:15 UTC

Member

marmarek commented Mar 8, 2015

Modified by joanna on 10 Sep 2011 08:15 UTC

@marmarek

This comment has been minimized.

Show comment
Hide comment
@marmarek

marmarek Mar 8, 2015

Member

Modified by joanna on 25 Sep 2011 10:53 UTC

Member

marmarek commented Mar 8, 2015

Modified by joanna on 25 Sep 2011 10:53 UTC

@marmarek marmarek added P: minor and removed P: major labels Mar 8, 2015

@marmarek

This comment has been minimized.

Show comment
Hide comment
@marmarek

marmarek Mar 8, 2015

Member

Comment by marmarek on 10 Oct 2011 15:30 UTC
Starting 5 domains simultaneously takes more than 30s on my hardware, which results in killing domains because of qrexec_daemon timeout.

Anyway changed to allow starting more domain simultaneously.

Member

marmarek commented Mar 8, 2015

Comment by marmarek on 10 Oct 2011 15:30 UTC
Starting 5 domains simultaneously takes more than 30s on my hardware, which results in killing domains because of qrexec_daemon timeout.

Anyway changed to allow starting more domain simultaneously.

@marmarek

This comment has been minimized.

Show comment
Hide comment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment