In:

Modules::add_module_exports
Modules::add_module_exports_qualified
Modules::add_module_exports_to_all_unnamed

We first lookup a PackageEntry - taking and releasing Module_lock - to then, after some trivial checks - take the Module_lock again.

Coarsening the locking so we only take the mutex once per call brings a small improvement to bootstrap (reduction of ~200-250k instructions)