>
> We discussed this in an internal code review and I thought we resolved
> to do something with this but obviously nothing happened..
>
> I believe the code should really be:
>
> if (ipif->ipif_flags & IPIF_UP) {
> mutex_enter(&ill->ill_lock);
> ipif->ipif_flags &= ~IPIF_UP;
> mutex_exit(&ill->ill_lock);
> ASSERT(ill->ill_ipif_up_count > 0);
> --ill->ill_ipif_up_count;
> ipif_was_up = B_TRUE;
>
> if (ill->ill_ipif_up_count == 0) {
> ...
> (void) hook_run(hr, (hook_data_t)&info);
> }
> /* Update status in SCTP's list */
> sctp_update_ipif(ipif, SCTP_IPIF_DOWN);
> }
The routing socket messages are generated in ipif_down_tail(); is there
a reason we want to run the hook a different place (ipif_down())?
--
meem
_______________________________________________
networking-discuss mailing list
[email protected]
