Closed Bug 1129776 Opened 8 years ago Closed 8 years ago

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

Categories

(Firefox OS Graveyard :: GonkIntegration, defect)

Product:
Firefox OS Graveyard
Component:
GonkIntegration
Platform:
x86_64
Linux
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

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

Status:
RESOLVED FIXED
Milestone:
2.2 S5 (6feb)
Project Flags:
blocking-b2g 2.2+
Tracking Flags:
Tracking Status
b2g-v2.2 --- fixed
b2g-master --- fixed

People

(Reporter: shawnjohnjr, Assigned: cyu)

Details

Attachments

(3 files)

PIE gdbserver 7.7
357.70 KB, application/x-sharedlib
Details
The gdbserver v7.7 PIE binary
61 bytes, text/x-github-pull-request
seinlin
: review+
Details | Review
Update repo manifest
52 bytes, text/x-github-pull-request
seinlin
: review+
Details | Review 
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
Attached 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)

    
    

    
  

      
      
      
      

        Attached file
          
        Update repo manifest
        — 
      

    


  

        Attachment #8560368 -
        Flags: review?(kli)
blocking-b2g: --- → 2.2?

    
  
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

          status-b2g-v2.2:
          --- → fixed

          status-b2g-master:
          --- → fixed
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.