Re: [edk2-devel] [Patch 1/2] OvmfPkg: Change default to disable MptScsi and PvScsi

["Mike Maslenkin" <mike.maslenkin@gmail.com>]

Greetings All!

As I can see LSI_SCSI_ENABLE  related to LSI LSI_53C895A_PCI_DEVICE_ID
Vid/Did 0x1000:0x0012.
I guess it is some old Megaraid adapter.

A patch mentioned above set MPT_SCSI_ENABLE=FALSE, that removed
support for LSI 53C1030 and SAS1068.
These SCSI controllers were emulated  by VMware, Parallels and I guess
VitualBox.
This is generic setup for VMware VMs, as far as I remember.
So the booting of such VMs (probably migrated from VMware  and others)
was definitely broken.

Regards,
Mike.


On Tue, Dec 6, 2022 at 5:38 PM dann frazier <dann.frazier@canonical.com> wrote:
>
> On Tue, Dec 06, 2022 at 06:56:49AM +0100, Gerd Hoffmann wrote:
> > On Mon, Dec 05, 2022 at 04:36:15PM -0700, dann frazier wrote:
> > > On Tue, Jul 26, 2022 at 12:46:39PM -0700, Michael D Kinney wrote:
> > > > The email addresses for the reviewers of the MptScsi and
> > > > PvScsi are no longer valid.  Disable the MptScsi and PvScsi
> > > > drivers in all DSC files until new maintainers/reviewers can
> > > > be identified.
> > >
> > > Hi Michael,
> > >
> > >   This seems likely to be the reason for the following regression
> > > report in Debian:
> > >
> > >   https://bugs.debian.org/1016359
> >
> > I'm not so sure about that.
> >
> > > > -  DEFINE PVSCSI_ENABLE           = TRUE
> > > > -  DEFINE MPT_SCSI_ENABLE         = TRUE
> > > > +  DEFINE PVSCSI_ENABLE           = FALSE
> > > > +  DEFINE MPT_SCSI_ENABLE         = FALSE
> > > >    DEFINE LSI_SCSI_ENABLE         = FALSE
> >
> > The bug report talks about lsilogic and virtio-scsi.
> >
> > lsilogic was already disabled by default before this patch.
> >
> > virtio-scsi support is included and there are no plans to change
> > that because it is a rather essential driver.  It works just fine
> > upstream, and there isn't even a config switch to disable it.
>
> Thanks Gerd - I'll work with the users to clarify via the bug (thanks
> for responding there as well btw).
>
>   -dann
>
>
> 
>
>