1
0
mirror of https://git.FreeBSD.org/src.git synced 2024-12-23 11:18:54 +00:00
freebsd/sys/dev/aic
Joerg Wunsch 31f639ecdf Back out the hack from rev 1.13 that was done to initiate a bus rescan
at insert time.  When asking gibbs for approval for an MFC, this was
his reply:

1) It leaks memory if it can't allocate a path.

2) It defers allocation of aic->path until the call to scan the
   bus.  This means the path may be NULL when an interrupt occurs
   prior to the call to scan the bus (stray bus reset for instance),
   which will lead to a panic.

3) The driver in current doesn't recover from the failure to allocate
   aic->path.  The driver doesn't check during normal operation if
   the path is NULL, so again a panic will result.

4) aic_cam_rescan calls malloc with M_WAITOK.  aic_cam_rescan is called
   from attach where it isn't necessarily safe to sleep.

5) And most importantly, it co-opts the xpt_periph from the driver level.
   This was never part of the design (xpt_periph used to be static).  Making
   a call of this type may completely confuse the XPT if other XPT operations
   are ongoing.

In the long term, Justin and Warner agreed to implement solution where
CAM itself will initiate the bus rescan if a new bus is added.  For
the time being (and in particular in light of the upcoming 4.5
release), we now have camcontrol available on the boot floppy, and can
have pccardd initiate the rescan through it.
2002-01-17 20:34:58 +00:00
..
aic6360reg.h Fix a few things in the aic(4) driver: 2001-03-16 22:20:19 +00:00
aic_cbus.c Merged from sys/dev/aic/aic_isa.c revision 1.8. 2001-06-26 11:35:46 +00:00
aic_isa.c Add PnP IDs for AHA-1530 and AHA-1520 cards. 2001-06-06 22:32:29 +00:00
aic_pccard.c Migrate to PCMCIA_CARD() macros 2001-11-15 07:52:49 +00:00
aic.c Back out the hack from rev 1.13 that was done to initiate a bus rescan 2002-01-17 20:34:58 +00:00
aicvar.h Fix a few things in the aic(4) driver: 2001-03-16 22:20:19 +00:00