Evolution-X-Devices/kernel_google_wahoo
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

msm: mdss: Fix scalar LUT handling

Browse Source 
Add lock to serialize access between userspace
and kernel. Fix error handling for LUT allocation.

Bug: 111127974
Test: boot, suspend/resume, video playback, camera, youtube, atest StagefrightTest
Change-Id: Ie86a8eb3e2a11852ae16d87ebc851afb6566732f
Signed-off-by: Animesh Kishore <animeshk@codeaurora.org>
This commit is contained in:
Animesh Kishore
2018-02-23 18:04:28 +05:30
committed by Raylin Hsu
parent d736a34c98
commit 9f2fd90ec4
4 changed files with 45 additions and 17 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
drivers/video/fbdev/msm/mdss.h
View File

@@ -1,4 +1,4 @@
/* Copyright (c) 2012-2017, The Linux Foundation. All rights reserved.
/* Copyright (c) 2012-2018, The Linux Foundation. All rights reserved.
*
* 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
@@ -254,6 +254,13 @@ struct mdss_scaler_block {
u32 *dest_scaler_off;
u32 *dest_scaler_lut_off;
struct mdss_mdp_qseed3_lut_tbl lut_tbl;
/*
* Lock is mainly to serialize access to LUT.
* LUT values come asynchronously from userspace
* via ioctl.
*/
struct mutex scaler_lock;
};
struct mdss_data_type;

4
drivers/video/fbdev/msm/mdss_mdp.c
View File

@@ -1,7 +1,7 @@
/*
* MDSS MDP Interface (used by framebuffer core)
*
* Copyright (c) 2007-2017, The Linux Foundation. All rights reserved.
* Copyright (c) 2007-2018, The Linux Foundation. All rights reserved.
* Copyright (C) 2007 Google Incorporated
*
* This software is licensed under the terms of the GNU General Public
@@ -2429,6 +2429,8 @@ static u32 mdss_mdp_scaler_init(struct mdss_data_type *mdata,
ret = mdss_mdp_ds_addr_setup(mdata);
}
mutex_init(&mdata->scaler_off->scaler_lock);
return ret;
}

40
drivers/video/fbdev/msm/mdss_mdp_overlay.c
View File

@@ -1,4 +1,4 @@
/* Copyright (c) 2012-2017, The Linux Foundation. All rights reserved.
/* Copyright (c) 2012-2018, The Linux Foundation. All rights reserved.
*
* 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
@@ -6577,14 +6577,18 @@ static int mdss_mdp_scaler_lut_init(struct mdss_data_type *mdata,
if (!mdata->scaler_off)
return -EFAULT;
mutex_lock(&mdata->scaler_off->scaler_lock);
qseed3_lut_tbl = &mdata->scaler_off->lut_tbl;
if ((lut_tbl->dir_lut_size !=
DIR_LUT_IDX * DIR_LUT_COEFFS * sizeof(uint32_t)) ||
(lut_tbl->cir_lut_size !=
CIR_LUT_IDX * CIR_LUT_COEFFS * sizeof(uint32_t)) ||
(lut_tbl->sep_lut_size !=
SEP_LUT_IDX * SEP_LUT_COEFFS * sizeof(uint32_t)))
return -EINVAL;
SEP_LUT_IDX * SEP_LUT_COEFFS * sizeof(uint32_t))) {
mutex_unlock(&mdata->scaler_off->scaler_lock);
return -EINVAL;
}
if (!qseed3_lut_tbl->dir_lut) {
qseed3_lut_tbl->dir_lut = devm_kzalloc(&mdata->pdev->dev,
@@ -6592,7 +6596,7 @@ static int mdss_mdp_scaler_lut_init(struct mdss_data_type *mdata,
GFP_KERNEL);
if (!qseed3_lut_tbl->dir_lut) {
ret = -ENOMEM;
goto fail;
goto err;
}
}
@@ -6602,7 +6606,7 @@ static int mdss_mdp_scaler_lut_init(struct mdss_data_type *mdata,
GFP_KERNEL);
if (!qseed3_lut_tbl->cir_lut) {
ret = -ENOMEM;
goto fail;
goto fail_free_dir_lut;
}
}
@@ -6612,44 +6616,52 @@ static int mdss_mdp_scaler_lut_init(struct mdss_data_type *mdata,
GFP_KERNEL);
if (!qseed3_lut_tbl->sep_lut) {
ret = -ENOMEM;
goto fail;
goto fail_free_cir_lut;
}
}
/* Invalidate before updating */
qseed3_lut_tbl->valid = false;
if (copy_from_user(qseed3_lut_tbl->dir_lut,
(void *)(unsigned long)lut_tbl->dir_lut,
lut_tbl->dir_lut_size)) {
ret = -EINVAL;
goto err;
goto fail_free_sep_lut;
}
if (copy_from_user(qseed3_lut_tbl->cir_lut,
(void *)(unsigned long)lut_tbl->cir_lut,
lut_tbl->cir_lut_size)) {
ret = -EINVAL;
goto err;
goto fail_free_sep_lut;
}
if (copy_from_user(qseed3_lut_tbl->sep_lut,
(void *)(unsigned long)lut_tbl->sep_lut,
lut_tbl->sep_lut_size)) {
ret = -EINVAL;
goto err;
goto fail_free_sep_lut;
}
qseed3_lut_tbl->valid = true;
mutex_unlock(&mdata->scaler_off->scaler_lock);
return ret;
fail:
kfree(qseed3_lut_tbl->dir_lut);
kfree(qseed3_lut_tbl->cir_lut);
kfree(qseed3_lut_tbl->sep_lut);
fail_free_sep_lut:
devm_kfree(&mdata->pdev->dev, qseed3_lut_tbl->sep_lut);
fail_free_cir_lut:
devm_kfree(&mdata->pdev->dev, qseed3_lut_tbl->cir_lut);
fail_free_dir_lut:
devm_kfree(&mdata->pdev->dev, qseed3_lut_tbl->dir_lut);
err:
qseed3_lut_tbl->dir_lut = NULL;
qseed3_lut_tbl->cir_lut = NULL;
qseed3_lut_tbl->sep_lut = NULL;
qseed3_lut_tbl->valid = false;
mutex_unlock(&mdata->scaler_off->scaler_lock);
return ret;
}

9
drivers/video/fbdev/msm/mdss_mdp_pp.c
View File

@@ -1,4 +1,4 @@
/* Copyright (c) 2012-2017, The Linux Foundation. All rights reserved.
/* Copyright (c) 2012-2018, The Linux Foundation. All rights reserved.
*
* 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
@@ -1605,11 +1605,16 @@ int mdss_mdp_scaler_lut_cfg(struct mdp_scale_data_v2 *scaler,
};
mdata = mdss_mdp_get_mdata();
mutex_lock(&mdata->scaler_off->scaler_lock);
lut_tbl = &mdata->scaler_off->lut_tbl;
if ((!lut_tbl) || (!lut_tbl->valid)) {
mutex_unlock(&mdata->scaler_off->scaler_lock);
pr_err("%s:Invalid QSEED3 LUT TABLE\n", __func__);
return -EINVAL;
}
if ((scaler->lut_flag & SCALER_LUT_DIR_WR) ||
(scaler->lut_flag & SCALER_LUT_Y_CIR_WR) ||
(scaler->lut_flag & SCALER_LUT_UV_CIR_WR) ||
@@ -1657,6 +1662,8 @@ int mdss_mdp_scaler_lut_cfg(struct mdp_scale_data_v2 *scaler,
}
}
mutex_unlock(&mdata->scaler_off->scaler_lock);
return 0;
}