eden-emu/eden
24
3
Fork 15
Code Issues 138 Pull requests 56 Projects 1 Releases Packages Wiki Activity Actions

[gpu/NVDRV] Unstub SetErrorNotifier and add PostErrorNotification #312

Closed
SDK-Chan wants to merge 3 commits from unstub-error-notifier into master
pull from: unstub-error-notifier
merge into: eden-emu:master
Conversation 1 Commits 3 Files changed 2 +95 -4
SDK-Chan commented 2025-08-25 16:52:41 +02:00
Member
Copy link

Add functionality to SetErrorNotifier.
Functionality should now be correctly written to memory.

Add PostErrorNotification.
This ensures that some functions with bad return values now properly set a error status.

Created a NotifierStatus enum to make error categories more descriptive.

Credits to JPikachu for creating the initial stub, and showing me how I should proceed to make that function complete.

Add functionality to SetErrorNotifier. Functionality should now be correctly written to memory. Add PostErrorNotification. This ensures that some functions with bad return values now properly set a error status. Created a NotifierStatus enum to make error categories more descriptive. Credits to JPikachu for creating the initial stub, and showing me how I should proceed to make that function complete.
SDK-Chan added 1 commit 2025-08-25 16:52:41 +02:00
[gpu/NVDRV] Unstub SetErrorNotifier and add PostErrorNotification
All checks were successful
eden-license / license-header (pull_request) Successful in 21s
Details
b668b94710
SDK-Chan added 1 commit 2025-08-25 17:16:19 +02:00
Update src/core/hle/service/nvdrv/devices/nvhost_gpu.cpp
All checks were successful
eden-license / license-header (pull_request) Successful in 21s
Details
610383262d
JPikachu commented 2025-08-25 17:19:49 +02:00
Member
Copy link

Looks good to me!
Well done @SDK-Chan

Looks good to me! Well done @SDK-Chan
SDK-Chan added 1 commit 2025-08-25 17:25:41 +02:00
Update src/core/hle/service/nvdrv/devices/nvhost_gpu.cpp
All checks were successful
eden-license / license-header (pull_request) Successful in 24s
Details
02c98ec5e2
CamilleLaVey force-pushed unstub-error-notifier from 02c98ec5e2 to 69bc01f47d 2025-08-29 00:13:49 +02:00 Compare
Shinmegumi closed this pull request 2025-09-17 19:24:04 +02:00
All checks were successful
eden-license / license-header (pull_request) Successful in 24s
Required
Details

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
android

Specific to android platform

  
audio

   
bug

Something is not working

  
ci

For changes to the ci/building

  
confirmed

Used when testers confirm a bug is reported and it is reproducable.

  
core

For core changes to the emulator

  
desktop

Specific to desktop PCs, handhelds, etc.

  
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
frontend

Frontend changes

  
fw

Changes related to firmware

  
git

For enhancements or changes regarding git.

  
graphics

   
loader

For changes to any of the loader implementations

  
nv gpu

For use with changes that involve the emulator GPU and any services.

  
ui
No labels
android
audio
bug
ci
confirmed
core
desktop
duplicate
enhancement
frontend
fw
git
graphics
loader
nv gpu
ui
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: eden-emu/eden#312
No description provided.
Delete branch "unstub-error-notifier"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?