View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0005285 | The Dark Mod | Graphics | public | 02.07.2020 16:40 | 12.03.2023 16:33 |
Reporter | cabalistic | Assigned To | cabalistic | ||
Priority | normal | Severity | normal | Reproducibility | have not tried |
Status | closed | Resolution | won't fix | ||
Product Version | TDM 2.09 | ||||
Target Version | TDM 2.12 | ||||
Summary | 0005285: Remove old backend's depth/interaction/stencil stages | ||||
Description | After 5238 is merged and deemed stable, remove the old stages it replicates, including their shaders. | ||||
Tags | No tags attached. | ||||
related to | 0004501 | resolved | duzenko | Remove unused obsolete rendering paths |
related to | 0005598 | resolved | cabalistic | Performance regression on hedge01_square_long |
related to | 0005828 | resolved | stgatilov | Try to reduce duplication among shader code. |
related to | 0005834 | closed | stgatilov | Refactor: merge interaction and ambient shaders |
parent of | 0005891 | resolved | stgatilov | Remove excessive code for bindless texture support |
related to | 0006271 | resolved | stgatilov | Merge old and new backend |
Done via rev 9023 to 9028 | |
Err, nope, that was something else. This is still open - it basically means to remove the r_useNewBackend switch entirely and everything that the old backend replicates from the new one. What the mentioned revisions removed was even older code :D | |
Wait a few months first. Most of the players have just started using TDM 2.09. As of now, I recall only these common issues: 1) new AMD bug with bindless textures, and 2) old AMD bug with 64-bit FBO. None of which are about the new backend, actually =) |
|
The investigation in 0005598 showed that new backend is sometimes a bit slower than the old one: (stgatilov) For me, it is: 190 FPS: default settings 210 FPS: +set r_fboColorBits 32 215 FPS: +set r_tonemap 0 245 FPS: +set r_useNewBackend 0 (cabalistic) As for the new backend, a good portion of the difference at least appears to lie within the depth stage. There were some concessions I made to support the multidraw approach (even when it's disabled) that might be at fault here. In any case, the drop with the new backend definitely needs to be fixed. It is not supposed to be a step backwards. I'll take care of it... somehow :) So perhaps old backend should live a bit longer... Here is an important discussion about the future of the new backend: https://forums.thedarkmod.com/index.php?/topic/20949-on-bindless-textures-multidraws-and-improvements-to-the-new-render-backend/ Basically, Cabalistic says bindless textures and multidrawindirect should be removed. Doing so will probably fix the performance issue... cite: "I sacrificed some scissoring in certain stages to be able to batch more draws - and while that makes no difference on fast cards, it might be hurting slower cards in certain situations." Also it will simplify our code a lot. I'd say removing bindless textures and multidrawindirect are the most actual tasks. |
|
Agreed; I'll find some time for it next year :) | |
Closing this since we plan to merge two backends and return to simpler approaches. Like uniforms, and later maybe per-surface, per-object, per-view, per-XXX UBOs . |
|
Date Modified | Username | Field | Change |
---|---|---|---|
02.07.2020 16:40 | cabalistic | New Issue | |
14.12.2020 16:16 | nbohr1more | Note Added: 0013121 | |
14.12.2020 16:16 | nbohr1more | Status | new => resolved |
14.12.2020 16:16 | nbohr1more | Resolution | open => fixed |
14.12.2020 16:16 | nbohr1more | Fixed in Version | => TDM 2.09 |
14.12.2020 16:21 | cabalistic | Status | resolved => confirmed |
14.12.2020 16:21 | cabalistic | Resolution | fixed => open |
14.12.2020 16:21 | cabalistic | Product Version | TDM 2.09 => TDM 2.10 |
14.12.2020 16:21 | cabalistic | Fixed in Version | TDM 2.09 => |
14.12.2020 16:21 | cabalistic | Target Version | TDM 2.09 => TDM 2.10 |
14.12.2020 16:21 | cabalistic | Note Added: 0013122 | |
14.12.2020 16:21 | cabalistic | Product Version | TDM 2.10 => TDM 2.09 |
14.12.2020 23:22 | nbohr1more | Relationship added | related to 0004501 |
16.02.2021 15:17 | stgatilov | Note Added: 0013690 | |
16.02.2021 15:18 | stgatilov | Assigned To | => cabalistic |
16.02.2021 15:18 | stgatilov | Status | confirmed => assigned |
04.10.2021 18:33 | nbohr1more | Target Version | TDM 2.10 => TDM 2.11 |
27.12.2021 03:48 | stgatilov | Relationship added | related to 0005598 |
27.12.2021 03:49 | stgatilov | Relationship added | related to 0005828 |
27.12.2021 03:49 | stgatilov | Relationship added | related to 0005834 |
27.12.2021 03:58 | stgatilov | Note Added: 0014609 | |
27.12.2021 07:50 | cabalistic | Note Added: 0014610 | |
24.01.2022 15:52 | nbohr1more | Relationship added | parent of 0005891 |
27.11.2022 21:05 | nbohr1more | Target Version | TDM 2.11 => TDM 2.12 |
12.03.2023 16:31 | stgatilov | Relationship added | related to 0006271 |
12.03.2023 16:33 | stgatilov | Status | assigned => closed |
12.03.2023 16:33 | stgatilov | Resolution | open => won't fix |
12.03.2023 16:33 | stgatilov | Note Added: 0015971 |