Closed
Bug 1158047
Opened 9 years ago
Closed 9 years ago
[Nexus5][Settings] USB storage can't be recognized by PC.
Categories
(Firefox OS Graveyard :: MTP/UMS, defect)
Tracking
(blocking-b2g:2.5?, firefox40 fixed, b2g-v2.2 unaffected, b2g-master verified)
Tracking | Status | |
---|---|---|
firefox40 | --- | fixed |
b2g-v2.2 | --- | unaffected |
b2g-master | --- | verified |
People
(Reporter: yi.zou, Assigned: alchen)
References
Details
(Whiteboard: [v2.2-nexus-5-l])
Attachments
(3 files, 1 obsolete file)
353.22 KB,
text/plain
|
Details | |
3.23 MB,
video/mp4
|
Details | |
2.66 KB,
patch
|
Details | Diff | Splinter Review |
[1.Description]: [Nexus5 v3.0][Settings] Launch settings app.Then turn on USB storage, and select MTP in transfer protocol. USB has been plugged in on PC. But USB storage can't be recognized by PC. Time:16:13 See attachment logcat_1613.txt and 1613.mp4 [2.Testing Steps]: 1.Launch settings app. 2.Turn on USB storage, and select MTP in transfer protocol. 3.USB plugged in on PC. [3.Expected Result]: 3.Device storage could be recognized by PC. [4.Actual Result]: 3.Device storage can't be recognized by PC. [5.Reproduction build]: Flame 2.2 version(Unaffected): Build ID 20150423002502 Gaia Revision b838d0e7c163e66660dcb6e387d8339944a7a30e Gaia Date 2015-04-23 02:32:46 Gecko Revision https://hg.mozilla.org/releases/mozilla-b2g37_v2_2/rev/8dce56574f28 Gecko Version 37.0 Device Name flame Firmware(Release) 4.4.2 Firmware(Incremental) eng.cltbld.20150423.035703 Firmware Date Thu Apr 23 03:57:16 EDT 2015 Bootloader L1TC000118D0 Flame 3.0 version(Unaffected): Build ID 20150423160207 Gaia Revision 0c5e2ee1173f3c53379ef3cd10de714836258fe8 Gaia Date 2015-04-23 16:10:10 Gecko Revision https://hg.mozilla.org/mozilla-central/rev/22a157f7feb7 Gecko Version 40.0a1 Device Name flame Firmware(Release) 4.4.2 Firmware(Incremental) eng.cltbld.20150423.193607 Firmware Date Thu Apr 23 19:36:18 EDT 2015 Bootloader L1TC000118D0 Nexus5 2.2 version(Unaffected): Build ID 20150423002502 Gaia Revision b838d0e7c163e66660dcb6e387d8339944a7a30e Gaia Date 2015-04-23 02:32:46 Gecko Revision https://hg.mozilla.org/releases/mozilla-b2g37_v2_2/rev/8dce56574f28 Gecko Version 37.0 Device Name hammerhead Firmware(Release) 5.1 Firmware(Incremental) eng.cltbld.20150423.035409 Firmware Date Thu Apr 23 03:54:27 EDT 2015 Bootloader HHZ12f Nexus5 3.0 version(Affected): Build ID 20150423160207 Gaia Revision 0c5e2ee1173f3c53379ef3cd10de714836258fe8 Gaia Date 2015-04-23 16:10:10 Gecko Revision https://hg.mozilla.org/mozilla-central/rev/22a157f7feb7 Gecko Version 40.0a1 Device Name hammerhead Firmware(Release) 5.1 Firmware(Incremental) eng.cltbld.20150423.192918 Firmware Date Thu Apr 23 19:29:36 EDT 2015 Bootloader HHZ12f [6.Reproduction Frequency]: Always Recurrence,10/10 [7.TCID]: Free Test [8.Note]: The bug can be repro on Mac and Ubuntu, Windows7
Reporter | ||
Updated•9 years ago
|
status-b2g-master:
--- → affected
Reporter | ||
Updated•9 years ago
|
status-b2g-v2.2:
--- → unaffected
Reporter | ||
Comment 1•9 years ago
|
||
Updated•9 years ago
|
Flags: needinfo?(ashiue)
Comment 2•9 years ago
|
||
We do a regression window wanted, and have narrowed the scope using nightly build, please see the below information. And we will continue doing b2g-inboud regression windows next week, but if someone can help with this, please help to do it in advance. Hope these information can helpfull. Thanks. 20150421010201 (unaffected) 20150421160203 (affected)
Comment 3•9 years ago
|
||
[Blocking Requested - why for this release]: Regression bug, basic function cannot work
blocking-b2g: --- → 3.0?
QA Whiteboard: [COM=MTP/UMS]
Component: Gaia::Settings → MTP/UMS
Flags: needinfo?(ashiue)
Keywords: regressionwindow-wanted
Assignee | ||
Comment 5•9 years ago
|
||
From log, automounter is in a weird mode. (ums Active state is weird, should be 1 or 0) The automounter state stay in "UMS_CONFIGURING". I will try to flash the image on Nexus 5 and check the status. 04-24 16:11:49.349 I/AutoMounter( 197): UpdateState: ums:A138C0E0 mtp:A1C0E1 mode:3 usb:1 tryToShare:0 state:UMS_CONFIGURING 04-24 16:13:25.014 I/AutoMounter( 197): UpdateState: ums:A211C0E0 mtp:A1C0E1 mode:3 usb:0 tryToShare:0 state:UMS_CONFIGURING 04-24 08:34:41.580 I/AutoMounter( 197): UpdateState: ums:A165C0E0 mtp:A1C0E0 mode:0 usb:1 tryToShare:0 state:UMS_CONFIGURING 04-24 08:34:41.580 I/AutoMounter( 197): UpdateState: ums:A20C0E0 mtp:A1C0E0 mode:0 usb:1 tryToShare:0 state:UMS_CONFIGURING
Flags: needinfo?(alchen)
Mozilla-inbound regression window Last Working Environmental Variables Device Nexus 5 Build ID 20150421133006 Gaia Revision 0645bbed4d6cbd8064652eebafe011edc3e417fd Gaia Date 2015-04-21 02:55:48 Gecko Revision https://hg.mozilla.org/integration/mozilla-inbound/rev/8b99d282cd87 Gecko Version 40.0a1 Device Name hammerhead Firmware(Release) 5.1 Firmware(Incremental) eng.cltbld.20150421.120212 Firmware Date Tue Apr 21 12:02:29 EDT 2015 Bootloader HHZ12f First Broken Environmental Variables Device Nexus 5 Build ID 20150421163005 Gaia Revision 15134b080b5f406e5aa36f5136c17dafb4e31f64 Gaia Date 2015-04-21 19:52:45 Gecko Revision https://hg.mozilla.org/integration/mozilla-inbound/rev/e01af01b7015 Gecko Version 40.0a1 Device Name hammerhead Firmware(Release) 5.1 Firmware(Incremental) eng.cltbld.20150421.120212 Firmware Date Tue Apr 21 12:02:29 EDT 2015 Bootloader HHZ12f First Broken Gaia & Last Working Gecko - issue DOES NOT repro Gaia 15134b080b5f406e5aa36f5136c17dafb4e31f64 Gecko 8b99d282cd87 First Broken Gecko & Last Working Gaia - issue DOES repro Gaia 0645bbed4d6cbd8064652eebafe011edc3e417fd Gecko e01af01b7015
Comment 7•9 years ago
|
||
Hi Ben, Could you check again according to comment 6? Thanks :)
Flags: needinfo?(btian)
Keywords: regressionwindow-wanted
Comment 8•9 years ago
|
||
Alphan, please help on this bug.
Assignee: nobody → alchen
Flags: needinfo?(alchen)
Updated•9 years ago
|
Flags: needinfo?(btian)
Assignee | ||
Comment 9•9 years ago
|
||
Hi Dave, please review this patch. In this bug, automounter stuck in UMS_CONFIGURING state, even when user change the mode to MTP. So I move the checking of "mtpEnabled" outside to be a standalone judgement. Besides, umsAvail initialization is missing. I patch the fix as well.
Flags: needinfo?(alchen)
Attachment #8598405 -
Flags: review?(dhylands)
Comment 10•9 years ago
|
||
Comment on attachment 8598405 [details] [diff] [review] Resolve the problem of stucking in UMS_CONFIGURING state Review of attachment 8598405 [details] [diff] [review]: ----------------------------------------------------------------- Looks good and everything seems to still work properly with some testing. Just one change that I suggested. ::: dom/system/gonk/AutoMounter.cpp @@ +859,5 @@ > + // MTP was enabled. Start reconfiguring. > + SetState(STATE_MTP_CONFIGURING); > + SetUsbFunction(USB_FUNC_MTP); > + break; > + } If you move the if (mtpEnabled) before the if (umsConfigured) then we'll eliminate calling SetState(STATE_UMS_CONFIGURED) and then immediately calling SetState(STATE_MTP_CONFIGURED) In fact the if (umsConfigured) should be moved after the if (rndisConfigured) test as well. That way SetState will only be called exactly once. @@ +1118,5 @@ > void AutoMounter::GetStatus(bool& umsAvail, bool& umsConfigured, bool& umsEnabled, > bool& mtpAvail, bool& mtpConfigured, bool& mtpEnabled, > bool& rndisConfigured) > { > + umsAvail = false; good catch
Attachment #8598405 -
Flags: review?(dhylands) → review+
Assignee | ||
Comment 11•9 years ago
|
||
Here is the try server result. It looks fine. https://treeherder.mozilla.org/#/jobs?repo=try&revision=32857bd8f197
Attachment #8598405 -
Attachment is obsolete: true
Assignee | ||
Updated•9 years ago
|
Keywords: checkin-needed
Comment 12•9 years ago
|
||
https://hg.mozilla.org/integration/b2g-inbound/rev/f16a1e24dbcd
Keywords: checkin-needed
Comment 13•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/f16a1e24dbcd
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox40:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 2.2 S12 (15may)
Comment 14•9 years ago
|
||
Verified on Build ID 20151021064220 Gaia Revision 32d827a70af90a05918f234e5b16b35d5d2a07e8 Gaia Date 2015-10-20 20:57:29 Gecko Revision https://hg.mozilla.org/mozilla-central/rev/473aefe5bd85842eeb142e0cde8e2cd21edbf40b Gecko Version 44.0a1 Device Name hammerhead Firmware(Release) 5.1 Firmware(Incremental) eng.cltbld.20151021.104001 Firmware Date Wed Oct 21 10:40:18 EDT 2015 Bootloader HHZ12f
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•