summaryrefslogtreecommitdiffstats
path: root/drivers/acpi/acpi_dbg.c
diff options
context:
space:
mode:
authorLv Zheng <lv.zheng@intel.com>2015-12-24 14:16:26 +0800
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>2015-12-28 01:29:40 +0100
commit73af2d590ab1b7bc7c3fc3e6933d5dcd379c9c2e (patch)
treef86662d59b468860e8a5c57dba13b614d092e6e8 /drivers/acpi/acpi_dbg.c
parent836d0830188a97d5c73e8eb514f346a857c086b9 (diff)
downloadlinux-73af2d590ab1b7bc7c3fc3e6933d5dcd379c9c2e.tar.gz
linux-73af2d590ab1b7bc7c3fc3e6933d5dcd379c9c2e.tar.bz2
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.c2
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");