New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 726789 link

Starred by 4 users

Issue metadata

Status: Verified
Owner:
Closed: Jun 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

[3.8, 3.10, 3.14] Partitions on USB sticks not detected/present in File manager

Project Member Reported by ka...@chromium.org, May 26 2017

Issue description

Some autotests alerted us on USB detection of some partitions like NTFS and ExFAT like Failures reported: ['STEP 1.3. PLUG FAILS - USB CHECKS DETAILS failed at ls -l /media/removable/ | grep -i ExFAT: - 60 sec', 'STEP 1.3. PLUG FAILS - USB CHECKS DETAILS failed at ls -l /media/removable/ | grep -i NTFS: - 60 sec']

Test results view screenshot - https://screenshot.googleplex.com/aJFTbrQrsiy

Started with R60-9590.0.0

The diff https://crosland.corp.google.com/log/9589.0.0..9590.0.0 shows devices on kernel version 3.8, 3.10 and 3.14 have this changes checked in:
https://chromium-review.googlesource.com/c/515207/
https://chromium-review.googlesource.com/c/515204/

+groeck@
 

Comment 1 by ka...@chromium.org, May 26 2017

Cc: helenzhang@chromium.org pgangishetty@chromium.org
Labels: M-60 OS-Chrome
Summary: [3.8, 3.10, 3.14] Partitions on USB sticks not detected/present in File manager (was: Partitions on USB sticks not detected/present in File manager)
Manually testing boards on these versions confirm the issue. In some cases the readings from /removable media is not consistent.

Comment 2 by ka...@chromium.org, May 26 2017

Labels: -Pri-3 Pri-1

Comment 3 by groeck@chromium.org, May 26 2017

The changes in question only affect configurations with CONFIG_USB_SERIAL_EDGEPORT_TI and/or CONFIG_USB_SERIAL_OMNINET enabled. Both are not enabled in any of the affected builds (it was only applied since it fixes a CVE). It is therefore very unlikely that the problem is caused by CL:515207 and/or CL:515204.
I'll do some more research, and may possibly revert the patches in question for simplicity in the affected releases, but that will very likely not fix the problem.

Comment 4 by groeck@chromium.org, May 26 2017

Owner: yueherngl@chromium.org
Given #3, reassigning to yueherngl@chromium.org to check if this could have been caused by the libusb update (https://chromium-review.googlesource.com/513502).

Comment 5 by dchan@google.com, May 26 2017

Status: Assigned (was: Untriaged)

Comment 6 by ka...@chromium.org, May 26 2017

Cc: josa...@chromium.org
Labels: ReleaseBlock-Beta
RBB, as it is consistent across these boards

Comment 7 by son...@google.com, May 30 2017

Still able to reproduce this issue on build 9592.0.0
Reproducible in Chrome OS 9592.3.0, 60.0.3112.10 on Pi, 9592.5.0, 60.0.3112.10 on spring.  
yueherngl@ can you look at CL in c#4 to confirm if this is the culprit here?
I will try to see if I can reproduce the problem, then revert the change to see how that goes to confirm.
Thanks.
Cc: vapier@chromium.org
Status: Verified (was: Assigned)
This got resolved with R61-9623.0.0
Not sure what fixed it.
Labels: Merge-TBD
[Auto-generated comment by a script] We noticed that this issue is targeted for M-60; it appears the fix may have landed after branch point, meaning a merge might be required. Please confirm if a merge is required here - if so add Merge-Request-60 label, otherwise remove Merge-TBD label. Thanks.
RE: #12

Thanks for updating. Not sure if this problem has anything to do with libusb then? Is it possible to bisect and narrow down the range? Thanks again.
Cc: benchan@chromium.org
Things changed on M60 too - at 9592.12.0 - https://screenshot.googleplex.com/3r7qwKHoM21
And the diff with .11 build shows no much at https://crosland.corp.google.com/log/9592.11.0..9592.12.0

Could it be fixed by https://chromium-review.googlesource.com/c/525055/  on M60?
It's a merge of the original change https://chromium-review.googlesource.com/c/523017, that went in with R61-9623.0.0 - the build we see M61 fixed.
Project Member

Comment 16 by sheriffbot@chromium.org, Jul 24 2017

Labels: -Merge-TBD

Sign in to add a comment