I can't understand. If it is the last bit. We should clear the bit then do foreach check? Or it will never deactive?

Clearing the bit up-front means hidbus_intr won't drain any last minute response to the device driver thaat hid_intr_stop (doing a usbd_transfer_drain) triggers. The device driver will either be discarding them anyways (e.g., u2f(4), hidraw(4)) or it's harmless; in either case, it's useful for debugging to be able to observe the response at that level even if it wll be ignored.

Ok. I think you want to trigger the hidbus_intr in the last minutes and let the decision pass to the underlying driver. But would it actually work? ivar->actiave = false set after the active |= tlc->active, it measn active would never equals to 0 if we don't deactive twice in the same node.

Right, that's why setting active in the loop iss predicated on tlc != ivar -- so that it's explicitly not accounted for, and if every other device is inactive then we will call hid_intr_stop() before clearing tlc->active

Do you think it worth a __unlikely here?