A-Team_Digital_Solutions/android_kernel_motorola_sm6375
Watch
2
0
Fork
You've already forked android_kernel_motorola_sm6375
 0
mirror of https://github.com/BobTheBlinker/android_kernel_motorola_sm6375.git synced 2026-10-09 21:59:12 -04:00
Code Issues Projects Releases Packages Wiki Activity

disp: msm: dsi: Fix potential data race in ctrl isr

Browse source 
Use atomic operations for shared variables to ensure
safe concurrent access from both ISR and task context.

Change-Id: I02d4bde88692d0be7027b158648a3ab8a5704b2d
Signed-off-by: Gopi Botlagunta <botlagun@qti.qualcomm.com>
This commit is contained in:
Gopi Botlagunta 2026-01-07 15:37:06 +05:30
commit a74bd43d86
2 changed files with 2 additions and 11 deletions
Download patch file Download diff file Expand all files Collapse all files

10
msm/dsi/dsi_ctrl.c
View file

@ -2889,12 +2889,8 @@ static irqreturn_t dsi_ctrl_isr(int irq, void *ptr)
if (status & DSI_CMD_MODE_DMA_DONE) {
if (dsi_ctrl->enable_cmd_dma_stats) {
if (dsi_ctrl->hw.ops.log_line_count[dsi_ctrl->disp_op])
reg =
dsi_ctrl->hw.ops.log_line_count[dsi_ctrl->disp_op](&dsi_ctrl->hw,
dsi_ctrl->cmd_mode);
else
reg = 0;
u32 reg = dsi_ctrl->hw.ops.log_line_count(&dsi_ctrl->hw,
dsi_ctrl->cmd_mode);
atomic_set(&dsi_ctrl->cmd_success_line, (reg & 0xFFFF));
atomic_set(&dsi_ctrl->cmd_success_frame, ((reg >> 16) & 0xFFFF));
SDE_EVT32(dsi_ctrl->cell_index, SDE_EVTLOG_FUNC_CASE1,
@ -2902,7 +2898,6 @@ static irqreturn_t dsi_ctrl_isr(int irq, void *ptr)
dsi_ctrl->cmd_success_frame);
}
atomic64_set(&dsi_ctrl->cmd_success_ts, ktime_get());
atomic_set(&dsi_ctrl->dma_irq_trig, 1);
dsi_ctrl_disable_status_interrupt(dsi_ctrl,
DSI_SINT_CMD_MODE_DMA_DONE);
@ -3487,7 +3482,6 @@ int dsi_ctrl_cmd_transfer(struct dsi_ctrl *dsi_ctrl,
rc);
}
cmd->ts = atomic64_read(&dsi_ctrl->cmd_success_ts);
dsi_ctrl_update_state(dsi_ctrl, DSI_CTRL_OP_CMD_TX, 0x0);
error:

3
msm/dsi/dsi_ctrl.h
View file

@ -312,9 +312,6 @@ struct dsi_ctrl {
u32 cmd_trigger_frame;
atomic_t cmd_success_line;
atomic_t cmd_success_frame;
u32 cmd_engine_refcount;
u32 pending_cmd_flags;
atomic64_t cmd_success_ts;
};
/**