public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Michael D Kinney" <michael.d.kinney@intel.com>
To: dann frazier <dann.frazier@canonical.com>,
	"devel@edk2.groups.io" <devel@edk2.groups.io>,
	"Kinney, Michael D" <michael.d.kinney@intel.com>
Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>,
	"Yao, Jiewen" <jiewen.yao@intel.com>,
	"Justen, Jordan L" <jordan.l.justen@intel.com>,
	"Gerd Hoffmann" <kraxel@redhat.com>,
	"Boeuf, Sebastien" <sebastien.boeuf@intel.com>,
	Brijesh Singh <brijesh.singh@amd.com>,
	"Aktas, Erdem" <erdemaktas@google.com>,
	James Bottomley <jejb@linux.ibm.com>,
	"Xu, Min M" <min.m.xu@intel.com>,
	Tom Lendacky <thomas.lendacky@amd.com>
Subject: Re: [edk2-devel] [Patch 1/2] OvmfPkg: Change default to disable MptScsi and PvScsi
Date: Tue, 6 Dec 2022 01:25:30 +0000	[thread overview]
Message-ID: <CO1PR11MB49296695F87EEE0BA08B647DD21B9@CO1PR11MB4929.namprd11.prod.outlook.com> (raw)
In-Reply-To: <Y46Ab0lEQUKFsy4A@xps13.dannf>

Hi Dann,

The purpose of this first step was due to lack of support/maintainers/reviewers.

If there are issues, then we need to find maintainers/reviewers starting with
those that are using and depend on these features.

If there are no issues from disabling these by default, then the 2nd step
would be deleting the code and the flags.

Are these features actually being used?

Thanks,

Mike

