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

TF Dynamic kernel design. #2

Merged
merged 2 commits into from
Jul 27, 2018
Merged

TF Dynamic kernel design. #2

merged 2 commits into from
Jul 27, 2018

Conversation

gunan
Copy link
Contributor

@gunan gunan commented Jun 5, 2018

Please leave comments to this PR. Comment period will be open until 2018-06-19

cc @ewilderj @martinwicke

Dynamic Loading of Kernels in TensorFlow

Status Proposed
Author(s) Gunhan Gulsoy (Google)
Sponsor Martin Wicke (Google)
Updated 2018-06-04

Objective

This document describes a new way to create and deploy new kernels for
TensorFlow. We propose deploying kernels in separate shared libraries (dso,
dylib or dll) and loading these at runtime. While at the moment the scope of
this document only covers TensorFlow Python distribution, we aim to
generalize this approach for all TF distributions. With this mechanism, we
would like to create the following capabilities:

  • Loading kernels dynamically at runtime from shared libraries.
  • Being able to load multiple kernels for the same op/device pair, and pick the
    best one in terms of hardware compatibility and performance.
  • Check the hardware and load the compatible kernels.
  • Check compiler options used and load the compatible kernels.

@ewilderj
Copy link
Contributor

@gunan if this passed Google internal design review, then we can merge it into rfcs/ - what's the current status of the design?

@martinwicke
Copy link
Member

Yeah, should be merged, not closed, I think.

@ewilderj
Copy link
Contributor

@gunan can you reopen this?

@gunan
Copy link
Contributor Author

gunan commented Jul 13, 2018

Ah, sorry looks like an update in my copy of the repository caused this. Did not realize I closed this.
I will try to see how I can reopen this.

@ewilderj ewilderj added RFC: Accepted RFC Design Document: Accepted by Review and removed RFC: Proposed RFC Design Document labels Jul 27, 2018
@ewilderj ewilderj merged commit c0ee5c2 into tensorflow:master Jul 27, 2018
theadactyl pushed a commit that referenced this pull request Oct 3, 2019
theadactyl pushed a commit that referenced this pull request Nov 26, 2019
Relax API requirements.
ematejska pushed a commit that referenced this pull request Mar 4, 2020
ematejska pushed a commit that referenced this pull request Apr 15, 2020
Update 20191016-dlpack-support.md with VoVAllen's changes
penpornk pushed a commit to penpornk/community that referenced this pull request Sep 3, 2020
Update 20200612-stream-executor-c-api.md
omalleyt12 pushed a commit to omalleyt12/community that referenced this pull request Nov 30, 2020
ematejska pushed a commit that referenced this pull request Mar 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
RFC: Accepted RFC Design Document: Accepted by Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants