Merge "Trim root path from output" am: b7c162435b am: 89b12b6f0b

Change-Id: Ic8b6bf9400f312efd3b4df56ebd36b37c2887d89

Merge "Trim root path from output" am: b7c162435b

Change-Id: If7db732fa5a1e267c777c0610608dc3492a77749

Merge "Define vndk_product namespace for product apps" am: 6eb18354c1 am: 7df20a75d3

Change-Id: I98e7a8604acbc0d324d3ed831f92b8b84788b944

Merge "Trim root path from output"

Merge "Define vndk_product namespace for product apps" am: 6eb18354c1

Change-Id: Ifde5ffcae2bdad31e068f2fabfbe9a1838b14a21

Merge "Define vndk_product namespace for product apps"

Merge "Make /linkerconfig/<apexname> directories accessible to others" am: b52f719faf am: bc9db82bf5

Change-Id: Id37a319f81c2b66968fd3d818d325aaab7c0956e

Merge "Make /linkerconfig/<apexname> directories accessible to others" am: b52f719faf

Change-Id: Id755473867942a98d93d1091a3e7556fccbd43bd

Merge "Make /linkerconfig/<apexname> directories accessible to others"

Trim root path from output

When linkerconfig runs on host with --root argument, it prints root as
well like following:

$ linkerconfig --vndk R --root /tmp/test_root
...
namespace.foo.search_paths = /tmp/test_root/product/${LIB}
..

This change trims root from path variables and apex paths so the output
look more like real output.

Bug: n/a
Test: build/boot
      run linkerconfig on host and compare outputs

Change-Id: Icc91f297047f059684a42d825048e50e26873520

Make /linkerconfig/<apexname> directories accessible to others

When linkerconfig is exec'ed by init, umask is by default set to 0x0077.
As a result, even though we create the directories with mkdir(...,
0755), they are marked as inaccessible for group and others.

Fixing the issue by explicitly setting umask to 0x0022.

Bug: 144914078
Test: build a device with flattened APEX. Check the mode bits of the
directories of under /linkerconfig.

$ ls -al /linkerconfig
total 96
drwxr-xr-x  9 root root   200 2020-02-21 13:47 .
drwxr-xr-x 22 root root  4096 2020-02-21 13:54 ..
drwx--xr-x  2 root root    60 2020-02-21 13:47 com.android.adbd
drwx--xr-x  2 root root    60 2020-02-21 13:47 com.android.art
drwx--xr-x  2 root root    60 2020-02-21 13:47 com.android.conscrypt
drwx--xr-x  2 root root    60 2020-02-21 13:47 com.android.media.swcodec
drwx--xr-x  2 root root    60 2020-02-21 13:47 com.android.os.statsd
drwx--xr-x  2 root root    60 2020-02-21 13:47 com.android.runtime
drwx--xr-x  2 root root    60 2020-02-21 13:47 com.android.sdkext
-rw-r--r--  1 root root 91651 2020-02-21 13:47 ld.config.txt

Change-Id: Id2f7f8a363ba239943942ba6e3bc984cd8f630b0

Define vndk_product namespace for product apps

Unbundled apps may use vndk-sp libs from the system section. However,
product apps must use the vndk-sp libs for the product vndk version
that can be different from the vendor vndk version. So we must define
a vndk namespace for product partition.

For this purpose, BuildVndkNamespace() function requires an
additional parameter to indicate which partition is using the
namespace.
Using this function, the system section has an additional namespace
called "vndk_product" which is used only for unbundled product apps.

Test: atest linkerconfig_modules_unittest
Bug: 149063221
Change-Id: I3658efdc75e5c9a0ac3b92b9afc68aff2d4cb300

Merge "Rename APEX Namespace" am: 52120b489b am: 6e1b2ec7ac

Change-Id: Ie27efc805d25e509576981f7ae03cc70cb555153

Merge "Rename APEX Namespace" am: 52120b489b

Change-Id: I1ad4220320ce62274f553afa7e81b322e82e1f51

Merge "Rename APEX Namespace"

Merge "Fix full content test" am: 9fa8d06ecc am: 9873f73464

Change-Id: I12c6f0f14516ab2e0306d3bdf7a4779ecbd78a54

Merge "Fix full content test" am: 9fa8d06ecc

