View Issue Details
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0005468||The Dark Mod||Graphics||public||27.12.2020 10:30||27.02.2021 14:00|
|Product Version||TDM 2.09|
|Target Version||TDM 2.10||Fixed in Version||TDM 2.09|
|Summary||0005468: Debug rendering like s_drawSounds does not work under com_smp|
|Description||There are many useful debugging render tools under debug cvars.|
Some of them don't work under com_smp, some of them force-disable com_smp (see 0004408).
Most likely problem is that these debug rendering have lifetime = 0, which means "only render next frame".
But with SMP it is broken, and needs fixing.
|Steps To Reproduce||1) Start any FM.|
2) Enable "s_drawSounds 1".
3) Listen and look for sounds.
There should be box and text near every point where sounds originate.
It is printed with com_smp 0, but is not with com_smp 1.
|Tags||No tags attached.|
Ok, the main problem is that debug data like rb_debugLines is filled from game/frontend thread, while backend thread draw this data in parallel.
So in order to support it properly, some complicated double buffering would be necessary.
For now, I'll add s_drawSounds to the condition on disabling com_smp.
Also, I guess I'll commit some refactoring of debug geometry drawing (after 2.09 beta is over, I guess).
But I'm not sure this issue is worth fixing.
Cabalistic did two related commits:
r9079 Make debug primitives usable with com_smp
r9084 Properly copy debug primitives over to backend
It is quite likely that they solve the original problem with SMP by copying the debug data.
I guess we need to review the hacks for 2.10 and probably remove them.
Two more commits:
r9153. Some refactoring of debug rendering primitives.
r9154. Removed "com_smp 0" workaround for most of debug primitives.
The com_smp force-disabling hack is still left for:
r_showSurfaceInfo: this is clearly buggy with SMP
r_showDepth: could not make it work at all
P.S.Also r_showSilhouette only works in deprecated GL profile.
|27.12.2020 10:30||stgatilov||New Issue|
|27.12.2020 10:30||stgatilov||Status||new => assigned|
|27.12.2020 10:30||stgatilov||Assigned To||=> stgatilov|
|27.12.2020 10:30||stgatilov||Relationship added||related to 0004408|
|31.12.2020 11:43||stgatilov||Note Added: 0013334|
|24.01.2021 10:40||stgatilov||Note Added: 0013503|
|16.02.2021 14:45||stgatilov||Priority||normal => low|
|27.02.2021 13:58||stgatilov||Note Added: 0013737|
|27.02.2021 13:59||stgatilov||Status||assigned => resolved|
|27.02.2021 13:59||stgatilov||Resolution||open => fixed|
|27.02.2021 13:59||stgatilov||Fixed in Version||=> TDM 2.10|
|27.02.2021 14:00||stgatilov||Assigned To||stgatilov => cabalistic|
|27.02.2021 14:00||stgatilov||Fixed in Version||TDM 2.10 => TDM 2.09|