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

Issue 653195 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

eCTS: Implement Interrupt Suite

Project Member Reported by dnojiri@chromium.org, Oct 5 2016

Issue description

(eCTS: https://docs.google.com/document/d/1VFZgGV6uBxOWS7mJHlOAWWmkvqz6ctdgcbCaDo7RYhc/edit)

Umbrella bug for all interrupt tests, which include nested/pre-empting interrupts, priorities.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Oct 8 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform/ec/+/473ecbe2b36bc44da2552cd5814874a495913fc1

commit 473ecbe2b36bc44da2552cd5814874a495913fc1
Author: Daisuke Nojiri <dnojiri@chromium.org>
Date: Wed Jul 27 22:46:42 2016

cts: Add real interrupt test

Interrupt test checks whether DUT can be interrupted by an interrupt
and an interrupt handler can be invoked as expected.

Note the previous interrupt test ported from test/interrupt.c runs in
an emulated environment on the host, thus does not test the real
interrupt capability of the chip.

BUG=chromium:653195
BRANCH=none
TEST=Run cts.py -m interrupt

Change-Id: I21cecff07594f048633d1c1b699fb3a1876379e0
Signed-off-by: Daisuke Nojiri <dnojiri@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/363943
Reviewed-by: Randall Spangler <rspangler@chromium.org>

[modify] https://crrev.com/473ecbe2b36bc44da2552cd5814874a495913fc1/board/nucleo-f072rb/board.c
[modify] https://crrev.com/473ecbe2b36bc44da2552cd5814874a495913fc1/cts/interrupt/th.c
[modify] https://crrev.com/473ecbe2b36bc44da2552cd5814874a495913fc1/cts/common/cts_common.h
[modify] https://crrev.com/473ecbe2b36bc44da2552cd5814874a495913fc1/board/nucleo-f072rb/gpio.inc
[add] https://crrev.com/473ecbe2b36bc44da2552cd5814874a495913fc1/cts/interrupt/cts.testlist
[modify] https://crrev.com/473ecbe2b36bc44da2552cd5814874a495913fc1/cts/interrupt/dut.c

Project Member

Comment 2 by bugdroid1@chromium.org, Nov 10 2016

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

commit b2f14a26b9e5b72486e9a7ad0e232fed269704c2
Author: Daisuke Nojiri <dnojiri@chromium.org>
Date: Tue Oct 18 16:27:34 2016

eCTS: Add nested interrupt test (Low->High)

Add a nested interrupt test to eCTS. Lower priority IRQ is fired,
followed by higher priority IRQ. Handler executions should be nested.

P1                    *-----*
                     /       \
P2             *----*         *----*
              /                     \
task_cts ----*                       *----
               A      B     C      D

BUG=chromium:653195
BRANCH=none
TEST=cts.py -m gpio, interrupt, timer; make buildall

Change-Id: I34dc7b4e819051b9070a11e69d13d6be704f2e5f
Reviewed-on: https://chromium-review.googlesource.com/408797
Commit-Ready: Daisuke Nojiri <dnojiri@chromium.org>
Tested-by: Daisuke Nojiri <dnojiri@chromium.org>
Reviewed-by: Randall Spangler <rspangler@chromium.org>

[modify] https://crrev.com/b2f14a26b9e5b72486e9a7ad0e232fed269704c2/board/nucleo-f072rb/board.c
[modify] https://crrev.com/b2f14a26b9e5b72486e9a7ad0e232fed269704c2/cts/interrupt/th.c
[modify] https://crrev.com/b2f14a26b9e5b72486e9a7ad0e232fed269704c2/board/stm32l476g-eval/gpio.inc
[modify] https://crrev.com/b2f14a26b9e5b72486e9a7ad0e232fed269704c2/board/nucleo-f072rb/board.h
[modify] https://crrev.com/b2f14a26b9e5b72486e9a7ad0e232fed269704c2/board/nucleo-f072rb/gpio.inc
[modify] https://crrev.com/b2f14a26b9e5b72486e9a7ad0e232fed269704c2/cts/interrupt/cts.testlist
[modify] https://crrev.com/b2f14a26b9e5b72486e9a7ad0e232fed269704c2/cts/interrupt/dut.c

Project Member

Comment 3 by bugdroid1@chromium.org, Nov 10 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform/ec/+/5488976a20d1036ba425b4061e435a53a022f876

commit 5488976a20d1036ba425b4061e435a53a022f876
Author: Daisuke Nojiri <dnojiri@chromium.org>
Date: Tue Oct 18 16:27:34 2016

eCTS: Add nested interrupt test (High->Low)

Add a nested interrupt test to eCTS. Higher priority IRQ is fired,
followed by lower priority IRQ. Handlers should be executed
sequentially.

P1               *-----*
                /       \
P2             /         *-----*
              /                 \
task_cts ----*                   *----
                 B     C A     D

BUG=chromium:653195
BRANCH=none
TEST=cts.py -m interrupt; make buildall

Change-Id: Ia9f1bf4205cefe8bdc11cc0aa3ad2057359b73ef
Signed-off-by: Daisuke Nojiri <dnojiri@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/409611
Reviewed-by: Randall Spangler <rspangler@chromium.org>

[modify] https://crrev.com/5488976a20d1036ba425b4061e435a53a022f876/cts/interrupt/dut.c
[modify] https://crrev.com/5488976a20d1036ba425b4061e435a53a022f876/cts/interrupt/th.c
[modify] https://crrev.com/5488976a20d1036ba425b4061e435a53a022f876/cts/cts.py
[modify] https://crrev.com/5488976a20d1036ba425b4061e435a53a022f876/cts/interrupt/cts.testlist

Status: Assigned (was: Untriaged)
Labels: eCTS

Sign in to add a comment