diff options
| author | Lv Zheng <lv.zheng@intel.com> | 2015-12-24 07:16:26 +0100 |
|---|---|---|
| committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2015-12-28 01:29:40 +0100 |
| commit | 73af2d590ab1b7bc7c3fc3e6933d5dcd379c9c2e (patch) | |
| tree | f86662d59b468860e8a5c57dba13b614d092e6e8 /drivers/acpi/acpi_dbg.c | |
| parent | ACPI / debugger: Add module support for ACPI debugger (diff) | |
| download | linux-73af2d590ab1b7bc7c3fc3e6933d5dcd379c9c2e.tar.xz linux-73af2d590ab1b7bc7c3fc3e6933d5dcd379c9c2e.zip | |
ACPI / debugger: Fix an issue a flag is modified without locking
There is one line of code, executed out of locking due to rebase mistakes.
This patch fixes this issue.
Signed-off-by: Lv Zheng <lv.zheng@intel.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Diffstat (limited to 'drivers/acpi/acpi_dbg.c')
| -rw-r--r-- | drivers/acpi/acpi_dbg.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/acpi/acpi_dbg.c b/drivers/acpi/acpi_dbg.c index 381beb299bf7..fa18bd0162f4 100644 --- a/drivers/acpi/acpi_dbg.c +++ b/drivers/acpi/acpi_dbg.c @@ -516,10 +516,10 @@ static int acpi_aml_open(struct inode *inode, struct file *file) ret = -EINVAL; goto err_lock; } - acpi_aml_io.flags |= ACPI_AML_OPENED; pr_debug("Debugger thread initialized.\n"); mutex_lock(&acpi_aml_io.lock); + acpi_aml_io.flags |= ACPI_AML_OPENED; acpi_aml_io.out_crc.head = acpi_aml_io.out_crc.tail = 0; acpi_aml_io.in_crc.head = acpi_aml_io.in_crc.tail = 0; pr_debug("Debugger interface opened.\n"); |