From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) by mx.groups.io with SMTP id smtpd.web11.29896.1683754178148005392 for ; Wed, 10 May 2023 14:29:38 -0700 Authentication-Results: mx.groups.io; dkim=fail reason="unable to parse pub key" header.i=@intel.com header.s=intel header.b=I+U7kXvI; spf=pass (domain: intel.com, ip: 134.134.136.24, mailfrom: gua.guo@intel.com) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1683754178; x=1715290178; h=from:to:cc:subject:date:message-id:mime-version: content-transfer-encoding; bh=gKGkaoSA1w3kxA4nTWAiqBC6Fb23IcfHv49fYk/JorU=; b=I+U7kXvIaxXJzZR9QCQj41xDKz5TX/c0u8f3SmltdccPXEW326q8nPjQ MM7bkua2Fz/PO4ffuNIUeMDxFWbqYLmj7ibE7lCWyAkSjYJNyE/htLwAd lumj3nv1oazKgInm9uiZ7ya+qCZL43JAmnUpeLYfBx1x81gB0YPzMi4Y1 Rr/7iPJkDjgg2+PKohBMC4QGMT8iImRhZJyz0LrKTRwfswTz4MUM8qCN+ pNSXCpmP2UZ1+s8GWg7cdwQnhDbTo3ziljWCB+SM9ZodFPe+xM6CoomfP 6OqsUUqSVhyoRTL6hBdoYnAriLQj6uvtvHmoHOgsNHo6ywSgp5wzL/zAm A==; X-IronPort-AV: E=McAfee;i="6600,9927,10706"; a="352528793" X-IronPort-AV: E=Sophos;i="5.99,265,1677571200"; d="scan'208";a="352528793" Received: from orsmga005.jf.intel.com ([10.7.209.41]) by orsmga102.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 10 May 2023 14:29:37 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6600,9927,10706"; a="873699769" X-IronPort-AV: E=Sophos;i="5.99,265,1677571200"; d="scan'208";a="873699769" Received: from gguo-desk.gar.corp.intel.com ([10.5.215.23]) by orsmga005.jf.intel.com with ESMTP; 10 May 2023 14:29:36 -0700 From: "Guo, Gua" To: devel@edk2.groups.io Cc: gua.guo@intel.com Subject: [PATCH v6 0/4] Trace Hub debug library support Date: Thu, 11 May 2023 05:29:27 +0800 Message-Id: <20230510212931.311-1-gua.guo@intel.com> X-Mailer: git-send-email 2.39.2.windows.1 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit From: Gua Guo V6: - https://github.com/tianocore/edk2/pull/3901 Split submodule into new patch Fix formatting issue Add Review-by message Offline check with Victor to use my sign-off name. V5: if no other open, it will be final change - https://github.com/tianocore/edk2/pull/3901 Fix random exception when long run catalog debug message V4 - https://github.com/tianocore/edk2/pull/3901 - Done Enhance SwapBytesGuid to use CopyGuid instead of CopyMem, to make implement code more simple. V3 - https://github.com/tianocore/edk2/pull/3901 - Done - Open: MdeModulePkg/Include/Guid/TraceHubDebugInfoHob.h: why MAX_TRACE_HUB_DEBUG_INSTANCE hardcoded to 5? Solution: Remove this macro, use Library Constructor to allocate it dynamiclly. - Open: MdeModulePkg/Library/TraceHubDebugSysTLib/InternalTraceHubApiCommon.c: SwapBytesGuid () algorithm wrong. Solution: Follow correct algorithm to implement it. VOID EFIAPI SwapBytesGuid ( IN GUID *Guid, <----------- In PreMem, guid is global data so region is readonly, add output data to support it. OUT GUID *ConvertedGuid ); - Open: Merge MSFT and GCC CC_FLAGS as they both supports -D Solution: use *_*_*_CC_FLAGS = -DMIPI_SYST_STATIC to unified both. V2 - https://github.com/tianocore/edk2/pull/3901 - Open: MdeModulePkg/Include/Guid/TraceHubDebugInfoHob.h: why MAX_TRACE_HUB_DEBUG_INSTANCE hardcoded to 5? - Open: MdeModulePkg/Library/TraceHubDebugSysTLib/InternalTraceHubApiCommon.c: SwapBytesGuid () algorithm wrong. - Open: Merge MSFT and GCC CC_FLAGS as they both supports -D V1 Previous PR: - https://github.com/tianocore/edk2/pull/3613 - TraceHubDebugLib without submodule - Reject - https://github.com/tianocore/edk2/pull/3793 - TraceHubDebugLib with submodule and without seperate into MipiSysTLib and TraceHubDebugLib - Reject Gua Guo (4): MdePkg: Add MipiSysTLib library MdePkg: Add NULL library of TraceHubDebugSysTLib MdeModulePkg: Add TraceHubDebugSysTLib library Maintainers.txt: Update reviewers and maintainers for TraceHubDebugLib. Maintainers.txt | 18 + .../Include/Guid/TraceHubDebugInfoHob.h | 24 + .../BaseTraceHubDebugSysTLib.c | 245 ++++++ .../BaseTraceHubDebugSysTLib.inf | 44 + .../DxeSmmTraceHubDebugSysTLib.c | 263 ++++++ .../DxeSmmTraceHubDebugSysTLib.inf | 51 ++ .../InternalTraceHubApi.c | 74 ++ .../InternalTraceHubApi.h | 37 + .../InternalTraceHubApiCommon.c | 200 +++++ .../InternalTraceHubApiCommon.h | 119 +++ .../PeiTraceHubDebugSysTLib.c | 282 +++++++ .../PeiTraceHubDebugSysTLib.inf | 50 ++ .../Library/TraceHubDebugSysTLib/Readme.md | 26 + MdeModulePkg/MdeModulePkg.dec | 21 + MdeModulePkg/MdeModulePkg.dsc | 3 + MdeModulePkg/MdeModulePkg.uni | 18 + MdePkg/Include/Library/MipiSysTLib.h | 66 ++ MdePkg/Include/Library/TraceHubDebugSysTLib.h | 81 ++ MdePkg/Library/MipiSysTLib/GenMipiSystH.py | 132 +++ MdePkg/Library/MipiSysTLib/MipiSysTLib.c | 123 +++ MdePkg/Library/MipiSysTLib/MipiSysTLib.inf | 52 ++ MdePkg/Library/MipiSysTLib/Platform.c | 164 ++++ MdePkg/Library/MipiSysTLib/Platform.h | 138 +++ MdePkg/Library/MipiSysTLib/Readme.md | 25 + MdePkg/Library/MipiSysTLib/mipi_syst.h | 789 ++++++++++++++++++ .../TraceHubDebugSysTLibNull.c | 76 ++ .../TraceHubDebugSysTLibNull.inf | 29 + MdePkg/MdePkg.ci.yaml | 12 +- MdePkg/MdePkg.dec | 9 + MdePkg/MdePkg.dsc | 2 + 30 files changed, 3170 insertions(+), 3 deletions(-) create mode 100644 MdeModulePkg/Include/Guid/TraceHubDebugInfoHob.h create mode 100644 MdeModulePkg/Library/TraceHubDebugSysTLib/BaseTraceHubDebugSysTLib.c create mode 100644 MdeModulePkg/Library/TraceHubDebugSysTLib/BaseTraceHubDebugSysTLib.inf create mode 100644 MdeModulePkg/Library/TraceHubDebugSysTLib/DxeSmmTraceHubDebugSysTLib.c create mode 100644 MdeModulePkg/Library/TraceHubDebugSysTLib/DxeSmmTraceHubDebugSysTLib.inf create mode 100644 MdeModulePkg/Library/TraceHubDebugSysTLib/InternalTraceHubApi.c create mode 100644 MdeModulePkg/Library/TraceHubDebugSysTLib/InternalTraceHubApi.h create mode 100644 MdeModulePkg/Library/TraceHubDebugSysTLib/InternalTraceHubApiCommon.c create mode 100644 MdeModulePkg/Library/TraceHubDebugSysTLib/InternalTraceHubApiCommon.h create mode 100644 MdeModulePkg/Library/TraceHubDebugSysTLib/PeiTraceHubDebugSysTLib.c create mode 100644 MdeModulePkg/Library/TraceHubDebugSysTLib/PeiTraceHubDebugSysTLib.inf create mode 100644 MdeModulePkg/Library/TraceHubDebugSysTLib/Readme.md create mode 100644 MdePkg/Include/Library/MipiSysTLib.h create mode 100644 MdePkg/Include/Library/TraceHubDebugSysTLib.h create mode 100644 MdePkg/Library/MipiSysTLib/GenMipiSystH.py create mode 100644 MdePkg/Library/MipiSysTLib/MipiSysTLib.c create mode 100644 MdePkg/Library/MipiSysTLib/MipiSysTLib.inf create mode 100644 MdePkg/Library/MipiSysTLib/Platform.c create mode 100644 MdePkg/Library/MipiSysTLib/Platform.h create mode 100644 MdePkg/Library/MipiSysTLib/Readme.md create mode 100644 MdePkg/Library/MipiSysTLib/mipi_syst.h create mode 100644 MdePkg/Library/TraceHubDebugSysTLibNull/TraceHubDebugSysTLibNull.c create mode 100644 MdePkg/Library/TraceHubDebugSysTLibNull/TraceHubDebugSysTLibNull.inf -- 2.39.2.windows.1