From ee613097bee89eec370c2118614ac39f65ea1f21 Mon Sep 17 00:00:00 2001 From: Vishnu Swaroop Duddu Date: Thu, 12 May 2016 20:07:29 +0530 Subject: [PATCH] tools: patch-image: Added optional size option Added optional command line option for patch-image tool Default 16KB size is still maintained as this is an optional argument. if one wants to specify or increase size they can provide this option. sample usage: patch-dtb [dtb max size] Signed-off-by: Vishnu Swaroop Duddu Signed-off-by: Hauke Mehrtens --- tools/patch-image/src/patch-cmdline.c | 14 ++++++++++---- tools/patch-image/src/patch-dtb.c | 22 ++++++++++++++-------- 2 files changed, 24 insertions(+), 12 deletions(-) diff --git a/tools/patch-image/src/patch-cmdline.c b/tools/patch-image/src/patch-cmdline.c index d3a5b119e4..8d1fce9172 100644 --- a/tools/patch-image/src/patch-cmdline.c +++ b/tools/patch-image/src/patch-cmdline.c @@ -35,10 +35,16 @@ int main(int argc, char **argv) { int fd, found = 0, len, ret = -1; char *ptr, *p; + unsigned int search_space; - if (argc != 3) { - fprintf(stderr, "Usage: %s \n", argv[0]); + if (argc <= 2 || argc > 4) { + fprintf(stderr, "Usage: %s [size]\n", argv[0]); goto err1; + } else if (argc == 3) { + fprintf(stdout, "search space used is default of 16KB\n"); + search_space = SEARCH_SPACE; + } else { + search_space = atoi(argv[3]); } len = strlen(argv[2]); if (len + 9 > CMDLINE_MAX) { @@ -47,12 +53,12 @@ int main(int argc, char **argv) } if (((fd = open(argv[1], O_RDWR)) < 0) || - (ptr = (char *) mmap(0, SEARCH_SPACE + CMDLINE_MAX, PROT_READ|PROT_WRITE, MAP_SHARED, fd, 0)) == (void *) (-1)) { + (ptr = (char *) mmap(0, search_space + CMDLINE_MAX, PROT_READ|PROT_WRITE, MAP_SHARED, fd, 0)) == (void *) (-1)) { fprintf(stderr, "Could not open kernel image"); goto err2; } - for (p = ptr; p < (ptr + SEARCH_SPACE); p += 4) { + for (p = ptr; p < (ptr + search_space); p += 4) { if (memcmp(p, "CMDLINE:", 8) == 0) { found = 1; p += 8; diff --git a/tools/patch-image/src/patch-dtb.c b/tools/patch-image/src/patch-dtb.c index 7812eb9b39..8d186236c1 100644 --- a/tools/patch-image/src/patch-dtb.c +++ b/tools/patch-image/src/patch-dtb.c @@ -30,7 +30,6 @@ #include #include -#define SEARCH_SPACE (16 * 1024) #define DTB_MAX (16 * 1024) int main(int argc, char **argv) @@ -38,11 +37,18 @@ int main(int argc, char **argv) int fd, fddtb, found = 0, len, ret = -1; char *ptr, *ptrdtb, *p; struct stat s; + unsigned int search_space , dtb_max_size; - if (argc != 3) { - fprintf(stderr, "Usage: %s \n", argv[0]); + if (argc <= 2 || argc > 4) { + fprintf(stderr, "Usage: %s [size]\n", argv[0]); goto err1; + } else if (argc == 3) { + fprintf(stdout, "DT size used is default of 16KB\n"); + search_space = dtb_max_size = DTB_MAX; + } else { + search_space = dtb_max_size = atoi(argv[3]); } + fddtb = open(argv[1], O_RDONLY); if (!fddtb) goto err1; @@ -53,24 +59,24 @@ int main(int argc, char **argv) } len = s.st_size; - if (len + 8 > DTB_MAX) { + if (len + 8 > dtb_max_size) { fprintf(stderr, "DTB too big\n"); goto err1; } if (((fddtb = open(argv[2], O_RDONLY)) < 0) || - (ptrdtb = (char *) mmap(0, DTB_MAX, PROT_READ, MAP_SHARED, fddtb, 0)) == (void *) (-1)) { + (ptrdtb = (char *) mmap(0, dtb_max_size, PROT_READ, MAP_SHARED, fddtb, 0)) == (void *) (-1)) { fprintf(stderr, "Could not open DTB"); goto err2; } if (((fd = open(argv[1], O_RDWR)) < 0) || - (ptr = (char *) mmap(0, SEARCH_SPACE + DTB_MAX, PROT_READ|PROT_WRITE, MAP_SHARED, fd, 0)) == (void *) (-1)) { + (ptr = (char *) mmap(0, search_space + dtb_max_size, PROT_READ|PROT_WRITE, MAP_SHARED, fd, 0)) == (void *) (-1)) { fprintf(stderr, "Could not open kernel image"); goto err3; } - for (p = ptr; p < (ptr + SEARCH_SPACE); p += 4) { + for (p = ptr; p < (ptr + search_space); p += 4) { if (memcmp(p, "OWRTDTB:", 8) == 0) { found = 1; p += 8; @@ -82,7 +88,7 @@ int main(int argc, char **argv) goto err4; } - memset(p, 0, DTB_MAX - 8); + memset(p, 0, dtb_max_size - 8); memcpy(p, ptrdtb, len); msync(p, len, MS_SYNC|MS_INVALIDATE); ret = 0;