Project

General

Profile

Actions

Bug #7707

closed

ixgbe crashes on debug builds when it fails to attach

Added by Robert Mustacchi almost 5 years ago. Updated almost 5 years ago.

Status:
Closed
Priority:
Normal
Category:
driver - device drivers
Start date:
2016-12-29
Due date:
% Done:

100%

Estimated time:
Difficulty:
Medium
Tags:
Gerrit CR:

Description

When ixgbe fails to map registers, due to 64-bit addresses being used for example, it will end up panicking by blowing an assertion when it calls ddi_cb_unregister(). ddi_cb_unrgister() is asserting that the handle is not NULL. However, in this case, we've never called ddi_cb_register() because we failed to map our registers.

ddi_cb_unregister() is a little bit different in that it does actually explicitly handle the case where the entry is not NULL; however, I believe the assertion is there for the same reason that the !servicing_interrupt() one is there. Mainly that it's trying to help folks who are on debug builds who would normally cast these errors to void and drive on and therefore we shouldn't remove this assertion, but instead fix it.

Actions

Also available in: Atom PDF