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

Process hanging when using TF_SessionRun with multiple times the same input #4084

Closed
LaurentMazare opened this issue Aug 28, 2016 · 3 comments
Assignees
Labels

Comments

@LaurentMazare
Copy link
Contributor

It seems that if the same input appears multiple times in the inputs argument of TF_SessionRun (from c_api.h) then the TF_SessionRun call never returns.
This issue can be reproduced by modifying c_api_test.cc and replacing the line:
csession.SetInputs({{feed, Int32Tensor(3)}});
With:
csession.SetInputs({{feed, Int32Tensor(3)}, {feed, Int32Tensor(3)}});
According to gdb, the process is waiting for a mutex in the RunState destructor from DirectSession.

What related GitHub issues or StackOverflow threads have you found by searching the web for your problem?

none

Environment info

Operating System: Linux 4.4

Installed version of CUDA and cuDNN: none

If installed from source, provide

  1. The commit hash (git rev-parse HEAD)
    008bcae
  2. The output of bazel version
    Build label: 0.3.1
    Build target: bazel-out/local-fastbuild/bin/src/main/java/com/google/devtools/build/lib/bazel/BazelServer_deploy.jar
    Build time: Fri Jul 29 09:09:52 2016 (1469783392)
    Build timestamp: 1469783392
    Build timestamp as int: 1469783392

If possible, provide a minimal reproducible example (We usually don't have time to read hundreds of lines of your code)

See above.

What other attempted solutions have you tried?

Logs or other output that would be helpful

(If logs are large, please upload as attachment or provide link).

@poxvoculi poxvoculi added the bug label Aug 30, 2016
@poxvoculi
Copy link
Contributor

Thanks for the report.

@mrry mrry self-assigned this Aug 31, 2016
@mrry
Copy link
Contributor

mrry commented Aug 31, 2016

In general the C API is unforgiving when it comes to usage, but this should be an easy and cheap piece of validation code to add.

@mrry
Copy link
Contributor

mrry commented Aug 31, 2016

Running tests now... it should appear after the next push.

@aselle aselle added type:bug Bug and removed bug labels Feb 9, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants