cleanup, still doesn't seem to work

getting attributes work, setting them (e.g. with mark_boot_successful)
doesn't seem to work properly.

The IOCTL required to set the boot LUN doesn't work, some kernel driver
is missing for it.
merge-requests/7/head
Caleb Connolly 3 years ago
parent 2f5bf10269
commit b28a87499c
No known key found for this signature in database
GPG Key ID: 0583312B195F64B6
  1. 3
      BootControl.cpp
  2. 16
      boot_control_impl_qcom.cpp
  3. 53
      gpt-utils.cpp

@ -87,6 +87,9 @@ int main (int argc, char * argv []) {
printf("======= Current slot: %d\n", bootctl->getCurrentSlot()); printf("======= Current slot: %d\n", bootctl->getCurrentSlot());
printf("======= isslotbootable: a = %d, b = %d\n", bootctl->isSlotBootable(0), printf("======= isslotbootable: a = %d, b = %d\n", bootctl->isSlotBootable(0),
bootctl->isSlotBootable(1)); bootctl->isSlotBootable(1));
printf("======= markBootSuccessful\n", bootctl->markBootSuccessful());
printf("======= isSlotMarkedSuccessful: a = %d, b = %d\n", bootctl->isSlotMarkedSuccessful(0), printf("======= isSlotMarkedSuccessful: a = %d, b = %d\n", bootctl->isSlotMarkedSuccessful(0),
bootctl->isSlotMarkedSuccessful(1)); bootctl->isSlotMarkedSuccessful(1));
// printf("\n\n\n trying to switch to slot b: %d\n",
// bootctl->setActiveBootSlot(1));
} }

@ -125,15 +125,19 @@ static int get_partition_attribute(char *partname,
goto error; goto error;
} }
attr = pentry + AB_FLAG_OFFSET; attr = pentry + AB_FLAG_OFFSET;
printf("gpt_disk_get_pentry: 0x%x", *attr); printf("get_partition_attribute() partname = %s, attr = 0x%x\n", partname, *attr);
if (part_attr == ATTR_SLOT_ACTIVE) if (part_attr == ATTR_SLOT_ACTIVE) {
retval = !!(*attr & AB_PARTITION_ATTR_SLOT_ACTIVE); retval = !!(*attr & AB_PARTITION_ATTR_SLOT_ACTIVE);
else if (part_attr == ATTR_BOOT_SUCCESSFUL) printf("ATTR_SLOT_ACTIVE, retval = %d\n", retval);
} else if (part_attr == ATTR_BOOT_SUCCESSFUL) {
retval = !!(*attr & AB_PARTITION_ATTR_BOOT_SUCCESSFUL); retval = !!(*attr & AB_PARTITION_ATTR_BOOT_SUCCESSFUL);
else if (part_attr == ATTR_UNBOOTABLE) printf("AB_PARTITION_ATTR_BOOT_SUCCESSFUL, retval = %d\n", retval);
} else if (part_attr == ATTR_UNBOOTABLE) {
retval = !!(*attr & AB_PARTITION_ATTR_UNBOOTABLE); retval = !!(*attr & AB_PARTITION_ATTR_UNBOOTABLE);
else printf("AB_PARTITION_ATTR_UNBOOTABLE, retval = %d\n", retval);
} else {
retval = -1; retval = -1;
}
gpt_disk_free(disk); gpt_disk_free(disk);
return retval; return retval;
error: error:
@ -685,7 +689,6 @@ error:
int is_slot_marked_successful(struct boot_control_module *module, unsigned slot) int is_slot_marked_successful(struct boot_control_module *module, unsigned slot)
{ {
printf("%s\n", __func__);
int attr = 0; int attr = 0;
char bootPartition[MAX_GPT_NAME_SIZE + 1] = {0}; char bootPartition[MAX_GPT_NAME_SIZE + 1] = {0};
@ -697,6 +700,7 @@ int is_slot_marked_successful(struct boot_control_module *module, unsigned slot)
sizeof(bootPartition) - 1, sizeof(bootPartition) - 1,
"boot%s", slot_suffix_arr[slot]); "boot%s", slot_suffix_arr[slot]);
attr = get_partition_attribute(bootPartition, ATTR_BOOT_SUCCESSFUL); attr = get_partition_attribute(bootPartition, ATTR_BOOT_SUCCESSFUL);
printf("%s(): slot = %d, attr = 0x%x\n", __func__, slot, attr);
if (attr >= 0) if (attr >= 0)
return attr; return attr;
error: error:

