New issue
Advanced search Search tips

Issue 778995 link

Starred by 1 user
Status: Fixed
Owner:
yawano@chromium.org
Closed: Nov 2017
Cc:
vapier@chromium.org
hashimoto@chromium.org
nya@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

Connecting a phone to a Chromebook via MTP does not work

Project Member Reported by hashimoto@chromium.org, Oct 27 2017 
Chrome Version: ToT
OS: Chrome

What steps will reproduce the problem?
(1) Connect an Android phone to a Chromebook via MTP

What is the expected result?
Files in the phone is shown w/ Files app.

What happens instead?
Nothing happens.

Please use labels and text to provide additional information.
This worked on a kevin running 63.0.3239.20 (Platform 10032.17.0), and a chell running 62.0.320.74 (Platform 9901.54.0).


For graphics-related bugs, please copy/paste the contents of the about:gpu
page at the end of this report.

Comment 1 by hashimoto@chromium.org, Oct 30 2017 

Tried on a samus, and it seems R64-10052.0.0 (Chrome 64.0.3244.0) is the first bad build.
R64-10051.0.0 also comes with Chrome 64.0.3244.0, but this build looks OK.

Looking at the diffs between 10052 and 10051, I guess chromium-review.googlesource.com/393166 is the culprit?
https://crosland.corp.google.com/log/10051.0.0..10052.0.0

Comment 2 by yawano@chromium.org, Oct 30 2017

Cc: nya@chromium.org vapier@chromium.org hashimoto@chromium.org
Status: Started (was: Assigned)
Reverted the CL locally, and it fixed the issue. I'll investigate what is causing this issue.

Comment 3 by yawano@chromium.org, Oct 30 2017 

Removing -e option fixes the issue. I'll prepare a CL to remove the option.

Comment 4 by yawano@chromium.org, Oct 30 2017 

Uploaded a fix at https://crrev.com/c/742827

mtpd is using udev_monitor_new_from_netlink.

https://chromium.googlesource.com/chromiumos/platform/mtpd/+/master/device_manager.cc#84

Comment 5 by vapier@chromium.org, Oct 30 2017 

it was working on my device, but maybe the way i launched it changed things.  searching around doesn't find clear documentation on the behavior of netlink sockets across net namespaces, so better to drop it to get things working again.
Project Member

Comment 6 by bugdroid1@chromium.org, Nov 2 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform/mtpd/+/ec50966c1cdb7cc2114f08217c541ea6f1405720

commit ec50966c1cdb7cc2114f08217c541ea6f1405720
Author: Yuki Awano <yawano@google.com>
Date: Thu Nov 02 07:13:02 2017

Do not enter new network namespace

BUG= chromium:778995 
TEST=Connect a mtp device and confirm that it's available from Files
     app.

Change-Id: I69ecef8b5ddee35227cad04495ca8e0ced1de6b4
Reviewed-on: https://chromium-review.googlesource.com/742827
Commit-Ready: Yuki Awano <yawano@chromium.org>
Tested-by: Yuki Awano <yawano@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/ec50966c1cdb7cc2114f08217c541ea6f1405720/mtpd.conf
[modify] https://crrev.com/ec50966c1cdb7cc2114f08217c541ea6f1405720/mtpd.service

Comment 7 by yawano@chromium.org, Nov 2 2017

Status: Fixed (was: Started)
►

Sign in to add a comment