From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) by mx.groups.io with SMTP id smtpd.web11.201803.1672304296055402778 for ; Thu, 29 Dec 2022 00:58:16 -0800 Authentication-Results: mx.groups.io; dkim=fail reason="unable to parse pub key" header.i=@intel.com header.s=intel header.b=CSaWDyi4; spf=pass (domain: intel.com, ip: 134.134.136.100, mailfrom: min.m.xu@intel.com) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1672304296; x=1703840296; h=from:to:cc:subject:date:message-id:mime-version: content-transfer-encoding; bh=e3PBtvC2WbEOBvRWRtrwasQOhzdFiSUYY1dFmc1ZMrk=; b=CSaWDyi4rSH7LVKMsrDf2z/f8cGJL+6v4jgKju7J+6dlDk/Ecn5yNZ/6 mP+ZpToA0UW0XncJcQ1VMpCR23bw36Zk68QDJ6BJyPfGhYpX5pxQkjGs0 CegBvZfGCpfN1wJykLggd4IfO3EUrNMLApb5HEpsTRyZCWSxVO7gN1MVD AvY0awYGW3ww5v5GxioB52wf7vULp72xiMY1ZOtRPqH5wgi16Q5T8TGT1 +Pu5eIeadoo80kBKRa7DdYYYSxr8s6BAYa4Msxq4DqmMoEHzVFOufIN9v qgYenm+6zd9LMkYiMKgxdEzxN84SiwNsd8BL4qdTi45lgqRk67e3H0vl6 Q==; X-IronPort-AV: E=McAfee;i="6500,9779,10574"; a="385437705" X-IronPort-AV: E=Sophos;i="5.96,283,1665471600"; d="scan'208";a="385437705" Received: from orsmga001.jf.intel.com ([10.7.209.18]) by orsmga105.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 29 Dec 2022 00:58:15 -0800 X-IronPort-AV: E=McAfee;i="6500,9779,10574"; a="685925578" X-IronPort-AV: E=Sophos;i="5.96,283,1665471600"; d="scan'208";a="685925578" Received: from mxu9-mobl1.ccr.corp.intel.com ([10.249.175.30]) by orsmga001-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 29 Dec 2022 00:58:12 -0800 From: "Min Xu" To: devel@edk2.groups.io Cc: Min Xu , Erdem Aktas , James Bottomley , Jiewen Yao , Gerd Hoffmann , Tom Lendacky , Ryan Afranji Subject: [PATCH V1 0/2] Refactor TDX MmioExit Date: Thu, 29 Dec 2022 16:55:46 +0800 Message-Id: <20221229085548.476-1-min.m.xu@intel.com> X-Mailer: git-send-email 2.29.2.windows.2 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=4169 The previous TDX MmioExit doesn't handle the Mmio instructions correctly in some scenarios. This patch-set refactors the implementation to fix the issues. Before the refactoring, common X86 instruction codes in CcExitVcHandler.c are moved to separate files (CcInstruction.h / CcInstruction.c) so that these codes can be re-used in TDX. Cc: Erdem Aktas Cc: James Bottomley Cc: Jiewen Yao Cc: Gerd Hoffmann Cc: Tom Lendacky Cc: Ryan Afranji Reported-by: Ryan Afranji Signed-off-by: Min Xu Min M Xu (2): OvmfPkg/CcExitLib: Move common X86 instruction code to separate file OvmfPkg/CcExitLib: Refactor TDX MmioExit OvmfPkg/Library/CcExitLib/CcExitLib.inf | 1 + OvmfPkg/Library/CcExitLib/CcExitVcHandler.c | 697 +++----------------- OvmfPkg/Library/CcExitLib/CcExitVeHandler.c | 498 +++++++++----- OvmfPkg/Library/CcExitLib/CcInstruction.c | 454 +++++++++++++ OvmfPkg/Library/CcExitLib/CcInstruction.h | 197 ++++++ OvmfPkg/Library/CcExitLib/SecCcExitLib.inf | 1 + 6 files changed, 1082 insertions(+), 766 deletions(-) create mode 100644 OvmfPkg/Library/CcExitLib/CcInstruction.c create mode 100644 OvmfPkg/Library/CcExitLib/CcInstruction.h -- 2.29.2.windows.2