Sync some things that were split out into pybind#5665

Author: b-pass

docs/advanced/embedding.rst

@@ -212,12 +212,6 @@ naturally:
         assert(locals["message"].cast<std::string>() == "1 + 2 = 3");
     }
 
-``PYBIND11_EMBEDDED_MODULE`` also accepts
-:func:`py::mod_gil_not_used()`,
-:func:`py::multiple_interpreters::per_interpreter_gil()`, and
-:func:`py::multiple_interpreters::shared_gil()` tags just like ``PYBIND11_MODULE``.
-See :ref:`misc_subinterp` and :ref:`misc_free_threading` for more information.
-
 
 Interpreter lifetime
 ====================

docs/advanced/misc.rst

@@ -155,8 +155,6 @@ following checklist.
   within pybind11 that will throw exceptions on certain GIL handling errors
   (reference counting operations).
 
-.. _misc_free_threading:
-
 Free-threading support
 ==================================================================
 
@@ -180,8 +178,6 @@ your code is thread safe.  Modules must still be built against the Python free-t
 enable free-threading, even if they specify this tag.  Adding this tag does not break
 compatibility with non-free-threaded Python.
 
-.. _misc_subinterp:
-
 Sub-interpreter support
 ==================================================================
 

tests/test_embed/test_subinterpreter.cpp

@@ -339,7 +339,13 @@ TEST_CASE("Per-Subinterpreter GIL") {
             T_REQUIRE(caught);
 
             // widget_module did provide the per_interpreter_gil tag, so it this does not throw
-            py::module_::import("widget_module");
+            try {
+                py::module_::import("widget_module");
+                caught = false;
+            } catch (pybind11::error_already_set &) {
+                caught = true;
+            }
+            T_REQUIRE(!caught);
 
             T_REQUIRE(!py::hasattr(py::module_::import("external_module"), "multi_interp"));
             py::module_::import("external_module").attr("multi_interp") = std::to_string(num);