Describe what router_reset_descriptor_download_failures() actually does

Comment-only change.

Part of #23813.
This commit is contained in:
teor 2017-10-10 14:36:08 -04:00 committed by Nick Mathewson
parent 51e47481fc
commit eaf9c0f25a

View File

@ -5358,8 +5358,10 @@ update_extrainfo_downloads(time_t now)
smartlist_free(wanted); smartlist_free(wanted);
} }
/** Reset the descriptor download failure count on all routers, so that we /** Reset the consensus and extra-info download failure count on all routers.
* can retry any long-failed routers immediately. * When we get a new consensus,
* routers_update_status_from_consensus_networkstatus() will reset the
* download statuses on the descriptors in that consensus.
*/ */
void void
router_reset_descriptor_download_failures(void) router_reset_descriptor_download_failures(void)
@ -5371,6 +5373,8 @@ router_reset_descriptor_download_failures(void)
last_descriptor_download_attempted = 0; last_descriptor_download_attempted = 0;
if (!routerlist) if (!routerlist)
return; return;
/* We want to download *all* extra-info descriptors, not just those in
* the consensus we currently have (or are about to have) */
SMARTLIST_FOREACH(routerlist->routers, routerinfo_t *, ri, SMARTLIST_FOREACH(routerlist->routers, routerinfo_t *, ri,
{ {
download_status_reset(&ri->cache_info.ei_dl_status); download_status_reset(&ri->cache_info.ei_dl_status);