Message ID | 20231201102954.47582-1-d-gole@ti.com |
---|---|
Headers | show |
Series | docs: AM62x: Remove SW_PRNG Flag for OPTEE | expand |
On 15:59-20231201, Dhruva Gole wrote: > The CFG_WITH_SOFTWARE_PRNG option was needed once when there were some > issues with system crashes/ hangs after a suspend-resume cycle. However > this seems to no longer be the case with newer firmwares, and this > config is not needed for basic boot support of the SoC either, hence > remove it from the docs. > > Cc: Kamlesh <kamlesh@ti.com> > Cc: Vibhore Vardhan <vibhore@ti.com> > Cc: Wadim Egorov <w.egorov@phytec.de> > > Dhruva Gole (2): > doc: board: beagle: am62x_beagleplay: Delete SW_PRNG flag for OPTEE > doc: board: ti: am62x_sk: Remove SW_PRNG Flag for OPTEE > > doc/board/beagle/am62x_beagleplay.rst | 1 - > doc/board/ti/am62x_sk.rst | 1 - > 2 files changed, 2 deletions(-) > NAK to the series. RNG seed is needed for KASLR. either enable h/w RNG or at least enable S/W RNG in OPTEE.
Hi Nishanth, On 05/12/23 01:00, Nishanth Menon wrote: > On 15:59-20231201, Dhruva Gole wrote: >> The CFG_WITH_SOFTWARE_PRNG option was needed once when there were some >> issues with system crashes/ hangs after a suspend-resume cycle. However >> this seems to no longer be the case with newer firmwares, and this >> config is not needed for basic boot support of the SoC either, hence >> remove it from the docs. >> >> Cc: Kamlesh <kamlesh@ti.com> >> Cc: Vibhore Vardhan <vibhore@ti.com> >> Cc: Wadim Egorov <w.egorov@phytec.de >> >> Dhruva Gole (2): >> doc: board: beagle: am62x_beagleplay: Delete SW_PRNG flag for OPTEE >> doc: board: ti: am62x_sk: Remove SW_PRNG Flag for OPTEE >> >> doc/board/beagle/am62x_beagleplay.rst | 1 - >> doc/board/ti/am62x_sk.rst | 1 - >> 2 files changed, 2 deletions(-) >> > > NAK to the series. RNG seed is needed for KASLR. either enable h/w RNG @Wadim: Please take note and ignore my previous suggestion to you to remove this line, keep this line as is. > or at least enable S/W RNG in OPTEE. > Agreed, I will send a patch to use HWRNG later on then. For now let's just keep the PRNG enabled in these docs.