Re: [edk2] [PATCH 0/5] MdeModulePkg: UDF fixes and cleanups

2017-09-12 Thread Laszlo Ersek
On 09/12/17 17:38, Ard Biesheuvel wrote: > On 12 September 2017 at 03:14, Laszlo Ersek wrote: >> On 09/10/17 02:12, Laszlo Ersek wrote: >>> Repo: https://github.com/lersek/edk2.git >>> Branch: udf_fixes_cleanups >>> >>> Patches #2, #3 and #4 are needed (and enough) for me to

Re: [edk2] [PATCH 0/5] MdeModulePkg: UDF fixes and cleanups

2017-09-12 Thread Ard Biesheuvel
On 12 September 2017 at 03:14, Laszlo Ersek wrote: > On 09/10/17 02:12, Laszlo Ersek wrote: >> Repo: https://github.com/lersek/edk2.git >> Branch: udf_fixes_cleanups >> >> Patches #2, #3 and #4 are needed (and enough) for me to build OVMF for >> IA32 and X64 with clang-3.8,

Re: [edk2] [PATCH 0/5] MdeModulePkg: UDF fixes and cleanups

2017-09-11 Thread Shi, Steven
;star.z...@intel.com>; Ard Biesheuvel <ard.biesheu...@linaro.org> > Subject: Re: [edk2] [PATCH 0/5] MdeModulePkg: UDF fixes and cleanups > > Hi Steven, > > On 11/09/2017 10:07, Shi, Steven wrote: > > Hi Laszlo, > > Your code is good. But there is a "Null Pointer U

Re: [edk2] [PATCH 0/5] MdeModulePkg: UDF fixes and cleanups

2017-09-11 Thread Paulo Alcantara
On 11/09/2017 03:58, Laszlo Ersek wrote: On 09/10/17 17:51, Paulo Alcantara wrote: On 10/09/2017 11:27, Shi, Steven wrote: OK. Does the UDF image you created correctly show up as CD-ROM content in Linux, e.g Fedora? The Fedora image I have (Fedora-Workstation-Live-x86_64-26-1.5.iso) does

Re: [edk2] [PATCH 0/5] MdeModulePkg: UDF fixes and cleanups

2017-09-11 Thread Paulo Alcantara
.org> Cc: Ni, Ruiyu <ruiyu...@intel.com>; Dong, Eric <eric.d...@intel.com>; Zeng, Star <star.z...@intel.com>; Ard Biesheuvel <ard.biesheu...@linaro.org> Subject: Re: [edk2] [PATCH 0/5] MdeModulePkg: UDF fixes and cleanups On 10/09/2017 11:27, Shi, Steven wrote: OK. Does the UDF i

Re: [edk2] [PATCH 0/5] MdeModulePkg: UDF fixes and cleanups

2017-09-11 Thread Laszlo Ersek
>> From: Paulo Alcantara [mailto:pca...@zytor.com] >> Sent: Sunday, September 10, 2017 11:52 PM >> To: Shi, Steven <steven....@intel.com>; Laszlo Ersek <ler...@redhat.com>; >> edk2-devel-01 <edk2-devel@lists.01.org> >> Cc: Ni, Ruiyu <ruiyu...@inte

Re: [edk2] [PATCH 0/5] MdeModulePkg: UDF fixes and cleanups

2017-09-11 Thread Ni, Ruiyu
ntel.com>; Dong, Eric <eric.d...@intel.com>; Zeng, Star <star.z...@intel.com>; Ard Biesheuvel <ard.biesheu...@linaro.org> Subject: RE: [edk2] [PATCH 0/5] MdeModulePkg: UDF fixes and cleanups Hi Laszlo, Your code is good. But there is a "Null Pointer Use" issue

Re: [edk2] [PATCH 0/5] MdeModulePkg: UDF fixes and cleanups

