am 44df726a: am cb079c89: Merge "Add instruction-set arguments to installd commands."

* commit '44df726a6cf4fe1119e146c9a18c78966a6305d3':
  Add instruction-set arguments to installd commands.
This commit is contained in:
Narayan Kamath 2014-05-01 14:05:10 +00:00 committed by Android Git Automerger
commit fc4e378877
3 changed files with 49 additions and 35 deletions

View File

@ -387,7 +387,7 @@ int free_cache(int64_t free_size)
return data_disk_free() >= free_size ? 0 : -1; 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 src_dex[PKG_PATH_MAX];
char dst_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(src)) return -1;
if (validate_apk_path(dst)) return -1; if (validate_apk_path(dst)) return -1;
if (create_cache_path(src_dex, src)) return -1; if (create_cache_path(src_dex, src, instruction_set)) return -1;
if (create_cache_path(dst_dex, dst)) return -1; if (create_cache_path(dst_dex, dst, instruction_set)) return -1;
ALOGV("move %s -> %s\n", src_dex, dst_dex); ALOGV("move %s -> %s\n", src_dex, dst_dex);
if (rename(src_dex, dst_dex) < 0) { 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]; char dex_path[PKG_PATH_MAX];
if (validate_apk_path(path)) return -1; 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); ALOGV("unlink %s\n", dex_path);
if (unlink(dex_path) < 0) { 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, int get_size(const char *pkgname, userid_t userid, const char *apkpath,
const char *libdirpath, const char *fwdlock_apkpath, const char *asecpath, const char *libdirpath, const char *fwdlock_apkpath, const char *asecpath,
int64_t *_codesize, int64_t *_datasize, int64_t *_cachesize, const char *instruction_set, int64_t *_codesize, int64_t *_datasize,
int64_t* _asecsize) int64_t *_cachesize, int64_t* _asecsize)
{ {
DIR *d; DIR *d;
int dfd; int dfd;
@ -456,7 +456,7 @@ int get_size(const char *pkgname, userid_t userid, const char *apkpath,
} }
} }
/* count the cached dexfile as code */ /* 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) { if (stat(path, &s) == 0) {
codesize += stat_size(&s); 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; char *tmp;
int srclen; int srclen;
@ -561,18 +561,20 @@ int create_cache_path(char path[PKG_PATH_MAX], const char *src)
} }
dstlen = srclen + strlen(DALVIK_CACHE_PREFIX) + dstlen = srclen + strlen(DALVIK_CACHE_PREFIX) +
strlen(DALVIK_CACHE_POSTFIX) + 1; strlen(instruction_set) +
strlen(DALVIK_CACHE_POSTFIX) + 2;
if (dstlen > PKG_PATH_MAX) { if (dstlen > PKG_PATH_MAX) {
return -1; return -1;
} }
sprintf(path,"%s%s%s", sprintf(path,"%s%s/%s%s",
DALVIK_CACHE_PREFIX, DALVIK_CACHE_PREFIX,
instruction_set,
src + 1, /* skip the leading / */ src + 1, /* skip the leading / */
DALVIK_CACHE_POSTFIX); 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 == '/') { if (*tmp == '/') {
*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, 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]; char dex2oat_flags[PROPERTY_VALUE_MAX];
property_get("dalvik.vm.dex2oat-flags", dex2oat_flags, ""); 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 char* DEX2OAT_BIN = "/system/bin/dex2oat";
static const int MAX_INT_LEN = 12; // '-'+10dig+'\0' -OR- 0x+8dig 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_fd_arg[strlen("--zip-fd=") + MAX_INT_LEN];
char zip_location_arg[strlen("--zip-location=") + PKG_PATH_MAX]; char zip_location_arg[strlen("--zip-location=") + PKG_PATH_MAX];
char oat_fd_arg[strlen("--oat-fd=") + MAX_INT_LEN]; char oat_fd_arg[strlen("--oat-fd=") + MAX_INT_LEN];
char oat_location_arg[strlen("--oat-name=") + PKG_PATH_MAX]; 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_fd_arg, "--zip-fd=%d", zip_fd);
sprintf(zip_location_arg, "--zip-location=%s", input_file_name); sprintf(zip_location_arg, "--zip-location=%s", input_file_name);
sprintf(oat_fd_arg, "--oat-fd=%d", oat_fd); sprintf(oat_fd_arg, "--oat-fd=%d", oat_fd);
sprintf(oat_location_arg, "--oat-location=%s", output_file_name); sprintf(oat_location_arg, "--oat-location=%s", output_file_name);
sprintf(instruction_set_arg, "--instruction-set=%s", instruction_set);
if (strcmp(pkgname, "*") != 0) { 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); DALVIK_CACHE_PREFIX "profiles", pkgname);
} else { } 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); ALOGV("Running %s in=%s out=%s\n", DEX2OAT_BIN, input_file_name, output_file_name);
execl(DEX2OAT_BIN, DEX2OAT_BIN, execl(DEX2OAT_BIN, DEX2OAT_BIN,
zip_fd_arg, zip_location_arg, zip_fd_arg, zip_location_arg,
oat_fd_arg, oat_location_arg, oat_fd_arg, oat_location_arg,
profile_file, profile_file_arg, instruction_set_arg,
strlen(dex2oat_flags) > 0 ? dex2oat_flags : NULL, strlen(dex2oat_flags) > 0 ? dex2oat_flags : NULL,
(char*) NULL); (char*) NULL);
ALOGE("execl(%s) failed: %s\n", DEX2OAT_BIN, strerror(errno)); 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, 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 utimbuf ut;
struct stat apk_stat, dex_stat; 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; 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) { if (strncmp(persist_sys_dalvik_vm_lib, "libdvm", 6) == 0) {
run_dexopt(zip_fd, out_fd, apk_path, out_path); run_dexopt(zip_fd, out_fd, apk_path, out_path);
} else if (strncmp(persist_sys_dalvik_vm_lib, "libart", 6) == 0) { } 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 { } else {
exit(69); /* Unexpected persist.sys.dalvik.vm.lib value */ exit(69); /* Unexpected persist.sys.dalvik.vm.lib value */
} }

