From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mx0a-001b2d01.pphosted.com (mx0a-001b2d01.pphosted.com [148.163.156.1]) by mx.groups.io with SMTP id smtpd.web10.6917.1623241123061228311 for ; Wed, 09 Jun 2021 05:18:43 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@ibm.com header.s=pp1 header.b=V0qABfmT; spf=pass (domain: linux.ibm.com, ip: 148.163.156.1, mailfrom: dovmurik@linux.ibm.com) Received: from pps.filterd (m0098393.ppops.net [127.0.0.1]) by mx0a-001b2d01.pphosted.com (8.16.0.43/8.16.0.43) with SMTP id 159CEwAh042496; Wed, 9 Jun 2021 08:18:42 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ibm.com; h=from : to : cc : subject : date : message-id : in-reply-to : references : mime-version : content-transfer-encoding; s=pp1; bh=V5rDAhw6JavRZdns8M+1RD0Vz2TBMw0eQZjcTo+fLjo=; b=V0qABfmTZp/0F1a2+BKwJyEp0Kbay7aAkJyHabHS5aNDif6JmYmrz7IKZ1+G7pL9FwZU TSYiyL0Dc7qDlxdpOHWB6We/rw6iZcT5xR6Dn6NHnvmU04jQ+AuxoGcT9kMO5JcUniMD KrdZQC1nmwtrNYWL/an7UvuZz1ZcfDtovlQisoitTXeF2FtirrYLGxZlaxmIC67yMbjA r2ygsmsjFAANMBOcvTzerd/qP+5VkTixcYPlXStk9/zZVRmXWfsOlqJoGwfyTY3+ToAZ BQHPintC3EWSnpiklvdPtKq020yH/L165dxve+tXHWZPm5aoEthFQA9glzr/xA1iyXne PA== Received: from pps.reinject (localhost [127.0.0.1]) by mx0a-001b2d01.pphosted.com with ESMTP id 392wfwr3ny-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Wed, 09 Jun 2021 08:18:42 -0400 Received: from m0098393.ppops.net (m0098393.ppops.net [127.0.0.1]) by pps.reinject (8.16.0.43/8.16.0.43) with SMTP id 159CGJvs050900; Wed, 9 Jun 2021 08:18:41 -0400 Received: from ppma01dal.us.ibm.com (83.d6.3fa9.ip4.static.sl-reverse.com [169.63.214.131]) by mx0a-001b2d01.pphosted.com with ESMTP id 392wfwr3n9-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Wed, 09 Jun 2021 08:18:41 -0400 Received: from pps.filterd (ppma01dal.us.ibm.com [127.0.0.1]) by ppma01dal.us.ibm.com (8.16.1.2/8.16.1.2) with SMTP id 159CDInZ022596; Wed, 9 Jun 2021 12:18:41 GMT Received: from b01cxnp23032.gho.pok.ibm.com (b01cxnp23032.gho.pok.ibm.com [9.57.198.27]) by ppma01dal.us.ibm.com with ESMTP id 3900w9q339-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Wed, 09 Jun 2021 12:18:40 +0000 Received: from b01ledav002.gho.pok.ibm.com (b01ledav002.gho.pok.ibm.com [9.57.199.107]) by b01cxnp23032.gho.pok.ibm.com (8.14.9/8.14.9/NCO v10.0) with ESMTP id 159CIeha30081448 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Wed, 9 Jun 2021 12:18:40 GMT Received: from b01ledav002.gho.pok.ibm.com (unknown [127.0.0.1]) by IMSVA (Postfix) with ESMTP id EBDD7124054; Wed, 9 Jun 2021 12:18:39 +0000 (GMT) Received: from b01ledav002.gho.pok.ibm.com (unknown [127.0.0.1]) by IMSVA (Postfix) with ESMTP id B81DB124052; Wed, 9 Jun 2021 12:18:39 +0000 (GMT) Received: from localhost.localdomain (unknown [9.2.130.16]) by b01ledav002.gho.pok.ibm.com (Postfix) with ESMTP; Wed, 9 Jun 2021 12:18:39 +0000 (GMT) From: "Dov Murik" To: devel@edk2.groups.io Cc: Dov Murik , Laszlo Ersek , Ard Biesheuvel , Jordan Justen , James Bottomley , Tobin Feldman-Fitzthum Subject: [PATCH v1 2/3] OvmfPkg/GenericQemuLoadImageLib: Read cmdline from QemuKernelLoaderFs Date: Wed, 9 Jun 2021 12:18:27 +0000 Message-Id: <20210609121828.1884825-3-dovmurik@linux.ibm.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20210609121828.1884825-1-dovmurik@linux.ibm.com> References: <20210609121828.1884825-1-dovmurik@linux.ibm.com> MIME-Version: 1.0 X-TM-AS-GCONF: 00 X-Proofpoint-GUID: DXv_hsXAkMVyJ2BeG808PcZdIs3fbtS3 X-Proofpoint-ORIG-GUID: 2br_elFY053P7KniHzS5pxb74q_90-d6 X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10434:6.0.391,18.0.761 definitions=2021-06-09_04:2021-06-04,2021-06-09 signatures=0 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 spamscore=0 phishscore=0 impostorscore=0 clxscore=1015 adultscore=0 priorityscore=1501 malwarescore=0 mlxlogscore=999 lowpriorityscore=0 mlxscore=0 bulkscore=0 suspectscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2104190000 definitions=main-2106090059 Content-Transfer-Encoding: quoted-printable Remove the QemuFwCfgLib interface used to read the QEMU cmdline (-append argument) and the initrd size. Instead, use the synthetic filesystem QemuKernelLoaderFs which has three files: "kernel", "initrd", and "cmdline". Cc: Laszlo Ersek Cc: Ard Biesheuvel Cc: Jordan Justen Cc: James Bottomley Cc: Tobin Feldman-Fitzthum Signed-off-by: Dov Murik --- OvmfPkg/Library/GenericQemuLoadImageLib/GenericQemuLoadImageLib.inf | 2 = +- OvmfPkg/Library/GenericQemuLoadImageLib/GenericQemuLoadImageLib.c | 132 = ++++++++++++++++++-- 2 files changed, 126 insertions(+), 8 deletions(-) diff --git a/OvmfPkg/Library/GenericQemuLoadImageLib/GenericQemuLoadImageLi= b.inf b/OvmfPkg/Library/GenericQemuLoadImageLib/GenericQemuLoadImageLib.inf index b262cb926a4d..f462fd6922cf 100644 --- a/OvmfPkg/Library/GenericQemuLoadImageLib/GenericQemuLoadImageLib.inf +++ b/OvmfPkg/Library/GenericQemuLoadImageLib/GenericQemuLoadImageLib.inf @@ -27,12 +27,12 @@ [LibraryClasses] DebugLib=0D MemoryAllocationLib=0D PrintLib=0D - QemuFwCfgLib=0D UefiBootServicesTableLib=0D =0D [Protocols]=0D gEfiDevicePathProtocolGuid=0D gEfiLoadedImageProtocolGuid=0D + gEfiSimpleFileSystemProtocolGuid=0D =0D [Guids]=0D gQemuKernelLoaderFsMediaGuid=0D diff --git a/OvmfPkg/Library/GenericQemuLoadImageLib/GenericQemuLoadImageLi= b.c b/OvmfPkg/Library/GenericQemuLoadImageLib/GenericQemuLoadImageLib.c index 114db7e8441f..7d26c912e14f 100644 --- a/OvmfPkg/Library/GenericQemuLoadImageLib/GenericQemuLoadImageLib.c +++ b/OvmfPkg/Library/GenericQemuLoadImageLib/GenericQemuLoadImageLib.c @@ -11,9 +11,9 @@ #include =0D #include =0D #include =0D +#include =0D #include =0D #include =0D -#include =0D #include =0D #include =0D #include =0D @@ -30,6 +30,11 @@ typedef struct { KERNEL_FILE_DEVPATH FileNode;=0D EFI_DEVICE_PATH_PROTOCOL EndNode;=0D } KERNEL_VENMEDIA_FILE_DEVPATH;=0D +=0D +typedef struct {=0D + VENDOR_DEVICE_PATH VenMediaNode;=0D + EFI_DEVICE_PATH_PROTOCOL EndNode;=0D +} SINGLE_VENMEDIA_NODE_DEVPATH;=0D #pragma pack ()=0D =0D STATIC CONST KERNEL_VENMEDIA_FILE_DEVPATH mKernelDevicePath =3D {=0D @@ -51,6 +56,78 @@ STATIC CONST KERNEL_VENMEDIA_FILE_DEVPATH mKernelDeviceP= ath =3D { }=0D };=0D =0D +STATIC CONST SINGLE_VENMEDIA_NODE_DEVPATH mQemuKernelLoaderFileSystemDevic= ePath =3D {=0D + {=0D + {=0D + MEDIA_DEVICE_PATH, MEDIA_VENDOR_DP,=0D + { sizeof (VENDOR_DEVICE_PATH) }=0D + },=0D + QEMU_KERNEL_LOADER_FS_MEDIA_GUID=0D + }, {=0D + END_DEVICE_PATH_TYPE, END_ENTIRE_DEVICE_PATH_SUBTYPE,=0D + { sizeof (EFI_DEVICE_PATH_PROTOCOL) }=0D + }=0D +};=0D +=0D +STATIC=0D +EFI_STATUS=0D +GetQemuKernelBlobSize (=0D + IN EFI_FILE_HANDLE Root,=0D + IN CHAR16 *FileName,=0D + OUT UINTN *Size=0D + )=0D +{=0D + EFI_STATUS Status;=0D + EFI_FILE_HANDLE FileHandle;=0D + UINT64 FileSize;=0D +=0D + Status =3D Root->Open (Root, &FileHandle, FileName, EFI_FILE_MODE_READ, = 0);=0D + if (EFI_ERROR (Status)) {=0D + return Status;=0D + }=0D + Status =3D FileHandleGetSize (FileHandle, &FileSize);=0D + if (EFI_ERROR (Status)) {=0D + goto CloseFile;=0D + }=0D + *Size =3D FileSize;=0D + Status =3D EFI_SUCCESS;=0D +CloseFile:=0D + FileHandle->Close (FileHandle);=0D + return Status;=0D +}=0D +=0D +STATIC=0D +EFI_STATUS=0D +ReadWholeQemuKernelBlob (=0D + IN EFI_FILE_HANDLE Root,=0D + IN CHAR16 *FileName,=0D + IN UINTN Size,=0D + OUT VOID *Buffer=0D + )=0D +{=0D + EFI_STATUS Status;=0D + EFI_FILE_HANDLE FileHandle;=0D + UINTN ReadSize;=0D +=0D + Status =3D Root->Open (Root, &FileHandle, FileName, EFI_FILE_MODE_READ, = 0);=0D + if (EFI_ERROR (Status)) {=0D + return Status;=0D + }=0D + ReadSize =3D Size;=0D + Status =3D FileHandle->Read (FileHandle, &ReadSize, Buffer);=0D + if (EFI_ERROR (Status)) {=0D + goto CloseFile;=0D + }=0D + if (ReadSize !=3D Size) {=0D + Status =3D EFI_PROTOCOL_ERROR;=0D + goto CloseFile;=0D + }=0D + Status =3D EFI_SUCCESS;=0D +CloseFile:=0D + FileHandle->Close (FileHandle);=0D + return Status;=0D +}=0D +=0D /**=0D Download the kernel, the initial ramdisk, and the kernel command line fr= om=0D QEMU's fw_cfg. The kernel will be instructed via its command line to loa= d=0D @@ -79,6 +156,10 @@ QemuLoadKernelImage ( EFI_STATUS Status;=0D EFI_HANDLE KernelImageHandle;=0D EFI_LOADED_IMAGE_PROTOCOL *KernelLoadedImage;=0D + EFI_DEVICE_PATH_PROTOCOL *DevicePathNode;=0D + EFI_HANDLE FsVolumeHandle;=0D + EFI_SIMPLE_FILE_SYSTEM_PROTOCOL *FsProtocol;=0D + EFI_FILE_HANDLE Root;=0D UINTN CommandLineSize;=0D CHAR8 *CommandLine;=0D UINTN InitrdSize;=0D @@ -124,8 +205,38 @@ QemuLoadKernelImage ( );=0D ASSERT_EFI_ERROR (Status);=0D =0D - QemuFwCfgSelectItem (QemuFwCfgItemCommandLineSize);=0D - CommandLineSize =3D (UINTN)QemuFwCfgRead32 ();=0D + //=0D + // Open the Qemu Kernel Loader abstract filesystem (volume) which will b= e=0D + // used to read the "initrd" and "cmdline" synthetic files.=0D + //=0D + DevicePathNode =3D (EFI_DEVICE_PATH_PROTOCOL *) &mQemuKernelLoaderFileSy= stemDevicePath;=0D + Status =3D gBS->LocateDevicePath (=0D + &gEfiSimpleFileSystemProtocolGuid,=0D + &DevicePathNode,=0D + &FsVolumeHandle=0D + );=0D + if (EFI_ERROR (Status)) {=0D + return Status;=0D + }=0D +=0D + Status =3D gBS->HandleProtocol (=0D + FsVolumeHandle,=0D + &gEfiSimpleFileSystemProtocolGuid,=0D + (VOID **)&FsProtocol=0D + );=0D + if (EFI_ERROR (Status)) {=0D + return Status;=0D + }=0D +=0D + Status =3D FsProtocol->OpenVolume (FsVolumeHandle, &Root);=0D + if (EFI_ERROR (Status)) {=0D + return Status;=0D + }=0D +=0D + Status =3D GetQemuKernelBlobSize (Root, L"cmdline", &CommandLineSize);=0D + if (EFI_ERROR (Status)) {=0D + goto CloseRoot;=0D + }=0D =0D if (CommandLineSize =3D=3D 0) {=0D KernelLoadedImage->LoadOptionsSize =3D 0;=0D @@ -136,8 +247,10 @@ QemuLoadKernelImage ( goto UnloadImage;=0D }=0D =0D - QemuFwCfgSelectItem (QemuFwCfgItemCommandLineData);=0D - QemuFwCfgReadBytes (CommandLineSize, CommandLine);=0D + Status =3D ReadWholeQemuKernelBlob (Root, L"cmdline", CommandLineSize,= CommandLine);=0D + if (EFI_ERROR (Status)) {=0D + goto FreeCommandLine;=0D + }=0D =0D //=0D // Verify NUL-termination of the command line.=0D @@ -155,8 +268,10 @@ QemuLoadKernelImage ( KernelLoadedImage->LoadOptionsSize =3D (UINT32)((CommandLineSize - 1) = * 2);=0D }=0D =0D - QemuFwCfgSelectItem (QemuFwCfgItemInitrdSize);=0D - InitrdSize =3D (UINTN)QemuFwCfgRead32 ();=0D + Status =3D GetQemuKernelBlobSize (Root, L"initrd", &InitrdSize);=0D + if (EFI_ERROR (Status)) {=0D + goto FreeCommandLine;=0D + }=0D =0D if (InitrdSize > 0) {=0D //=0D @@ -193,6 +308,7 @@ QemuLoadKernelImage ( }=0D =0D *ImageHandle =3D KernelImageHandle;=0D + Root->Close (Root);=0D return EFI_SUCCESS;=0D =0D FreeCommandLine:=0D @@ -201,6 +317,8 @@ FreeCommandLine: }=0D UnloadImage:=0D gBS->UnloadImage (KernelImageHandle);=0D +CloseRoot:=0D + Root->Close (Root);=0D =0D return Status;=0D }=0D --=20 2.25.1