[PATCH v1] ufs: core: Fix possible NULL pointer dereference in ufshcd_add_command_trace()
Bart Van Assche
bvanassche at acm.org
Tue Feb 24 08:48:58 PST 2026
On 2/23/26 9:29 PM, Peter Wang (王信友) wrote:
> Therefore, there is no need to assign hwq_id separately.
Hi Peter,
That's not what I proposed. When ignoring the NULL test, this is what
the current ufshcd_add_command_trace() implementation does:
hwq_id = hba->uhq[READ_ONCE(req->mq_hctx)->queue_num].id;
That's more complicated than necessary. This should be sufficient
(again ignoring the NULL test):
hwq_id = READ_ONCE(req->mq_hctx)->queue_num;
Anyway, since the proposed change probably only results in a small
performance improvement, let's proceed with the current patch.
Bart.
More information about the Linux-mediatek
mailing list