Change-Id: I41c27ed81cfb803954d773a07a4d0747bea4b33f

Merge "Fix full content test"

Fix full content test

Full content test was not working properly because the way of checking
if VNDK is enabled has been changed. This fix generates test context to
mock it available for VNDK.

Test: atest passed
Change-Id: I46064d1d5505dc5c6af438e69faae166dee9dc58

Rename APEX Namespace

Current APEX Namespace is named with APEX name itself, which also uses
.(dot) so linker configuration can keep the syntax safe. Update name of
namespace from APEX by replacing '_' with '.' so keep configuration safe
for future syntax update.

Bug: 148826508
Test: m -j passed
Test: boot succeeded from cuttlefish and walleye
Change-Id: I9185c909678e2e39d539622d51d8f378b819b466

Add com.android.os.statsd to legacy linkerconfig am: 2b465ce881 am: 6f3a53b4ec

Change-Id: Id4711bdabc9a4eb5f123a83d53afb029fc948b4b

Add com.android.os.statsd to legacy linkerconfig am: 2b465ce881

Change-Id: Ib2a29f358385d1b67a27f58e900dcd76a2efa20f

Add com.android.os.statsd to legacy linkerconfig

Similar to aosp/1237667, but for legacy stuff.

Bug: None
Test: Legacy stuff boots.
Change-Id: Iad17f5529ea6bb520c0ee5a4071a82ffd7c2cc27

Merge "Update linkerconfig for libs in statsd" am: 5504faa670 am: 1730a33997

Change-Id: I01042fe7ee802e5c224ff00c001219b87f32e419

Merge "Update linkerconfig for libs in statsd" am: 5504faa670

Change-Id: I969a8e8828d5e9273c863688c48d6d2ba9297d95

Merge "Update linkerconfig for libs in statsd"

Merge "Add com.android.os.statsd to linkerconfig" am: 91fccce607 am: 365c145906

Change-Id: I245cf83033159721babcc7c176d3290e68d51aef

Merge "Add com.android.os.statsd to linkerconfig" am: 91fccce607

Change-Id: Ia2aa4670b96557434b1c1c8675409e17515f00a3

Merge "Add com.android.os.statsd to linkerconfig"

Add com.android.os.statsd to linkerconfig

Adds com.android.os.statsd to System BuildSection. I believe this is to
get the namespace the proper visibility for jni.

Test: build/boots
Change-Id: Id516e79e12fde608888587ec15ab0f3358f4918d
Merged-In: Id516e79e12fde608888587ec15ab0f3358f4918d
(cherry picked from commit 91454166a50d86d2367c8ec5b4686087ecc13184)

Merge "Remove extra using statements" am: 28133def11 am: de106d8a2d

Change-Id: Ida676cfd0249a0134df4f8afa79e7e9bfd42721e

Merge "Remove extra using statements" am: 28133def11

Change-Id: Iab0964273993949a3eb3c0c331eb58bc75e3c5e1

Merge "Remove extra using statements"

Update linkerconfig for libs in statsd

Bug: 148605160
Test: m -j
Change-Id: Idb6308e8f518536e97106c445b1945945040a4c3

Merge "Fix Camera bug" am: 5c811ebf1c am: 4291671637

Change-Id: I7a4bd4b68772d59e1085799bdadfe8ddfd1562f8

Merge "Fix Camera bug" am: 5c811ebf1c

Change-Id: I78f49c50833251698c1fc5351e3a5ee66819e872

Merge "Fix Camera bug"

Remove extra using statements

android::base::Errorf and android::base::ErrnoErrorf are now #define's
instead of functions, so remove their using statements.

Test: build
Change-Id: I7333b916ee87668fb9fd15b9293de5da41c43409

Fix Camera bug

Bug: 149538115
Test: Flash and take a picture, verified no crash
Change-Id: Idd455f8803cf6f1b1f4c95d854a6077e33bae5e1

Allow libincident.so to be linked from apex am: 4e15bf53f0 am: b741643a5c

Change-Id: I965b1fba687d5b8a82a11a5ce6f52ec9eea185e1

Allow libincident.so to be linked from apex am: 4e15bf53f0

Change-Id: Iea0a310c93367544a21a0964a7985647f888ef52

Allow libincident.so to be linked from apex

