That's not surprising, none of our internal workflows involve flashing the built userdata.img, as that would effectively wipe all the data on the device. We always use adb push (that's what the atest tool does) or adb sync to run tests and benchmarks.
I think a patch to update fs_config would be fine though. On Tue, Oct 15, 2019 at 8:27 AM Paul Elliott <[email protected]> wrote: > Hi Colin, > > We're just building the data image and loading that, not using adb sync, > which potentially explains the differences. If I look at the data image I > can definitely see that the permissions are wrong. > > Paul. > > On Monday, October 14, 2019 at 11:22:28 PM UTC+1, Colin Cross wrote: >> >> How are you getting the benchmarks on to the device? In my testing (`m >> libjavacore-benchmarks`, `adb sync data` they are marked executable. >> >> On Fri, Oct 11, 2019 at 10:49 AM Paul Elliott <[email protected]> wrote: >> >>> Hi Colin, >>> >>> The curious bit is that even as root I am having to change the tests >>> inside benchmarktest(64) to be executable because they too have no >>> permissions, which is why I was curious about this as it clearly breaks >>> automation for us, and therefore likely for anyone else. We could >>> potentially submit a patch for this to add the directory to fs_config.cpp, >>> however given the lack of mention by anyone else, was unsure if we were >>> missing something, basically. >>> >>> Paul. >>> >>> On Thursday, October 10, 2019 at 9:12:11 PM UTC+1, Colin Cross wrote: >>>> >>>> Likely an oversight. It has been this way for benchmarks defined in >>>> Make since 2016. Maybe the benchmark test runners are running as root? >>>> >>>> On Thu, Oct 10, 2019 at 10:22 AM Paul Elliott <[email protected]> wrote: >>>> >>>>> Hi all, >>>>> >>>>> Prior to android 9, all benchmarks and test suites were generally >>>>> built into /data/nativetest(64), however recently in Android 9, some (but >>>>> not all) tests have been ending up getting built into >>>>> /data/benchmarktest(64), due to a change aligning soong with make - >>>>> https://android-review.googlesource.com/c/platform/build/soong/+/479681 >>>>> >>>>> This would be fine, however due to its lack of inclusion >>>>> in system/core/libcutils/fs_config.cpp, this folder is created without any >>>>> permissions whatsoever. >>>>> >>>>> I have checked this is still the case in Android 10, and indeed >>>>> master. Is this an oversight, or am I missing something? >>>>> >>>>> Thanks in advance, >>>>> >>>>> Paul. >>>>> >>>>> -- >>>>> -- >>>>> You received this message because you are subscribed to the "Android >>>>> Building" mailing list. >>>>> To post to this group, send email to [email protected] >>>>> To unsubscribe from this group, send email to >>>>> [email protected] >>>>> For more options, visit this group at >>>>> http://groups.google.com/group/android-building?hl=en >>>>> >>>>> --- >>>>> You received this message because you are subscribed to the Google >>>>> Groups "Android Building" group. >>>>> To unsubscribe from this group and stop receiving emails from it, send >>>>> an email to [email protected]. >>>>> To view this discussion on the web visit >>>>> https://groups.google.com/d/msgid/android-building/989e4b75-4366-4b18-b534-dd12f2c022a4%40googlegroups.com >>>>> <https://groups.google.com/d/msgid/android-building/989e4b75-4366-4b18-b534-dd12f2c022a4%40googlegroups.com?utm_medium=email&utm_source=footer> >>>>> . >>>>> >>>> -- >>> -- >>> You received this message because you are subscribed to the "Android >>> Building" mailing list. >>> To post to this group, send email to [email protected] >>> To unsubscribe from this group, send email to >>> [email protected] >>> For more options, visit this group at >>> http://groups.google.com/group/android-building?hl=en >>> >>> --- >>> You received this message because you are subscribed to the Google >>> Groups "Android Building" group. >>> To unsubscribe from this group and stop receiving emails from it, send >>> an email to [email protected]. >>> To view this discussion on the web visit >>> https://groups.google.com/d/msgid/android-building/fce01460-04f9-49a1-b73b-f4c89c8a4ab4%40googlegroups.com >>> <https://groups.google.com/d/msgid/android-building/fce01460-04f9-49a1-b73b-f4c89c8a4ab4%40googlegroups.com?utm_medium=email&utm_source=footer> >>> . >>> >> -- > -- > You received this message because you are subscribed to the "Android > Building" mailing list. > To post to this group, send email to [email protected] > To unsubscribe from this group, send email to > [email protected] > For more options, visit this group at > http://groups.google.com/group/android-building?hl=en > > --- > You received this message because you are subscribed to the Google Groups > "Android Building" group. > To unsubscribe from this group and stop receiving emails from it, send an > email to [email protected]. > To view this discussion on the web visit > https://groups.google.com/d/msgid/android-building/4c6adeb4-5d1b-44a3-af50-aa0e2c75ccbe%40googlegroups.com > <https://groups.google.com/d/msgid/android-building/4c6adeb4-5d1b-44a3-af50-aa0e2c75ccbe%40googlegroups.com?utm_medium=email&utm_source=footer> > . > -- -- You received this message because you are subscribed to the "Android Building" mailing list. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [email protected] For more options, visit this group at http://groups.google.com/group/android-building?hl=en --- You received this message because you are subscribed to the Google Groups "Android Building" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To view this discussion on the web visit https://groups.google.com/d/msgid/android-building/CAMbhsRTQyGmG%2BD5%2B2%3D-oX46EPuJvcvLMK0UpFL3AxWspnUtnpw%40mail.gmail.com.