@ -76,10 +76,10 @@
//the boot critical luns lie between sda to sdz which is acceptable because //the boot critical luns lie between sda to sdz which is acceptable because
//only user added external disks,etc would lie beyond that limit which do not //only user added external disks,etc would lie beyond that limit which do not
//contain partitions that interest us here. //contain partitions that interest us here.
#define PATH_TRUNCATE_LOC (sizeof("/dev/disk/sda") - 1) #define PATH_TRUNCATE_LOC (sizeof("/dev/sda") - 1)
//From /dev/disk/sda get just sda //From /dev/disk/sda get just sda
#define LUN_NAME_START_LOC (sizeof("/dev/disk/") - 1) #define LUN_NAME_START_LOC (sizeof("/dev/") - 1)
#define BOOT_LUN_A_ID 1 #define BOOT_LUN_A_ID 1
#define BOOT_LUN_B_ID 2 #define BOOT_LUN_B_ID 2
/****************************************************************************** /******************************************************************************
@ -125,6 +125,36 @@ struct update_data {
/****************************************************************************** /******************************************************************************
* FUNCTIONS * FUNCTIONS
******************************************************************************/ ******************************************************************************/
void DumpHex(const void* data, size_t size) {
char ascii[17];
size_t i, j;
ascii[16] = '\0';
for (i = 0; i < size; ++i) {
printf("%02X ", ((unsigned char*)data)[i]);
if (((unsigned char*)data)[i] >= ' ' && ((unsigned char*)data)[i] <= '~') {
ascii[i % 16] = ((unsigned char*)data)[i];
} else {
ascii[i % 16] = '.';
}
if ((i+1) % 8 == 0 || i+1 == size) {
printf(" ");
if ((i+1) % 16 == 0) {
printf("| %s \n", ascii);
} else if (i+1 == size) {
ascii[(i+1) % 16] = '\0';
if ((i+1) % 16 <= 8) {
printf(" ");
}
for (j = (i+1) % 16; j < 16; ++j) {
printf(" ");
}
printf("| %s \n", ascii);
}
}
}
}
/** /**
* ========================================================================== * ==========================================================================
* *
@ -205,7 +235,7 @@ static uint8_t *gpt_pentry_seek(const char *ptn_name,
for (i = 0; i < sizeof(name8); i++) for (i = 0; i < sizeof(name8); i++)
name8[i] = pentry_name[i * 2]; name8[i] = pentry_name[i * 2];
if (!strncmp(ptn_name, name8, len)) if (!strncmp(ptn_name, name8, len))
if (name8[len] == 0; || !strcmp(&name8[len], BAK_PTN_NAME_EXT)) if (name8[len] == 0 || !strcmp(&name8[len], BAK_PTN_NAME_EXT))
return (uint8_t *) (pentry_name - PARTITION_NAME_OFFSET); return (uint8_t *) (pentry_name - PARTITION_NAME_OFFSET);
} }
@ -540,7 +570,7 @@ int get_scsi_node_from_bootdevice(const char *bootdev_path,
__func__); __func__);
goto error; goto error;
} }
if (readlink(bootdev_path, real_path, sizeof(real_path) - 1) < 0) { if (realpath(bootdev_path, real_path) < 0) {
fprintf(stderr, "failed to resolve link for %s(%s)\n", fprintf(stderr, "failed to resolve link for %s(%s)\n",
bootdev_path, bootdev_path,
strerror(errno)); strerror(errno));
@ -559,6 +589,8 @@ int get_scsi_node_from_bootdevice(const char *bootdev_path,
real_path); real_path);
goto error; goto error;
} }
// snprintf(sg_node_path, PATH_MAX, "%s", real_path);
// return 0;
//This will give us /dev/disk/sdb/device/scsi_generic //This will give us /dev/disk/sdb/device/scsi_generic
//which contains a file sgY whose name gives us the path //which contains a file sgY whose name gives us the path
//to /dev/sgY which we return //to /dev/sgY which we return
@ -614,6 +646,7 @@ int set_boot_lun(char *sg_dev, uint8_t boot_lun_id)
__func__); __func__);
goto error; goto error;
} }
printf("%s(): sg_dev = %s\n", __func__, sg_dev);
memset(data, 0, ioctl_data_size); memset(data, 0, ioctl_data_size);
data->opcode = UPIU_QUERY_OPCODE_WRITE_ATTR; data->opcode = UPIU_QUERY_OPCODE_WRITE_ATTR;
data->idn = QUERY_ATTR_IDN_BOOT_LU_EN; data->idn = QUERY_ATTR_IDN_BOOT_LU_EN;
@ -1017,9 +1050,9 @@ int prepare_boot_update(enum boot_update_stage stage)
if (stat(buf, &ufs_dir_stat)) { if (stat(buf, &ufs_dir_stat)) {
continue; continue;
} }
if (readlink(buf, real_path, sizeof(real_path) - 1) < 0) if (realpath(buf, real_path) < 0)
{ {
fprintf(stderr, "%s: readlink error. Skipping %s\n", fprintf(stderr, "%s: realpath error. Skipping %s\n",
__func__, __func__,
strerror(errno)); strerror(errno));
} else { } else {
@ -1076,7 +1109,6 @@ static int get_dev_path_from_partition_name(const char *partname,
"%s/%s", "%s/%s",
BOOT_DEV_DIR, BOOT_DEV_DIR,
partname); partname);
printf("PATH: #%s#\n", path);
// if (rc = stat(path, &st)) { // if (rc = stat(path, &st)) {
// printf("stat failed: errno=%d\n", errno); // printf("stat failed: errno=%d\n", errno);
// goto error; // goto error;
@ -1084,12 +1116,12 @@ static int get_dev_path_from_partition_name(const char *partname,
buf = realpath(path, buf); buf = realpath(path, buf);
if (!buf) if (!buf)
{ {
printf("readlink failed\n"); printf("realpath failed\n");
goto error; goto error;
} else { } else {
buf[PATH_TRUNCATE_LOC] = '\0'; buf[PATH_TRUNCATE_LOC] = '\0';
} }
printf("readlink path: %s, rc = %d, errno=%d\n", buf, rc, errno); printf("PATH: %s, realpath: %s\n", path, buf);
} else { } else {
snprintf(buf, buflen, "/dev/mmcblk0"); snprintf(buf, buflen, "/dev/mmcblk0");
} }
@ -1228,7 +1260,6 @@ static uint8_t* gpt_get_header(const char *partname, enum gpt_instance instance)
} }
hdr = (uint8_t*)calloc(block_size, 1); hdr = (uint8_t*)calloc(block_size, 1);
printf("%s(): devpath = %s, block_size = %u, hdr[0] = 0x%x, hdr[1] = 0x%x\n", __func__, devpath, block_size, hdr[0], hdr[1]);
if (!hdr) { if (!hdr) {
fprintf(stderr, "%s: Failed to allocate memory for gpt header\n", fprintf(stderr, "%s: Failed to allocate memory for gpt header\n",
__func__); __func__);
@ -1248,6 +1279,7 @@ static uint8_t* gpt_get_header(const char *partname, enum gpt_instance instance)
__func__); __func__);
goto error; goto error;
} }
//DumpHex(hdr, block_size);
close(fd); close(fd);
return hdr; return hdr;
error: error:
@ -1294,7 +1326,6 @@ static uint8_t* gpt_get_pentry_arr(uint8_t *hdr, int fd)
__func__); __func__);
goto error; goto error;
} }
printf("hdr + PENTRIES_OFFSET = %lu, pentries_start: %lu\n", GET_8_BYTES(hdr + PENTRIES_OFFSET), pentries_start);
rc = blk_rw(fd, 0, rc = blk_rw(fd, 0,
pentries_start, pentries_start,
pentry_arr, pentry_arr,

Loading…
Cancel
Save