Bug: 145923087
Test: m -j
Change-Id: I787f15750e19c5028b334161c5a7d4fa13003c6f

Fix linkerconfig_backward_compatibility_test am: 2dd4e4d2cb am: b94abc04ad

Change-Id: Ifa52778d8d084fd1b0828a6c0f1e04479eb0cf1f

Fix linkerconfig_backward_compatibility_test am: 2dd4e4d2cb

Change-Id: I978ed22941f49942163419544398f8925af3ddaa

Fix linkerconfig_backward_compatibility_test

The current GetValue() falls back to GetProperty(). This makes the test
show different result on different running environment. For consistent
test result, MockVariables() now overrides all product/vendor variables.

Bug: n/a
Test: atest --host linkerconfig_backward_compatibility_test
      atest linkerconfig_backward_compatibility_test

Change-Id: I4b7533e10058b0236a8aef4c54fbb58f3bec0ccc

Read vndk libraries txt files from vndk apex am: 7c2b73a71f am: 1b5d7bf6e8

Change-Id: Ibe7a89be985750a6362ea047844c6829ef9c0508

Generate vendor stuff only if vndk is avilable am: 3f83cfb602 am: 3ea9bdb598

Change-Id: Ia13c12dff6af00263d8b69c70f3a34ba2ce82315

Read vndk libraries txt files from vndk apex am: 7c2b73a71f

Change-Id: I24a5c1482e9a059025dac3aba1ea1e1cf485b7b2

Generate vendor stuff only if vndk is avilable am: 3f83cfb602

Change-Id: I90769d8e5ae9ea3116454aa3bf4d43d04021a877

Read vndk libraries txt files from vndk apex

Vndk apex contains some txt files which list libraries it contains.
- llndk.libraries.txt
- vndksp.libraries.txt
- vndkcore.libraries.txt
- vndkprivate.libraries.txt

These files can be read from vndk apex instead of /system/etc.

Bug: 145184886
Test: build / flash / boot
Change-Id: I2b242b71768af2a30de519ccd4d73b1a86e46382

Generate vendor stuff only if vndk is avilable

Linkerconfig runs even when there is no apexes available. When vndk apex
is not available, having vendor section or namespaces requiring vndk
libraries is meaningless.

This is necessary because we'd like to load some variables from vndk
apex, but relying on those variables when vndk apex is not available may
fail to generate linker configuration.

Bug: 145184886
Test: build / flash / boot
Change-Id: I7c77757254a63c8bc7470c4ae87f2db3779e6465

Merge "Add required modules for linkerconfig" am: bade76eb13 am: abc420f2ba

Change-Id: I1bf9881155249401cf79d43505b0a8562945a54d

Merge "Add required modules for linkerconfig" am: bade76eb13

Change-Id: I9cad270d6fbfa1125f1524954f2b8f7765228499

Merge "2 more missing Result::ok() calls in system/linkerconfig" am: ffacdd9477 am: 284084a3d1

Change-Id: I9eacc7a5aa22b823d0d4bca8666d6f9eaf01282c

Merge "Add required modules for linkerconfig"

Merge "2 more missing Result::ok() calls in system/linkerconfig" am: ffacdd9477

Change-Id: I0fbc15f0cb04554567ee042fb97a91e09a6faf6f

Merge "2 more missing Result::ok() calls in system/linkerconfig"

Merge changes I2ead6a77,I6bf2204a am: 4add3c0b71 am: e9070610f2

Change-Id: If41dada9442fbd754ea1a963f69fb7551b3d9351

Merge changes I2ead6a77,I6bf2204a am: 4add3c0b71

Change-Id: I3b673660490dc97f6d080a9c0112a6b62a98c99c

Merge changes I2ead6a77,I6bf2204a

* changes:
  Remove ResolveVariables()
  Make ConfigWriter simpler

Add required modules for linkerconfig

linkerconfig relies on some txt files.
Adding those modules as required:
- llndk.libraries.txt
- vndkprivate.libraries.txt
- vndkcore.libraries.txt
- vndkcorevariant.libraries.txt
- vndksp.libraries.txt
- sanitizer.libraries.txt

Bug: N/A
Test: m installclean; m linkerconfig
      check if those txt files are installed together

Change-Id: I8ab98591026dad06b3140043c54b7134ca3dac33

