Detach Rx/Tx/IOV tasks upon iflib device or psuedo device register failure.

[stallamr_netapp.com]

Summary

Once MSIX interrupt assign is complete, Rx/Tx/IOV tasks gets attach to respective taskqueues. Upon IFDI_ATTACH_POST()
or any failure beyond interrupt assignment, only admin tasks gets detached so far. While I'm here, group taskqueue
detach tasks into single function and use new function instead.

Test Plan

At NetApp, tested with kernel failpoints on physical interfaces. Didnot test psuedo interfaces though.

[markj]

sys/net/iflib.c

5088 ↗

(On Diff #79907)

This calls iflib_rx_sds_free() on each rxq, so why is it necessary to keep freeing the freelist bitmaps in the loop immediately above?

5167 ↗

(On Diff #79907)

Same question as above.

[stallamr_netapp.com]

sys/net/iflib.c

5088 ↗

(On Diff #79907)

Thanks Mark for pointing this out. I agree. Above loop to free bitmap is not necessary. I will remove the loop and refresh the change.

5167 ↗

(On Diff #79907)

Agree. Will remove the loop and refresh the review.

[stallamr_netapp.com]

rx bitmap do get free in following function call. Remove explicit free of bitmap yet again.

[markj]

I think this is ok. I'll give other reviewers a few more days to have a look.