Closed Bug 1147266 Opened 5 years ago Closed 5 years ago

Update b2g repos for nexus-5-l to mr1

Categories

(Firefox OS Graveyard :: GonkIntegration, defect, major)

ARM
Gonk (Firefox OS)
defect
Not set
major

Tracking

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

RESOLVED FIXED
2.2 S10 (17apr)
blocking-b2g 2.2+
Tracking Status
b2g-v2.2 --- fixed
b2g-master --- fixed

People

(Reporter: seinlin, Assigned: seinlin)

References

Details

Attachments

(10 files, 2 obsolete files)

56 bytes, text/x-github-pull-request
mwu
: review+
Details | Review
59 bytes, text/x-github-pull-request
mwu
: review+
Details | Review
54 bytes, text/x-github-pull-request
mwu
: review+
Details | Review
61 bytes, text/x-github-pull-request
mwu
: review+
Details | Review
62 bytes, text/x-github-pull-request
mwu
: review+
Details | Review
65 bytes, text/x-github-pull-request
mwu
: review+
Details | Review
61 bytes, text/x-github-pull-request
cyu
: review+
Details | Review
58 bytes, text/x-github-pull-request
mwu
: review+
Details | Review
60 bytes, text/x-github-pull-request
jld
: review+
Details | Review
66 bytes, text/x-github-pull-request
mwu
: review+
Details | Review
No description provided.
Depends on: 1148324
Michael, I re-base all our modifications on 5.0 to 5.1, Would you mind review this? Thanks!
Attachment #8585342 - Flags: review?(mwu)
Attachment #8585343 - Flags: review?(mwu)
Attachment #8585344 - Flags: review?(mwu)
Attachment #8585350 - Flags: review?(mwu)
Attachment #8585352 - Flags: review?(mwu)
Attachment #8585354 - Flags: review?(mwu)
Summary: Update gonk-l to mr1 → Update nexus-5-l to mr1
Assignee: nobody → kli
Depends on: 1144512, 1144515
Attachment #8585342 - Flags: review?(mwu) → review+
Attachment #8585343 - Flags: review?(mwu) → review+
Attachment #8585344 - Flags: review?(mwu) → review+
Attachment #8585345 - Flags: review?(mwu) → review+
Attachment #8585346 - Flags: review?(mwu) → review+
Attachment #8585347 - Flags: review?(mwu) → review+
Comment on attachment 8585348 [details] [review]
Update platform_prebuilts_misc to lmr1.

To Cervantes, to verify if we still need to upgrade our gdb.
Attachment #8585348 - Flags: review?(mwu) → review?(cyu)
Attachment #8585350 - Flags: review?(mwu) → review+
Attachment #8585354 - Flags: review?(mwu) → review+
Comment on attachment 8585352 [details] [review]
Update codeaurora_kernel_msm to lmr1.

Did seccomp-bpf land upstream? Not sure why there's only two commits here..

jld, does this look ok to you?
Attachment #8585352 - Flags: review?(mwu) → review?(jld)
Comment on attachment 8585353 [details] [review]
Update device_lge_hammerhead-kernel to lmr1.

r=me assuming the corresponding source branch is r+'d. No need to request review for new builds if the source branch is updated.
Attachment #8585353 - Flags: review?(mwu) → review+
Blocks: 1149387
Depends on: 1144513, 1144514
Attachment #8585348 - Flags: review?(cyu) → review+
(In reply to Michael Wu [:mwu] from comment #12)
> Comment on attachment 8585348 [details] [review]
> Update platform_prebuilts_misc to lmr1.
> 
> To Cervantes, to verify if we still need to upgrade our gdb.

5.1 still uses gdbserver 7.6 so we still need to upgrade to 7.7.
(In reply to Michael Wu [:mwu] from comment #13)
> Comment on attachment 8585352 [details] [review]
> Update codeaurora_kernel_msm to lmr1.
> 
> Did seccomp-bpf land upstream? Not sure why there's only two commits here..

Looks like it did, as of lollipop-mr1: https://github.com/android/kernel_msm/commit/53e0661faaa2c23db1c7b38732b3f3ab27b7a6bb

Also… I just noticed that SECCOMP_FILTER defaults to "y" if SECCOMP is enabled, which explains this upstream commit: https://github.com/android/kernel_msm/commit/072e22762cb5e70a5a4c5e64cbef71195e23c225 (also new as of lollipop-mr1)

So I think we can drop the "enable seccomp-bpf" commit, because it's already enabled.
Comment on attachment 8585352 [details] [review]
Update codeaurora_kernel_msm to lmr1.

r=me with the "enable seccomp-bpf" patch removed; see previous comment.

Testing whether B2G boots is enough to verify that change, because we require seccomp-bpf support and check for it very early (bug 1009995).
Attachment #8585352 - Flags: review?(jld) → review+
Thanks! I removed "enable seccomp-bpf" patch and verified that b2g can boot up correctly. PR of device_lge_hammerhead-kernel is also updated accordingly.
Depends on: 1150345
Depends on: 1150904
No longer blocks: gonk-L
Blocks: gonk-L
No longer blocks: gonk-L
Depends on: 1151787
nexus-5-l need this update to verify v2.2 branch.
blocking-b2g: --- → 2.2?
Need to Support L MR1 in 2.2
blocking-b2g: 2.2? → 2.2+
Reverted. The v2.2 PR did not have driver approval and dependent patches were not reviewed.

V2.2: https://github.com/mozilla-b2g/b2g-manifest/commit/e79c9709e6c94733b5090afdb5eb0f360b83e90c
Master: https://github.com/mozilla-b2g/b2g-manifest/commit/ba3b8dc6ebc441fbaf197a795fc3ab53f926758c
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Summary: Update nexus-5-l to mr1 → Update b2g repos for nexus-5-l to mr1
To avoid build bustage, base-l.xml and all manifests depend on it should be updated at the same time. Bug 1153631 will update this. As all repos for nexus-5-l are updated as comment 19. Let's resolve this bug.
Status: REOPENED → RESOLVED
Closed: 5 years ago5 years ago
Resolution: --- → FIXED
Blocks: 1153631
No longer depends on: 1150904, 1151787
Comment on attachment 8585354 [details] [review]
Update nexus-5-l manifest to b2g-5.1.0_r1.

Bug 1153631 will follow up to update the manifest.
Attachment #8585354 - Attachment is obsolete: true
Attachment #8590291 - Attachment is obsolete: true
Target Milestone: --- → 2.2 S10 (17apr)
You need to log in before you can comment on or make changes to this bug.