On 10/11/2022 17:26, Bart Van Assche wrote:
n 11/10/22 03:15, John Garry wrote:
Current method means a store (in scsi_exec_args struct), a load, a
comparison, and a mov value to register whose value depends on
comparison. That's most relevant on performance being a concern.
Hi John,
Is there any code that calls scsi_execute() from a code path in which
performance matters?
Eh, I don't know. Does performance matter for the touched ioctls or
probe lun code?
Anyway, this code I mention is just the same as before. It just bugs me
when I see code which accepts a hardcoded value (dma dir, in this case)
and translates into another value, when the translated value could be
just passed.
Thanks,
John