Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 791781bf authored by Narayan Kamath's avatar Narayan Kamath
Browse files

Add instruction-set arguments to installd commands. 

The instruction-set is used to construct the dalvik cache
path and is also passed down to dex2oat.

Change-Id: Ie480841b3e917dabe23ad4d5f4a0b08990500a51
parent 2eefe69f

cmds/installd/commands.c

+32 −20
Original line number Diff line number Diff line
@@ -387,7 +387,7 @@ int free_cache(int64_t free_size) 
    return data_disk_free() >= free_size ? 0 : -1;

}



int move_dex(const char *src, const char *dst)

int move_dex(const char *src, const char *dst, const char *instruction_set)

{

    char src_dex[PKG_PATH_MAX];

    char dst_dex[PKG_PATH_MAX];
@@ -395,8 +395,8 @@ int move_dex(const char *src, const char *dst) 
    if (validate_apk_path(src)) return -1;

    if (validate_apk_path(dst)) return -1;



    if (create_cache_path(src_dex, src)) return -1;

    if (create_cache_path(dst_dex, dst)) return -1;

    if (create_cache_path(src_dex, src, instruction_set)) return -1;

    if (create_cache_path(dst_dex, dst, instruction_set)) return -1;



    ALOGV("move %s -> %s\n", src_dex, dst_dex);

    if (rename(src_dex, dst_dex) < 0) {
@@ -407,12 +407,12 @@ int move_dex(const char *src, const char *dst) 
    }

}



int rm_dex(const char *path)

int rm_dex(const char *path, const char *instruction_set)

