858853 - BRANCH=master emulator broken, due to new kernel?

Status: RESOLVED FIXED

(Firefox OS Graveyard :: GonkIntegration, defect)

Description

[Jonathan Griffin (:jgriffin)]

If you |BRANCH=master ./config.sh emulator|, the resulting emulator build you get will not launch.  It will hang with a black screen, without reporting any errors to stdout.  |adb devices| will show the device as online, but all other adb commands (adb logcat, adb shell) will hang.

Attached are two sources.xml, one from a good build, and one from a bad build generated a short time later.  Aside from gecko and gaia differences, the only difference is a different qemu kernel:

good:
  <project name="platform/prebuilts/qemu-kernel" path="prebuilts/qemu-kernel" revision="ec4a882d411d0d4ceadc5912ab4ce6fd4df23ab7"/>

bad:
  <project name="platform/prebuilts/qemu-kernel" path="prebuilts/qemu-kernel" revision="cc1a057747814c4d47fa5cc083b86c24bb9a74cd"/>

Comment 1

[Jonathan Griffin (:jgriffin)]

Attachment: good sources.xml

Comment 2

[Jonathan Griffin (:jgriffin)]

Attachment: bad sources.xml

Comment 3

[Michael Wu [:mwu]]

Guess we'll have to force the kernel back to the old one.

Comment 5

[Vicamo Yang [:vicamo][:vyang]]

Attachment: Github PR for manifest

Comment 6

[Michael Wu [:mwu]]

Comment on attachment 734479 [details]
Github PR for manifest

r- because we don't need a mirror. We just need to point to the last good revision - ec4a882d411d0d4ceadc5912ab4ce6fd4df23ab7.

Comment 7

[Vicamo Yang [:vicamo][:vyang]]

(In reply to Michael Wu [:mwu] from comment #6)
> Comment on attachment 734479 [details]
> Github PR for manifest
> 
> r- because we don't need a mirror. We just need to point to the last good
> revision - ec4a882d411d0d4ceadc5912ab4ce6fd4df23ab7.

Please consider we'll need our own kernel image for bug 852853, Bluetooth emulation.

Comment 8

[Vicamo Yang [:vicamo][:vyang]]

(In reply to Vicamo Yang [:vicamo][:vyang] from comment #7)
> Please consider we'll need our own kernel image for bug 852853, Bluetooth
> emulation.

Sorry, bug 852583.

Comment 9

[Martijn Wargers (dead)]

I would like to have a working emulator again, how do I revert back to ec4a882d411d0d4ceadc5912ab4ce6fd4df23ab7 ?
Or how do I apply https://bugzilla.mozilla.org/attachment.cgi?id=734479 ?

Comment 10

[Jonathan Griffin (:jgriffin)]

In your B2G folder, go to prebuilts/qemu-kernel, and run the command:

git checkout ec4a882d411d0d4ceadc5912ab4ce6fd4df23ab7 

then /build.sh.  You'll have to do this each time you ./config.sh emulator, since ./config.sh will reset the state of that tree.

Comment 11

[Vicamo Yang [:vicamo][:vyang]]

(In reply to Martijn Wargers [:mw22] (QA - IRC nick: mw22) from comment #9)
> I would like to have a working emulator again, how do I revert back to
> ec4a882d411d0d4ceadc5912ab4ce6fd4df23ab7 ?
> Or how do I apply https://bugzilla.mozilla.org/attachment.cgi?id=734479 ?

I've update the pull request to point the last green commit instead.

Comment 12

[Michael Wu [:mwu]]

(In reply to Vicamo Yang [:vicamo][:vyang] from comment #11)
> (In reply to Martijn Wargers [:mw22] (QA - IRC nick: mw22) from comment #9)
> > I would like to have a working emulator again, how do I revert back to
> > ec4a882d411d0d4ceadc5912ab4ce6fd4df23ab7 ?
> > Or how do I apply https://bugzilla.mozilla.org/attachment.cgi?id=734479 ?
> 
> I've update the pull request to point the last green commit instead.

Merged, thanks.

We want support for bluetooth emulation but that's a separate issue from this.

Comment 13

[Vicamo Yang [:vicamo][:vyang]]

(In reply to Michael Wu [:mwu] from comment #12)
> We want support for bluetooth emulation but that's a separate issue from
> this.

Sure. Just don't block other people.