evie/android_kernel_oneplus_msm8998
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Merge "coresight-remote-etm: Adds missing lock to avoid race condition"

Browse source
This commit is contained in:
Linux Build Service Account 2017-05-25 15:51:40 -07:00 • committed by Gerrit - the friendly Code Review server
commit c1a961ce2f
1 changed files with 7 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
drivers/hwtracing/coresight/coresight-remote-etm.c
View file

@ -1,4 +1,4 @@
/* Copyright (c) 2013-2016, The Linux Foundation. All rights reserved. /* Copyright (c) 2013-2017, The Linux Foundation. All rights reserved.
* *
* This program is free software; you can redistribute it and/or modify * This program is free software; you can redistribute it and/or modify
* it under the terms of the GNU General Public License version 2 and * it under the terms of the GNU General Public License version 2 and
@ -186,10 +186,12 @@ static void remote_etm_rcv_msg(struct work_struct *work)
struct remote_etm_drvdata *drvdata = container_of(work, struct remote_etm_drvdata *drvdata = container_of(work,
struct remote_etm_drvdata, struct remote_etm_drvdata,
work_rcv_msg); work_rcv_msg);
mutex_lock(&drvdata->mutex);
if (qmi_recv_msg(drvdata->handle) < 0) if (qmi_recv_msg(drvdata->handle) < 0)
dev_err(drvdata->dev, "%s: Error receiving QMI message\n", dev_err(drvdata->dev, "%s: Error receiving QMI message\n",
__func__); __func__);
mutex_unlock(&drvdata->mutex);
} }
static void remote_etm_notify(struct qmi_handle *handle, static void remote_etm_notify(struct qmi_handle *handle,
@ -227,6 +229,7 @@ static void remote_etm_svc_arrive(struct work_struct *work)
return; return;
} }
mutex_lock(&drvdata->mutex);
if (qmi_connect_to_service(drvdata->handle, CORESIGHT_QMI_SVC_ID, if (qmi_connect_to_service(drvdata->handle, CORESIGHT_QMI_SVC_ID,
CORESIGHT_QMI_VERSION, CORESIGHT_QMI_VERSION,
drvdata->inst_id) < 0) { drvdata->inst_id) < 0) {
@ -236,7 +239,6 @@ static void remote_etm_svc_arrive(struct work_struct *work)
drvdata->handle = NULL; drvdata->handle = NULL;
} }
mutex_lock(&drvdata->mutex);
if (drvdata->inst_id < sizeof(int)*BITS_PER_BYTE if (drvdata->inst_id < sizeof(int)*BITS_PER_BYTE
&& (boot_enable & BIT(drvdata->inst_id))) { && (boot_enable & BIT(drvdata->inst_id))) {
if (!drvdata->enable) if (!drvdata->enable)
@ -251,9 +253,10 @@ static void remote_etm_svc_exit(struct work_struct *work)
struct remote_etm_drvdata *drvdata = container_of(work, struct remote_etm_drvdata *drvdata = container_of(work,
struct remote_etm_drvdata, struct remote_etm_drvdata,
work_svc_exit); work_svc_exit);
mutex_lock(&drvdata->mutex);
qmi_handle_destroy(drvdata->handle); qmi_handle_destroy(drvdata->handle);
drvdata->handle = NULL; drvdata->handle = NULL;
mutex_unlock(&drvdata->mutex);
} }
static int remote_etm_svc_event_notify(struct notifier_block *this, static int remote_etm_svc_event_notify(struct notifier_block *this,