On Thu, Feb 06, 2025 at 05:19:34PM -0800, Jay Vosburgh wrote: > I think this now deserves some commentary in the code. Not > because this function itself is unclear, but because there's the > similarly-named slave_set_ns_maddr() (singular, not plural as in this > patch) that will behave in a subtly different manner after this patch is > applied. Yes, slave_set_ns_maddr() also need to be changed. The names are similar. I will add some comments for the functions. Thanks Hangbin