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.5085.1585888656321183632 for ; Thu, 02 Apr 2020 21:37:36 -0700 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: intel.com, ip: 134.134.136.24, mailfrom: liming.gao@intel.com) IronPort-SDR: 0l88Yopc8/XGpskjIYHwqAURduzTqMEPpqfHfL/ZPVM8oBISIqzazkB55a3FHAjUjZO6AZDh4L 3Xd45U/zNJLA== X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga004.jf.intel.com ([10.7.209.38]) by orsmga102.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 02 Apr 2020 21:37:35 -0700 IronPort-SDR: USBr5ZIZUJXJAPtpnmRquuEi43w5cRgqfHAKPfVs6uSNf8+USpHtmA8HfLCN3KvflJXwvSju00 U5pxUHZjyuYg== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.72,338,1580803200"; d="scan'208";a="396604962" Received: from fmsmsx105.amr.corp.intel.com ([10.18.124.203]) by orsmga004.jf.intel.com with ESMTP; 02 Apr 2020 21:37:35 -0700 Received: from shsmsx601.ccr.corp.intel.com (10.109.6.141) by FMSMSX105.amr.corp.intel.com (10.18.124.203) with Microsoft SMTP Server (TLS) id 14.3.439.0; Thu, 2 Apr 2020 21:37:35 -0700 Received: from shsmsx604.ccr.corp.intel.com (10.109.6.214) by SHSMSX601.ccr.corp.intel.com (10.109.6.141) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.1713.5; Fri, 3 Apr 2020 12:37:33 +0800 Received: from shsmsx604.ccr.corp.intel.com ([10.109.6.214]) by SHSMSX604.ccr.corp.intel.com ([10.109.6.214]) with mapi id 15.01.1713.004; Fri, 3 Apr 2020 12:37:33 +0800 From: "Liming Gao" To: "Zhang, Shenglei" , "devel@edk2.groups.io" CC: "Wang, Jian J" , "Wu, Hao A" , "Feng, Bob C" Subject: Re: [PATCH v2 0/2] Use submodule way to access brotli Thread-Topic: [PATCH v2 0/2] Use submodule way to access brotli Thread-Index: AQHWCAAGN53jdUvyOUyMNFkoWLSTbKhm0yYg Date: Fri, 3 Apr 2020 04:37:32 +0000 Message-ID: <8e23fcacda4c4b6a8abf6417c9324c83@intel.com> References: <20200401083157.23960-1-shenglei.zhang@intel.com> In-Reply-To: <20200401083157.23960-1-shenglei.zhang@intel.com> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: dlp-version: 11.2.0.6 dlp-product: dlpe-windows dlp-reaction: no-action x-originating-ip: [10.239.127.36] MIME-Version: 1.0 Return-Path: liming.gao@intel.com Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Reviewed-by: Liming Gao > -----Original Message----- > From: Zhang, Shenglei > Sent: Wednesday, April 1, 2020 4:32 PM > To: devel@edk2.groups.io > Cc: Gao, Liming ; Wang, Jian J ; Wu, Hao A ; Feng, Bob C > > Subject: [PATCH v2 0/2] Use submodule way to access brotli >=20 > Currently brotli is used and customized by edk2 in BaseTools > and MdeModulePkg. These two patches make brotli a submodule in > edk2. > https://bugzilla.tianocore.org/show_bug.cgi?id=3D2558 > https://bugzilla.tianocore.org/show_bug.cgi?id=3D2559 >=20 > v2: Add submodule path in CISettings.py >=20 > Cc: Liming Gao > Cc: Jian J Wang > Cc: Hao A Wu > Cc: Bob Feng > Shenglei Zhang (2): > MdeModulePkg/BrotliCustomDecompressLib: Make brotli a submodule > BaseTools: Make brotli a submodule >=20 > .../C/BrotliCompress/common/dictionary.c | 5905 ----------------- > .../C/BrotliCompress/common/transform.c | 235 - > .../Source/C/BrotliCompress/dec/bit_reader.c | 48 - > .../Source/C/BrotliCompress/dec/decode.c | 2497 ------- > .../Source/C/BrotliCompress/dec/huffman.c | 356 - > BaseTools/Source/C/BrotliCompress/dec/state.c | 164 - > .../BrotliCompress/enc/backward_references.c | 144 - > .../enc/backward_references_hq.c | 830 --- > .../Source/C/BrotliCompress/enc/bit_cost.c | 35 - > .../C/BrotliCompress/enc/block_splitter.c | 194 - > .../C/BrotliCompress/enc/brotli_bit_stream.c | 1331 ---- > .../Source/C/BrotliCompress/enc/cluster.c | 56 - > .../C/BrotliCompress/enc/compress_fragment.c | 790 --- > .../enc/compress_fragment_two_pass.c | 645 -- > .../C/BrotliCompress/enc/dictionary_hash.c | 1120 ---- > .../Source/C/BrotliCompress/enc/encode.c | 1864 ------ > .../C/BrotliCompress/enc/encoder_dict.c | 32 - > .../C/BrotliCompress/enc/entropy_encode.c | 501 -- > .../Source/C/BrotliCompress/enc/histogram.c | 100 - > .../C/BrotliCompress/enc/literal_cost.c | 175 - > .../Source/C/BrotliCompress/enc/memory.c | 170 - > .../Source/C/BrotliCompress/enc/metablock.c | 667 -- > .../Source/C/BrotliCompress/enc/static_dict.c | 486 -- > .../Source/C/BrotliCompress/enc/utf8_util.c | 85 - > .../Source/C/BrotliCompress/tools/brotli.c | 1143 ---- > .../BrotliDecUefiSupport.c | 31 + > .../common/dictionary.c | 5905 ----------------- > .../common/transform.c | 235 - > .../dec/bit_reader.c | 48 - > .../BrotliCustomDecompressLib/dec/decode.c | 2500 ------- > .../BrotliCustomDecompressLib/dec/huffman.c | 356 - > .../BrotliCustomDecompressLib/dec/state.c | 164 - > .gitmodules | 6 + > .pytool/CISettings.py | 4 + > BaseTools/Source/C/BrotliCompress/GNUmakefile | 54 +- > BaseTools/Source/C/BrotliCompress/LICENSE | 19 - > BaseTools/Source/C/BrotliCompress/Makefile | 52 +- > BaseTools/Source/C/BrotliCompress/README.md | 26 - > BaseTools/Source/C/BrotliCompress/ReadMe.txt | 2 - > BaseTools/Source/C/BrotliCompress/brotli | 1 + > .../C/BrotliCompress/common/constants.h | 64 - > .../Source/C/BrotliCompress/common/context.h | 261 - > .../C/BrotliCompress/common/dictionary.h | 64 - > .../Source/C/BrotliCompress/common/platform.h | 558 -- > .../C/BrotliCompress/common/transform.h | 80 - > .../Source/C/BrotliCompress/common/version.h | 26 - > .../Source/C/BrotliCompress/dec/bit_reader.h | 309 - > .../Source/C/BrotliCompress/dec/huffman.h | 72 - > .../Source/C/BrotliCompress/dec/prefix.h | 750 --- > BaseTools/Source/C/BrotliCompress/dec/state.h | 258 - > .../brotli-comparison-study-2015-09-22.pdf | Bin 215208 -> 0 bytes > .../BrotliCompress/enc/backward_references.h | 38 - > .../enc/backward_references_hq.h | 93 - > .../enc/backward_references_inc.h | 153 - > .../Source/C/BrotliCompress/enc/bit_cost.h | 63 - > .../C/BrotliCompress/enc/bit_cost_inc.h | 127 - > .../C/BrotliCompress/enc/block_encoder_inc.h | 34 - > .../C/BrotliCompress/enc/block_splitter.h | 51 - > .../C/BrotliCompress/enc/block_splitter_inc.h | 431 -- > .../C/BrotliCompress/enc/brotli_bit_stream.h | 84 - > .../Source/C/BrotliCompress/enc/cluster.h | 48 - > .../Source/C/BrotliCompress/enc/cluster_inc.h | 317 - > .../Source/C/BrotliCompress/enc/command.h | 190 - > .../C/BrotliCompress/enc/compress_fragment.h | 61 - > .../enc/compress_fragment_two_pass.h | 54 - > .../C/BrotliCompress/enc/dictionary_hash.h | 24 - > .../C/BrotliCompress/enc/encoder_dict.h | 41 - > .../C/BrotliCompress/enc/entropy_encode.h | 122 - > .../enc/entropy_encode_static.h | 539 -- > .../Source/C/BrotliCompress/enc/fast_log.h | 147 - > .../C/BrotliCompress/enc/find_match_length.h | 80 - > BaseTools/Source/C/BrotliCompress/enc/hash.h | 497 -- > .../C/BrotliCompress/enc/hash_composite_inc.h | 133 - > .../enc/hash_forgetful_chain_inc.h | 254 - > .../enc/hash_longest_match64_inc.h | 266 - > .../enc/hash_longest_match_inc.h | 258 - > .../enc/hash_longest_match_quickly_inc.h | 235 - > .../C/BrotliCompress/enc/hash_rolling_inc.h | 215 - > .../enc/hash_to_binary_tree_inc.h | 327 - > .../Source/C/BrotliCompress/enc/histogram.h | 63 - > .../C/BrotliCompress/enc/histogram_inc.h | 51 - > .../C/BrotliCompress/enc/literal_cost.h | 30 - > .../Source/C/BrotliCompress/enc/memory.h | 102 - > .../Source/C/BrotliCompress/enc/metablock.h | 105 - > .../C/BrotliCompress/enc/metablock_inc.h | 183 - > .../Source/C/BrotliCompress/enc/params.h | 44 - > .../Source/C/BrotliCompress/enc/prefix.h | 53 - > .../Source/C/BrotliCompress/enc/quality.h | 165 - > .../Source/C/BrotliCompress/enc/ringbuffer.h | 164 - > .../Source/C/BrotliCompress/enc/static_dict.h | 40 - > .../C/BrotliCompress/enc/static_dict_lut.h | 5864 ---------------- > .../Source/C/BrotliCompress/enc/utf8_util.h | 32 - > .../Source/C/BrotliCompress/enc/write_bits.h | 85 - > .../C/BrotliCompress/include/brotli/decode.h | 344 - > .../C/BrotliCompress/include/brotli/encode.h | 427 -- > .../C/BrotliCompress/include/brotli/port.h | 274 - > .../C/BrotliCompress/include/brotli/types.h | 83 - > .../Source/C/BrotliCompress/tools/brotli.md | 107 - > .../BrotliCustomDecompressLib.inf | 48 +- > .../BrotliDecUefiSupport.h | 43 + > .../BrotliDecompressLibInternal.h | 23 +- > .../Library/BrotliCustomDecompressLib/LICENSE | 19 - > .../BrotliCustomDecompressLib/README.md | 26 - > .../BrotliCustomDecompressLib/ReadMe.txt | 2 - > .../Library/BrotliCustomDecompressLib/brotli | 1 + > .../BrotliCustomDecompressLib/brotli/decode.h | 344 - > .../BrotliCustomDecompressLib/brotli/port.h | 274 - > .../BrotliCustomDecompressLib/brotli/types.h | 92 - > .../common/constants.h | 64 - > .../common/context.h | 261 - > .../common/dictionary.h | 64 - > .../common/platform.h | 559 -- > .../common/transform.h | 80 - > .../common/version.h | 26 - > .../dec/bit_reader.h | 309 - > .../BrotliCustomDecompressLib/dec/huffman.h | 72 - > .../BrotliCustomDecompressLib/dec/prefix.h | 750 --- > .../BrotliCustomDecompressLib/dec/state.h | 258 - > .../brotli-comparison-study-2015-09-22.pdf | Bin 215208 -> 0 bytes > .../BrotliCustomDecompressLib/stddef.h | 9 + > .../BrotliCustomDecompressLib/stdint.h | 9 + > .../BrotliCustomDecompressLib/stdlib.h | 9 + > .../BrotliCustomDecompressLib/string.h | 9 + > MdeModulePkg/MdeModulePkg.dec | 2 + > Readme.md | 6 +- > 125 files changed, 212 insertions(+), 47663 deletions(-) > delete mode 100644 BaseTools/Source/C/BrotliCompress/common/dictionary.c > delete mode 100644 BaseTools/Source/C/BrotliCompress/common/transform.c > delete mode 100644 BaseTools/Source/C/BrotliCompress/dec/bit_reader.c > delete mode 100644 BaseTools/Source/C/BrotliCompress/dec/decode.c > delete mode 100644 BaseTools/Source/C/BrotliCompress/dec/huffman.c > delete mode 100644 BaseTools/Source/C/BrotliCompress/dec/state.c > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/backward_refere= nces.c > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/backward_refere= nces_hq.c > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/bit_cost.c > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/block_splitter.= c > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/brotli_bit_stre= am.c > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/cluster.c > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/compress_fragme= nt.c > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/compress_fragme= nt_two_pass.c > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/dictionary_hash= .c > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/encode.c > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/encoder_dict.c > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/entropy_encode.= c > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/histogram.c > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/literal_cost.c > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/memory.c > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/metablock.c > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/static_dict.c > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/utf8_util.c > delete mode 100644 BaseTools/Source/C/BrotliCompress/tools/brotli.c > create mode 100644 MdeModulePkg/Library/BrotliCustomDecompressLib/Brotli= DecUefiSupport.c > delete mode 100644 MdeModulePkg/Library/BrotliCustomDecompressLib/common= /dictionary.c > delete mode 100644 MdeModulePkg/Library/BrotliCustomDecompressLib/common= /transform.c > delete mode 100644 MdeModulePkg/Library/BrotliCustomDecompressLib/dec/bi= t_reader.c > delete mode 100644 MdeModulePkg/Library/BrotliCustomDecompressLib/dec/de= code.c > delete mode 100644 MdeModulePkg/Library/BrotliCustomDecompressLib/dec/hu= ffman.c > delete mode 100644 MdeModulePkg/Library/BrotliCustomDecompressLib/dec/st= ate.c > delete mode 100644 BaseTools/Source/C/BrotliCompress/LICENSE > delete mode 100644 BaseTools/Source/C/BrotliCompress/README.md > delete mode 100644 BaseTools/Source/C/BrotliCompress/ReadMe.txt > create mode 160000 BaseTools/Source/C/BrotliCompress/brotli > delete mode 100644 BaseTools/Source/C/BrotliCompress/common/constants.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/common/context.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/common/dictionary.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/common/platform.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/common/transform.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/common/version.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/dec/bit_reader.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/dec/huffman.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/dec/prefix.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/dec/state.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/docs/brotli-compari= son-study-2015-09-22.pdf > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/backward_refere= nces.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/backward_refere= nces_hq.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/backward_refere= nces_inc.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/bit_cost.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/bit_cost_inc.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/block_encoder_i= nc.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/block_splitter.= h > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/block_splitter_= inc.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/brotli_bit_stre= am.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/cluster.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/cluster_inc.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/command.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/compress_fragme= nt.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/compress_fragme= nt_two_pass.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/dictionary_hash= .h > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/encoder_dict.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/entropy_encode.= h > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/entropy_encode_= static.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/fast_log.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/find_match_leng= th.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/hash.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/hash_composite_= inc.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/hash_forgetful_= chain_inc.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/hash_longest_ma= tch64_inc.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/hash_longest_ma= tch_inc.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/hash_longest_ma= tch_quickly_inc.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/hash_rolling_in= c.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/hash_to_binary_= tree_inc.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/histogram.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/histogram_inc.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/literal_cost.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/memory.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/metablock.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/metablock_inc.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/params.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/prefix.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/quality.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/ringbuffer.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/static_dict.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/static_dict_lut= .h > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/utf8_util.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/write_bits.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/include/brotli/deco= de.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/include/brotli/enco= de.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/include/brotli/port= .h > delete mode 100644 BaseTools/Source/C/BrotliCompress/include/brotli/type= s.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/tools/brotli.md > create mode 100644 MdeModulePkg/Library/BrotliCustomDecompressLib/Brotli= DecUefiSupport.h > delete mode 100644 MdeModulePkg/Library/BrotliCustomDecompressLib/LICENS= E > delete mode 100644 MdeModulePkg/Library/BrotliCustomDecompressLib/README= .md > delete mode 100644 MdeModulePkg/Library/BrotliCustomDecompressLib/ReadMe= .txt > create mode 160000 MdeModulePkg/Library/BrotliCustomDecompressLib/brotli > delete mode 100644 MdeModulePkg/Library/BrotliCustomDecompressLib/brotli= /decode.h > delete mode 100644 MdeModulePkg/Library/BrotliCustomDecompressLib/brotli= /port.h > delete mode 100644 MdeModulePkg/Library/BrotliCustomDecompressLib/brotli= /types.h > delete mode 100644 MdeModulePkg/Library/BrotliCustomDecompressLib/common= /constants.h > delete mode 100644 MdeModulePkg/Library/BrotliCustomDecompressLib/common= /context.h > delete mode 100644 MdeModulePkg/Library/BrotliCustomDecompressLib/common= /dictionary.h > delete mode 100644 MdeModulePkg/Library/BrotliCustomDecompressLib/common= /platform.h > delete mode 100644 MdeModulePkg/Library/BrotliCustomDecompressLib/common= /transform.h > delete mode 100644 MdeModulePkg/Library/BrotliCustomDecompressLib/common= /version.h > delete mode 100644 MdeModulePkg/Library/BrotliCustomDecompressLib/dec/bi= t_reader.h > delete mode 100644 MdeModulePkg/Library/BrotliCustomDecompressLib/dec/hu= ffman.h > delete mode 100644 MdeModulePkg/Library/BrotliCustomDecompressLib/dec/pr= efix.h > delete mode 100644 MdeModulePkg/Library/BrotliCustomDecompressLib/dec/st= ate.h > delete mode 100644 MdeModulePkg/Library/BrotliCustomDecompressLib/docs/b= rotli-comparison-study-2015-09-22.pdf > create mode 100644 MdeModulePkg/Library/BrotliCustomDecompressLib/stddef= .h > create mode 100644 MdeModulePkg/Library/BrotliCustomDecompressLib/stdint= .h > create mode 100644 MdeModulePkg/Library/BrotliCustomDecompressLib/stdlib= .h > create mode 100644 MdeModulePkg/Library/BrotliCustomDecompressLib/string= .h >=20 > -- > 2.18.0.windows.1