{

    char dex_path[PKG_PATH_MAX];



    if (validate_apk_path(path)) return -1;

    if (create_cache_path(dex_path, path)) return -1;

    if (create_cache_path(dex_path, path, instruction_set)) return -1;



    ALOGV("unlink %s\n", dex_path);

    if (unlink(dex_path) < 0) {
@@ -425,8 +425,8 @@ int rm_dex(const char *path) 


int get_size(const char *pkgname, userid_t userid, const char *apkpath,

             const char *libdirpath, const char *fwdlock_apkpath, const char *asecpath,

             int64_t *_codesize, int64_t *_datasize, int64_t *_cachesize,

             int64_t* _asecsize)

             const char *instruction_set, int64_t *_codesize, int64_t *_datasize,

             int64_t *_cachesize, int64_t* _asecsize)

{

    DIR *d;

    int dfd;
@@ -456,7 +456,7 @@ int get_size(const char *pkgname, userid_t userid, const char *apkpath, 
        }

    }

        /* count the cached dexfile as code */

    if (!create_cache_path(path, apkpath)) {

    if (!create_cache_path(path, apkpath, instruction_set)) {

        if (stat(path, &s) == 0) {

            codesize += stat_size(&s);

        }
@@ -543,7 +543,7 @@ done: 
}





int create_cache_path(char path[PKG_PATH_MAX], const char *src)

int create_cache_path(char path[PKG_PATH_MAX], const char *src, const char *instruction_set)

{

    char *tmp;

    int srclen;
@@ -561,18 +561,20 @@ int create_cache_path(char path[PKG_PATH_MAX], const char *src) 
    }



    dstlen = srclen + strlen(DALVIK_CACHE_PREFIX) +

        strlen(DALVIK_CACHE_POSTFIX) + 1;

        strlen(instruction_set) +

        strlen(DALVIK_CACHE_POSTFIX) + 2;



    if (dstlen > PKG_PATH_MAX) {

        return -1;

    }



    sprintf(path,"%s%s%s",

    sprintf(path,"%s%s/%s%s",

            DALVIK_CACHE_PREFIX,

            instruction_set,

            src + 1, /* skip the leading / */

            DALVIK_CACHE_POSTFIX);



    for(tmp = path + strlen(DALVIK_CACHE_PREFIX); *tmp; tmp++) {

    for(tmp = path + strlen(DALVIK_CACHE_PREFIX) + strlen(instruction_set) + 1; *tmp; tmp++) {

        if (*tmp == '/') {

            *tmp = '@';

        }
@@ -604,7 +606,7 @@ static void run_dexopt(int zip_fd, int odex_fd, const char* input_file_name, 
}



static void run_dex2oat(int zip_fd, int oat_fd, const char* input_file_name,

    const char* output_file_name, const char *pkgname)

    const char* output_file_name, const char *pkgname, const char *instruction_set)

{

    char dex2oat_flags[PROPERTY_VALUE_MAX];

    property_get("dalvik.vm.dex2oat-flags", dex2oat_flags, "");
@@ -612,28 +614,38 @@ static void run_dex2oat(int zip_fd, int oat_fd, const char* input_file_name, 


    static const char* DEX2OAT_BIN = "/system/bin/dex2oat";

    static const int MAX_INT_LEN = 12;      // '-'+10dig+'\0' -OR- 0x+8dig

    static const unsigned int MAX_INSTRUCTION_SET_LEN = 32;



    if (strlen(instruction_set) >= MAX_INSTRUCTION_SET_LEN) {

        ALOGE("Instruction set %s longer than max length of %d",

              instruction_set, MAX_INSTRUCTION_SET_LEN);

        return;

    }



    char zip_fd_arg[strlen("--zip-fd=") + MAX_INT_LEN];

    char zip_location_arg[strlen("--zip-location=") + PKG_PATH_MAX];

    char oat_fd_arg[strlen("--oat-fd=") + MAX_INT_LEN];

    char oat_location_arg[strlen("--oat-name=") + PKG_PATH_MAX];

    char profile_file[strlen("--profile-file=") + PKG_PATH_MAX];

    char profile_file_arg[strlen("--profile-file=") + PKG_PATH_MAX];

    char instruction_set_arg[strlen("--instruction-set=") + MAX_INSTRUCTION_SET_LEN];



    sprintf(zip_fd_arg, "--zip-fd=%d", zip_fd);

    sprintf(zip_location_arg, "--zip-location=%s", input_file_name);

    sprintf(oat_fd_arg, "--oat-fd=%d", oat_fd);

    sprintf(oat_location_arg, "--oat-location=%s", output_file_name);

    sprintf(instruction_set_arg, "--instruction-set=%s", instruction_set);

    if (strcmp(pkgname, "*") != 0) {

        snprintf(profile_file, sizeof(profile_file), "--profile-file=%s/%s",

        snprintf(profile_file_arg, sizeof(profile_file_arg), "--profile-file=%s/%s",

                 DALVIK_CACHE_PREFIX "profiles", pkgname);

    } else {

        strcpy(profile_file, "--no-profile-file");

        strcpy(profile_file_arg, "--no-profile-file");

    }



    ALOGV("Running %s in=%s out=%s\n", DEX2OAT_BIN, input_file_name, output_file_name);

    execl(DEX2OAT_BIN, DEX2OAT_BIN,

          zip_fd_arg, zip_location_arg,

          oat_fd_arg, oat_location_arg,

          profile_file,

          profile_file_arg, instruction_set_arg,

          strlen(dex2oat_flags) > 0 ? dex2oat_flags : NULL,

          (char*) NULL);

    ALOGE("execl(%s) failed: %s\n", DEX2OAT_BIN, strerror(errno));
@@ -666,7 +678,7 @@ static int wait_child(pid_t pid) 
}



int dexopt(const char *apk_path, uid_t uid, int is_public,

           const char *pkgname)

           const char *pkgname, const char *instruction_set)

{

    struct utimbuf ut;

    struct stat apk_stat, dex_stat;
@@ -694,7 +706,7 @@ int dexopt(const char *apk_path, uid_t uid, int is_public, 
        }

    }



    if (create_cache_path(out_path, apk_path)) {

    if (create_cache_path(out_path, apk_path, instruction_set)) {

        return -1;

    }
@@ -762,7 +774,7 @@ int dexopt(const char *apk_path, uid_t uid, int is_public, 
        if (strncmp(persist_sys_dalvik_vm_lib, "libdvm", 6) == 0) {

            run_dexopt(zip_fd, out_fd, apk_path, out_path);

        } else if (strncmp(persist_sys_dalvik_vm_lib, "libart", 6) == 0) {

            run_dex2oat(zip_fd, out_fd, apk_path, out_path, pkgname);

            run_dex2oat(zip_fd, out_fd, apk_path, out_path, pkgname, instruction_set);

        } else {

            exit(69);   /* Unexpected persist.sys.dalvik.vm.lib value */

        }

cmds/installd/installd.c

+9 −9
Original line number Diff line number Diff line
@@ -38,18 +38,18 @@ static int do_install(char **arg, char reply[REPLY_MAX]) 


static int do_dexopt(char **arg, char reply[REPLY_MAX])

{

        /* apk_path, uid, is_public, pkgname */

    return dexopt(arg[0], atoi(arg[1]), atoi(arg[2]), arg[3]);

        /* apk_path, uid, is_public, pkgname, instruction_set */

    return dexopt(arg[0], atoi(arg[1]), atoi(arg[2]), arg[3], arg[4]);

}



static int do_move_dex(char **arg, char reply[REPLY_MAX])

{

    return move_dex(arg[0], arg[1]); /* src, dst */

    return move_dex(arg[0], arg[1], arg[2]); /* src, dst, instruction_set */

}



static int do_rm_dex(char **arg, char reply[REPLY_MAX])

{

    return rm_dex(arg[0]); /* pkgname */

    return rm_dex(arg[0], arg[1]); /* pkgname, instruction_set */

}



static int do_remove(char **arg, char reply[REPLY_MAX])
@@ -87,7 +87,7 @@ static int do_get_size(char **arg, char reply[REPLY_MAX]) 


        /* pkgdir, userid, apkpath */

    res = get_size(arg[0], atoi(arg[1]), arg[2], arg[3], arg[4], arg[5],

            &codesize, &datasize, &cachesize, &asecsize);

            arg[6], &codesize, &datasize, &cachesize, &asecsize);



    /*

     * Each int64_t can take up 22 characters printed out. Make sure it
@@ -144,15 +144,15 @@ struct cmdinfo { 
struct cmdinfo cmds[] = {

    { "ping",                 0, do_ping },

    { "install",              4, do_install },

    { "dexopt",               4, do_dexopt },

    { "movedex",              2, do_move_dex },

    { "rmdex",                1, do_rm_dex },

    { "dexopt",               5, do_dexopt },

    { "movedex",              3, do_move_dex },

    { "rmdex",                2, do_rm_dex },

    { "remove",               2, do_remove },

    { "rename",               2, do_rename },

    { "fixuid",               3, do_fixuid },

    { "freecache",            1, do_free_cache },

    { "rmcache",              2, do_rm_cache },

    { "getsize",              6, do_get_size },

    { "getsize",              7, do_get_size },

    { "rmuserdata",           2, do_rm_user_data },

    { "movefiles",            0, do_movefiles },

    { "linklib",              3, do_linklib },

cmds/installd/installd.h

+8 −6
Original line number Diff line number Diff line
@@ -152,7 +152,8 @@ int create_move_path(char path[PKG_PATH_MAX], 


int is_valid_package_name(const char* pkgname);



int create_cache_path(char path[PKG_PATH_MAX], const char *src);

int create_cache_path(char path[PKG_PATH_MAX], const char *src,

                      const char *instruction_set);



int delete_dir_contents(const char *pathname,

                        int also_delete_dir,
@@ -202,14 +203,15 @@ int delete_user_data(const char *pkgname, userid_t userid); 
int make_user_data(const char *pkgname, uid_t uid, userid_t userid, const char* seinfo);

int delete_user(userid_t userid);

int delete_cache(const char *pkgname, userid_t userid);

int move_dex(const char *src, const char *dst);

int rm_dex(const char *path);

int move_dex(const char *src, const char *dst, const char *instruction_set);

int rm_dex(const char *path, const char *instruction_set);

int protect(char *pkgname, gid_t gid);

int get_size(const char *pkgname, userid_t userid, const char *apkpath, const char *libdirpath,

             const char *fwdlock_apkpath, const char *asecpath, int64_t *codesize,

             int64_t *datasize, int64_t *cachesize, int64_t *asecsize);

             const char *fwdlock_apkpath, const char *asecpath, const char *instruction_set,

             int64_t *codesize, int64_t *datasize, int64_t *cachesize, int64_t *asecsize);

int free_cache(int64_t free_size);

int dexopt(const char *apk_path, uid_t uid, int is_public, const char *pkgName);

int dexopt(const char *apk_path, uid_t uid, int is_public, const char *pkgName,

           const char *instruction_set);

int movefiles();

int linklib(const char* target, const char* source, int userId);

int idmap(const char *target_path, const char *overlay_path, uid_t uid);