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

Issue 633694 link

Starred by 1 user

Issue metadata

Status: Archived
Owner:
Closed: Aug 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

EC: MKBP: Host interrupt isn't cleared when there are no pending events.

Project Member Reported by aaboagye@chromium.org, Aug 2 2016

Issue description

I was taking a look at the EC output on Shelley's kevin and discovered that the EC was getting spammed with host commands. The host command was EC_CMD_GET_NEXT_EVENT, but the EC was returning that there were no more events to process.

I think that the kernel kept calling the EC for the next event, but there was no more events. However, the EC failed to clear the host interrupt which caused the kernel to continually call down to the EC.

I have a fix in place.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Aug 3 2016

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

commit f5ec39ee0f70ec041eb739f931608e6ee1f96860
Author: Aseda Aboagye <aaboagye@google.com>
Date: Tue Aug 02 20:31:41 2016

mkbp: Clear host interrupt if no more events.

BUG= chromium:633694 
BRANCH=None
TEST=Flash kevin; verify that no more console spam is present on the EC.

Change-Id: I240fbe330952b82e2a5f97d0be7ebe4b2a8e2b46
Signed-off-by: Aseda Aboagye <aaboagye@google.com>
Reviewed-on: https://chromium-review.googlesource.com/365470
Commit-Ready: Aseda Aboagye <aaboagye@chromium.org>
Tested-by: Aseda Aboagye <aaboagye@chromium.org>
Reviewed-by: Randall Spangler <rspangler@chromium.org>

[modify] https://crrev.com/f5ec39ee0f70ec041eb739f931608e6ee1f96860/common/mkbp_event.c

Status: Fixed (was: Started)
Labels: VerifyIn-54

Comment 4 by dchan@chromium.org, Oct 7 2016

Labels: VerifyIn-55

Comment 5 by dchan@google.com, Nov 19 2016

Labels: VerifyIn-56

Comment 6 by dchan@google.com, Jan 21 2017

Labels: VerifyIn-57
Project Member

Comment 7 by bugdroid1@chromium.org, Feb 13 2017

Labels: merge-merged-firmware-veyron-6588.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform/ec/+/108c73bdb2873386e2382cdca9a9dd9dbbd43cd4

commit 108c73bdb2873386e2382cdca9a9dd9dbbd43cd4
Author: Aseda Aboagye <aaboagye@google.com>
Date: Mon Feb 13 09:30:22 2017

mkbp: Clear host interrupt if no more events.

BUG= chromium:633694 ,chrome-os-partner:60882
BRANCH=veyron
TEST=Flash kevin; verify that no more console spam is present on the EC.

Original-Change-Id: I240fbe330952b82e2a5f97d0be7ebe4b2a8e2b46
Signed-off-by: Aseda Aboagye <aaboagye@google.com>
Reviewed-on: https://chromium-review.googlesource.com/365470
Commit-Ready: Aseda Aboagye <aaboagye@chromium.org>
Tested-by: Aseda Aboagye <aaboagye@chromium.org>
Reviewed-by: Randall Spangler <rspangler@chromium.org>
(cherry picked from commit f5ec39ee0f70ec041eb739f931608e6ee1f96860)

Conflicts:
	common/mkbp_event.c

Change-Id: Ia9e641c230538f231b906d73cae548626adf50e6
Signed-off-by: Jeffy Chen <jeffy.chen@rock-chips.com>
Reviewed-on: https://chromium-review.googlesource.com/438067
Reviewed-by: Alexandru Stan <amstan@chromium.org>
Reviewed-by: Simon Glass <sjg@google.com>

[modify] https://crrev.com/108c73bdb2873386e2382cdca9a9dd9dbbd43cd4/common/mkbp_event.c

Comment 8 by dchan@google.com, Mar 4 2017

Labels: VerifyIn-58

Comment 9 by dchan@google.com, Apr 17 2017

Labels: VerifyIn-59

Comment 10 by dchan@google.com, May 30 2017

Labels: VerifyIn-60
Labels: VerifyIn-61

Comment 12 by dchan@chromium.org, Oct 14 2017

Status: Archived (was: Fixed)

Sign in to add a comment