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

hipGraph API fixes & "native" Graph implementation #560

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Commits on Jul 24, 2023

  1. Update OpenCL headers in include/CL

    this is required for cl_khr_command_buffer
    franz committed Jul 24, 2023
    Configuration menu
    Copy the full SHA
    36b6feb View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    905ff02 View commit details
    Browse the repository at this point in the history
  3. add include/CL/cl_ext_pocl.h (required for command buffer extensions)

    CL/opencl.h: include cl_ext_pocl.h but exclude cl_gl_ext.h
    (now just redirects to cl_ext.h)
    franz committed Jul 24, 2023
    Configuration menu
    Copy the full SHA
    cdc8282 View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    2104c36 View commit details
    Browse the repository at this point in the history
  5. Implement chipstar::GraphNative

    This class is used to implement Graphs that execute "natively"
    in the backend, using OpenCL command-buffers or LevelZero command-lists
    and only synchronizing with the host when required. Fallback to
    original Graph is provided in chipstar::GraphExec::launch()
    
    Add more error-checking to the hipGraphXYZ APIs & bugfixes
    
    Additionally, samples/graph + samples/graphMatrixMultiply
    work using the "native graphs" (cl_command_buffer), not
    the original chip-spv's graph execution.
    
    Merged style fixes by Henry.
    
    Co-authored-by: Henry Linjamäki <linehill@users.noreply.github.com>
    franz and linehill committed Jul 24, 2023
    Configuration menu
    Copy the full SHA
    7be5794 View commit details
    Browse the repository at this point in the history
  6. update unit tests with failing/passing Graph tests

    move graph tests to FAILING_FOR_ALL
    franz committed Jul 24, 2023
    Configuration menu
    Copy the full SHA
    5f6a368 View commit details
    Browse the repository at this point in the history

Commits on Jul 25, 2023

  1. Configuration menu
    Copy the full SHA
    4e2d624 View commit details
    Browse the repository at this point in the history

Commits on Aug 2, 2023

  1. fix issue with GraphNodeKernel constructors

    the constructors were copying the value of Args pointer,
    instead of making an actual copy of the arguments. This
    can lead to a crash if the Args pointer lives on the stack,
    later goes out of scope, and setupAllArgs() is called.
    franz committed Aug 2, 2023
    Configuration menu
    Copy the full SHA
    83ee3d8 View commit details
    Browse the repository at this point in the history