Title "megasas: Fix" is no good, possibly an editing accident. Suggest something like "megasas: Fix check for msi_init() failure"
Cao jin <[email protected]> writes: > msi_init returns non-zero value on both failure and success. > > cc: Hannes Reinecke <[email protected]> > cc: Paolo Bonzini <[email protected]> > cc: Marcel Apfelbaum <[email protected]> > > Reviewed-by: Marcel Apfelbaum <[email protected]> > Signed-off-by: Cao jin <[email protected]> > --- > hw/scsi/megasas.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/hw/scsi/megasas.c b/hw/scsi/megasas.c > index a63a581..56fb645 100644 > --- a/hw/scsi/megasas.c > +++ b/hw/scsi/megasas.c > @@ -2348,7 +2348,7 @@ static void megasas_scsi_realize(PCIDevice *dev, Error > **errp) > "megasas-queue", 0x40000); > > if (megasas_use_msi(s) && > - msi_init(dev, 0x50, 1, true, false)) { > + msi_init(dev, 0x50, 1, true, false) < 0) { > s->flags &= ~MEGASAS_MASK_USE_MSI; > } > if (megasas_use_msix(s) &&
