View Issue Details

IDProjectCategoryView StatusLast Update
0005285The Dark ModGraphicspublic12.03.2023 16:33
Reportercabalistic Assigned Tocabalistic  
PrioritynormalSeveritynormalReproducibilityhave not tried
Status closedResolutionwon't fix 
Product VersionTDM 2.09 
Target VersionTDM 2.12 
Summary0005285: Remove old backend's depth/interaction/stencil stages
DescriptionAfter 5238 is merged and deemed stable, remove the old stages it replicates, including their shaders.
TagsNo tags attached.

Relationships

related to 0004501 resolvedduzenko Remove unused obsolete rendering paths 
related to 0005598 resolvedcabalistic Performance regression on hedge01_square_long 
related to 0005828 resolvedstgatilov Try to reduce duplication among shader code. 
related to 0005834 closedstgatilov Refactor: merge interaction and ambient shaders 
parent of 0005891 resolvedstgatilov Remove excessive code for bindless texture support 
related to 0006271 resolvedstgatilov Merge old and new backend 

Activities

nbohr1more

nbohr1more

14.12.2020 16:16

developer   ~0013121

Done via rev 9023 to 9028
cabalistic

cabalistic

14.12.2020 16:21

developer   ~0013122

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
stgatilov

stgatilov

16.02.2021 15:17

administrator   ~0013690

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 =)
stgatilov

stgatilov

27.12.2021 03:58

administrator   ~0014609

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.
cabalistic

cabalistic

27.12.2021 07:50

developer   ~0014610

Agreed; I'll find some time for it next year :)
stgatilov

stgatilov

12.03.2023 16:33

administrator   ~0015971

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 .

Issue History

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