Re: [PATCH v2] scsi: sr: get rid of sr global mutex

[Simon Arlott]

On 24/02/2020 21:20, Merlijn B.W. Wajer wrote:
> Just wanted to check if you planned to apply this v2 (you tried to apply
> v1 but it didn't compile, so I rebased it onto 5.7/scsi-queue as you
> requested). Please let me know if there's anything you'd like to see
> changed.

There's a missing call to mutex_destroy(&cd->lock) in sr_probe() if
there are any failures after calling mutex_init(&cd->lock).

> On 18/02/2020 20:21, Merlijn B.W. Wajer wrote:
>> Perhaps I or someone else can work on removing the usage of the locks,
>> but as it stands I think this addresses the performance issue present in
>> the current kernel, and removing locks and the associated testing
>> required with that is something I am not entirely comfortable doing.

I have a patch to move the locks into the cdrom driver and I've got
multiple PATA drives to test it with.

-- 
Simon Arlott