From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by mx.groups.io with SMTP id smtpd.web11.2137.1685642982744279881 for ; Thu, 01 Jun 2023 11:09:42 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=D5JNEnts; spf=pass (domain: kernel.org, ip: 139.178.84.217, mailfrom: ardb@kernel.org) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 36EA3648A2 for ; Thu, 1 Jun 2023 18:09:42 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 98761C4339E for ; Thu, 1 Jun 2023 18:09:41 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1685642981; bh=7zEpEQ3l6DpsE9P3Nx7NRhN36A6jYmnlTm3MmTqfAow=; h=References:In-Reply-To:From:Date:Subject:To:Cc:From; b=D5JNEntsofQmREC6KHA06oiX0hj+4vgTuEI5JH6sdRScXgeQKzFKcqJH6nZn1S2Xu rj+U8XbQHo6wJaUMedU4H6NKB3J+Gkg9dAsWB+tXIxb9Ps1SGyk1uRUxDDBznsQ1he vkwyw+ZmdznnzcH+EkVoHt3eIJXUJz7X8ZPnej0JaSusuaIUPLlEM8ZrBpqIh6VfFA Y5/i6ujHxBRBurSd5z0uQDGuKAcMvx2lGQDHEhKgweHJ1/lQGDPhAx+QVV42Rs1AcT teV4LntgYVtyHQH1UIRPsiD1X5mhW1XEskwoP/4Rj6jbOtuQ4iKZQBYVBWLKEKrEnW PgEufpSiPYL5Q== Received: by mail-lf1-f41.google.com with SMTP id 2adb3069b0e04-4f3baf04f0cso1526482e87.1 for ; Thu, 01 Jun 2023 11:09:41 -0700 (PDT) X-Gm-Message-State: AC+VfDygRJZfiCoQkF2glh7RMLTmweZ+2ImSGFHoSCVB3hRQJXC/vNJX V6zes9FbbWZmF1jZ1xmlnQKx+9FHBOuqtOAPhgA= X-Google-Smtp-Source: ACHHUZ47alaHK33iT8ABYRWyAVIDwgtQNoUkp/tNNA036Tt+FXFJ2Hs5Z3E+DkxubSsZlNl1NSWdauQbfzq4Yrubcnc= X-Received: by 2002:ac2:5322:0:b0:4f3:b207:65ff with SMTP id f2-20020ac25322000000b004f3b20765ffmr444406lfh.24.1685642979650; Thu, 01 Jun 2023 11:09:39 -0700 (PDT) MIME-Version: 1.0 References: <20230601172740.9165-1-pedro.falcato@gmail.com> In-Reply-To: <20230601172740.9165-1-pedro.falcato@gmail.com> From: "Ard Biesheuvel" Date: Thu, 1 Jun 2023 20:09:28 +0200 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH v3 00/12] OvmfPkg: Replace the OVMF-specific SataControllerDxe with the generic one To: Pedro Falcato Cc: devel@edk2.groups.io, Ard Biesheuvel , Jiewen Yao , Gerd Hoffmann , Erdem Aktas , James Bottomley , Min Xu , Tom Lendacky , Michael Roth , Rebecca Cran , Peter Grehan , =?UTF-8?Q?Corvin_K=C3=B6hne?= , Sebastien Boeuf , Anthony Perard , Julien Grall , Laszlo Ersek Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Thu, 1 Jun 2023 at 19:28, Pedro Falcato wrote: > > This patch-set replaces the OVMF specific SataControllerDxe with the MdeM= odulePkg/Bus/Pci one. > They were both forked from the same code, and are code-and-functionality = similar. As such, there > seems to be no need for duplication here. > > The first four pages both replay OvmfPkg/SataControllerDxe commits and fi= x up a couple of details. > The rest of the patches (minus the last) replace OvmfPkg/SataControllerDx= e with the MdeModulePkg variant. > The last patch, finally, removes the thing. > > Tested by booting in QEMU (Q35 (AHCI) and PC (IDE)). > More testing from other, alternative platforms is desired, although break= age seems unlikely. > > This patchset (or at least, the last patch) depends on the edk2-platforms= patch series to get merged. > > (+CC Laszlo as the author of the original SataControllerDxe patches) > > PR: https://github.com/tianocore/edk2/pull/4370 > > Cc: Ard Biesheuvel > Cc: Jiewen Yao > Cc :Jordan Justen > Cc: Gerd Hoffmann > Cc: Erdem Aktas > Cc: James Bottomley > Cc: Min Xu > Cc: Tom Lendacky > Cc: Michael Roth > Cc: Rebecca Cran > Cc: Peter Grehan > Cc: Corvin K=C3=B6hne > Cc: Sebastien Boeuf > Cc: Anthony Perard > Cc: Julien Grall > Cc: Laszlo Ersek > > Changes: > v3: > - Pick up Reviewed-by and Acked-by tags > v2: > - Address Laszlo's feedback wrt commit to port into MdeModulePkg > - Address Laszlo's feedback wrt how to split the removal of the driver = and the patches for each sub-platform maintainer > - Address Mike Maslenkin's feedback on the preexisting ASSERTs (Private= !=3D NULL) > - Add Gerd's ACK and Tested-by (conservatively) to patches 6 and 12 (si= nce they are remotely QEMU related) > > Pedro Falcato (12): > MdeModulePkg/SataControllerDxe: Clean up error handling in Start() > MdeModulePkg/SataControllerDxe: Log expected errors at DEBUG_INFO > level > MdeModulePkg/SataControllerDxe: Remove useless null check > MdeModulePkg/SataControllerDxe: Fix up ASSERTS (Private !=3D NULL) > OvmfPkg: Replace the OVMF-specific SataControllerDxe > OvmfPkg/Microvm: Replace the OVMF-specific SataControllerDxe > OvmfPkg/Bhyve: Replace the OVMF-specific SataControllerDxe > OvmfPkg/CloudHv: Replace the OVMF-specific SataControllerDxe > OvmfPkg/IntelTdx: Replace the OVMF-specific SataControllerDxe > OvmfPkg/AmdSev: Replace the OVMF-specific SataControllerDxe > OvmfPkg/Xen: Replace the OVMF-specific SataControllerDxe > OvmfPkg: Remove SataControllerDxe > Merged as #4466 Thanks!