[PATCHv4 4/4] block/sed: Embed function data into the function sequence
Jon Derrick
jonathan.derrick at intel.com
Wed Feb 22 06:35:36 PST 2017
On 02/22/2017 12:13 AM, Christoph Hellwig wrote:
>> + if (!lock_held)
>> + mutex_lock(&dev->dev_lock);
>
> No conditional locking, please. I guess I causesd this by asking you
> to remove __opal_lock_unlock, but it seems we'd either need to keep it
> in the end.
>
> Except for that the series looks fine to me.
>
> Jens: given that 1-3 are the important fixes how about you pick those
> up ASAP? They all also had my Reviewed-by for previous postings.
>
Thanks
I'll respin just 4/4 shortly with __opal_lock_unlock
More information about the Linux-nvme
mailing list