fix typo

this was split into multiple diffs: D16265-D16270

pass a pointer to the processor in ThreadImport

we can pass a pointer to ThreadImport in anticipation of not initializing the avalanche processor when avalanche is not needed (see review D16224)

make it possible to run the RPC when the processor is not initialized

don't use EnsureAvalanche in the RPC calls, make it possible to use the RPCs with the processor not initialized

don't use EnsureAvalanche in the RPC code, just assume that the processor can be uninitialized.

@bot build-asan

rebase and fix summary

This can be split into smaller diffs

pass avalanche down to all the ActivateBestChain callsites in non-test code

call ActiveBestChain in net_processing, improve FindMostWorkChain
still a WIP

@bot build-tsan

This is not ready. I'm getting test failures.

try passing the processor to chainstate functions that need it

rebase after splitting the diff into smaller diffs. This is now just the g_avalanche -> node.avalanche part of the original diff

I confirm that I cannot make the processor const here. It fails to compile.

review

use return; instead of assert in code that responds to remote messages
Remove a couple of asserts in functions that are now only called by code that verifies that the function exists.

rebase after splitting out a lot of the changes. Still a WIP, we can probably extract more smaller diffs from this.

remove the now redundant assertion