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

Arm64/Sve: Implement the SVE APIs that takes 32-bit address as input #103297

Open
kunalspathak opened this issue Jun 11, 2024 · 3 comments
Open

Arm64/Sve: Implement the SVE APIs that takes 32-bit address as input #103297

kunalspathak opened this issue Jun 11, 2024 · 3 comments
Labels
area-System.Runtime.Intrinsics arm-sve Work related to arm64 SVE/SVE2 support
Milestone
Future

Comments

@kunalspathak
Copy link
Member

There are several APIs that operates on truncated address such that it is valid in 4GB memory range. We implemented the APIs, but realized that adding test might not be straight forward because it will involve calling VirtualAlloc or equivalent to make sure that the address is in that range. As such we decided to skip such APIs and add them opportunistically if time permits.

Related discussion: #103159 (comment)
@kunalspathak kunalspathak added Priority:3 Work that is nice to have arm-sve Work related to arm64 SVE/SVE2 support labels Jun 11, 2024
@dotnet-issue-labeler dotnet-issue-labeler bot added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Jun 11, 2024
@dotnet-policy-service dotnet-policy-service bot added the untriaged New issue has not been triaged by the area owner label Jun 11, 2024
@kunalspathak kunalspathak mentioned this issue Jun 11, 2024
Merged
@kunalspathak
Copy link
Member Author

@dotnet/arm64-contrib

@vcsjones vcsjones added area-System.Runtime.Intrinsics and removed needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners labels Jun 12, 2024
@dotnet-policy-service Dotnet Policy Service
Copy link
Contributor

Tagging subscribers to this area: @dotnet/area-system-runtime-intrinsics
See info in area-owners.md if you want to be subscribed.

@kunalspathak kunalspathak mentioned this issue Jul 15, 2024
Merged
@jeffhandley
Copy link
Member

@kunalspathak Is the 9.0.0 milestone appropriate for this, or should we set it to 10.0.0?

@kunalspathak kunalspathak added this to the 10.0.0 milestone Jul 20, 2024
@jeffhandley jeffhandley removed the untriaged New issue has not been triaged by the area owner label Jul 20, 2024
kunalspathak added a commit to kunalspathak/runtime that referenced this issue Sep 12, 2024
@kunalspathak
Remove entry of APIs because of dotnet#103297
7170938
kunalspathak added a commit that referenced this issue Sep 13, 2024
@kunalspathak
Arm64/Sve: Remove entries of SVE APIs take 32-bit address (#107758)
c8ae841 
* Remove entry of APIs because of #103297

* comment out the test cases for unsupported APIs
kunalspathak added a commit to kunalspathak/runtime that referenced this issue Sep 13, 2024
@kunalspathak
Arm64/Sve: Remove entries of SVE APIs take 32-bit address (dotnet#107758
913470d 
)

* Remove entry of APIs because of dotnet#103297

* comment out the test cases for unsupported APIs
kunalspathak added a commit to kunalspathak/runtime that referenced this issue Sep 13, 2024
@kunalspathak
Arm64/Sve: Remove entries of SVE APIs take 32-bit address (dotnet#107758
c8806ba 
)

* Remove entry of APIs because of dotnet#103297

* comment out the test cases for unsupported APIs
jeffschwMSFT added a commit that referenced this issue Sep 13, 2024
@kunalspathak @jeffschwMSFT
Arm64/Sve: Remove entries of SVE APIs take 32-bit address (#107758) (#…
15ef393 
…107780)

* Remove entry of APIs because of #103297

* comment out the test cases for unsupported APIs

Co-authored-by: Jeff Schwartz <jeffschw@microsoft.com>
jtschuster pushed a commit to jtschuster/runtime that referenced this issue Sep 17, 2024
@kunalspathak @jtschuster
Arm64/Sve: Remove entries of SVE APIs take 32-bit address (dotnet#107758
83cb4d6 
)

* Remove entry of APIs because of dotnet#103297

* comment out the test cases for unsupported APIs
sirntar pushed a commit to sirntar/runtime that referenced this issue Sep 30, 2024
@kunalspathak @sirntar
Arm64/Sve: Remove entries of SVE APIs take 32-bit address (dotnet#107758
bd36644 
)

* Remove entry of APIs because of dotnet#103297

* comment out the test cases for unsupported APIs
@kunalspathak kunalspathak removed the Priority:3 Work that is nice to have label Oct 8, 2024
@kunalspathak kunalspathak modified the milestones: 10.0.0, Future Oct 8, 2024
mikelle-rogers pushed a commit to mikelle-rogers/runtime that referenced this issue Dec 10, 2024
@kunalspathak @mikelle-rogers
Arm64/Sve: Remove entries of SVE APIs take 32-bit address (dotnet#107758
dc43625 
)

* Remove entry of APIs because of dotnet#103297

* comment out the test cases for unsupported APIs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area-System.Runtime.Intrinsics arm-sve Work related to arm64 SVE/SVE2 support
Projects
None yet
Milestone
Future
Development

No branches or pull requests
3 participants
@vcsjones @jeffhandley @kunalspathak