commit | 35ec8408d8b40d74693edc32495b392a525dc238 | [log] [tgz] |
---|---|---|
author | mukesh agrawal <quiche@chromium.org> | Fri Jul 19 14:49:08 2013 -0700 |
committer | ChromeBot <chrome-bot@google.com> | Mon Jul 22 23:14:08 2013 -0700 |
tree | 931729eb4a3f7cab847941fef790d8796cb75664 | |
parent | 4823183bd3a6e31868fb54972c7ef9bc49b52ec3 [diff] |
shill: PPP dongles: cleanly move Service from Cellular to PPPDevice In CL:61602, I added a CHECK that the Cellular object doesn't have a selected_service() when PPP comes up. The idea was to ensure that we only have one Device trying to drive the state of the CellularService. This CHECK fails on devices with an embryonic cdc_ether or cdc_ncm interface, because we SelectService() in LinkEvent(). Resolve this by dropping the selected_service() in StartPPP(). We do this using DropConnection(), which may be what I should have done all along. (DropConnection() also takes care of killing dhcpcd, if it is running.) While there: fix up a couple of buggy log statements in device_info. BUG=chromium:261364 TEST=unit tests, manual Manual testing -------------- - plug in a dongle with an embryonic cdc_ether or cdc_ncm interface. e.g. the huawei UMG1691 dongle - check that the cellular connection comes up (IP address is configured, can load web pages) - check that shill did not crash in the process (grep crash /var/log/messages) - repeat steps 1-3 with a ppp-only dongle (e.g. huawei e369) Change-Id: Ia270f00746dc321b41a223e9f848ffc796dd8528 Reviewed-on: https://gerrit.chromium.org/gerrit/62712 Reviewed-by: mukesh agrawal <quiche@chromium.org> Tested-by: mukesh agrawal <quiche@chromium.org> Reviewed-by: Ben Chan <benchan@chromium.org> Commit-Queue: mukesh agrawal <quiche@chromium.org>