From mboxrd@z Thu Jan 1 00:00:00 1970 From: Allen Pais To: op-tee@lists.trustedfirmware.org Subject: Re: [PATCH v2 1/2] optee: fix tee out of memory failure seen during kexec reboot Date: Mon, 22 Mar 2021 13:29:40 +0530 Message-ID: In-Reply-To: < > MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="===============2533482730001293859==" List-Id: --===============2533482730001293859== Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit >>>> >>>> [ 0.368428] tee_bnxt_fw optee-clnt0: tee_shm_alloc failed >>>> [ 0.368461] tee_bnxt_fw: probe of optee-clnt0 failed with error -22 >>>> >>>> tee_shm_release() is not invoked on dma shm buffer. >>>> >>>> Implement .shutdown() method to handle the release of the buffers >>>> correctly. >>>> >>>> More info: >>>> https://github.com/OP-TEE/optee_os/issues/3637 >>>> >>>> Signed-off-by: Allen Pais >>>> --- >>>> drivers/tee/optee/core.c | 20 ++++++++++++++++++++ >>>> 1 file changed, 20 insertions(+) >>> >>> This looks good to me. Do you have a practical way of testing this on >>> QEMU for instance? >>> >> >> Jens, >> >> I could not reproduce nor create a setup using QEMU, I could only >> do it on a real h/w. >> >> I have extensively tested the fix and I don't see any issues. > > I did a few test runs too, seems OK. Thank you very much. --===============2533482730001293859==--