On 11/30/21 11:15 AM, Adrian Hunter wrote:
On 30/11/2021 19:51, Bart Van Assche wrote:
By using the block layer request allocation functions the block layer guarantees
that each tag is in use in only one context. When bypassing the block layer code
would have to be inserted in ufshcd_exec_dev_cmd() and ufshcd_issue_devman_upiu_cmd()
to serialize these functions.
They already are serialized, but you are essentially saying the functionality
being duplicated is just a lock. What you are proposing seems awfully
complicated just to get the functionality of a lock.
Are you perhaps referring to hba->dev_cmd.lock? I had overlooked that lock
when I wrote my previous email. I will take a closer look at the reserved
slot approach.
Thanks,
Bart.
