| commit | 6e41879e12f1697a02474be2def14ba81b79e36a | [log] [tgz] |
|---|---|---|
| author | Jim Pollock <jmpollock@chromium.org> | Sun May 02 23:36:16 2021 +0000 |
| committer | Commit Bot <commit-bot@chromium.org> | Mon May 10 10:13:32 2021 +0000 |
| tree | 46b1354e5c5c4f019329e1b937c9c0f23adbf78e | |
| parent | 08ef5a3d3e1aecc71cd4fc09e69c336c3a9ab3d6 [diff] |
Reland "nn: use cryptohome daemon storage location for cache files" This reverts commit 2f8f5dd0f6ef971301ffbcf7528fc63e293ff0a0. Reason for revert: Fix found, applied in next CL Original change's description: > Revert "nn: use cryptohome daemon storage location for cache files" > > This reverts commit a69a9e160ff4db6cd8971d30d023aae1379a5c9a. > > Reason for revert: Has broken relesae builds: https://logs.chromium.org/logs/chromeos/buildbucket/cr-buildbucket.appspot.com/8848535058247719152/+/steps/BuildPackages/0/stdout > > Original change's description: > > nn: use cryptohome daemon storage location for cache files > > > > This CL forces any request that sets up compilation caching to store > > the files in the cryptohome daemon storage location of the ml_service > > process. This is done due to the docs indicating that cache files should > > be stored in a secure location. See: > > https://source.android.com/devices/neural-networks/compilation-caching#security > > > > We can't do this in configuration since the daemon storage is per-user > > and the username hash needs to be queried at runtime. > > > > BUG=b:178762478 > > TEST=FEATURES=test emerge-volteer aosp-frameworks-ml-nn > > > > Change-Id: I09d5d4be01a3435cedda4e822a4f423f8723e5ce > > Reviewed-on: https://chromium-review.googlesource.com/c/aosp/platform/frameworks/ml/+/2817759 > > Tested-by: Jim Pollock <jmpollock@chromium.org> > > Auto-Submit: Jim Pollock <jmpollock@chromium.org> > > Commit-Queue: Jim Pollock <jmpollock@chromium.org> > > Reviewed-by: Stuart Langley <slangley@chromium.org> > > Reviewed-by: Michael Pishchagin <mblsha@google.com> > > Bug: b:178762478 > Change-Id: Iac9913c54332c5fb5f1205d7b6ed9c58825b866c > Reviewed-on: https://chromium-review.googlesource.com/c/aosp/platform/frameworks/ml/+/2859271 > Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> > Reviewed-by: David Riley <davidriley@chromium.org> > Commit-Queue: Jim Pollock <jmpollock@chromium.org> > Auto-Submit: Jim Pollock <jmpollock@chromium.org> Bug: b:178762478 Change-Id: Ieb0ad679b1fb5b401866101b80ca668d5d2180b1 Reviewed-on: https://chromium-review.googlesource.com/c/aosp/platform/frameworks/ml/+/2865825 Tested-by: Jim Pollock <jmpollock@chromium.org> Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Commit-Queue: Jim Pollock <jmpollock@chromium.org>