On Wed, Nov 11, 2026 at 12:26:41PM +0000, Hayato Kuroda (Fujitsu) wrote: > Found issue > ========== > Injection points attached after injection_points_set_local() are removed via > before_shmem_exit callback. This cleanup identifies an injection point only > by name. > If another backend detached the point and attached a new callback with the > same > name before the original backend exited, the cleanup could detach the new > callback > instead.
Yeah, sure. Or just make sure that the first backend has exited before attaching something else to the same point. Or just use a differently-named point for the second backend, in the same path. There are a couple of options. FWIW, I've read your patch, and I am absolutely unconvinced about the fact that we need to complicate the server side injpoint functionality in core for only this purpose. Test facilities like this one should remain simple in design, even if they require tweaks sometimes. Complicating them would lead to bugs in the facility. I know that I am pretty adamant about this part of the code, but injpoints are used for many cases across the whole tree, and what you are doing is changing one property for another, complicating the code even further, while on it. -- Michael
signature.asc
Description: PGP signature
