[Nexus-5-l] Provide gdbserver 7.7 for Lollipop

RESOLVED FIXED in 2.2 S5 (6feb)

Status

defect
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: shawnjohnjr, Assigned: cyu)

Tracking

unspecified
2.2 S5 (6feb)
x86_64
Linux

Firefox Tracking Flags

(blocking-b2g:2.2+, b2g-v2.2 fixed, b2g-master fixed)

Details

Attachments

(3 attachments)

Current v2.2 uses
 <project name="platform/prebuilts/misc" path="prebuilts/misc" revision="de282aa3cc6a3917fa42968fc85676846db776b0"/>

But it missed Bug 1078383: Update gdbserver to v7.7
Summary: [Neuxs-5-l] Update manifest to use correct gdb server for v2.2 → [Neuxs-5-l] Provie gdbserver 7.7 for Lollipop
We cannot use this commit for L.

 Bug 1078383: Update gdbserver to v7.7. r=mwu
    
    This gdbserver binary is built using:

Due to L has enforced PIE security check, this caused L cannot use gdbserver to restart b2g process.

Error log:
./run-gdb.sh attach ADB Location:  adb error: only position independent executables (PIE) are supported. prebuilts/gcc/linux-x86/arm/arm-linux-androideabi-4.8/bin/arm-linux-androideabi-gdb -x /tmp/b2g.gdbinit.shawnjohnjr.8462 /code/nexus5-l/B2G/objdir-gecko-v2_2/dist/bin/b2g GNU gdb (GDB) 7.6
ni Cervantes, he knows better gdbserver according to bug 1078383.
Flags: needinfo?(cyu)
Summary: [Neuxs-5-l] Provie gdbserver 7.7 for Lollipop → [Neuxs-5-l] Provide gdbserver 7.7 for Lollipop
Posted file PIE gdbserver 7.7
The PIE version of gdbserver 7.7.
prebuilt and manifest to be updated using this file.
Flags: needinfo?(cyu)
Great, now new gdbserver works. :)
I think we can use this new gdbserver in base-l.xml as how base-kk.xml.
For the record, the configure command I used in building this binary is:
CFLAGS="--sysroot=$B2G_REPO/prebuilts/ndk/8/platforms/android-14/arch-arm -fPIE" LDFLAGS="-pie" ./configure --host=arm-linux-androideabi
Attachment #8560354 - Flags: review?(kli)
Attachment #8560368 - Flags: review?(kli)
blocking-b2g: --- → 2.2?

Updated

4 years ago
blocking-b2g: 2.2? → 2.2+
Attachment #8560354 - Flags: review?(kli) → review+
Attachment #8560368 - Flags: review?(kli) → review+
BTW, we need to update run-gdb.sh to use gdb in lollipop base device. I filed bug 1130673 to update it.
Any reason we're bypassing the regular approval process for these?
Assignee: nobody → cyu
Flags: needinfo?(kli)
Summary: [Neuxs-5-l] Provide gdbserver 7.7 for Lollipop → [Nexus-5-l] Provide gdbserver 7.7 for Lollipop
Target Milestone: --- → 2.2 S5 (6feb)
I asked bajaj about the repos other than gecko/gaia and know that 2.2+ is fine.
Flags: needinfo?(kli)
You need to log in before you can comment on or make changes to this bug.