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.

Sign up for GitHub

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

Jump to bottom

Support adapterless execution via kernel source file parsing #10

Open
eyalroz opened this issue Apr 26, 2022 · 0 comments
Open

Support adapterless execution via kernel source file parsing #10

eyalroz opened this issue Apr 26, 2022 · 0 comments
Labels
enhancement New feature or request

Comments

@eyalroz
Copy link
Owner

eyalroz commented Apr 26, 2022

At the moment, one can't simply feed a new kernel to gpu-kernel-runner and expect it to work. Instead, the kernel runner must be built with support for this kernel, knowing about its signature, its shared memory requirements etc. - via an adapter class.

It should probably be possible, by parsing the kernel's source file, to determine most or all of these parameters automatically, so that the user need only provide the source, the arguments, and the grid dimensions.

The cppast library may be useful for this purpose; but it may also be overkill and more crude parsing is in order.
@eyalroz eyalroz added the enhancement New feature or request label Apr 26, 2022
@eyalroz eyalroz mentioned this issue Aug 5, 2022
Closed
eyalroz added a commit that referenced this issue Aug 13, 2022
@eyalroz
Fixes #46, regards #3, #10: Dropping two-phase argument parsing
a1feaec 
* Arguments are now passed as `-a name=value` or `--argument name=value`
* No longer printing kernel-specific usage information
* Some log message tweaks
* Removed some unused utility code
* Renamed `kernel_inspecific_cmdline_options_t` -> `parsed_cmdline_options_t`
* Renamed some of the parsing-related functions
eyalroz added a commit that referenced this issue Aug 14, 2022
@eyalroz
Fixes #46, regards #3, #10: Dropping two-phase argument parsing
7f09808 
* Arguments are now passed as `-a name=value` or `--argument name=value`
* No longer printing kernel-specific usage information
* Some log message tweaks
* Removed some unused utility code
* Renamed `kernel_inspecific_cmdline_options_t` -> `parsed_cmdline_options_t`
* Renamed some of the parsing-related functions
* `README.md` changes (some related to the other changes, some just phrasing tweaks)
eyalroz added a commit that referenced this issue Aug 14, 2022
@eyalroz
Fixes #46, regards #3, #10: Dropping two-phase argument parsing
781a05b 
* Arguments are now passed as `-a name=value` or `--argument name=value`
* No longer printing kernel-specific usage information
* Some log message tweaks
* Removed some unused utility code
* Renamed `kernel_inspecific_cmdline_options_t` -> `parsed_cmdline_options_t`
* Renamed some of the parsing-related functions
* `README.md` changes (some related to the other changes, some just phrasing tweaks)
eyalroz added a commit that referenced this issue Aug 15, 2022
@eyalroz
Fixes #46, regards #3, #10: Dropping two-phase argument parsing
25fc7af 
* Arguments are now passed as `-a name=value` or `--argument name=value`
* No longer printing kernel-specific usage information
* Some log message tweaks
* Removed some unused utility code
* Renamed `kernel_inspecific_cmdline_options_t` -> `parsed_cmdline_options_t`
* Renamed some of the parsing-related functions
* `README.md` changes (some related to the other changes, some just phrasing tweaks)
eyalroz added a commit that referenced this issue Aug 15, 2022
@eyalroz
Fixes #46, regards #3, #10: Dropping two-phase argument parsing
c1b048a 
* Arguments are now passed as `-a name=value` or `--argument name=value`
* No longer printing kernel-specific usage information
* Some log message tweaks
* Removed some unused utility code
* Renamed `kernel_inspecific_cmdline_options_t` -> `parsed_cmdline_options_t`
* Renamed some of the parsing-related functions
* `README.md` changes (some related to the other changes, some just phrasing tweaks)
eyalroz added a commit that referenced this issue Aug 30, 2022
@eyalroz
Fixes #46, regards #3, #10: Dropping two-phase argument parsing
6e975ae 
* Arguments are now passed as `-a name=value` or `--argument name=value`
* No longer printing kernel-specific usage information
* Some log message tweaks
* Removed some unused utility code
* Renamed `kernel_inspecific_cmdline_options_t` -> `parsed_cmdline_options_t`
* Renamed some of the parsing-related functions
* `README.md` changes (some related to the other changes, some just phrasing tweaks)
eyalroz added a commit that referenced this issue Jan 15, 2023
@eyalroz
Fixes #46, regards #3, #10: Dropping two-phase argument parsing
9865f61 
* Arguments are now passed as `-a name=value` or `--argument name=value`
* No longer printing kernel-specific usage information
* Some log message tweaks
* Removed some unused utility code
* Renamed `kernel_inspecific_cmdline_options_t` -> `parsed_cmdline_options_t`
* Renamed some of the parsing-related functions
* `README.md` changes (some related to the other changes, some just phrasing tweaks)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@eyalroz