[illumos-Developer] webrev: 896 sata module can panic on smartctl issued in vbox

[Albert Lee]

On Mon, Apr 11, 2011 at 2:27 PM, Garrett D'Amore <garrett at nexenta.com> wrote:
>  This is a Nexenta bug (I'll fix the hg comments, so don't worry about
> this), but it affects illumos too.  In fact, the bug causes illumos to panic
> in vbox if you run certain smartctl commands, or do anything to cause an
> initiator-initiated reset.
>
> http://mexico.purplecow.org/gdamore/webrev/satapanic/
>
> The panic is 100% reproducible.  And good news is, I can't reproduce it in
> vbox anymore after this change.  (Nexentians: this will be a change for NCP
> as well. :-)
>
> Thanks.
>
>    - Garrett
>

Looks good, thanks for the explanation (pkt freed after sata_hba_start).

-Albert