2 more missing Result::ok() calls in system/linkerconfig

Test: cd system/linkerconfig && atest
Change-Id: I2c6f8eb4b81065b90512f88a9f248affecacf019

Merge "Use apex_name for the name of linker namespace" am: 2fa82ebd25 am: 0449c685fb

Change-Id: Icaed65a2bfc461da6c6ea196a164f4eb75bf2b82

Merge "Use apex_name for the name of linker namespace" am: 2fa82ebd25

Change-Id: I92d3bdbf307e06d8439d957d779312498c12e20f

Merge "Use apex_name for the name of linker namespace"

Remove ResolveVariables()

ResolveVariables() takes long because it uses regex pattern search and
replace them with values. It can be avoided by resolving variables while
configurations are being built.

For easy transition, "Var()" utility function is introduced which should
be called after LoadVariables(). This will be enforced by moving Var()
into Context so that it only be called when Context is available (which
is, after LoadVariables() is called)

Bug: 149020353
Test: atest --test-mapping system/linkerconfig
      build / flash / boot
Change-Id: I2ead6a77dcc9ec14b4e083897a2818714438beb5

Make ConfigWriter simpler

- Remove varg WriteLine
- No prefix
- Add WriteVars for writing multiple lines using "="/ "+="

This refactoring is a prep for removing "ResolveVariables()" which is very slow.

Bug: 149020353
Test: atest --test-mapping system/linkerconfig
Change-Id: I6bf2204a9ac0794b0bdf8028999a3543821a26be

Last missing Result::ok() calls in system/linkerconfig am: 51be4fe748 am: 5beb96375e

Change-Id: I653eebadb4580e9712bd2776754712ce9c45a972

Last missing Result::ok() calls in system/linkerconfig am: 51be4fe748

Change-Id: I71de5d9c437049101bdf9c185242058776981c35

Last missing Result::ok() calls in system/linkerconfig

No functionality change.

Test: cd system/linkerconfig && atest
Change-Id: I5ffa14d1d935c8e01c694338400e366a19895ba6

Remove unused files am: 35df8b6140 am: 71239daad0

Change-Id: I92ac09a5d326fe0e06859192782d73d000ce1a07

Remove unused files am: 35df8b6140

Change-Id: I3d3de0df7afb119313feb7704d5e03a054b09510

Use apex_name for the name of linker namespace

Till now short names are used for apex namespaces but there are no
explicit rule for this. Instead of extracting namespace name from apex
name (com.android.foo => foo), we'd better use apex name as it is
because it is well-established identifier for apexes.

Bug: 148826508
Test: build/flash/boot
      atest

Change-Id: I4f77eb85652a999274844436ba14b5c05831cf5b

Remove unused files

conscrypt and neuralnetworks namespaces are now generated automatically.

Bug: N/A
Test: build
Change-Id: Ic13d3f4206ecfda1937adf2111977b279fba05ef

Auto-generate runtime namespace am: b818280515 am: a9b9bcfd10

Change-Id: I9feeae2be4f8ddb34a379d281510baa2fe1bfc58

Auto-generate media namespace am: 834efc8c45 am: 3a3e8a5e81

Change-Id: Ie37051dbb631537f8b061acb728d37a82c5e7635

Auto-generate apex namespaces: cronet,nn,conscrypt am: 9960aca469 am: 0a034f67e0

Change-Id: I1bfb32e3c30f7b083f70df444eab94e394fc0c32

Auto-generate runtime namespace am: b818280515

Change-Id: I9ae5f026915c98aa1070560fd1923536c0a8d3b7

Auto-generate media namespace am: 834efc8c45

Change-Id: Idff81f99b6a7baf98882615653563e7cc69b09f2

Auto-generate apex namespaces: cronet,nn,conscrypt am: 9960aca469

Change-Id: I7e6ecab4463e0c50864ef186952d549b6a63b40f

Auto-generate runtime namespace

runtime apex provides bionic(libc/libm/libdl) libraries. But actually
they are supposed to be accessed via symlinks in /system/lib (i.e.
system namespace).

By amending soong not to emit those bionic libs as providing libs of
runtime apex, now runtime apex can be auto-generated by linkerconfig.

Bug: 144664390
Test: build / flash / boot
Change-Id: Idda42801a608e5fcb495bfccdd07ca6c0666a53d