2017-09-11 Thread Shi, Steven
Dong, Eric <eric.d...@intel.com>; Zeng, > Star <star.z...@intel.com>; Ard Biesheuvel <ard.biesheu...@linaro.org> > Subject: Re: [edk2] [PATCH 0/5] MdeModulePkg: UDF fixes and cleanups > > > > On 10/09/2017 11:27, Shi, Steven wrote: > > OK. Does the

Re: [edk2] [PATCH 0/5] MdeModulePkg: UDF fixes and cleanups

2017-09-11 Thread Ard Biesheuvel
On 10 September 2017 at 01:12, Laszlo Ersek wrote: > Repo: https://github.com/lersek/edk2.git > Branch: udf_fixes_cleanups > > Patches #2, #3 and #4 are needed (and enough) for me to build OVMF for > IA32 and X64 with clang-3.8, after the UDF introduction. > > Patches #1 and

Re: [edk2] [PATCH 0/5] MdeModulePkg: UDF fixes and cleanups

2017-09-11 Thread Laszlo Ersek
On 09/10/17 17:51, Paulo Alcantara wrote: > > > On 10/09/2017 11:27, Shi, Steven wrote: >> OK. Does the UDF image you created correctly show up as CD-ROM content >> in Linux, e.g Fedora? > > The Fedora image I have (Fedora-Workstation-Live-x86_64-26-1.5.iso) does > not contain a valid UDF file

Re: [edk2] [PATCH 0/5] MdeModulePkg: UDF fixes and cleanups

2017-09-10 Thread Paulo Alcantara
c <eric.d...@intel.com>; Zeng, Star <star.z...@intel.com>; Ard Biesheuvel <ard.biesheu...@linaro.org> Subject: Re: [edk2] [PATCH 0/5] MdeModulePkg: UDF fixes and cleanups Hi Steven, On 09/10/17 10:38, Laszlo Ersek wrote: On 09/10/17 06:24, Shi, Steven wrote: Hi Laszlo, How coul

Re: [edk2] [PATCH 0/5] MdeModulePkg: UDF fixes and cleanups

2017-09-10 Thread Paulo Alcantara
On 09/09/2017 21:12, Laszlo Ersek wrote: Repo: https://github.com/lersek/edk2.git Branch: udf_fixes_cleanups Patches #2, #3 and #4 are needed (and enough) for me to build OVMF for IA32 and X64 with clang-3.8, after the UDF introduction. Patches #1 and #5 are cleanups that I felt fit before

Re: [edk2] [PATCH 0/5] MdeModulePkg: UDF fixes and cleanups

2017-09-10 Thread Shi, Steven
7 9:52 PM > To: Shi, Steven <steven@intel.com>; edk2-devel-01 de...@lists.01.org> > Cc: Ni, Ruiyu <ruiyu...@intel.com>; Dong, Eric <eric.d...@intel.com>; Zeng, > Star <star.z...@intel.com>; Ard Biesheuvel <ard.biesheu...@linaro.org> > Subject: Re: [

Re: [edk2] [PATCH 0/5] MdeModulePkg: UDF fixes and cleanups

2017-09-10 Thread Laszlo Ersek
Hi Steven, On 09/10/17 10:38, Laszlo Ersek wrote: > On 09/10/17 06:24, Shi, Steven wrote: >> Hi Laszlo, >> How could we configure the Qemu and test the UDF driver on OVMF? > > I guess you would format e.g. a DVD image with UDF, and attach it to > QEMU like any other CD-ROM. I tried to look

Re: [edk2] [PATCH 0/5] MdeModulePkg: UDF fixes and cleanups

2017-09-09 Thread Shi, Steven
Hi Laszlo, How could we configure the Qemu and test the UDF driver on OVMF? BTW, how could we configure the Qemu to create a full feature scope machine to include all possible devices in it, e.g. USB, ISA, SD/MMC, network etc. Steven Shi Intel\SSG\STO\UEFI Firmware Tel: +86 021-61166522