On Tue, Oct 19, 2021 at 12:54:54PM -0700, David Rientjes wrote:
> > -                                              1 << get_order(size));
> > -                   if (err)
> > -                           goto out_free_pages;
> > -           }
> > +           err = dma_set_decrypted(dev, ret, size);
> 
> Should be
> 
>       if (dma_set_decrypted(dev, ret, size))
>               goto out_free_pages;
> 
> ?  Otherwise we ignore the value?

Yes.  Except that we also need to undo the remap as well.
_______________________________________________
iommu mailing list
iommu@lists.linux-foundation.org
https://lists.linuxfoundation.org/mailman/listinfo/iommu

Reply via email to