From: "Andrew Fish" <afish@apple.com>
To: devel@edk2.groups.io, Mike Kinney <michael.d.kinney@intel.com>
Cc: Jordan Justen <jordan.l.justen@intel.com>, "Ni, Ray" <ray.ni@intel.com>
Subject: Re: [edk2-devel] [Patch V4 06/10] EmulatorPkg: Fix XCODE5 lldb issues
Date: Fri, 16 Aug 2019 10:30:49 -0700 [thread overview]
Message-ID: <2BB17527-DAF7-4C52-B77D-D03511160F71@apple.com> (raw)
In-Reply-To: <E92EE9817A31E24EB0585FDF735412F5B9D8A077@ORSMSX113.amr.corp.intel.com>
[-- Attachment #1: Type: text/plain, Size: 6554 bytes --]
Tested-by: Andrew Fish <afish@apple.com>
I tested the XCODE builds of the emulator and made sure that source level debugging worked probably for builds that generate symbols.
Thanks,
Andrew Fish
> On Aug 16, 2019, at 8:09 AM, Michael D Kinney <michael.d.kinney@intel.com> wrote:
>
> Jordan,
>
> It is not a typo.
>
> Andrew generated the XCODE specific changes, so they have
> been tested by him. I have also reviewed and tested the XCODE
> changes and verified that all 6 combinations build and boot
> to shell (IA32/X64 for RELEASE/DEBUG/NOOPT). Since they are
> all related to making EmulatorPkg work, we decided to fold
> them into the same patch set that was already being reviewed.
>
> I also verified build and boot to shell for 6 combinations
> on GCC5 (IA32/X64 for RELEASE/DEBUG/NOOPT) and the 12
> combinations of VS2015/VS2017, IA323/X64, RELEASE/DEBUG/NOOPT.
>
> I have been working on some CI experiments using Azure Pipelines.
> Here is a pointer to the build logs for all the combinations
> listed above.
>
> https://dev.azure.com/mikekinney/edk2-ci/_build/results?buildId=312
>
> Mike
>
>> -----Original Message-----
>> From: Justen, Jordan L
>> Sent: Friday, August 16, 2019 12:41 AM
>> To: Kinney, Michael D <michael.d.kinney@intel.com>;
>> devel@edk2.groups.io
>> Cc: Ni, Ray <ray.ni@intel.com>; Andrew Fish
>> <afish@apple.com>
>> Subject: Re: [Patch V4 06/10] EmulatorPkg: Fix XCODE5
>> lldb issues
>>
>> On 2019-08-15 19:14:33, Michael D Kinney wrote:
>>> Fix scripts to support lldb symbolic debugging when
>> using XCODE5 tool
>>> chain.
>>>
>>> Cc: Jordan Justen <jordan.l.justen@intel.com>
>>> Cc: Ray Ni <ray.ni@intel.com>
>>> Cc: Michael D Kinney <michael.d.kinney@intel.com>
>>> Signed-off-by: Andrew Fish <afish@apple.com>
>>
>> Is this a Cc/Signed-off-by typo? (See also, patches 7-
>> 10).
>>
>> This makes me wonder if you are taking advantage of the
>> git commit -s switch to add your Signed-off-by.
>>
>> Also, I'm wondering if you are taking advantage of git-
>> send-email automatically Cc'ing the addresses you
>> listed in the commit message.
>> (I thought it Cc'd for the author and Cc tags, but I
>> wasn't sure about the Signed-off-by tag, and yet I see
>> Andrew was Cc'd.)
>>
>> There's a couple long lines below. You could use \ at
>> the end of the line to split the .sh line. I think the
>> cd can be a separate command in a shell script. (Not in
>> make)
>>
>> I hope someone that uses the XCODE toolchain could
>> review/check the XCODE patches.
>>
>> -Jordan
>>
>>> ---
>>> EmulatorPkg/Unix/lldbefi.py | 8 +++++---
>>> EmulatorPkg/build.sh | 17 ++---------------
>>> 2 files changed, 7 insertions(+), 18 deletions(-)
>>>
>>> diff --git a/EmulatorPkg/Unix/lldbefi.py
>> b/EmulatorPkg/Unix/lldbefi.py
>>> index 218326b8cb..099192d8b5 100755
>>> --- a/EmulatorPkg/Unix/lldbefi.py
>>> +++ b/EmulatorPkg/Unix/lldbefi.py
>>> @@ -346,6 +346,7 @@ def TypePrintFormating(debugger):
>>> debugger.HandleCommand("type summary add CHAR8 -
>> -python-function lldbefi.CHAR8_TypeSummary")
>>> debugger.HandleCommand('type summary add --regex
>> "CHAR8
>>> \[[0-9]+\]" --python-function
>> lldbefi.CHAR8_TypeSummary')
>>>
>>> + debugger.HandleCommand('setting set frame-format
>> "frame
>>> + #${frame.index}: ${frame.pc}{
>>> +
>> ${module.file.basename}{:${function.name}()${function.p
>> c-offset}}}{
>>> + at ${line.file.fullpath}:${line.number}}\n"')
>>>
>>> gEmulatorBreakWorkaroundNeeded = True
>>>
>>> @@ -381,15 +382,16 @@ def
>> LoadEmulatorEfiSymbols(frame, bp_loc , internal_dict):
>>> Error = lldb.SBError()
>>> FileNamePtr = frame.FindVariable
>> ("FileName").GetValueAsUnsigned()
>>> FileNameLen = frame.FindVariable
>>> ("FileNameLength").GetValueAsUnsigned()
>>> +
>>> FileName =
>> frame.thread.process.ReadCStringFromMemory
>> (FileNamePtr, FileNameLen, Error)
>>> if not Error.Success():
>>> print "!ReadCStringFromMemory() did not find
>> a %d byte C string at %x" % (FileNameLen, FileNamePtr)
>>> # make breakpoint command contiue
>>> - frame.GetThread().GetProcess().Continue()
>>> + return False
>>>
>>> debugger = frame.thread.process.target.debugger
>>> if frame.FindVariable
>> ("AddSymbolFlag").GetValueAsUnsigned() == 1:
>>> - LoadAddress = frame.FindVariable
>> ("LoadAddress").GetValueAsUnsigned()
>>> + LoadAddress = frame.FindVariable
>>> + ("LoadAddress").GetValueAsUnsigned() - 0x240
>>>
>>> debugger.HandleCommand ("target modules add
>> %s" % FileName)
>>> print "target modules load --slid 0x%x %s" %
>> (LoadAddress,
>>> FileName) @@ -405,7 +407,7 @@ def
>> LoadEmulatorEfiSymbols(frame, bp_loc , internal_dict):
>>> print "!lldb.target.RemoveModule
>> (%s) FAILED" %
>>> SBModule
>>>
>>> # make breakpoint command contiue
>>> - frame.thread.process.Continue()
>>> + return False
>>>
>>> def GuidToCStructStr (guid, Name=False):
>>> #
>>> diff --git a/EmulatorPkg/build.sh
>> b/EmulatorPkg/build.sh index
>>> 60056e1b6c..35912a7775 100755
>>> --- a/EmulatorPkg/build.sh
>>> +++ b/EmulatorPkg/build.sh
>>> @@ -209,21 +209,8 @@ fi
>>> if [[ "$RUN_EMULATOR" == "yes" ]]; then
>>> case `uname` in
>>> Darwin*)
>>> - #
>>> - # On Darwin we can't use dlopen, so we have to
>> load the real PE/COFF images.
>>> - # This .gdbinit script sets a breakpoint that
>> loads symbols for the PE/COFFEE
>>> - # images that get loaded in Host
>>> - #
>>> - if [[ "$CLANG_VER" == *-ccc-host-triple* ]]
>>> - then
>>> - # only older versions of Xcode support -ccc-
>> host-tripe, for newer versions
>>> - # it is -target
>>> - cp $WORKSPACE/EmulatorPkg/Unix/lldbefi.py
>> "$BUILD_OUTPUT_DIR/${BUILDTARGET}_$TARGET_TOOLS/$PROCES
>> SOR"
>>> - cd $BUILD_ROOT_ARCH; /usr/bin/lldb --source
>> $WORKSPACE/EmulatorPkg/Unix/lldbinit Host
>>> - exit $?
>>> - else
>>> - cp $WORKSPACE/EmulatorPkg/Unix/.gdbinit
>> "$BUILD_OUTPUT_DIR/${BUILDTARGET}_$TARGET_TOOLS/$PROCES
>> SOR"
>>> - fi
>>> + cd $BUILD_ROOT_ARCH; /usr/bin/lldb -o "command
>> script import $WORKSPACE/EmulatorPkg/Unix/lldbefi.py" -
>> o 'script lldb.debugger.SetAsync(True)' -o "run" ./Host
>>> + exit $?
>>> ;;
>>> esac
>>>
>>> --
>>> 2.21.0.windows.1
>>>
>
>
>
[-- Attachment #2: Type: text/html, Size: 11923 bytes --]
next prev parent reply other threads:[~2019-08-16 17:30 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-08-16 2:14 [Patch V4 00/10] EmulatorPkg: Fix VS20xx IA32 boot and simplify build config Michael D Kinney
2019-08-16 2:14 ` [Patch V4 01/10] EmulatorPkg: Fix VS20xx IA32 boot failure Michael D Kinney
2019-08-16 2:14 ` [Patch V4 02/10] EmulatorPkg: Remove UNIX_SEC_BUILD/WIN_SEC_BUILD Michael D Kinney
2019-08-16 2:14 ` [Patch V4 03/10] EmulatorPkg: Add -D DISABLE_NEW_DEPRECATED_INTERFACES Michael D Kinney
2019-08-16 2:14 ` [Patch V4 04/10] EmulatorPkg: Add support for NOOPT target Michael D Kinney
2019-08-16 2:14 ` [Patch V4 05/10] EmulatorPkg/Unix/Host: Disable inline/optimizations Michael D Kinney
2019-08-16 2:14 ` [Patch V4 06/10] EmulatorPkg: Fix XCODE5 lldb issues Michael D Kinney
2019-08-16 7:40 ` Jordan Justen
2019-08-16 15:09 ` Michael D Kinney
2019-08-16 17:30 ` Andrew Fish [this message]
2019-08-16 18:21 ` Jordan Justen
2019-08-16 21:14 ` Michael D Kinney
2019-08-16 2:14 ` [Patch V4 07/10] EmulatorPkg/Unix/Host: Fix BerkeleyPacketFilter.c issues Michael D Kinney
2019-08-16 7:47 ` Jordan Justen
2019-08-16 2:14 ` [Patch V4 08/10] EmulatorPkg: Disable TftpDynamicCommand and LogoDxe for XCODE5 Michael D Kinney
2019-08-16 2:14 ` [Patch V4 09/10] EmulatorPkg/Sec: Change local variable scope " Michael D Kinney
2019-08-16 18:29 ` Jordan Justen
2019-08-16 2:14 ` [Patch V4 10/10] BaseTools/tools_def.template: Add -gdwarf to XCODE5 X64 Michael D Kinney
2019-08-16 4:50 ` [edk2-devel] " Liming Gao
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=2BB17527-DAF7-4C52-B77D-D03511160F71@apple.com \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox