Closed sunshine-Chun closed 12 months ago
@sunshine-Chun has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.
@sunshine-Chun has updated the pull request. You must reimport the pull request before landing.
@sunshine-Chun has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.
@sunshine-Chun, please close the PR
This is a fix for the overshooting upstream bug. The active_task is calculated in a wrong way. In the current calculation, the active_task will always be 1 and the task_target will be always equal to target_speed, which will lead up to the case that all threads use their max bandwidth. (No throttling at all). The fix is to change the active_task to the number of threads, since all threads should be running and active.