Messages in this thread | | | From | Simon Arlott <> | Subject | Re: [PATCH v2] scsi: sr: get rid of sr global mutex | Date | Fri, 6 Mar 2020 22:43:02 +0000 |
| |
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
| |