merged [13035] vi: manage resources independently of nvnflinger and refactor #13035

Closed
opened 2026-01-02 20:42:45 -03:00 by Accord · 0 comments
Owner

Branch: __refs_pull_13035_head

Labels: hle-fix, mainline-merge, android-merge

liamwhite opened issue at 2024-02-17 05:13:

While in the process of working on the am changes in #13018, I had to work around some structural issues with the way display resources are managed. This resolves those issues at the macro level, as am is no longer talking directly to nvnflinger and vi now does all the management (ideally in the future we would have omm talk to vi).
Unfortunately there ended up being so many issues to resolve to move and separate things out of nvnflinger that it was not really feasible to split up the commits, so this is what it is for now.

saad-script commented at 2024-02-18 16:51:

Did this break fps mod support?
Because fps mods are not working on latest mainline. I haven't tested to see if it's specifically this PR that causes it tho 🤔

liamwhite commented at 2024-02-18 16:57:

Yeah it did, I just pushed a fix

[Export of Github issue for yuzu-emu/yuzu. Generated on 2024.02.28 at 10:04:32.]

**Branch:** [__refs_pull_13035_head](https://accord.dravee.dev/Accord/yuzu-mirror/src/branch/__refs_pull_13035_head) > **Labels**: `hle-fix`, `mainline-merge`, `android-merge` #### <div><img width="50" height="50" src="https://avatars.githubusercontent.com/u/9658600?v=4" width="50"></div>[liamwhite](https://github.com/liamwhite) opened issue at [2024-02-17 05:13](https://github.com/yuzu-emu/yuzu/pull/13035): > While in the process of working on the am changes in #13018, I had to work around some structural issues with the way display resources are managed. This resolves those issues at the macro level, as am is no longer talking directly to nvnflinger and vi now does all the management (ideally in the future we would have omm talk to vi). > Unfortunately there ended up being so many issues to resolve to move and separate things out of nvnflinger that it was not really feasible to split up the commits, so this is what it is for now. #### <div><img width="50" height="50" src="https://avatars.githubusercontent.com/u/89614721?v=4" width="50"></div>[saad-script](https://github.com/saad-script) commented at [2024-02-18 16:51](https://github.com/yuzu-emu/yuzu/pull/13035#issuecomment-1951383411): > Did this break fps mod support? > Because fps mods are not working on latest mainline. I haven't tested to see if it's specifically this PR that causes it tho 🤔 #### <div><img width="50" height="50" src="https://avatars.githubusercontent.com/u/9658600?v=4" width="50"></div>[liamwhite](https://github.com/liamwhite) commented at [2024-02-18 16:57](https://github.com/yuzu-emu/yuzu/pull/13035#issuecomment-1951384793): > Yeah it did, I just pushed a fix > ------------------------------------------------------------------------------- > [Export of Github issue for [yuzu-emu/yuzu](https://github.com/yuzu-emu/yuzu). Generated on 2024.02.28 at 10:04:32.]
Sign in to join this conversation.
No labels
No milestone
No project
No assignees
1 participant
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
Accord/yuzu-mirror#13035
No description provided.