View File

@ -38,18 +38,18 @@ static int do_install(char **arg, char reply[REPLY_MAX])
static int do_dexopt(char **arg, char reply[REPLY_MAX]) static int do_dexopt(char **arg, char reply[REPLY_MAX])
{ {
/* apk_path, uid, is_public, pkgname */ /* apk_path, uid, is_public, pkgname, instruction_set */
return dexopt(arg[0], atoi(arg[1]), atoi(arg[2]), arg[3]); 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]) 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]) 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]) 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 */ /* pkgdir, userid, apkpath */
res = get_size(arg[0], atoi(arg[1]), arg[2], arg[3], arg[4], arg[5], 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 * Each int64_t can take up 22 characters printed out. Make sure it
@ -144,15 +144,15 @@ struct cmdinfo {
struct cmdinfo cmds[] = { struct cmdinfo cmds[] = {
{ "ping", 0, do_ping }, { "ping", 0, do_ping },
{ "install", 4, do_install }, { "install", 4, do_install },
{ "dexopt", 4, do_dexopt }, { "dexopt", 5, do_dexopt },
{ "movedex", 2, do_move_dex }, { "movedex", 3, do_move_dex },
{ "rmdex", 1, do_rm_dex }, { "rmdex", 2, do_rm_dex },
{ "remove", 2, do_remove }, { "remove", 2, do_remove },
{ "rename", 2, do_rename }, { "rename", 2, do_rename },
{ "fixuid", 3, do_fixuid }, { "fixuid", 3, do_fixuid },
{ "freecache", 1, do_free_cache }, { "freecache", 1, do_free_cache },
{ "rmcache", 2, do_rm_cache }, { "rmcache", 2, do_rm_cache },
{ "getsize", 6, do_get_size }, { "getsize", 7, do_get_size },
{ "rmuserdata", 2, do_rm_user_data }, { "rmuserdata", 2, do_rm_user_data },
{ "movefiles", 0, do_movefiles }, { "movefiles", 0, do_movefiles },
{ "linklib", 3, do_linklib }, { "linklib", 3, do_linklib },

View File

@ -152,7 +152,8 @@ int create_move_path(char path[PKG_PATH_MAX],
int is_valid_package_name(const char* pkgname); 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 delete_dir_contents(const char *pathname,
int also_delete_dir, 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 make_user_data(const char *pkgname, uid_t uid, userid_t userid, const char* seinfo);
int delete_user(userid_t userid); int delete_user(userid_t userid);
int delete_cache(const char *pkgname, userid_t userid); int delete_cache(const char *pkgname, userid_t userid);
int move_dex(const char *src, const char *dst); int move_dex(const char *src, const char *dst, const char *instruction_set);
int rm_dex(const char *path); int rm_dex(const char *path, const char *instruction_set);
int protect(char *pkgname, gid_t gid); int protect(char *pkgname, gid_t gid);
int get_size(const char *pkgname, userid_t userid, const char *apkpath, const char *libdirpath, 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, const char *fwdlock_apkpath, const char *asecpath, const char *instruction_set,
int64_t *datasize, int64_t *cachesize, int64_t *asecsize); int64_t *codesize, int64_t *datasize, int64_t *cachesize, int64_t *asecsize);
int free_cache(int64_t free_size); 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 movefiles();
int linklib(const char* target, const char* source, int userId); int linklib(const char* target, const char* source, int userId);
int idmap(const char *target_path, const char *overlay_path, uid_t uid); int idmap(const char *target_path, const char *overlay_path, uid_t uid);