From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) (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 D2CBB20945B83 for ; Sun, 17 Sep 2017 18:01:40 -0700 (PDT) Received: from fmsmga003.fm.intel.com ([10.253.24.29]) by orsmga105.jf.intel.com with ESMTP; 17 Sep 2017 18:04:43 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.42,410,1500966000"; d="scan'208";a="901167949" Received: from fmsmsx103.amr.corp.intel.com ([10.18.124.201]) by FMSMGA003.fm.intel.com with ESMTP; 17 Sep 2017 18:04:43 -0700 Received: from shsmsx152.ccr.corp.intel.com (10.239.6.52) by FMSMSX103.amr.corp.intel.com (10.18.124.201) with Microsoft SMTP Server (TLS) id 14.3.319.2; Sun, 17 Sep 2017 18:04:43 -0700 Received: from shsmsx151.ccr.corp.intel.com ([169.254.3.98]) by SHSMSX152.ccr.corp.intel.com ([169.254.6.93]) with mapi id 14.03.0319.002; Mon, 18 Sep 2017 09:04:41 +0800 From: "Ni, Ruiyu" To: Paulo Alcantara , "edk2-devel@lists.01.org" Thread-Topic: [edk2] [PATCH 0/3] UDF partition driver fix Thread-Index: AQHTLzRZesTImxrR0E2867ngzAIqmaK51KjA Date: Mon, 18 Sep 2017 01:04:40 +0000 Message-ID: <734D49CCEBEEF84792F5B80ED585239D5BA442B4@SHSMSX151.ccr.corp.intel.com> References: In-Reply-To: Accept-Language: en-US, zh-CN X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.239.127.40] MIME-Version: 1.0 Subject: Re: [PATCH 0/3] UDF partition driver fix X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.22 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 18 Sep 2017 01:01:41 -0000 Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Paulo, I checked carefully of patch #1, 50% carefully of #2, 20% carefully of #3. I could only give comments from the EDKII coding style perspective. I do provide some other comments, but please understand they are from a person that knows very little about UDF. (I know the concept of Volume. But just know that there are so many different types of descriptors.) If my understanding is wrong, please correct me! Again, thanks for the contribution. Thanks/Ray > -----Original Message----- > From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of > Paulo Alcantara > Sent: Sunday, September 17, 2017 5:37 AM > To: edk2-devel@lists.01.org > Subject: [edk2] [PATCH 0/3] UDF partition driver fix >=20 > This series fixes an UDF issue with Partition driver as discussed in ML: >=20 > https://lists.01.org/pipermail/edk2-devel/2017-September/014694.html >=20 > Thanks! > Paulo >=20 > Repo: https://github.com/pcacjr/edk2.git > Branch: udf-partition-fix >=20 > Paulo Alcantara (3): > MdePkg: Add UDF volume structure definitions > MdeModulePkg/PartitionDxe: Fix creation of UDF logical partition > MdeModulePkg/UdfDxe: Rework driver to support PartitionDxe changes >=20 > MdeModulePkg/Universal/Disk/PartitionDxe/Udf.c | 307 +++++++++++- > MdeModulePkg/Universal/Disk/UdfDxe/File.c | 13 +- > .../Universal/Disk/UdfDxe/FileSystemOperations.c | 525 ++++++++-------= --- > --- > MdeModulePkg/Universal/Disk/UdfDxe/Udf.c | 7 - > MdeModulePkg/Universal/Disk/UdfDxe/Udf.h | 88 +--- > MdePkg/Include/IndustryStandard/Udf.h | 63 +++ > 6 files changed, 560 insertions(+), 443 deletions(-) >=20 > -- > 2.11.0 >=20 > _______________________________________________ > edk2-devel mailing list > edk2-devel@lists.01.org > https://lists.01.org/mailman/listinfo/edk2-devel