From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=192.55.52.93; helo=mga11.intel.com; envelope-from=star.zeng@intel.com; receiver=edk2-devel@lists.01.org Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 8263D21A07A92 for ; Mon, 8 Oct 2018 18:50:15 -0700 (PDT) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga005.jf.intel.com ([10.7.209.41]) by fmsmga102.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 08 Oct 2018 18:50:14 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.54,358,1534834800"; d="scan'208";a="264014677" Received: from fmsmsx104.amr.corp.intel.com ([10.18.124.202]) by orsmga005.jf.intel.com with ESMTP; 08 Oct 2018 18:50:14 -0700 Received: from fmsmsx121.amr.corp.intel.com (10.18.125.36) by fmsmsx104.amr.corp.intel.com (10.18.124.202) with Microsoft SMTP Server (TLS) id 14.3.319.2; Mon, 8 Oct 2018 18:50:13 -0700 Received: from shsmsx103.ccr.corp.intel.com (10.239.4.69) by fmsmsx121.amr.corp.intel.com (10.18.125.36) with Microsoft SMTP Server (TLS) id 14.3.319.2; Mon, 8 Oct 2018 18:50:13 -0700 Received: from shsmsx102.ccr.corp.intel.com ([169.254.2.217]) by SHSMSX103.ccr.corp.intel.com ([169.254.4.245]) with mapi id 14.03.0319.002; Tue, 9 Oct 2018 09:50:10 +0800 From: "Zeng, Star" To: "Gao, Liming" , "edk2-devel@lists.01.org" CC: "Zeng, Star" Thread-Topic: [edk2] [Patch 0/2] Update BrotliCompress to the latest version 1.0.5 Thread-Index: AQHUSJ6VKvKA3Q+QHEe44AMXbJComKUUvXeggABYHgCAAT3nMA== Date: Tue, 9 Oct 2018 01:50:09 +0000 Message-ID: <0C09AFA07DD0434D9E2A0C6AEB0483103BC104D8@shsmsx102.ccr.corp.intel.com> References: <1536539845-13332-1-git-send-email-liming.gao@intel.com> <0C09AFA07DD0434D9E2A0C6AEB0483103BC0CDFE@shsmsx102.ccr.corp.intel.com> <4A89E2EF3DFEDB4C8BFDE51014F606A14E333653@SHSMSX104.ccr.corp.intel.com> In-Reply-To: <4A89E2EF3DFEDB4C8BFDE51014F606A14E333653@SHSMSX104.ccr.corp.intel.com> Accept-Language: zh-CN, en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.239.127.40] MIME-Version: 1.0 Subject: Re: [Patch 0/2] Update BrotliCompress to the latest version 1.0.5 X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 09 Oct 2018 01:50:15 -0000 Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Reviewed-by: Star Zeng -----Original Message----- From: Gao, Liming=20 Sent: Monday, October 8, 2018 10:52 PM To: Zeng, Star ; edk2-devel@lists.01.org Subject: RE: [edk2] [Patch 0/2] Update BrotliCompress to the latest version= 1.0.5 Star: Thank you for review. I just find Brotli 1.0.6 has been released. Based o= n v1.0.5, there is the minor change in source file. They have no functional= ity impact. I have pushed the additional patches in https://github.com/lgao= 4/edk2/tree/Brotli to integrate 1.0.6 change. Could you help review them? A= fter review, I will combine the patches to single serial to update BrotliCo= mpress to the latest version 1.0.6.=20 Thanks Liming > -----Original Message----- > From: Zeng, Star > Sent: Monday, October 8, 2018 9:37 AM > To: Gao, Liming ; edk2-devel@lists.01.org > Cc: Zeng, Star > Subject: RE: [edk2] [Patch 0/2] Update BrotliCompress to the latest=20 > version 1.0.5 >=20 > Reviewed-by: Star Zeng >=20 > -----Original Message----- > From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of=20 > Liming Gao > Sent: Monday, September 10, 2018 8:37 AM > To: edk2-devel@lists.01.org > Subject: [edk2] [Patch 0/2] Update BrotliCompress to the latest=20 > version 1.0.5 >=20 > Because the patch is too big, the change is placed in=20 > https://github.com/lgao4/edk2/tree/Brotli >=20 > Update Brotli to the latest version 1.0.5=20 > https://github.com/google/brotli Verify VS2017, GCC5 build. > Verify Decompression boot functionality. >=20 > Liming Gao (2): > BaseTools: Update Brotli Compress to the latest one 1.0.5 > MdeModulePkg: Update Brotli DecompressLib to the latest v1.0.5 >=20 > BaseTools/BinWrappers/PosixLike/BrotliCompress | 13 +- > .../WindowsLike}/BrotliCompress.bat | 12 +- > BaseTools/Source/C/BrotliCompress/GNUmakefile | 8 +- > BaseTools/Source/C/BrotliCompress/Makefile | 20 +- > BaseTools/Source/C/BrotliCompress/ReadMe.txt | 2 +- > .../Source/C/BrotliCompress/common/constants.h | 25 +- > .../Source/C/BrotliCompress/common}/context.h | 356 +- > .../Source/C/BrotliCompress/common/dictionary.c | 15341 +++++++------= ---- > .../Source/C/BrotliCompress/common/dictionary.h | 47 +- > .../Source/C/BrotliCompress/common/platform.h | 509 + > BaseTools/Source/C/BrotliCompress/common/port.h | 107 - > .../Source/C/BrotliCompress/common/transform.c | 235 + > .../Source/C/BrotliCompress/common/transform.h | 80 + > BaseTools/Source/C/BrotliCompress/common/types.h | 58 - > BaseTools/Source/C/BrotliCompress/common/version.h | 26 + > BaseTools/Source/C/BrotliCompress/dec/bit_reader.c | 6 +- > BaseTools/Source/C/BrotliCompress/dec/bit_reader.h | 162 +- > BaseTools/Source/C/BrotliCompress/dec/decode.c | 1208 +- > BaseTools/Source/C/BrotliCompress/dec/decode.h | 188 - > BaseTools/Source/C/BrotliCompress/dec/huffman.c | 59 +- > BaseTools/Source/C/BrotliCompress/dec/huffman.h | 24 +- > BaseTools/Source/C/BrotliCompress/dec/port.h | 159 - > BaseTools/Source/C/BrotliCompress/dec/prefix.h | 9 +- > BaseTools/Source/C/BrotliCompress/dec/state.c | 90 +- > BaseTools/Source/C/BrotliCompress/dec/state.h | 82 +- > BaseTools/Source/C/BrotliCompress/dec/transform.h | 300 - > .../C/BrotliCompress/enc/backward_references.c | 822 +- > .../C/BrotliCompress/enc/backward_references.h | 77 +- > .../C/BrotliCompress/enc/backward_references_hq.c | 830 + > .../C/BrotliCompress/enc/backward_references_hq.h | 93 + > .../C/BrotliCompress/enc/backward_references_inc.h | 70 +- > BaseTools/Source/C/BrotliCompress/enc/bit_cost.c | 4 +- > BaseTools/Source/C/BrotliCompress/enc/bit_cost.h | 12 +- > .../C/BrotliCompress/enc/block_encoder_inc.h | 13 +- > .../Source/C/BrotliCompress/enc/block_splitter.c | 11 +- > .../Source/C/BrotliCompress/enc/block_splitter.h | 4 +- > .../C/BrotliCompress/enc/block_splitter_inc.h | 35 +- > .../C/BrotliCompress/enc/brotli_bit_stream.c | 223 +- > .../C/BrotliCompress/enc/brotli_bit_stream.h | 63 +- > BaseTools/Source/C/BrotliCompress/enc/cluster.c | 4 +- > BaseTools/Source/C/BrotliCompress/enc/cluster.h | 4 +- > .../Source/C/BrotliCompress/enc/cluster_inc.h | 2 + > BaseTools/Source/C/BrotliCompress/enc/command.h | 83 +- > .../C/BrotliCompress/enc/compress_fragment.c | 189 +- > .../C/BrotliCompress/enc/compress_fragment.h | 9 +- > .../enc/compress_fragment_two_pass.c | 296 +- > .../enc/compress_fragment_two_pass.h | 9 +- > BaseTools/Source/C/BrotliCompress/enc/compressor.h | 161 - > BaseTools/Source/C/BrotliCompress/enc/context.h | 184 - > .../Source/C/BrotliCompress/enc/dictionary_hash.c | 1120 ++ =20 > .../Source/C/BrotliCompress/enc/dictionary_hash.h | 4101 +---- > BaseTools/Source/C/BrotliCompress/enc/encode.c | 1050 +- > BaseTools/Source/C/BrotliCompress/enc/encode.h | 221 - > .../Source/C/BrotliCompress/enc/encode_parallel.h | 27 - > .../Source/C/BrotliCompress/enc/encoder_dict.c | 32 + > .../Source/C/BrotliCompress/enc/encoder_dict.h | 41 + > .../Source/C/BrotliCompress/enc/entropy_encode.c | 46 +- > .../Source/C/BrotliCompress/enc/entropy_encode.h | 18 +- > .../C/BrotliCompress/enc/entropy_encode_static.h | 10 +- > BaseTools/Source/C/BrotliCompress/enc/fast_log.h | 8 +- > .../C/BrotliCompress/enc/find_match_length.h | 22 +- > BaseTools/Source/C/BrotliCompress/enc/hash.h | 732 +- > .../C/BrotliCompress/enc/hash_composite_inc.h | 133 + > .../BrotliCompress/enc/hash_forgetful_chain_inc.h | 145 +- > .../BrotliCompress/enc/hash_longest_match64_inc.h | 266 + > .../C/BrotliCompress/enc/hash_longest_match_inc.h | 229 +- > .../enc/hash_longest_match_quickly_inc.h | 163 +- > .../Source/C/BrotliCompress/enc/hash_rolling_inc.h | 215 + > .../C/BrotliCompress/enc/hash_to_binary_tree_inc.h | 327 + > BaseTools/Source/C/BrotliCompress/enc/histogram.c | 13 +- > BaseTools/Source/C/BrotliCompress/enc/histogram.h | 11 +- > .../Source/C/BrotliCompress/enc/histogram_inc.h | 2 +- > .../Source/C/BrotliCompress/enc/literal_cost.c | 21 +- > .../Source/C/BrotliCompress/enc/literal_cost.h | 8 +- > BaseTools/Source/C/BrotliCompress/enc/memory.c | 23 +- > BaseTools/Source/C/BrotliCompress/enc/memory.h | 46 +- > BaseTools/Source/C/BrotliCompress/enc/metablock.c | 427 +- > BaseTools/Source/C/BrotliCompress/enc/metablock.h | 35 +- > .../Source/C/BrotliCompress/enc/metablock_inc.h | 2 +- > BaseTools/Source/C/BrotliCompress/enc/params.h | 44 + > BaseTools/Source/C/BrotliCompress/enc/port.h | 168 - > BaseTools/Source/C/BrotliCompress/enc/prefix.h | 11 +- > BaseTools/Source/C/BrotliCompress/enc/quality.h | 91 +- > BaseTools/Source/C/BrotliCompress/enc/ringbuffer.h | 38 +- > .../Source/C/BrotliCompress/enc/static_dict.c | 118 +- > .../Source/C/BrotliCompress/enc/static_dict.h | 9 +- > .../Source/C/BrotliCompress/enc/static_dict_lut.h | 17035 +++++++------= ------ > BaseTools/Source/C/BrotliCompress/enc/streams.h | 121 - > BaseTools/Source/C/BrotliCompress/enc/utf8_util.c | 42 +- > BaseTools/Source/C/BrotliCompress/enc/utf8_util.h | 6 +- > BaseTools/Source/C/BrotliCompress/enc/write_bits.h | 41 +- > .../C/BrotliCompress/include/brotli/decode.h | 344 + > .../C/BrotliCompress/include/brotli/encode.h | 427 + > .../Source/C/BrotliCompress/include/brotli/port.h | 274 + > .../Source/C/BrotliCompress/include/brotli/types.h | 83 + > BaseTools/Source/C/BrotliCompress/tools/bro.c | 554 - > BaseTools/Source/C/BrotliCompress/tools/brotli.c | 1067 ++ > BaseTools/Source/C/BrotliCompress/tools/brotli.md | 107 + > BaseTools/Source/C/BrotliCompress/tools/version.h | 14 - > .../BrotliCustomDecompressLib.inf | 14 + > .../BrotliCustomDecompressLib/BrotliDecompress.c | 26 +- > .../BrotliDecompressLibInternal.h | 4 +- > .../Library/BrotliCustomDecompressLib/ReadMe.txt | 2 +- > .../BrotliCustomDecompressLib/brotli/decode.h | 344 + > .../BrotliCustomDecompressLib/brotli/port.h | 274 + > .../BrotliCustomDecompressLib/brotli/types.h | 96 + > .../BrotliCustomDecompressLib/common/constants.h | 25 +- > .../BrotliCustomDecompressLib/common}/context.h | 356 +- > .../BrotliCustomDecompressLib/common/dictionary.c | 15341 +++++++------= ---- > .../BrotliCustomDecompressLib/common/dictionary.h | 47 +- > .../BrotliCustomDecompressLib/common/platform.h | 510 + > .../BrotliCustomDecompressLib/common/port.h | 107 - > .../BrotliCustomDecompressLib/common/transform.c | 235 + > .../BrotliCustomDecompressLib/common/transform.h | 80 + > .../BrotliCustomDecompressLib/common/types.h | 72 - > .../BrotliCustomDecompressLib/common/version.h | 26 + > .../BrotliCustomDecompressLib/dec/bit_reader.c | 6 +- > .../BrotliCustomDecompressLib/dec/bit_reader.h | 163 +- > .../Library/BrotliCustomDecompressLib/dec/decode.c | 1222 +- > .../Library/BrotliCustomDecompressLib/dec/decode.h | 188 - > .../BrotliCustomDecompressLib/dec/huffman.c | 59 +- > .../BrotliCustomDecompressLib/dec/huffman.h | 25 +- > .../Library/BrotliCustomDecompressLib/dec/port.h | 159 - > .../Library/BrotliCustomDecompressLib/dec/prefix.h | 9 +- > .../Library/BrotliCustomDecompressLib/dec/state.c | 91 +- > .../Library/BrotliCustomDecompressLib/dec/state.h | 82 +- > .../BrotliCustomDecompressLib/dec/transform.h | 300 - > 127 files changed, 30174 insertions(+), 42101 deletions(-) rename=20 > BaseTools/{Source/C/BrotliCompress =3D>=20 > BinWrappers/WindowsLike}/BrotliCompress.bat (76%) rename=20 > {MdeModulePkg/Library/BrotliCustomDecompressLib/dec =3D>=20 > BaseTools/Source/C/BrotliCompress/common}/context.h (58%) create mode=20 > 100644 BaseTools/Source/C/BrotliCompress/common/platform.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/common/port.h > create mode 100644=20 > BaseTools/Source/C/BrotliCompress/common/transform.c > create mode 100644=20 > BaseTools/Source/C/BrotliCompress/common/transform.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/common/types.h > create mode 100644 BaseTools/Source/C/BrotliCompress/common/version.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/dec/decode.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/dec/port.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/dec/transform.h > create mode 100644=20 > BaseTools/Source/C/BrotliCompress/enc/backward_references_hq.c > create mode 100644=20 > BaseTools/Source/C/BrotliCompress/enc/backward_references_hq.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/compressor.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/context.h > create mode 100644=20 > BaseTools/Source/C/BrotliCompress/enc/dictionary_hash.c > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/encode.h > delete mode 100644=20 > BaseTools/Source/C/BrotliCompress/enc/encode_parallel.h > create mode 100644=20 > BaseTools/Source/C/BrotliCompress/enc/encoder_dict.c > create mode 100644=20 > BaseTools/Source/C/BrotliCompress/enc/encoder_dict.h > create mode 100644=20 > BaseTools/Source/C/BrotliCompress/enc/hash_composite_inc.h > create mode 100644=20 > BaseTools/Source/C/BrotliCompress/enc/hash_longest_match64_inc.h > create mode 100644=20 > BaseTools/Source/C/BrotliCompress/enc/hash_rolling_inc.h > create mode 100644=20 > BaseTools/Source/C/BrotliCompress/enc/hash_to_binary_tree_inc.h > create mode 100644 BaseTools/Source/C/BrotliCompress/enc/params.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/port.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/enc/streams.h > create mode 100644=20 > BaseTools/Source/C/BrotliCompress/include/brotli/decode.h > create mode 100644=20 > BaseTools/Source/C/BrotliCompress/include/brotli/encode.h > create mode 100644=20 > BaseTools/Source/C/BrotliCompress/include/brotli/port.h > create mode 100644=20 > BaseTools/Source/C/BrotliCompress/include/brotli/types.h > delete mode 100644 BaseTools/Source/C/BrotliCompress/tools/bro.c > create mode 100644 BaseTools/Source/C/BrotliCompress/tools/brotli.c > create mode 100644 BaseTools/Source/C/BrotliCompress/tools/brotli.md > delete mode 100644 BaseTools/Source/C/BrotliCompress/tools/version.h > create mode 100644=20 > MdeModulePkg/Library/BrotliCustomDecompressLib/brotli/decode.h > create mode 100644=20 > MdeModulePkg/Library/BrotliCustomDecompressLib/brotli/port.h > create mode 100644=20 > MdeModulePkg/Library/BrotliCustomDecompressLib/brotli/types.h > rename {BaseTools/Source/C/BrotliCompress/dec =3D>=20 > MdeModulePkg/Library/BrotliCustomDecompressLib/common}/context.h (58%) =20 > create mode 100644=20 > MdeModulePkg/Library/BrotliCustomDecompressLib/common/platform.h > delete mode 100644=20 > MdeModulePkg/Library/BrotliCustomDecompressLib/common/port.h > create mode 100644=20 > MdeModulePkg/Library/BrotliCustomDecompressLib/common/transform.c > create mode 100644=20 > MdeModulePkg/Library/BrotliCustomDecompressLib/common/transform.h > delete mode 100644=20 > MdeModulePkg/Library/BrotliCustomDecompressLib/common/types.h > create mode 100644=20 > MdeModulePkg/Library/BrotliCustomDecompressLib/common/version.h > delete mode 100644=20 > MdeModulePkg/Library/BrotliCustomDecompressLib/dec/decode.h > delete mode 100644=20 > MdeModulePkg/Library/BrotliCustomDecompressLib/dec/port.h > delete mode 100644=20 > MdeModulePkg/Library/BrotliCustomDecompressLib/dec/transform.h >=20 > -- > 2.10.0.windows.1 >=20 > _______________________________________________ > edk2-devel mailing list > edk2-devel@lists.01.org > https://lists.01.org/mailman/listinfo/edk2-devel