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

Fixes for the remote-mqpu platform #1158

Merged
merged 4 commits into from
Feb 4, 2024

Conversation

1tnguyen
Copy link
Collaborator

@1tnguyen 1tnguyen commented Feb 1, 2024

Description

(1) Clean shutdown behavior

  • Client side: enforce cleanup order: qpus first then any auto-launch server instances. This prevents some stray errors during shut down. For example, the server process is shut down before the client, which causes the client to report an error (lost connection to the server) momentarily before it being destructed.

  • Server app: make sure that we always have a loaded simulator in the runtime to prevent segfault during shutdown when the runtime would try to find one.

(2) Harden the tensornet scratch memory allocation against race condition.
Related to #1136

(1) Clean shutdown behavior

- Client side: enforce cleanup order: qpus first then any auto-launch
  server instances. This prevents some stray errors during shut down:
  the server is shut down before the client, which causes the client to
report an error just moment before it being destructed anyway.

- Server app: make sure that we always have a loaded simulator in the
  runtime to prevent segfault during shutdow when the runtime would try
to find one.

(2) Harden the tensornet scratch memory allocation against race
condition.
Copy link

github-actions bot commented Feb 2, 2024

CUDA Quantum Docs Bot: A preview of the documentation can be found here.

github-actions bot pushed a commit that referenced this pull request Feb 2, 2024
Copy link
Collaborator

@bettinaheim bettinaheim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

Copy link

github-actions bot commented Feb 2, 2024

CUDA Quantum Docs Bot: A preview of the documentation can be found here.

github-actions bot pushed a commit that referenced this pull request Feb 2, 2024
Copy link

github-actions bot commented Feb 2, 2024

CUDA Quantum Docs Bot: A preview of the documentation can be found here.

github-actions bot pushed a commit that referenced this pull request Feb 2, 2024
Copy link

github-actions bot commented Feb 4, 2024

CUDA Quantum Docs Bot: A preview of the documentation can be found here.

github-actions bot pushed a commit that referenced this pull request Feb 4, 2024
@1tnguyen 1tnguyen merged commit ac8ce9e into NVIDIA:main Feb 4, 2024
133 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Feb 4, 2024
@bettinaheim bettinaheim added this to the release 0.7.0 milestone Mar 9, 2024
@bettinaheim bettinaheim added the bug fix To be listed under Bug Fixes in the release notes label Mar 9, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug fix To be listed under Bug Fixes in the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants