Skip to content

Conversation

@Jokeren
Copy link
Contributor

No description provided.

@JokerenJokeren marked this pull request as ready for review November 5, 2024 02:51
@JokerenJokeren requested a review from ptillet as a code ownerNovember 5, 2024 02:51
@ThomasRaoux
Copy link
Collaborator

Can you update the readme?

@Jokeren
Copy link
ContributorAuthor

Updated

@JokerenJokeren merged commit 7275ff7 into mainNov 5, 2024
7 checks passed
@JokerenJokeren deleted the keren/disable-mmav1 branch November 5, 2024 18:59
@Isotr0py
Copy link

Isotr0py commented Mar 3, 2025

Hi @Jokeren@ThomasRaoux, I noticed that the supported Nvidia GPU in README was bumped to requiring cc 8.0+ in this PR.

Did it mean the Volta and Turing support including FMA fallback will be removed in the future? Or did it just mean MMA will be disabled on pre-Ampere GPUs with FMA fallback?

I used T4 to run the triton core tests and it passed with latest triton 0.3.2, but I'm not sure if it just still worked by coincidence since it's no longer in the supported hardware list.

@Jokeren
Copy link
ContributorAuthor

Jokeren commented Mar 3, 2025

Did it mean the Volta and Turing support including FMA fallback will be removed in the future? Or did it just mean MMA will be disabled on pre-Ampere GPUs with FMA fallback?

IMO, community contribution to pre ampere architectures with MMA is welcome if it doesn't break backward compatibility. The FMA code path will still be maintained in triton/main.

@Isotr0py
Copy link

@Jokeren Got it! Thanks for your explanation!

@ThomasRaoux
Copy link
Collaborator

yes we are not going to break support for older platform but the support will be best effort and won't be tested on our side. As Keren mentioned contributions are welcome to fix older platform however we are unlikely to accept complex performance related patches for those.

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@Jokeren@ThomasRaoux@Isotr0py