8000 Add architecture to XPU device property by guangyey · Pull Request #138186 · pytorch/pytorch · GitHub
[go: up one dir, main page]

Skip to content

Add architecture to XPU device property #138186

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

Closed
wants to merge 8 commits into from

Conversation

guangyey
Copy link
Collaborator
@guangyey guangyey commented Oct 17, 2024

Stack from ghstack (oldest at bottom):

Motivation

Add architecture to XPU device property.
In some cases, low-level application code can use special features or do specific optimizations depending on the device architecture, and this PR enables such applications.
Modified from https://github.com/pytorch/pytorch/pull/129675/files

cc @gujinghui @EikanWang @fengyuan14

Copy link
pytorch-bot bot commented Oct 17, 2024

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/138186

Note: Links to docs will display an error until the docs builds have been completed.

✅ No Failures

As of commit 1976202 with merge base 565a794 (image):
💚 Looks good so far! There are no failures yet. 💚

This comment was automatically generated by Dr. CI and updates every 15 minutes.

guangyey added a commit that referenced this pull request Oct 17, 2024
ghstack-source-id: 013f224
Pull Request resolved: #138186
@guangyey guangyey changed the title Add architecture to XPU device property [WIP] Add architecture to XPU device property Oct 17, 2024
@guangyey guangyey marked this pull request as draft October 17, 2024 08:09
@guangyey guangyey added ciflow/xpu Run XPU CI tasks release notes: xpu release notes category labels Oct 17, 2024
[ghstack-poisoned]
guangyey added a commit that referenced this pull request Nov 11, 2024
ghstack-source-id: 15a83b1
Pull Request resolved: #138186
@guangyey guangyey marked this pull request as ready for review November 11, 2024 08:35
@guangyey guangyey changed the title [WIP] Add architecture to XPU device property Add architecture to XPU device property Nov 11, 2024
guangyey added a commit that referenced this pull request Nov 11, 2024
ghstack-source-id: c55e5d5
Pull Request resolved: #138186
@guangyey guangyey added the module: xpu Intel XPU related issues label Nov 11, 2024
guangyey added a commit that referenced this pull request Nov 11, 2024
ghstack-source-id: eb5a106
Pull Request resolved: #138186
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
guangyey added a commit that referenced this pull request Nov 12, 2024
ghstack-source-id: fa7ea42
Pull Request resolved: #138186
guangyey added a commit that referenced this pull request Nov 12, 2024
ghstack-source-id: 5aa95bf
Pull Request resolved: #138186
@guangyey guangyey added the ciflow/trunk Trigger trunk jobs on your pull request label Nov 12, 2024
guangyey added a commit that referenced this pull request Nov 12, 2024
ghstack-source-id: b8c4c9f
Pull Request resolved: #138186
@guangyey guangyey added this to the 2.6.0 milestone Nov 12, 2024
[ghstack-poisoned]
[ghstack-poisoned]
guangyey added a commit that referenced this pull request Nov 12, 2024
ghstack-source-id: f19d1df
Pull Request resolved: #138186
[ghstack-poisoned]
[ghstack-poisoned]
@ezyang
Copy link
Contributor
ezyang commented Nov 13, 2024

@pytorchbot merge

@pytorchmergebot
Copy link
Collaborator

Merge started

Your change will be merged once all checks pass (ETA 0-4 Hours).

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team

Advanced Debugging
Check the merge workflow status
here

zero000064 pushed a commit to zero000064/pytorch that referenced this pull request Nov 14, 2024
# Motivation
Add `architecture` to XPU device property.
In some cases, low-level application code can use special features or do specific optimizations depending on the device architecture, and this PR enables such applications.
Modified from https://github.com/pytorch/pytorch/pull/129675/files

Pull Request resolved: pytorch#138186
Approved by: https://github.com/ezyang
Ryo-not-rio pushed a commit to Ryo-not-rio/pytorch that referenced this pull request Dec 2, 2024
# Motivation
Add `architecture` to XPU device property.
In some cases, low-level application code can use special features or do specific optimizations depending on the device architecture, and this PR enables such applications.
Modified from https://github.com/pytorch/pytorch/pull/129675/files

Pull Request resolved: pytorch#138186
Approved by: https://github.com/ezyang
pobin6 pushed a commit to pobin6/pytorch that referenced this pull request Dec 5, 2024
# Motivation
Add `architecture` to XPU device property.
In some cases, low-level application code can use special features or do specific optimizations depending on the device architecture, and this PR enables such applications.
Modified from https://github.com/pytorch/pytorch/pull/129675/files

Pull Request resolved: pytorch#138186
Approved by: https://github.com/ezyang
fmo-mt pushed a commit to fmo-mt/pytorch that referenced this pull request Dec 11, 2024
# Motivation
Add `architecture` to XPU device property.
In some cases, low-level application code can use special features or do specific optimizations depending on the device architecture, and this PR enables such applications.
Modified from https://github.com/pytorch/pytorch/pull/129675/files

Pull Request resolved: pytorch#138186
Approved by: https://github.com/ezyang
whitneywhtsang added a commit to intel/intel-xpu-backend-for-triton that referenced this pull request Dec 12, 2024
With pytorch/pytorch#138186, `architecture` is
added to XPU device property.
Instead of hard coding `pvc` when invoking `ocloc`, this PR changed to
dynamically passing the device architecture parsed.

---------

Signed-off-by: Whitney Tsang <whitney.tsang@intel.com>
@github-actions github-actions bot deleted the gh/guangyey/78/head branch December 14, 2024 02:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ciflow/trunk Trigger trunk jobs on your pull request ciflow/xpu Run XPU CI tasks Merged module: xpu Intel XPU related issues open source release notes: xpu release notes category
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants
0