Skip to content

Conversation

@ZhiweiYan-96
Copy link
Collaborator

@ZhiweiYan-96 ZhiweiYan-96 commented Jul 10, 2024

Motivation

cdist calls cdist_stub for each backend. However, solely using REGISTER_XPU_DISPATCH could not goes into cdist_stub since there is another device list incdist_impl func and XPU is not in it.

This PR add XPU device into the supported list of cdist_impl so as to allow calling xpu kernel via cdist_stub

Stack from ghstack (oldest at bottom):

cc @gujinghui @EikanWang @fengyuan14 @guangyey

[ghstack-poisoned]
@pytorch-bot
Copy link

pytorch-bot bot commented Jul 10, 2024

🔗 Helpful Links

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

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

❗ 1 Active SEVs

There are 1 currently active SEVs. If your PR is affected, please view them below:

✅ No Failures

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

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

ZhiweiYan-96 added a commit that referenced this pull request Jul 10, 2024
ghstack-source-id: 8cf9221
Pull Request resolved: #130412
@ZhiweiYan-96 ZhiweiYan-96 marked this pull request as draft July 10, 2024 02:54
@ZhiweiYan-96 ZhiweiYan-96 changed the title Add XPU into device list of cdist_impl [Intel GPU]Add XPU into device list of cdist_impl Jul 10, 2024
@ZhiweiYan-96 ZhiweiYan-96 requested a review from EikanWang July 10, 2024 06:01
[ghstack-poisoned]
ZhiweiYan-96 added a commit that referenced this pull request Jul 10, 2024
ghstack-source-id: c457a42
Pull Request resolved: #130412
@ZhiweiYan-96 ZhiweiYan-96 added ciflow/xpu Run XPU CI tasks ciflow/trunk Trigger trunk jobs on your pull request module: xpu Intel XPU related issues labels Jul 10, 2024
@ZhiweiYan-96 ZhiweiYan-96 marked this pull request as ready for review July 12, 2024 06:17
ZhiweiYan-96 added a commit that referenced this pull request Jul 16, 2024
ghstack-source-id: 8618122
Pull Request resolved: #130088

Add XPU into device list of cdist_impl

ghstack-source-id: 8618122
Pull Request resolved: #130412
@ZhiweiYan-96 ZhiweiYan-96 changed the title [Intel GPU]Add XPU into device list of cdist_impl [Intel GPU]Add XPU into device list of cdist_impl Jul 17, 2024
ZhiweiYan-96 added a commit that referenced this pull request Jul 19, 2024
ghstack-source-id: 1c7ebf9
Pull Request resolved: #130088

Add XPU into device list of cdist_impl

ghstack-source-id: 1c7ebf9
Pull Request resolved: #130412
ZhiweiYan-96 added a commit that referenced this pull request Jul 23, 2024
ghstack-source-id: 0d1ad88
Pull Request resolved: #130088

Add XPU into device list of cdist_impl

ghstack-source-id: 0d1ad88
Pull Request resolved: #130412
ZhiweiYan-96 added a commit that referenced this pull request Jul 26, 2024
ghstack-source-id: 21876e0
Pull Request resolved: #130088

Add XPU into device list of cdist_impl

ghstack-source-id: 21876e0
Pull Request resolved: #130412
ZhiweiYan-96 added a commit that referenced this pull request Jul 26, 2024
ghstack-source-id: e657758
Pull Request resolved: #130088

Add XPU into device list of cdist_impl

ghstack-source-id: e657758
Pull Request resolved: #130412
ZhiweiYan-96 added a commit that referenced this pull request Aug 3, 2024
ghstack-source-id: 36a7439
Pull Request resolved: #130088

Add XPU into device list of cdist_impl

ghstack-source-id: 36a7439
Pull Request resolved: #130412
ZhiweiYan-96 added a commit that referenced this pull request Aug 5, 2024
ghstack-source-id: 7fed97a
Pull Request resolved: #130088

Add XPU into device list of cdist_impl

ghstack-source-id: 7fed97a
Pull Request resolved: #130412
@EikanWang
Copy link
Collaborator

EikanWang commented Aug 12, 2024

@ZhiweiYan-96 , Is this PR still valid?

@github-actions github-actions bot deleted the gh/ZhiweiYan-96/20/head branch September 17, 2024 01:53
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 module: xpu Intel XPU related issues open source

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants