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

Issue 737937 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Jul 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Feature



Sign in to add a comment

atrusd: log number of connected Atruses

Project Member Reported by lndmrk@chromium.org, Jun 29 2017

Issue description

We have had issues where multiple Atruses have been connected simultaneously via USB instead of daisy-chained. We wan't to detect this by logging how many Atruses that are connected to a host.
 

Comment 1 by lndmrk@chromium.org, Jun 30 2017

Status: Started (was: Assigned)
Project Member

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

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/atrusctl/+/398b2ac413be2cb972c2e0bc80bd399b7b522e78

commit 398b2ac413be2cb972c2e0bc80bd399b7b522e78
Author: Karl Petersson <karl@limesaudio.com>
Date: Thu Jul 06 19:12:05 2017

Log number of connected devices

So we can track if users attempt to connect multiple Atrus devices via
USB

BUG= chromium:737937 
TEST=Started atrusd, added/removed multiple devices and confirmed that it
logged the correct amount of devices connected
CQ-DEPEND=CL:552152

Change-Id: I9d909cd18b6db5fad8a9df6b80b7b57d8bbba37f
Reviewed-on: https://chromium-review.googlesource.com/558082
Commit-Ready: Emil Lundmark <lndmrk@chromium.org>
Tested-by: Emil Lundmark <lndmrk@chromium.org>
Reviewed-by: Simon Glass <sjg@chromium.org>

[modify] https://crrev.com/398b2ac413be2cb972c2e0bc80bd399b7b522e78/src/atrusd.cc
[modify] https://crrev.com/398b2ac413be2cb972c2e0bc80bd399b7b522e78/src/udev_device_manager.cc
[modify] https://crrev.com/398b2ac413be2cb972c2e0bc80bd399b7b522e78/src/diagnostics.cc
[modify] https://crrev.com/398b2ac413be2cb972c2e0bc80bd399b7b522e78/src/diagnostics.h

Status: Verified (was: Started)
rpattumani@ - could you please test / verify this.
Thanks
Labels: Proj-Bluestreak

Sign in to add a comment