On 2/9/21 11:01 AM, Bodo Stroesser wrote:
> On 09.02.21 13:38, Mike Christie wrote:
>>
>> +void target_queued_compl_work(struct work_struct *work)
>> +{
>> + struct se_cmd_queue *cq = container_of(work, struct se_cmd_queue,
>> + work);
>> + struct se_cmd *se_cmd, *next_cmd;
>> + struct llist_node *cmd_list;
>> +
>> + cmd_list = llist_del_all(&cq->cmd_list);
>
> Probably nit-picking: I'd like to reverse the list before processing like you did during submission.
>
Yeah, I flip flopped a lot on both. I'll just do it. I didn't notice
any perf differences.
