linux-acpi.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Lv Zheng <lv.zheng@intel.com>
To: "Rafael J. Wysocki" <rafael.j.wysocki@intel.com>,
	Len Brown <len.brown@intel.com>
Cc: Lv Zheng <lv.zheng@intel.com>, Lv Zheng <zetalog@gmail.com>,
	linux-kernel@vger.kernel.org, linux-acpi@vger.kernel.org
Subject: [UPDATE PATCH v2 0/6] ACPI/EC: Critical race fixes.
Date: Mon,  7 Jul 2014 11:09:49 +0800	[thread overview]
Message-ID: <cover.1404700569.git.lv.zheng@intel.com> (raw)
In-Reply-To: <cover.1402792240.git.lv.zheng@intel.com>

History:
v1:
 The collection of bug-fixes from kernel bugzilla bug 70891.
v2:
 1. Improves the patch descriptions to make the root cause of the 70891 bug
    more correct.
 2. Improves a race fix to make it more robust.
v2 UPDATE:
 Added "Reported-by" and "Tested-by" from kernel bugzilla bug 59911/63931
 as a result of EC bug triage. Only patch descriptions are updated.
 Removed the GPE fix, it is included by another GPE API fix series.

This patchset includes the following improvements of the ACPI/EC driver:
1. Fixes 2 race conditions in the current EC driver.
2. Cleans up the first command byte write as one of the race condition fix
   requires additional cleanup for the first command byte write. And this
   also helps us to do improvements in the BURST mode in the future.
3. Improves debugging message, register details can be enabled by defining
   DEBUG. And bug fixes and reports are based on this improvement.
The EC driver is upgraded to v2.2 to indicate the difference from the
previous buggy one, so that the future bug reports can be correctly
categorized.

All patches are tested to be valid fixes for the following bugs:
https://bugzilla.kernel.org/show_bug.cgi?id=59911
https://bugzilla.kernel.org/show_bug.cgi?id=63931
https://bugzilla.kernel.org/show_bug.cgi?id=70891

Lv Zheng (6):
  ACPI/EC: Fix an issue that advance_transaction() processes stale
    hardware status.
  ACPI/EC: Add asynchronous command byte write support.
  ACPI/EC: Remove duplicated ec_wait_ibf0() waiter.
  ACPI/EC: Fix a race condition in ec_transaction_completed().
  ACPI/EC: Update revision due to full asynchrnous command support.
  ACPI/EC: Add detailed fields debugging support of EC_SC(R).

 drivers/acpi/ec.c |  159 +++++++++++++++++++++++++++--------------------------
 1 file changed, 81 insertions(+), 78 deletions(-)

-- 
1.7.10


  parent reply	other threads:[~2014-07-07  3:09 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <cover.1402543079.git.lv.zheng@intel.com>
2014-06-15  0:40 ` [PATCH v2 0/7] ACPI/EC: Critical bug fixes related to EC and event handling Lv Zheng
2014-06-15  0:41   ` [PATCH v2 1/7] ACPI/EC: Fix an issue that advance_transaction() processes stale hardware status Lv Zheng
2014-06-15  0:41   ` [PATCH v2 2/7] ACPI/EC: Add asynchronous command byte write support Lv Zheng
2014-06-15  0:41   ` [PATCH v2 3/7] ACPI/EC: Remove duplicated ec_wait_ibf0() waiter Lv Zheng
2014-06-15  0:42   ` [PATCH v2 4/7] ACPI/EC: Fix a race condition in ec_transaction_completed() Lv Zheng
2014-06-15  0:42   ` [PATCH v2 5/7] ACPI/EC: Update revision due to full asynchrnous command support Lv Zheng
2014-06-15  0:42   ` [PATCH v2 6/7] ACPICA: Events: Fix edge-triggered GPE by disabling before acknowledging it Lv Zheng
2014-06-15  0:42   ` [PATCH v2 7/7] ACPI/EC: Add detailed fields debugging support of EC_SC(R) Lv Zheng
2014-07-07  3:09   ` Lv Zheng [this message]
2014-07-07  3:09     ` [UPDATE PATCH v2 1/6] ACPI/EC: Fix an issue that advance_transaction() processes stale hardware status Lv Zheng
2014-07-07  3:10     ` [UPDATE PATCH v2 2/6] ACPI/EC: Add asynchronous command byte write support Lv Zheng
2014-07-07  3:10     ` [UPDATE PATCH v2 3/6] ACPI/EC: Remove duplicated ec_wait_ibf0() waiter Lv Zheng
2014-07-07  3:10     ` [UPDATE PATCH v2 4/6] ACPI/EC: Fix a race condition in ec_transaction_completed() Lv Zheng
2014-07-07  3:10     ` [UPDATE PATCH v2 5/6] ACPI/EC: Update revision due to full asynchrnous command support Lv Zheng
2014-07-07  3:10     ` [UPDATE PATCH v2 6/6] ACPI/EC: Add detailed fields debugging support of EC_SC(R) Lv Zheng
2014-07-07 11:18     ` [UPDATE PATCH v2 0/6] ACPI/EC: Critical race fixes Rafael J. Wysocki
2014-07-08  0:00       ` Zheng, Lv

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=cover.1404700569.git.lv.zheng@intel.com \
    --to=lv.zheng@intel.com \
    --cc=len.brown@intel.com \
    --cc=linux-acpi@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=rafael.j.wysocki@intel.com \
    --cc=zetalog@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).