diff options
author | Vito Caputo <vcaputo@pengaru.com> | 2023-07-08 20:15:06 -0700 |
---|---|---|
committer | Vito Caputo <vcaputo@pengaru.com> | 2023-07-08 21:22:24 -0700 |
commit | 4bad0667dd6ddfa1a43339ae37b3535550e8e264 (patch) | |
tree | 33ee3f2077520ed5503c1a83b0612da96d2d40a1 /src/libs/ray/ray_gamma.h | |
parent | 83f390da70600491b30aa328ace166ccdfd02b50 (diff) |
modules/checkers: stop setting til_frame_plan_t.cpu_affinity
In the early days of checkers when I introduced fill_module= with
the per-cpu contexts to still allow threaded rendering, the whole
seed passing to contexts thing wasn't as well sorted out. This
meant the contexts often produced vastly different outputs
despite being the same module, same seed, and same settings.
The consequence of that was that w/fill_module checkers would
produce crazy randomized output when you expected the same output
in the filled cells. But by using .cpu_affinity (which I had to
implement just for this use case actually) at least the different
outputs would become stable. It was a band-aid over a different
problem that still needed sorting out.
Nowadays, it seems like this is improved enough at least for
alphazed to look correct without the affinity hack, so I'm
removing it because it really kills checkers threaded
performance.
Whatever modules remain uncooperative WRT seed reproducibility,
they'll just need to be fixed up.
Diffstat (limited to 'src/libs/ray/ray_gamma.h')
0 files changed, 0 insertions, 0 deletions