From nobody Wed May 8 14:43:16 2024 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zoho.com: domain of groups.io designates 66.175.222.12 as permitted sender) smtp.mailfrom=bounce+27952+43115+1787277+3901457@groups.io ARC-Seal: i=1; a=rsa-sha256; t=1561978234; cv=none; d=zoho.com; s=zohoarc; b=lmPDxf2xx2+axbSoTy2XlsfNuGnNmxT/nxwu5GexuvQtzOfXoNpyxjy7ZnlC8dt8Y0cymbcH2/MJ7p7PJpqKlPvZW1UuPIozatbY2PUtcYTWPtC3kroCT6e8hw/cG4Fl8Q7mWcQrBLBTrxz18gWg/IP3Usm0K+e4+4ioMc3fQtI= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zoho.com; s=zohoarc; t=1561978234; h=Content-Type:Content-Transfer-Encoding:Cc:Date:From:List-Id:List-Unsubscribe:MIME-Version:Message-ID:Reply-To:Sender:Subject:To:ARC-Authentication-Results; bh=kfRqZzv1wfmyAnk8Wh1njzBcAEDgU7I0MTJNRMyCCjU=; b=F8PvQCHgP3LTv1CTQAuhfjCaVqaaJxG6kha6JV9hVv0iUPf+XU6UvEnv+Sb2UUUEORkvKbCbMCkWq0iApXlSF/KOSqhdiT5nMsZz7bdIGtc1eOhpGCqCqQRriR1wqj349w3QmkrLxhKYVaOtMpXXY369aHZxozDPxEITff+plzY= ARC-Authentication-Results: i=1; mx.zoho.com; dkim=pass; spf=pass (zoho.com: domain of groups.io designates 66.175.222.12 as permitted sender) smtp.mailfrom=bounce+27952+43115+1787277+3901457@groups.io Received: from web01.groups.io (web01.groups.io [66.175.222.12]) by mx.zohomail.com with SMTPS id 1561978234839661.3067659430769; Mon, 1 Jul 2019 03:50:34 -0700 (PDT) Return-Path: X-Received: from esa2.hc3370-68.iphmx.com (esa2.hc3370-68.iphmx.com [216.71.145.153]) by groups.io with SMTP; Mon, 01 Jul 2019 03:50:34 -0700 Received-SPF: pass (zoho.com: domain of groups.io designates 66.175.222.12 as permitted sender) client-ip=66.175.222.12; envelope-from=bounce+27952+43115+1787277+3901457@groups.io; helo=web01.groups.io; Received-SPF: None (esa2.hc3370-68.iphmx.com: no sender authenticity information available from domain of anthony.perard@citrix.com) identity=pra; client-ip=162.221.158.21; receiver=esa2.hc3370-68.iphmx.com; envelope-from="anthony.perard@citrix.com"; x-sender="anthony.perard@citrix.com"; x-conformance=sidf_compatible Received-SPF: Pass (esa2.hc3370-68.iphmx.com: domain of anthony.perard@citrix.com designates 162.221.158.21 as permitted sender) identity=mailfrom; client-ip=162.221.158.21; receiver=esa2.hc3370-68.iphmx.com; envelope-from="anthony.perard@citrix.com"; x-sender="anthony.perard@citrix.com"; x-conformance=sidf_compatible; x-record-type="v=spf1"; x-record-text="v=spf1 ip4:209.167.231.154 ip4:178.63.86.133 ip4:195.66.111.40/30 ip4:85.115.9.32/28 ip4:199.102.83.4 ip4:192.28.146.160 ip4:192.28.146.107 ip4:216.52.6.88 ip4:216.52.6.188 ip4:162.221.158.21 ip4:162.221.156.83 ~all" Received-SPF: None (esa2.hc3370-68.iphmx.com: no sender authenticity information available from domain of postmaster@mail.citrix.com) identity=helo; client-ip=162.221.158.21; receiver=esa2.hc3370-68.iphmx.com; envelope-from="anthony.perard@citrix.com"; x-sender="postmaster@mail.citrix.com"; x-conformance=sidf_compatible IronPort-SDR: DfyOM9HgXRM3aubaEyi7SKB//V8Z70QUBAbmNUgbi2R7FZhNgo0nwxsAw43vw8OjmD0k6TFjar wwvAe0NYxOK+jhgLZr4EFKw5UI93oPQRDv7m4TA9olXzVtCfXcWCom6ApypN1UFlIGGqQicVOw SkiANl/aS9aXwib4RACNHCnWZBBfDdxo+6a9qJQ7ChFCKNPaPl+AWoieOeruWOK8qwJSH+C4M5 TMdVZJjm8FBc9teHqVLJ5WzCgXCLbTnuBs2J9NtBrl4+9tiKTB8c6hIoP5wE4j3MrqFBOOcRNV N2w= X-SBRS: 2.7 X-MesageID: 2425670 X-Ironport-Server: esa2.hc3370-68.iphmx.com X-Remote-IP: 162.221.158.21 X-Policy: $RELAYED X-IronPort-AV: E=Sophos;i="5.63,439,1557201600"; d="scan'208";a="2425670" From: "Anthony PERARD" To: CC: Jordan Justen , Laszlo Ersek , Ard Biesheuvel , Julien Grall , Anthony PERARD Subject: [edk2-devel] [PATCH v2] OvmfPkg/XenBusDxe: Close XenIoProtocol openned by children Date: Mon, 1 Jul 2019 11:50:12 +0100 Message-ID: <20190701105012.25758-1-anthony.perard@citrix.com> MIME-Version: 1.0 Precedence: Bulk List-Unsubscribe: Sender: devel@edk2.groups.io List-Id: Mailing-List: list devel@edk2.groups.io; contact devel+owner@edk2.groups.io Reply-To: devel@edk2.groups.io,anthony.perard@citrix.com Content-Transfer-Encoding: quoted-printable DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=groups.io; q=dns/txt; s=20140610; t=1561978234; bh=vz1PYKOucr1Zqz9bcL3yNbo1hjumNWv08aFxvVJd8Gw=; h=CC:Content-Type:Date:From:Reply-To:Subject:To; b=TwAu9xjB7VjhrRKTJGPX96iyZfjFkIhILN+zMJ/0qSatOs3968LOXzCZPjpyHrFaghz Xo1rdfUrWfnUVK+Rf8jRHLtQfYHmj9jDoCZ1EmVVtmwORHxHKAjy20KZU6oHwQWFwRu5e otx86aX4BWt7n0rbLUgH7JZCLGWpTYfRoeU= X-ZohoMail-DKIM: pass (identity @groups.io) Content-Type: text/plain; charset="utf-8" In XenBusDxe, the XenBusAddDevice() opens the gXenIoProtocolGuid on behalf of child controllers. It is never closed and prevents us from uninstalling the protocol. Close it where we stop all the children in XenBusDxe->Stop(). Signed-off-by: Anthony PERARD Reviewed-by: Laszlo Ersek --- OvmfPkg/XenBusDxe/XenBusDxe.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/OvmfPkg/XenBusDxe/XenBusDxe.c b/OvmfPkg/XenBusDxe/XenBusDxe.c index 0e63707f50..7c07a96650 100644 --- a/OvmfPkg/XenBusDxe/XenBusDxe.c +++ b/OvmfPkg/XenBusDxe/XenBusDxe.c @@ -453,6 +453,10 @@ XenBusDxeDriverBindingStop ( continue; } =20 + Status =3D gBS->CloseProtocol (Dev->ControllerHandle, &gXenIoProtocolG= uid, + Dev->This->DriverBindingHandle, ChildData->Handle); + ASSERT_EFI_ERROR (Status); + Status =3D gBS->UninstallMultipleProtocolInterfaces ( ChildData->Handle, &gEfiDevicePathProtocolGuid, ChildData->DevicePath, --=20 Anthony PERARD -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#43115): https://edk2.groups.io/g/devel/message/43115 Mute This Topic: https://groups.io/mt/32270096/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-