| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
In the interest of making long-term branch maintenance incur as little
technical debt on us as possible, we should not maintain any files on
the branch we are not actually using.
This has the added effect of making it extremely clear when merging CLs
from the main branch when changes have the possibility to affect us.
The follow-on CL adds a convenience script to actually pull updates from
the main branch and generate a CL for the update.
BUG=b:204206272
BRANCH=ish
TEST=make BOARD=arcada_ish && make BOARD=drallion_ish
Signed-off-by: Jack Rosenthal <jrosenth@chromium.org>
Change-Id: I17e4694c38219b5a0823e0a3e55a28d1348f4b18
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3262038
Reviewed-by: Jett Rink <jettrink@chromium.org>
Reviewed-by: Tom Hughes <tomhughes@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This requests that cros lint (and repo upload hook) use the new
Chromium OS 4 space indent policy.
Since legacy python scripts still use 2 space, I added pylint ignore
statements to the individual files to disable indentation checking.
Note: There are still valid pylint errors in some of these legacy scripts.
BRANCH=none
BUG=none
TEST=cros lint util/*.py
Signed-off-by: Craig Hesling <hesling@chromium.org>
Change-Id: I439f5a87bc50f1f43a4996e574bbc0626922a88e
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3064761
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Reviewed-by: Aseda Aboagye <aaboagye@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch makes uart_stress_tester.py import logging instead of
cros_logging from chromite.lib.
BUG=none
BRANCH=none
TEST=ran uart_stress_tester.py on local chroot and on labstation.
Signed-off-by: Namyoon Woo <namyoon@google.com>
Change-Id: I479e2b7a046d6c7b02fb4aca2c7e1fc83334e294
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2190826
Tested-by: Namyoon Woo <namyoon@chromium.org>
Reviewed-by: Ruben Rodriguez Buchillon <coconutruben@chromium.org>
Reviewed-by: Mary Ruthven <mruthven@chromium.org>
Commit-Queue: Namyoon Woo <namyoon@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Convert the code for python3, and fixed some pylint
errors.
BUG=none
BRANCH=none
TEST=manually ran it on Fleex.
'uart_stress_tester.py -d -t 300 /dev/ttyUSB2'
Also ran pylint.
'pylint --rcfile /mnt/host/source/chromite/pylintrc
uart_stress_tester.py'
Change-Id: Ie983eff06e0757af14ebc16878ca892fc1f629a5
Signed-off-by: Namyoon Woo <namyoon@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2015351
Reviewed-by: Mary Ruthven <mruthven@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
| |
BUG=chromium:992607
BRANCH=None
TEST=ran stress test on fleex.
./util/uart_stress_tester.py /dev/ttyUSB0 /dev/ttyUSB2 \
--time 120 --debug --usb
Change-Id: Iebe5e8b18116fcbd5b94bc5796983cac4fb40087
Signed-off-by: Namyoon Woo <namyoon@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/1891254
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch let the tester exit with an error code on test failure.
It would make easy the future autotest to detect a failure.
BUG=None
BRANCH=None
TEST=ran on fleex.
$ uart_stress_tester.py /dev/ttyUSB1 /dev/ttyUSB2 -t 120 -d
[before patch]
...
INFO | UartSerial| EC | 14888 char lost / 1382400 (1.1 %)
INFO | UartSerial| AP | 0 char lost / 1382400 (0.0 %)
ERROR | ChargenTest | FAIL: lost 14888 character(s) from the test
...
$ echo $?
0
[after patch]
...
INFO | UartSerial| EC | 14888 char lost / 1382400 (1.1 %)
INFO | UartSerial| AP | 0 char lost / 1382400 (0.0 %)
ERROR | ChargenTest | FAIL: lost 14888 character(s) from the test
Error: Test failed for losing 144888 character(s)
...
$ echo $?
1
$ ./util/uart_stress_tester.py /dev/ttyUSB1 /dev/ttyUSB2 -t 120 -d
Error: /dev/ttyUSB1 does not exist.
$ echo $?
1
Change-Id: I210efd4ad7fdb8eb612206624eda6c39c5bb3b1c
Signed-off-by: Namyoon Woo <namyoon@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/1696115
Reviewed-by: Vadim Bendebury <vbendeb@chromium.org>
Reviewed-by: Aseda Aboagye <aaboagye@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
During initialization, uart_stress_tester checks each of ports
exists, and raises an error if not.
BUG=None
BRANCH=None
TEST=ran on fleex.
$ uart_stress_tester.py /dev/ttyUSBX -t 120
Error: [Errno 2] No such file or directory: '/dev/ttyUSBX'
$ ./util/uart_stress_tester.py ./util/uart_stress_tester.py -t 120
Error: ./util/uart_stress_tester.py is not a character device.
Change-Id: I989a9e767796b04b059861aea2a3412877c1d739
Signed-off-by: Namyoon Woo <namyoon@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/1706626
Reviewed-by: Vadim Bendebury <vbendeb@chromium.org>
Reviewed-by: Aseda Aboagye <aaboagye@chromium.org>
|
|
Uart stress tester runs a 'chargen' UART command on EC and/or AP, and
checks if any characters are lost from UART output.
BUG=b:131340067
BRANCH=None
TEST=ran on Bob and Octopus (Fleex)
$ ./util/uart_stress_tester.py -h
usage: uart_stress_tester.py [-h] [-c] [-d] [-t TIME] [port [port ...]]
uart_stress_tester.py repeats sending a uart console command
to each UART device for a given time, and check if output
has any missing characters.
Examples:
uart_stress_tester.py /dev/ttyUSB2 --time 3600
uart_stress_tester.py /dev/ttyUSB1 /dev/ttyUSB2 --debug
uart_stress_tester.py /dev/ttyUSB1 /dev/ttyUSB2 --cr50
positional arguments:
port UART device path to test
optional arguments:
-h, --help show this help message and exit
-c, --cr50 generate TPM workload on cr50
-d, --debug enable debug messages
-t TIME, --time TIME Test duration in second
$ ./util/uart_stress_tester.py /dev/ttyUSB1 /dev/ttyUSB2 -t 120
INFO | UartSerial| EC | 0 char lost / 1382400 (0.0 %)
INFO | UartSerial| AP | 0 char lost / 1382400 (0.0 %)
INFO | ChargenTest | PASS: lost 0 character(s) from the test
$ ./util/uart_stress_tester.py /dev/ttyUSB1 /dev/ttyUSB2 -t 120 --cr50
INFO | UartSerial| EC | 0 char lost / 1382400 (0.0 %)
INFO | UartSerial| AP | 0 char lost / 1382400 (0.0 %)
INFO | ChargenTest | PASS: lost 0 character(s) from the test
Change-Id: I713fb0180db3ca5904bd7aae0dd26a4633733d2e
Signed-off-by: Namyoon Woo <namyoon@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/1683011
Reviewed-by: Mary Ruthven <mruthven@chromium.org>
|