On Thu, Aug 31 2017, Scott Bauer wrote:
> @@ -2345,6 +2371,11 @@ bool opal_unlock_from_suspend(struct opal_dev *dev)
>                                suspend->unlk.session.sum);
>                       was_failure = true;
>               }
> +             if (dev->mbr_enabled) {
> +                     ret = __opal_set_mbr_done(dev, 
> &suspend->unlk.session.opal_key);
> +                     if (ret)
> +                             pr_debug("Failed to set MBR Done in S3 
> resume\n");
> +             }

Should ret != 0 set was_failure = true here?

-- 
Jens Axboe

Reply via email to