Auto-generate media namespace

Now, media namespace is auto-generated.
This is done by adding /apex/<name>/lib as permitted path as well as
search path, because media apex needs a permitted path
/com.android.media/lib/extractors.

Bug: 144664390
Test: build / flash / boot
      atest --test-mapping system/linkerconfig
Change-Id: I9186b0768b3bce3ad19f3d14010725af0bbf5861

Auto-generate apex namespaces: cronet,nn,conscrypt

Some visible namespaces are now auto-generated using activated apex
info. This is quite a significant change for init/bootstrap stages.
For example, at init stage which has no apexes activated, linkerconfig
dosn't generate apex namespaces at all. Previously, it was statically
generated whether those apexes are available or not.

Amon remaining apexes, media and runtime will be removed soon.

Bug: 144664390
Test: compare generated ld.config.txt between changes
      build / flash / boot
      atest --test-mapping system/linkerconfig
Change-Id: I334071dd59273e194ccdca1ef2cff0b7ee1b5c35

Merge "Configuration for special APEX binaries" am: ce9da40b83 am: b2e5192c1f

Change-Id: I6cea090c1f2d00d10ea6e09ace322160d25156b6

Merge "Configuration for special APEX binaries" am: ce9da40b83

Change-Id: I182bca74f03de286823a6f0b61b04043e819920f

Merge "Configuration for special APEX binaries"

Configuration for special APEX binaries

Some of the linker configuration for APEX binaries are not enough just
to use auto-generated contents. This change implements some of the
additional namespaces for some special APEXes with binaries - ART and
Media SWCodec.

Bug: 147987608
Test: m -j passed
Test: Tested adbd and swcodec(Youtube) on Cuttlefish and Crosshatch
Change-Id: I58345dc31bb97138c9c632039a10ab000e23bcb6

Convert system/linkerconfig to Result::ok() am: 5c325567e2 am: 9dfd5dc389

Change-Id: Id387d0ff30bdc4269ca3881a313dd612050bbdb8

Convert system/linkerconfig to Result::ok() am: 5c325567e2

Change-Id: I3d10df9aaaa1dbb9456960c24b20c7d7c4f6bab4

Convert system/linkerconfig to Result::ok()

No functionality changes, this is a mechanical cleanup.

Test: m
Change-Id: I759398fdf01d1703b0c128fb4f34d33a6f1a2027

Move part of Context into modules::BaseContext am: 32c2ef7510 am: fded559ebc

Change-Id: Ib548efd43970c4f0bf2c7e5fbc40b6b580822d22

Add default "provides" list to system namespace am: 91079f0f9b am: 9e61bf8ea9

Change-Id: Ia4f3bcf0a2ae0a47f520d7872b8125db04b52f9e

Add "--strict" flag am: b4af747146 am: f5593d9e47

Change-Id: I103e823b93f8804dc3ead1c263907a320fdd00c8

Build adbd/resolv namespace automatically am: 7482996bd3 am: a5c0b57481

Change-Id: I6a13e46015e3bafda9613082b703b1511f247597

Move part of Context into modules::BaseContext am: 32c2ef7510

Change-Id: Ie1318021f558fce908222f5df2c174fe1db33812

Add default "provides" list to system namespace am: 91079f0f9b

Change-Id: Ie67d2b6d6bd4f5762a03f7559a6ff1ebcdd2c7ec

Add "--strict" flag am: b4af747146

Change-Id: Ic1e6d5a1e28fedc98470dc8a3f0ef9375fedf85f

Build adbd/resolv namespace automatically am: 7482996bd3

Change-Id: I7edb3004451f2b94e2f0f019ca603c4c1b423b72

Move part of Context into modules::BaseContext

ApexInfo/strict is used in Section::Resolve and it's very generic.
Moving it into BaseContext makes code simpler

Bug: N/A
Test: atest --test-mapping system/linkerconfig
Change-Id: I8acbdb7ed3368b6c10474be1c30a12fbc81c8ca6

Add default "provides" list to system namespace

Contrary to APEX modules, system doesn't have meta data represening
"provides". So adding the "provides" list to system namespaces.

Bug: 144664390
Test: build / flash / boot
Test: atest --test-mapping system/linkerconfig
Change-Id: I808736814221a0d0cff2f163d64da01b398dc265