> -----Original Message-----
> From: dann frazier <dann.frazier@canonical.com>
> Sent: Monday, December 5, 2022 3:36 PM
> To: devel@edk2.groups.io; Kinney, Michael D <michael.d.kinney@intel.com>
> Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>; Yao, Jiewen <jiewen.yao@intel.com>; Justen, Jordan L
> <jordan.l.justen@intel.com>; Gerd Hoffmann <kraxel@redhat.com>; Boeuf, Sebastien <sebastien.boeuf@intel.com>; Brijesh Singh
> <brijesh.singh@amd.com>; Aktas, Erdem <erdemaktas@google.com>; James Bottomley <jejb@linux.ibm.com>; Xu, Min M
> <min.m.xu@intel.com>; Tom Lendacky <thomas.lendacky@amd.com>
> Subject: Re: [edk2-devel] [Patch 1/2] OvmfPkg: Change default to disable MptScsi and PvScsi
> 
> 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 trying to decide whether or not this is something we should
> force back on for our users to avoid the regression, and/or if we need
> to actively move users off of it (mark it deprecated, etc). Do you know
> if there are known issues with this code, other than the lack of
> reviewers? Is there a plan to remove this code at some point?
> 
>   -dann
> 
> > Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>
> > Cc: Jiewen Yao <jiewen.yao@intel.com>
> > Cc: Jordan Justen <jordan.l.justen@intel.com>
> > Cc: Gerd Hoffmann <kraxel@redhat.com>
> > Cc: Sebastien Boeuf <sebastien.boeuf@intel.com>
> > Cc: Brijesh Singh <brijesh.singh@amd.com>
> > Cc: Erdem Aktas <erdemaktas@google.com>
> > Cc: James Bottomley <jejb@linux.ibm.com>
> > Cc: Jiewen Yao <jiewen.yao@intel.com>
> > Cc: Min Xu <min.m.xu@intel.com>
> > Cc: Tom Lendacky <thomas.lendacky@amd.com>
> > Signed-off-by: Michael D Kinney <michael.d.kinney@intel.com>
> > ---
> >  OvmfPkg/AmdSev/AmdSevX64.dsc     | 4 ++--
> >  OvmfPkg/CloudHv/CloudHvX64.dsc   | 4 ++--
> >  OvmfPkg/IntelTdx/IntelTdxX64.dsc | 4 ++--
> >  OvmfPkg/Microvm/MicrovmX64.dsc   | 4 ++--
> >  OvmfPkg/OvmfPkgIa32.dsc          | 4 ++--
> >  OvmfPkg/OvmfPkgIa32X64.dsc       | 4 ++--
> >  OvmfPkg/OvmfPkgX64.dsc           | 4 ++--
> >  7 files changed, 14 insertions(+), 14 deletions(-)
> >
> > diff --git a/OvmfPkg/AmdSev/AmdSevX64.dsc b/OvmfPkg/AmdSev/AmdSevX64.dsc
> > index 726521c94381..90e8a213ef77 100644
> > --- a/OvmfPkg/AmdSev/AmdSevX64.dsc
> > +++ b/OvmfPkg/AmdSev/AmdSevX64.dsc
> > @@ -43,8 +43,8 @@ [Defines]
> >    #
> >    # Device drivers
> >    #
> > -  DEFINE PVSCSI_ENABLE           = TRUE
> > -  DEFINE MPT_SCSI_ENABLE         = TRUE
> > +  DEFINE PVSCSI_ENABLE           = FALSE
> > +  DEFINE MPT_SCSI_ENABLE         = FALSE
> >    DEFINE LSI_SCSI_ENABLE         = FALSE
> >
> >    #
> > diff --git a/OvmfPkg/CloudHv/CloudHvX64.dsc b/OvmfPkg/CloudHv/CloudHvX64.dsc
> > index 0bfe542f8a88..f0d700f14477 100644
> > --- a/OvmfPkg/CloudHv/CloudHvX64.dsc
> > +++ b/OvmfPkg/CloudHv/CloudHvX64.dsc
> > @@ -49,8 +49,8 @@ [Defines]
> >    #
> >    # Device drivers
> >    #
> > -  DEFINE PVSCSI_ENABLE           = TRUE
> > -  DEFINE MPT_SCSI_ENABLE         = TRUE
> > +  DEFINE PVSCSI_ENABLE           = FALSE
> > +  DEFINE MPT_SCSI_ENABLE         = FALSE
> >    DEFINE LSI_SCSI_ENABLE         = FALSE
> >
> >    #
> > diff --git a/OvmfPkg/IntelTdx/IntelTdxX64.dsc b/OvmfPkg/IntelTdx/IntelTdxX64.dsc
> > index 144d50aa9dba..71b1cf8e7090 100644
> > --- a/OvmfPkg/IntelTdx/IntelTdxX64.dsc
> > +++ b/OvmfPkg/IntelTdx/IntelTdxX64.dsc
> > @@ -34,8 +34,8 @@ [Defines]
> >    #
> >    # Device drivers
> >    #
> > -  DEFINE PVSCSI_ENABLE           = TRUE
> > -  DEFINE MPT_SCSI_ENABLE         = TRUE
> > +  DEFINE PVSCSI_ENABLE           = FALSE
> > +  DEFINE MPT_SCSI_ENABLE         = FALSE
> >    DEFINE LSI_SCSI_ENABLE         = FALSE
> >
> >    #
> > diff --git a/OvmfPkg/Microvm/MicrovmX64.dsc b/OvmfPkg/Microvm/MicrovmX64.dsc
> > index 61db9b6e4c83..52498bbe90a8 100644
> > --- a/OvmfPkg/Microvm/MicrovmX64.dsc
> > +++ b/OvmfPkg/Microvm/MicrovmX64.dsc
> > @@ -47,8 +47,8 @@ [Defines]
> >    #
> >    # Device drivers
> >    #
> > -  DEFINE PVSCSI_ENABLE           = TRUE
> > -  DEFINE MPT_SCSI_ENABLE         = TRUE
> > +  DEFINE PVSCSI_ENABLE           = FALSE
> > +  DEFINE MPT_SCSI_ENABLE         = FALSE
> >    DEFINE LSI_SCSI_ENABLE         = FALSE
> >
> >    #
> > diff --git a/OvmfPkg/OvmfPkgIa32.dsc b/OvmfPkg/OvmfPkgIa32.dsc
> > index e708411076ca..725a01ae9a20 100644
> > --- a/OvmfPkg/OvmfPkgIa32.dsc
> > +++ b/OvmfPkg/OvmfPkgIa32.dsc
> > @@ -50,8 +50,8 @@ [Defines]
> >    #
> >    # Device drivers
> >    #
> > -  DEFINE PVSCSI_ENABLE           = TRUE
> > -  DEFINE MPT_SCSI_ENABLE         = TRUE
> > +  DEFINE PVSCSI_ENABLE           = FALSE
> > +  DEFINE MPT_SCSI_ENABLE         = FALSE
> >    DEFINE LSI_SCSI_ENABLE         = FALSE
> >
> >    #
> > diff --git a/OvmfPkg/OvmfPkgIa32X64.dsc b/OvmfPkg/OvmfPkgIa32X64.dsc
> > index 0b036d8bb53f..adc813ba2e1e 100644
> > --- a/OvmfPkg/OvmfPkgIa32X64.dsc
> > +++ b/OvmfPkg/OvmfPkgIa32X64.dsc
> > @@ -49,8 +49,8 @@ [Defines]
> >    #
> >    # Device drivers
> >    #
> > -  DEFINE PVSCSI_ENABLE           = TRUE
> > -  DEFINE MPT_SCSI_ENABLE         = TRUE
> > +  DEFINE PVSCSI_ENABLE           = FALSE
> > +  DEFINE MPT_SCSI_ENABLE         = FALSE
> >    DEFINE LSI_SCSI_ENABLE         = FALSE
> >
> >    #
> > diff --git a/OvmfPkg/OvmfPkgX64.dsc b/OvmfPkg/OvmfPkgX64.dsc
> > index 8ad04b50f74f..6e68f60dc90f 100644
> > --- a/OvmfPkg/OvmfPkgX64.dsc
> > +++ b/OvmfPkg/OvmfPkgX64.dsc
> > @@ -49,8 +49,8 @@ [Defines]
> >    #
> >    # Device drivers
> >    #
> > -  DEFINE PVSCSI_ENABLE           = TRUE
> > -  DEFINE MPT_SCSI_ENABLE         = TRUE
> > +  DEFINE PVSCSI_ENABLE           = FALSE
> > +  DEFINE MPT_SCSI_ENABLE         = FALSE
> >    DEFINE LSI_SCSI_ENABLE         = FALSE
> >
> >    #

  reply	other threads:[~2022-12-06  1:25 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-07-26 19:46 [Patch 0/2] Remove MptScsi and PvScsi reviewers Michael D Kinney
2022-07-26 19:46 ` [Patch 1/2] OvmfPkg: Change default to disable MptScsi and PvScsi Michael D Kinney
2022-12-05 23:36   ` [edk2-devel] " dann frazier
2022-12-06  1:25     ` Michael D Kinney [this message]
2022-12-06  2:07       ` dann frazier
2022-12-06  5:56     ` Gerd Hoffmann
2022-12-06 14:37       ` dann frazier
2022-12-06 14:55         ` Mike Maslenkin
2022-12-07  7:41           ` Gerd Hoffmann
2022-12-07 14:09             ` Ard Biesheuvel
2022-12-07 14:14               ` James Bottomley
2022-12-07 16:01                 ` Gerd Hoffmann
2022-12-07 16:04                   ` Ard Biesheuvel
2022-12-07 16:22                     ` James Bottomley
2022-12-07 21:59                       ` dann frazier
2022-07-26 19:46 ` [Patch 2/2] Maintainers.txt: Remove MptScsi and PvScsi reviewers Michael D Kinney
2022-07-26 20:39 ` [Patch 0/2] " Ard Biesheuvel
2022-07-27  0:38 ` Yao, Jiewen
2022-08-15 15:00 ` Gerd Hoffmann

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-list from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CO1PR11MB49296695F87EEE0BA08B647DD21B9@CO1PR11MB4929.namprd11.prod.outlook.com \
    --to=devel@edk2.groups.io \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox