mirrors/scst
1
0
Fork 0
mirror of https://github.com/SCST-project/scst.git synced 2026-08-19 05:36:22 +00:00
Code Issues Packages Projects Releases Wiki Activity

scst: Modify the code that decides whether or not to process in tasklet context

Browse Source 
Since the next patch will remove cpu_cmd_count, make this decision
independent of cpu_cmd_count.


git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@9320 d57e44dd-8a1f-0410-8b47-8ef2f437770f
This commit is contained in:
Bart Van Assche
2021-01-03 02:56:01 +00:00
parent cb8c3a4725
commit 7f0900b858
1 changed files with 12 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
scst/src/scst_targ.c
+12 -3
View File
@@ -59,7 +59,16 @@ static inline void scst_schedule_tasklet(struct scst_cmd *cmd)
i = &scst_percpu_infos[smp_processor_id()];
if (atomic_read(&i->cpu_cmd_count) <= scst_max_tasklet_cmd) {
/*
* Commands are removed from the list they are on before being
* processed. If both lists are empty that means that at most two
* commands are being processed and hence that processing a
* command in tasklet context is possible without making a CPU core
* spend all its time in interrupt and tasklet context and thereby
* starving threads scheduled on the same CPU core.
*/
if (list_empty_careful(&i->tasklet_cmd_list) &&
list_empty_careful(&cmd->cmd_threads->active_cmd_list)) {
spin_lock_irqsave(&i->tasklet_lock, flags);
TRACE_DBG("Adding cmd %p to tasklet %d cmd list", cmd,
smp_processor_id());
@@ -69,8 +78,8 @@ static inline void scst_schedule_tasklet(struct scst_cmd *cmd)
tasklet_schedule(&i->tasklet);
} else {
spin_lock_irqsave(&cmd->cmd_threads->cmd_list_lock, flags);
TRACE_DBG("Too many tasklet commands (%d), adding cmd %p to "
"active cmd list", atomic_read(&i->cpu_cmd_count), cmd);
TRACE_DBG("Too many tasklet commands, adding cmd %p to active cmd list",
cmd);
list_add_tail(&cmd->cmd_list_entry,
&cmd->cmd_threads->active_cmd_list);
wake_up(&cmd->cmd_threads->cmd_list_waitQ);