[edk2-devel] [edk2-platforms Patch 00/14] Vlv2TbltDevicePkg: Remove Intel Framework dependencies

Tue Jul 9 03:52:37 UTC 2019

Hi Gary,

I picked up the changes on your branch.  I re-enabled the BIOS ID
Feature and added support for IA32 GCC builds, so the builds
available from Windows and Linux are matched now.

I have posted my updated branch here:

https://github.com/mdkinney/edk2-platforms/tree/Bug_xxx_Vlv2_Linux_And_Smbios_Fixes_V2

I also verified that the SD Card support does work, so I am
keeping that change.

I agree the Readme is out of date.  I will work on that and
add to the branch referenced above tomorrow.

Thanks,

Mike

> -----Original Message-----
> From: devel at edk2.groups.io [mailto:devel at edk2.groups.io]
> On Behalf Of Gary Lin
> Sent: Tuesday, July 2, 2019 8:57 PM
> To: Kinney, Michael D <michael.d.kinney at intel.com>
> Cc: devel at edk2.groups.io; Qian, Yi <yi.qian at intel.com>;
> Sun, Zailiang <zailiang.sun at intel.com>
> Subject: Re: [edk2-devel] [edk2-platforms Patch 00/14]
> Vlv2TbltDevicePkg: Remove Intel Framework dependencies
> 
> On Tue, Jul 02, 2019 at 09:11:06PM +0000,  Kinney,
> Michael D  wrote:
> > Hi Gary,
> >
> Hi Mike,
> 
> > I have posted an update to your patches the removes the
> setting of
> > WORKSPACE and accesses all files relative to WORKSPACE
> in the Linux shell scripts.
> > It also fixes null pointer bugs in the SmbiosMiscDxe
> module.
> >
> > https://github.com/mdkinney/edk2-
> platforms/tree/Bug_xxx_Vlv2_Linux_And
> > _Smbios_Fixes
> >
> > With these changes I can boot to the UEFI Shell and to
> Linux.  Please
> > verify from your environments.
> >
> The fix for SmbiosMiscDxe works for me. Thanks for that!
> 
> I pushed another commit to my branch to check WORKSPACE
> to notify the
> user:
> 
> https://github.com/lcp/edk2-
> platforms/commit/fc530689e913e3de25ca2c553d616b9cef28901c
> 
> We probably have to update Readme.md later. It's already
> dated.
> 
> > I see you have added the SDMMC modules, but I do not
> see that peripheral yet.
> > That feature looks like it needs some more debug.  We
> may need to back
> > out those changes for now.
> >
> I saw the SD card settings in the South Cluster
> Configuration menu even without those SDMMC modules, so
> they are probably from the edk2-non-osi drivers. However,
> the SD card didn't work until I added the drivers from
> MdeModulePkg, so I need those drivers at least for my
> Minnowboard Turbot.
> 
> Thanks,
> 
> Gary Lin
> 
> > I agree there are ASL issues.  The ASL code needs to be
> updated to the
> > latest ASL compiler.
> >
> > Thanks,
> >
> > Mike
> >
> > > -----Original Message-----
> > > From: Kinney, Michael D
> > > Sent: Tuesday, July 2, 2019 9:50 AM
> > > To: Gary Lin <GLin at suse.com>; devel at edk2.groups.io;
> Kinney, Michael
> > > D <michael.d.kinney at intel.com>
> > > Cc: Qian, Yi <yi.qian at intel.com>; Sun, Zailiang
> > > <zailiang.sun at intel.com>
> > > Subject: RE: [edk2-devel] [edk2-platforms Patch
> 00/14]
> > > Vlv2TbltDevicePkg: Remove Intel Framework
> dependencies
> > >
> > > Hi Gary,
> > >
> > > Thanks for the branch.  I have the build working and
> have reproduced
> > > the SMBIOS issue and am debugging that now.
> > >
> > > Mike
> > >
> > > > -----Original Message-----
> > > > From: Gary Lin [mailto:GLin at suse.com]
> > > > Sent: Tuesday, July 2, 2019 2:48 AM
> > > > To: devel at edk2.groups.io; Kinney, Michael D
> > > > <michael.d.kinney at intel.com>
> > > > Cc: Qian, Yi <yi.qian at intel.com>; Sun, Zailiang
> > > > <zailiang.sun at intel.com>
> > > > Subject: Re: [edk2-devel] [edk2-platforms Patch
> 00/14]
> > > > Vlv2TbltDevicePkg: Remove Intel Framework
> dependencies
> > > >
> > > > On Mon, Jul 01, 2019 at 11:07:11PM +0000,  Michael
> D
> > > Kinney  wrote:
> > > > > Hi Gary,
> > > > >
> > > > > I recommend you rebase on top of this series.
> > > > >
> > > > > I have verified Linux boots and I have been
> working
> > > on
> > > > the GCC and
> > > > > Linux build scripts as well.
> > > > >
> > > > > Please let me know if you see any issues with
> this
> > > > series applied and
> > > > > I can help get them fixed.
> > > > >
> > > > Hi Mike,
> > > >
> > > > I'm rebasing my patches and they are available in
> my
> > > > branch:
> > > >
> > > > https://github.com/lcp/edk2-platforms/commits/vlv2-
> > > > fixes-wip
> > > >
> > > > It's based on
> > > > edk2(dcce9cbd48e07fcd11f7ef34cad48faf7bb61e6e)
> > > > and edk2-non-
> > > > osi(de0a6bc19c4ee0468f04e4d37035cdc59558137f).
> > > >
> > > > There are 3 problems I currently encountered, and
> maybe
> > > you can give
> > > > me some suggestions.
> > > >
> > > > 1. Missing DSDT
> > > >
> > > > When compiling DSDT, iasl failed with this error:
> > > >
> > > > DSDT.iiii   6077:
> > > > Or(ShiftLeft(Derefof(Index(CDCT,
> > > > \_SB.PCI0.GFX0.MCHK.DCFE)), 21),PARM, PARM)
> > > > Error    6163 -
> > > > ^ Object is created temporarily in another method
> and
> > > cannot be
> > > > accessed (\_SB.PCI0.GFX0. MCHK.DCFE)
> > > >
> > > > The code is from
> > > >
> > >
> Silicon/Intel/Vlv2DeviceRefCodePkg/AcpiTablesPCAT/IgdOGB
> > > > DA.ASL
> > > >
> > > > I have to comment out the line or iasl won't
> generate
> > > the aml file and
> > > > the OS would fail to get DSDT.
> > > >
> > > > My acpica version is 20190509, and I vaguely
> remember
> > > that the older
> > > > acpica seems to be more tolerant to this error.
> > > >
> > > > 2. Crash at SMBIOS driver
> > > >
> > > > When booting the system, I got the following error:
> > > >
> > > > Loading driver at 0x00078DBA000
> > > EntryPoint=0x00078DBEC3C
> > > > MiscSubclass.efi
> > > > InstallProtocolInterface: BC62157E-3E33-4FEC-9920-
> > > 2D3B36D750DF
> > > > 78FE8D18 ProtectUefiImageCommon -
> > > > 0x78FE8040
> > > >   - 0x0000000078DBA000 - 0x0000000000007B80
> > > > PlatformInfoHob->BoardId [0xB0]
> > > > GetBiosVersionDateTime :MNW2MAX1.X64.0084.D01
> > > 01/01/2019
> > > > 00:00
> > > > SmbiosAdd: Smbios type 0 with size 0x48 is added to
> 32-
> > > bit table
> > > > SmbiosAdd: Smbios type 0 with size 0x48 is added to
> 64-
> > > bit table
> > > > SmbiosCreateTable: Initialize 32-bit entry point
> > > structure
> > > > SmbiosCreateTable() re-allocate SMBIOS 32-bit table
> > > > SmbiosCreateTable: Initialize 64-bit entry point
> > > structure
> > > > SmbiosCreate64BitTable() re-allocate SMBIOS 64-bit
> > > table Create Smbios
> > > > Manu callback.
> > > > !!!! X64 Exception Type - 06(#UD - Invalid Opcode)
> CPU
> > > Apic ID -
> > > > 00000000 !!!!
> > > > RIP  - 0000000078DBEC26, CS  - 0000000000000038,
> RFLAGS
> > > > - 0000000000010206 RAX  - 0000000000000000, RCX -
> > > 000000007984A5A0,
> > > > RDX - 000000007A46E5FF RBX  - 000000007A46E920, RSP
> -
> > > > 000000007A46E8B0, RBP -
> > > > 0000000078DC0EC0 RSI  - 0000000000000012, RDI -
> > > > 0000000000000003
> > > > R8   - 800000000000000E, R9  - 000000007A46E920,
> R10 -
> > > > 0000000078FE8398
> > > > R11  - 0000000000000001, R12 - 0000000078FE8798,
> R13 -
> > > > 0000000078FE8898
> > > > R14  - 0000000078FE8818, R15 - 000000007A46EC60
> > > > DS   - 0000000000000030, ES  - 0000000000000030, FS
> -
> > > > 0000000000000030
> > > > GS   - 0000000000000030, SS  - 0000000000000030
> > > > CR0  - 0000000080010013, CR2 - 0000000000000000,
> CR3 -
> > > > 000000007A201000
> > > > CR4  - 0000000000000628, CR8 - 0000000000000000
> > > > DR0  - 0000000000000000, DR1 - 0000000000000000,
> DR2 -
> > > > 0000000000000000
> > > > DR3  - 0000000000000000, DR6 - 00000000FFFF0FF8,
> DR7 -
> > > > 0000000000000400 GDTR - 0000000079FFDA98
> > > 0000000000000047, LDTR -
> > > > 0000000000000000
> > > > IDTR - 0000000079862018 0000000000000FFF,   TR -
> > > > 0000000000000000
> > > > FXSAVE_STATE - 000000007A46E510
> > > > !!!! Find image based on IP(0x78DBEC26)
> > > > /home/gary/git/edk2-
> > > >
> > >
> workspace/Build/Vlv2TbltDevicePkg/DEBUG_GCC5/X64/Vlv2Tbl
> > > >
> > >
> tDevicePkg/SmBiosMiscDxe/SmBiosMiscDxe/DEBUG/MiscSubclas
> > > > s.
> > > > dll (ImageBase=0000000078DBA000,
> > > > EntryPoint=0000000078DBEC3C) !!!!
> > > >
> > > > I disaseembled MiscSubclass.debug and found that
> the
> > > address is a
> > > > "ud2"
> > > > instruction at
> > > >
> > >
> SmBiosMiscDxe/MiscBaseBoardManufacturerFunction.c:MiscBa
> > > > seBoardManufacturer()
> > > >
> > > > After adding a few more debug messages, the system
> > > crashed at another
> > > > function:
> > > >
> > >
> SmBiosMiscDxe/MiscSystemManufacturerFunction.c:AddSmbios
> > > > ManuCallback()
> > > >
> > > > If I added debug messages to both functions, the
> system
> > > booted
> > > > successfully. I currently didn't have any clue on
> this.
> > > >
> > > > 3. The firmware always booted into EFI shell.
> > > >
> > > > I guess something went wrong with BDS. I haven't
> looked
> > > into this
> > > > issue yet.
> > > >
> > > > Thanks,
> > > >
> > > > Gary Lin
> > > >
> > > > > Thanks,
> > > > >
> > > > > Mike
> > > > >
> > > > > > -----Original Message-----
> > > > > > From: Gary Lin [mailto:GLin at suse.com]
> > > > > > Sent: Sunday, June 30, 2019 9:04 PM
> > > > > > To: devel at edk2.groups.io; Kinney, Michael D
> > > > > > <michael.d.kinney at intel.com>
> > > > > > Cc: Qian, Yi <yi.qian at intel.com>; Sun, Zailiang
> > > > > > <zailiang.sun at intel.com>
> > > > > > Subject: Re: [edk2-devel] [edk2-platforms Patch
> > > > 00/14]
> > > > > > Vlv2TbltDevicePkg: Remove Intel Framework
> > > > dependencies
> > > > > >
> > > > > > On Sun, Jun 30, 2019 at 07:55:39PM -0700,
> Michael
> > > D
> > > > Kinney  wrote:
> > > > > > > The following changes remove all dependencies
> on
> > > > the
> > > > > > IntelFrameworkPkg
> > > > > > > and IntelFrameworkModulePkg from the
> > > > Vlv2TbltDevicePkg
> > > > > > platform
> > > > > > > builds.  These changes do depend on updates
> > > > binaries
> > > > > > on the edk2-non-osi repository.
> > > > > > >
> > > > > > Hi Michael,
> > > > > >
> > > > > > I am amending the bash scripts for gcc. The
> > > firmware
> > > > I built is
> > > > > > using the binaries from Development190216 and
> could
> > > > boot into Linux
> > > > > > successfully with some workarounds for Wpce791,
> > > Bds,
> > > > and
> > > > > > InteFspWrapper. The patches you posted
> eliminate
> > > > those parts I have
> > > > > > to work around.
> > > > > >
> > > > > > Just want to know if you need any assistance
> for
> > > the
> > > > gcc build? I
> > > > > > could rebase my work upon your patches and send
> > > them
> > > > later.
> > > > > >
> > > > > > Thanks,
> > > > > >
> > > > > > Gary Lin
> > > > > >
> > > > >
> > > > >
> > > > >
> >
> 
> 

-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#43418): https://edk2.groups.io/g/devel/message/43418
Mute This Topic: https://groups.io/mt/32267616/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-