* [PATCH RESEND v3] mmc-utils: Re-submit of the erase command addition plus remval of MMC_IOC_MULTI_CMD ifndef for erase. Re-committing the change per request by Avir.
@ 2021-04-22 16:12 luserhker
2021-04-23 5:19 ` Avri Altman
0 siblings, 1 reply; 8+ messages in thread
From: luserhker @ 2021-04-22 16:12 UTC (permalink / raw)
To: beanhuo, kenny.gibbons, kimito.sakata, rkamdar, chris,
ulf.hansson, avri.altman
Cc: linux-mmc, linux-kernel, Kimito Sakata
From: Kimito Sakata <kimito.sakata@oracle.com>
Signed-off-by: Kimito Sakata <ksakata@Kimitos-MBP.hsd1.co.comcast.net>
---
mmc.c | 8 ++++
mmc.h | 13 +++++-
mmc_cmds.c | 135 +++++++++++++++++++++++++++++++++++++++++++++++++++++
mmc_cmds.h | 1 +
4 files changed, 156 insertions(+), 1 deletion(-)
diff --git a/mmc.c b/mmc.c
index f3d724b..eb2638b 100644
--- a/mmc.c
+++ b/mmc.c
@@ -229,6 +229,14 @@ static struct Command commands[] = {
"Run Field Firmware Update with <image name> on <device>.\n",
NULL
},
+ { do_erase, -4,
+ "erase", "<type> " "<start address> " "<end address> " "<device>\n"
+ "Send Erase CMD38 with specific argument to the <device>\n\n"
+ "NOTE!: This will delete all user data in the specified region of the device\n"
+ "<type> must be: legacy | discard | secure-erase | "
+ "secure-trim1 | secure-trim2 | trim \n",
+ NULL
+ },
{ 0, 0, 0, 0 }
};
diff --git a/mmc.h b/mmc.h
index 5754a9d..e9766d7 100644
--- a/mmc.h
+++ b/mmc.h
@@ -35,7 +35,15 @@
#define MMC_SET_WRITE_PROT 28 /* ac [31:0] data addr R1b */
#define MMC_CLEAR_WRITE_PROT 29 /* ac [31:0] data addr R1b */
#define MMC_SEND_WRITE_PROT_TYPE 31 /* ac [31:0] data addr R1 */
-
+#define MMC_ERASE_GROUP_START 35 /* ac [31:0] data addr R1 */
+#define MMC_ERASE_GROUP_END 36 /* ac [31:0] data addr R1 */
+#define MMC_ERASE 38 /* ac [31] Secure request
+ [30:16] set to 0
+ [15] Force Garbage Collect request
+ [14:2] set to 0
+ [1] Discard Enable
+ [0] Identify Write Blocks for
+ Erase (or TRIM Enable) R1b */
/*
* EXT_CSD fields
*/
@@ -62,6 +70,7 @@
#define EXT_CSD_CACHE_SIZE_2 251
#define EXT_CSD_CACHE_SIZE_1 250
#define EXT_CSD_CACHE_SIZE_0 249
+#define EXT_CSD_SEC_FEATURE_SUPPORT 231
#define EXT_CSD_BOOT_INFO 228 /* R/W */
#define EXT_CSD_HC_ERASE_GRP_SIZE 224
#define EXT_CSD_HC_WP_GRP_SIZE 221
@@ -190,6 +199,8 @@
#define EXT_CSD_REV_V4_2 2
#define EXT_CSD_REV_V4_1 1
#define EXT_CSD_REV_V4_0 0
+#define EXT_CSD_SEC_GB_CL_EN (1<<4)
+#define EXT_CSD_SEC_ER_EN (1<<0)
/* From kernel linux/mmc/core.h */
diff --git a/mmc_cmds.c b/mmc_cmds.c
index 6c24cea..9340e3f 100644
--- a/mmc_cmds.c
+++ b/mmc_cmds.c
@@ -2514,6 +2514,141 @@ int do_cache_dis(int nargs, char **argv)
return do_cache_ctrl(0, nargs, argv);
}
+static int erase(int dev_fd, __u32 argin, __u32 start, __u32 end)
+{
+#ifndef MMC_IOC_MULTI_CMD
+ fprintf(stderr, "mmc-utils has been compiled without MMC_IOC_MULTI_CMD"
+ " support, needed by erase.\n");
+ return -ENOTSUP;
+#else
+ int ret = 0;
+ struct mmc_ioc_multi_cmd *multi_cmd;
+
+ multi_cmd = calloc(1, sizeof(struct mmc_ioc_multi_cmd) +
+ 3 * sizeof(struct mmc_ioc_cmd));
+ if (!multi_cmd) {
+ perror("Failed to allocate memory");
+ return -ENOMEM;
+ }
+
+ multi_cmd->num_of_cmds = 3;
+ /* Set erase start address */
+ multi_cmd->cmds[0].opcode = MMC_ERASE_GROUP_START;
+ multi_cmd->cmds[0].arg = start;
+ multi_cmd->cmds[0].flags = MMC_RSP_SPI_R1 | MMC_RSP_R1 | MMC_CMD_AC;
+ multi_cmd->cmds[0].write_flag = 1;
+
+ /* Set erase end address */
+ multi_cmd->cmds[1].opcode = MMC_ERASE_GROUP_END;
+ multi_cmd->cmds[1].arg = end;
+ multi_cmd->cmds[1].flags = MMC_RSP_SPI_R1 | MMC_RSP_R1 | MMC_CMD_AC;
+ multi_cmd->cmds[1].write_flag = 1;
+
+ /* Send Erase Command */
+ multi_cmd->cmds[2].opcode = MMC_ERASE;
+ multi_cmd->cmds[2].arg = argin;
+ multi_cmd->cmds[2].cmd_timeout_ms = 300*255*255;
+ multi_cmd->cmds[2].flags = MMC_RSP_SPI_R1B | MMC_RSP_R1B | MMC_CMD_AC;
+ multi_cmd->cmds[2].write_flag = 1;
+
+ /* send erase cmd with multi-cmd */
+ ret = ioctl(dev_fd, MMC_IOC_MULTI_CMD, multi_cmd);
+ if (ret)
+ perror("Erase multi-cmd ioctl");
+
+ free(multi_cmd);
+ return ret;
+#endif
+}
+
+int do_erase(int nargs, char **argv)
+{
+ int dev_fd, ret;
+ char *print_str;
+ char **eptr = NULL;
+ __u8 ext_csd[512], checkup_mask = 0;
+ __u32 arg, start, end;
+
+ if (nargs != 5) {
+ fprintf(stderr, "Usage: erase <type> <start addr> <end addr> </path/to/mmcblkX>\n");
+ exit(1);
+ }
+
+ if (strstr(argv[2], "0x") || strstr(argv[2], "0X"))
+ start = strtol(argv[2], eptr, 16);
+ else
+ start = strtol(argv[2], eptr, 10);
+
+ if (strstr(argv[3], "0x") || strstr(argv[3], "0X"))
+ end = strtol(argv[3], eptr, 16);
+ else
+ end = strtol(argv[3], eptr, 10);
+
+ if (end < start) {
+ fprintf(stderr, "erase start [0x%08x] > erase end [0x%08x]\n",
+ start, end);
+ exit(1);
+ }
+
+ if (strcmp(argv[1], "legacy") == 0) {
+ arg = 0x00000000;
+ print_str = "Legacy Erase";
+ } else if (strcmp(argv[1], "discard") == 0) {
+ arg = 0x00000003;
+ print_str = "Discard";
+ } else if (strcmp(argv[1], "secure-erase") == 0) {
+ print_str = "Secure Erase";
+ checkup_mask = EXT_CSD_SEC_ER_EN;
+ arg = 0x80000000;
+ } else if (strcmp(argv[1], "secure-trim1") == 0) {
+ print_str = "Secure Trim Step 1";
+ checkup_mask = EXT_CSD_SEC_ER_EN | EXT_CSD_SEC_GB_CL_EN;
+ arg = 0x80000001;
+ } else if (strcmp(argv[1], "secure-trim2") == 0) {
+ print_str = "Secure Trim Step 2";
+ checkup_mask = EXT_CSD_SEC_ER_EN | EXT_CSD_SEC_GB_CL_EN;
+ arg = 0x80008000;
+ } else if (strcmp(argv[1], "trim") == 0) {
+ print_str = "Trim";
+ checkup_mask = EXT_CSD_SEC_GB_CL_EN;
+ arg = 0x00000001;
+ } else {
+ fprintf(stderr, "Unknown erase type: %s\n", argv[1]);
+ exit(1);
+ }
+
+ dev_fd = open(argv[4], O_RDWR);
+ if (dev_fd < 0) {
+ perror(argv[4]);
+ exit(1);
+ }
+
+ if (checkup_mask) {
+ ret = read_extcsd(dev_fd, ext_csd);
+ if (ret) {
+ fprintf(stderr, "Could not read EXT_CSD from %s\n",
+ argv[4]);
+ goto out;
+ }
+ if ((checkup_mask & ext_csd[EXT_CSD_SEC_FEATURE_SUPPORT]) !=
+ checkup_mask) {
+ fprintf(stderr, "%s is not supported in %s\n",
+ print_str, argv[4]);
+ ret = -ENOTSUP;
+ goto out;
+ }
+
+ }
+ printf("Executing %s from 0x%08x to 0x%08x\n", print_str, start, end);
+
+ ret = erase(dev_fd, arg, start, end);
+out:
+ printf(" %s %s!\n\n", print_str, ret ? "Failed" : "Succeed");
+ close(dev_fd);
+ return ret;
+}
+
+
int do_ffu(int nargs, char **argv)
{
#ifndef MMC_IOC_MULTI_CMD
diff --git a/mmc_cmds.h b/mmc_cmds.h
index 9d3246c..8331ab2 100644
--- a/mmc_cmds.h
+++ b/mmc_cmds.h
@@ -45,3 +45,4 @@ int do_ffu(int nargs, char **argv);
int do_read_scr(int argc, char **argv);
int do_read_cid(int argc, char **argv);
int do_read_csd(int argc, char **argv);
+int do_erase(int nargs, char **argv);
--
2.24.1 (Apple Git-126)
^ permalink raw reply related [flat|nested] 8+ messages in thread
* RE: [PATCH RESEND v3] mmc-utils: Re-submit of the erase command addition plus remval of MMC_IOC_MULTI_CMD ifndef for erase. Re-committing the change per request by Avir.
2021-04-22 16:12 [PATCH RESEND v3] mmc-utils: Re-submit of the erase command addition plus remval of MMC_IOC_MULTI_CMD ifndef for erase. Re-committing the change per request by Avir luserhker
@ 2021-04-23 5:19 ` Avri Altman
2021-04-23 5:31 ` Avri Altman
2021-04-23 14:23 ` [External] : " kimito.sakata
0 siblings, 2 replies; 8+ messages in thread
From: Avri Altman @ 2021-04-23 5:19 UTC (permalink / raw)
To: luserhker, beanhuo, kenny.gibbons, kimito.sakata, rkamdar, chris,
ulf.hansson
Cc: linux-mmc, linux-kernel, Kimito Sakata
>
> From: Kimito Sakata <kimito.sakata@oracle.com>
>
> Signed-off-by: Kimito Sakata <ksakata@Kimitos-MBP.hsd1.co.comcast.net>
Hi Kimito,
Please use a proper subject and commit log body.
If you don't understand the difference between those two - please ask.
Also for consistency, you might want to use your oracle mail for your Signed-off-by tag.
You need to change your git configs for that.
Thanks,
Avri
> ---
> mmc.c | 8 ++++
> mmc.h | 13 +++++-
> mmc_cmds.c | 135
> +++++++++++++++++++++++++++++++++++++++++++++++++++++
> mmc_cmds.h | 1 +
> 4 files changed, 156 insertions(+), 1 deletion(-)
>
> diff --git a/mmc.c b/mmc.c
> index f3d724b..eb2638b 100644
> --- a/mmc.c
> +++ b/mmc.c
> @@ -229,6 +229,14 @@ static struct Command commands[] = {
> "Run Field Firmware Update with <image name> on <device>.\n",
> NULL
> },
> + { do_erase, -4,
> + "erase", "<type> " "<start address> " "<end address> " "<device>\n"
> + "Send Erase CMD38 with specific argument to the <device>\n\n"
> + "NOTE!: This will delete all user data in the specified region of the
> device\n"
> + "<type> must be: legacy | discard | secure-erase | "
> + "secure-trim1 | secure-trim2 | trim \n",
> + NULL
> + },
> { 0, 0, 0, 0 }
> };
>
> diff --git a/mmc.h b/mmc.h
> index 5754a9d..e9766d7 100644
> --- a/mmc.h
> +++ b/mmc.h
> @@ -35,7 +35,15 @@
> #define MMC_SET_WRITE_PROT 28 /* ac [31:0] data addr R1b */
> #define MMC_CLEAR_WRITE_PROT 29 /* ac [31:0] data addr R1b */
> #define MMC_SEND_WRITE_PROT_TYPE 31 /* ac [31:0] data addr R1 */
> -
> +#define MMC_ERASE_GROUP_START 35 /* ac [31:0] data addr R1 */
> +#define MMC_ERASE_GROUP_END 36 /* ac [31:0] data addr R1 */
> +#define MMC_ERASE 38 /* ac [31] Secure request
> + [30:16] set to 0
> + [15] Force Garbage Collect request
> + [14:2] set to 0
> + [1] Discard Enable
> + [0] Identify Write Blocks for
> + Erase (or TRIM Enable) R1b */
> /*
> * EXT_CSD fields
> */
> @@ -62,6 +70,7 @@
> #define EXT_CSD_CACHE_SIZE_2 251
> #define EXT_CSD_CACHE_SIZE_1 250
> #define EXT_CSD_CACHE_SIZE_0 249
> +#define EXT_CSD_SEC_FEATURE_SUPPORT 231
> #define EXT_CSD_BOOT_INFO 228 /* R/W */
> #define EXT_CSD_HC_ERASE_GRP_SIZE 224
> #define EXT_CSD_HC_WP_GRP_SIZE 221
> @@ -190,6 +199,8 @@
> #define EXT_CSD_REV_V4_2 2
> #define EXT_CSD_REV_V4_1 1
> #define EXT_CSD_REV_V4_0 0
> +#define EXT_CSD_SEC_GB_CL_EN (1<<4)
> +#define EXT_CSD_SEC_ER_EN (1<<0)
>
>
> /* From kernel linux/mmc/core.h */
> diff --git a/mmc_cmds.c b/mmc_cmds.c
> index 6c24cea..9340e3f 100644
> --- a/mmc_cmds.c
> +++ b/mmc_cmds.c
> @@ -2514,6 +2514,141 @@ int do_cache_dis(int nargs, char **argv)
> return do_cache_ctrl(0, nargs, argv);
> }
>
> +static int erase(int dev_fd, __u32 argin, __u32 start, __u32 end)
> +{
> +#ifndef MMC_IOC_MULTI_CMD
> + fprintf(stderr, "mmc-utils has been compiled without
> MMC_IOC_MULTI_CMD"
> + " support, needed by erase.\n");
> + return -ENOTSUP;
> +#else
> + int ret = 0;
> + struct mmc_ioc_multi_cmd *multi_cmd;
> +
> + multi_cmd = calloc(1, sizeof(struct mmc_ioc_multi_cmd) +
> + 3 * sizeof(struct mmc_ioc_cmd));
> + if (!multi_cmd) {
> + perror("Failed to allocate memory");
> + return -ENOMEM;
> + }
> +
> + multi_cmd->num_of_cmds = 3;
> + /* Set erase start address */
> + multi_cmd->cmds[0].opcode = MMC_ERASE_GROUP_START;
> + multi_cmd->cmds[0].arg = start;
> + multi_cmd->cmds[0].flags = MMC_RSP_SPI_R1 | MMC_RSP_R1 |
> MMC_CMD_AC;
> + multi_cmd->cmds[0].write_flag = 1;
> +
> + /* Set erase end address */
> + multi_cmd->cmds[1].opcode = MMC_ERASE_GROUP_END;
> + multi_cmd->cmds[1].arg = end;
> + multi_cmd->cmds[1].flags = MMC_RSP_SPI_R1 | MMC_RSP_R1 |
> MMC_CMD_AC;
> + multi_cmd->cmds[1].write_flag = 1;
> +
> + /* Send Erase Command */
> + multi_cmd->cmds[2].opcode = MMC_ERASE;
> + multi_cmd->cmds[2].arg = argin;
> + multi_cmd->cmds[2].cmd_timeout_ms = 300*255*255;
> + multi_cmd->cmds[2].flags = MMC_RSP_SPI_R1B | MMC_RSP_R1B |
> MMC_CMD_AC;
> + multi_cmd->cmds[2].write_flag = 1;
> +
> + /* send erase cmd with multi-cmd */
> + ret = ioctl(dev_fd, MMC_IOC_MULTI_CMD, multi_cmd);
> + if (ret)
> + perror("Erase multi-cmd ioctl");
> +
> + free(multi_cmd);
> + return ret;
> +#endif
> +}
> +
> +int do_erase(int nargs, char **argv)
> +{
> + int dev_fd, ret;
> + char *print_str;
> + char **eptr = NULL;
> + __u8 ext_csd[512], checkup_mask = 0;
> + __u32 arg, start, end;
> +
> + if (nargs != 5) {
> + fprintf(stderr, "Usage: erase <type> <start addr> <end addr>
> </path/to/mmcblkX>\n");
> + exit(1);
> + }
> +
> + if (strstr(argv[2], "0x") || strstr(argv[2], "0X"))
> + start = strtol(argv[2], eptr, 16);
> + else
> + start = strtol(argv[2], eptr, 10);
> +
> + if (strstr(argv[3], "0x") || strstr(argv[3], "0X"))
> + end = strtol(argv[3], eptr, 16);
> + else
> + end = strtol(argv[3], eptr, 10);
> +
> + if (end < start) {
> + fprintf(stderr, "erase start [0x%08x] > erase end [0x%08x]\n",
> + start, end);
> + exit(1);
> + }
> +
> + if (strcmp(argv[1], "legacy") == 0) {
> + arg = 0x00000000;
> + print_str = "Legacy Erase";
> + } else if (strcmp(argv[1], "discard") == 0) {
> + arg = 0x00000003;
> + print_str = "Discard";
> + } else if (strcmp(argv[1], "secure-erase") == 0) {
> + print_str = "Secure Erase";
> + checkup_mask = EXT_CSD_SEC_ER_EN;
> + arg = 0x80000000;
> + } else if (strcmp(argv[1], "secure-trim1") == 0) {
> + print_str = "Secure Trim Step 1";
> + checkup_mask = EXT_CSD_SEC_ER_EN | EXT_CSD_SEC_GB_CL_EN;
> + arg = 0x80000001;
> + } else if (strcmp(argv[1], "secure-trim2") == 0) {
> + print_str = "Secure Trim Step 2";
> + checkup_mask = EXT_CSD_SEC_ER_EN | EXT_CSD_SEC_GB_CL_EN;
> + arg = 0x80008000;
> + } else if (strcmp(argv[1], "trim") == 0) {
> + print_str = "Trim";
> + checkup_mask = EXT_CSD_SEC_GB_CL_EN;
> + arg = 0x00000001;
> + } else {
> + fprintf(stderr, "Unknown erase type: %s\n", argv[1]);
> + exit(1);
> + }
> +
> + dev_fd = open(argv[4], O_RDWR);
> + if (dev_fd < 0) {
> + perror(argv[4]);
> + exit(1);
> + }
> +
> + if (checkup_mask) {
> + ret = read_extcsd(dev_fd, ext_csd);
> + if (ret) {
> + fprintf(stderr, "Could not read EXT_CSD from %s\n",
> + argv[4]);
> + goto out;
> + }
> + if ((checkup_mask & ext_csd[EXT_CSD_SEC_FEATURE_SUPPORT]) !=
> + checkup_mask) {
> + fprintf(stderr, "%s is not supported in %s\n",
> + print_str, argv[4]);
> + ret = -ENOTSUP;
> + goto out;
> + }
> +
> + }
> + printf("Executing %s from 0x%08x to 0x%08x\n", print_str, start, end);
> +
> + ret = erase(dev_fd, arg, start, end);
> +out:
> + printf(" %s %s!\n\n", print_str, ret ? "Failed" : "Succeed");
> + close(dev_fd);
> + return ret;
> +}
> +
> +
> int do_ffu(int nargs, char **argv)
> {
> #ifndef MMC_IOC_MULTI_CMD
> diff --git a/mmc_cmds.h b/mmc_cmds.h
> index 9d3246c..8331ab2 100644
> --- a/mmc_cmds.h
> +++ b/mmc_cmds.h
> @@ -45,3 +45,4 @@ int do_ffu(int nargs, char **argv);
> int do_read_scr(int argc, char **argv);
> int do_read_cid(int argc, char **argv);
> int do_read_csd(int argc, char **argv);
> +int do_erase(int nargs, char **argv);
> --
> 2.24.1 (Apple Git-126)
^ permalink raw reply [flat|nested] 8+ messages in thread
* RE: [PATCH RESEND v3] mmc-utils: Re-submit of the erase command addition plus remval of MMC_IOC_MULTI_CMD ifndef for erase. Re-committing the change per request by Avir.
2021-04-23 5:19 ` Avri Altman
@ 2021-04-23 5:31 ` Avri Altman
2021-04-23 14:27 ` [External] : " kimito.sakata
2021-04-26 9:36 ` Bean Huo
2021-04-23 14:23 ` [External] : " kimito.sakata
1 sibling, 2 replies; 8+ messages in thread
From: Avri Altman @ 2021-04-23 5:31 UTC (permalink / raw)
To: luserhker, beanhuo, kenny.gibbons, kimito.sakata, rkamdar, chris,
ulf.hansson
Cc: linux-mmc, linux-kernel, Kimito Sakata
+Bean
> > From: Kimito Sakata <kimito.sakata@oracle.com>
> >
> > Signed-off-by: Kimito Sakata <ksakata@Kimitos-MBP.hsd1.co.comcast.net>
> Hi Kimito,
> Please use a proper subject and commit log body.
> If you don't understand the difference between those two - please ask.
> Also for consistency, you might want to use your oracle mail for your Signed-
> off-by tag.
> You need to change your git configs for that.
Also, can you refer to this patch submitted by Bean - https://www.spinics.net/lists/linux-mmc/msg63582.html
Are you co-developing this together?
Thanks,
Avri
>
> Thanks,
> Avri
>
> > ---
> > mmc.c | 8 ++++
> > mmc.h | 13 +++++-
> > mmc_cmds.c | 135
> > +++++++++++++++++++++++++++++++++++++++++++++++++++++
> > mmc_cmds.h | 1 +
> > 4 files changed, 156 insertions(+), 1 deletion(-)
> >
> > diff --git a/mmc.c b/mmc.c
> > index f3d724b..eb2638b 100644
> > --- a/mmc.c
> > +++ b/mmc.c
> > @@ -229,6 +229,14 @@ static struct Command commands[] = {
> > "Run Field Firmware Update with <image name> on <device>.\n",
> > NULL
> > },
> > + { do_erase, -4,
> > + "erase", "<type> " "<start address> " "<end address> " "<device>\n"
> > + "Send Erase CMD38 with specific argument to the <device>\n\n"
> > + "NOTE!: This will delete all user data in the specified region of the
> > device\n"
> > + "<type> must be: legacy | discard | secure-erase | "
> > + "secure-trim1 | secure-trim2 | trim \n",
> > + NULL
> > + },
> > { 0, 0, 0, 0 }
> > };
> >
> > diff --git a/mmc.h b/mmc.h
> > index 5754a9d..e9766d7 100644
> > --- a/mmc.h
> > +++ b/mmc.h
> > @@ -35,7 +35,15 @@
> > #define MMC_SET_WRITE_PROT 28 /* ac [31:0] data addr R1b */
> > #define MMC_CLEAR_WRITE_PROT 29 /* ac [31:0] data addr R1b */
> > #define MMC_SEND_WRITE_PROT_TYPE 31 /* ac [31:0] data addr R1 */
> > -
> > +#define MMC_ERASE_GROUP_START 35 /* ac [31:0] data addr R1 */
> > +#define MMC_ERASE_GROUP_END 36 /* ac [31:0] data addr R1 */
> > +#define MMC_ERASE 38 /* ac [31] Secure request
> > + [30:16] set to 0
> > + [15] Force Garbage Collect request
> > + [14:2] set to 0
> > + [1] Discard Enable
> > + [0] Identify Write Blocks for
> > + Erase (or TRIM Enable) R1b */
> > /*
> > * EXT_CSD fields
> > */
> > @@ -62,6 +70,7 @@
> > #define EXT_CSD_CACHE_SIZE_2 251
> > #define EXT_CSD_CACHE_SIZE_1 250
> > #define EXT_CSD_CACHE_SIZE_0 249
> > +#define EXT_CSD_SEC_FEATURE_SUPPORT 231
> > #define EXT_CSD_BOOT_INFO 228 /* R/W */
> > #define EXT_CSD_HC_ERASE_GRP_SIZE 224
> > #define EXT_CSD_HC_WP_GRP_SIZE 221
> > @@ -190,6 +199,8 @@
> > #define EXT_CSD_REV_V4_2 2
> > #define EXT_CSD_REV_V4_1 1
> > #define EXT_CSD_REV_V4_0 0
> > +#define EXT_CSD_SEC_GB_CL_EN (1<<4)
> > +#define EXT_CSD_SEC_ER_EN (1<<0)
> >
> >
> > /* From kernel linux/mmc/core.h */
> > diff --git a/mmc_cmds.c b/mmc_cmds.c
> > index 6c24cea..9340e3f 100644
> > --- a/mmc_cmds.c
> > +++ b/mmc_cmds.c
> > @@ -2514,6 +2514,141 @@ int do_cache_dis(int nargs, char **argv)
> > return do_cache_ctrl(0, nargs, argv);
> > }
> >
> > +static int erase(int dev_fd, __u32 argin, __u32 start, __u32 end)
> > +{
> > +#ifndef MMC_IOC_MULTI_CMD
> > + fprintf(stderr, "mmc-utils has been compiled without
> > MMC_IOC_MULTI_CMD"
> > + " support, needed by erase.\n");
> > + return -ENOTSUP;
> > +#else
> > + int ret = 0;
> > + struct mmc_ioc_multi_cmd *multi_cmd;
> > +
> > + multi_cmd = calloc(1, sizeof(struct mmc_ioc_multi_cmd) +
> > + 3 * sizeof(struct mmc_ioc_cmd));
> > + if (!multi_cmd) {
> > + perror("Failed to allocate memory");
> > + return -ENOMEM;
> > + }
> > +
> > + multi_cmd->num_of_cmds = 3;
> > + /* Set erase start address */
> > + multi_cmd->cmds[0].opcode = MMC_ERASE_GROUP_START;
> > + multi_cmd->cmds[0].arg = start;
> > + multi_cmd->cmds[0].flags = MMC_RSP_SPI_R1 | MMC_RSP_R1 |
> > MMC_CMD_AC;
> > + multi_cmd->cmds[0].write_flag = 1;
> > +
> > + /* Set erase end address */
> > + multi_cmd->cmds[1].opcode = MMC_ERASE_GROUP_END;
> > + multi_cmd->cmds[1].arg = end;
> > + multi_cmd->cmds[1].flags = MMC_RSP_SPI_R1 | MMC_RSP_R1 |
> > MMC_CMD_AC;
> > + multi_cmd->cmds[1].write_flag = 1;
> > +
> > + /* Send Erase Command */
> > + multi_cmd->cmds[2].opcode = MMC_ERASE;
> > + multi_cmd->cmds[2].arg = argin;
> > + multi_cmd->cmds[2].cmd_timeout_ms = 300*255*255;
> > + multi_cmd->cmds[2].flags = MMC_RSP_SPI_R1B | MMC_RSP_R1B |
> > MMC_CMD_AC;
> > + multi_cmd->cmds[2].write_flag = 1;
> > +
> > + /* send erase cmd with multi-cmd */
> > + ret = ioctl(dev_fd, MMC_IOC_MULTI_CMD, multi_cmd);
> > + if (ret)
> > + perror("Erase multi-cmd ioctl");
> > +
> > + free(multi_cmd);
> > + return ret;
> > +#endif
> > +}
> > +
> > +int do_erase(int nargs, char **argv)
> > +{
> > + int dev_fd, ret;
> > + char *print_str;
> > + char **eptr = NULL;
> > + __u8 ext_csd[512], checkup_mask = 0;
> > + __u32 arg, start, end;
> > +
> > + if (nargs != 5) {
> > + fprintf(stderr, "Usage: erase <type> <start addr> <end addr>
> > </path/to/mmcblkX>\n");
> > + exit(1);
> > + }
> > +
> > + if (strstr(argv[2], "0x") || strstr(argv[2], "0X"))
> > + start = strtol(argv[2], eptr, 16);
> > + else
> > + start = strtol(argv[2], eptr, 10);
> > +
> > + if (strstr(argv[3], "0x") || strstr(argv[3], "0X"))
> > + end = strtol(argv[3], eptr, 16);
> > + else
> > + end = strtol(argv[3], eptr, 10);
> > +
> > + if (end < start) {
> > + fprintf(stderr, "erase start [0x%08x] > erase end [0x%08x]\n",
> > + start, end);
> > + exit(1);
> > + }
> > +
> > + if (strcmp(argv[1], "legacy") == 0) {
> > + arg = 0x00000000;
> > + print_str = "Legacy Erase";
> > + } else if (strcmp(argv[1], "discard") == 0) {
> > + arg = 0x00000003;
> > + print_str = "Discard";
> > + } else if (strcmp(argv[1], "secure-erase") == 0) {
> > + print_str = "Secure Erase";
> > + checkup_mask = EXT_CSD_SEC_ER_EN;
> > + arg = 0x80000000;
> > + } else if (strcmp(argv[1], "secure-trim1") == 0) {
> > + print_str = "Secure Trim Step 1";
> > + checkup_mask = EXT_CSD_SEC_ER_EN | EXT_CSD_SEC_GB_CL_EN;
> > + arg = 0x80000001;
> > + } else if (strcmp(argv[1], "secure-trim2") == 0) {
> > + print_str = "Secure Trim Step 2";
> > + checkup_mask = EXT_CSD_SEC_ER_EN | EXT_CSD_SEC_GB_CL_EN;
> > + arg = 0x80008000;
> > + } else if (strcmp(argv[1], "trim") == 0) {
> > + print_str = "Trim";
> > + checkup_mask = EXT_CSD_SEC_GB_CL_EN;
> > + arg = 0x00000001;
> > + } else {
> > + fprintf(stderr, "Unknown erase type: %s\n", argv[1]);
> > + exit(1);
> > + }
> > +
> > + dev_fd = open(argv[4], O_RDWR);
> > + if (dev_fd < 0) {
> > + perror(argv[4]);
> > + exit(1);
> > + }
> > +
> > + if (checkup_mask) {
> > + ret = read_extcsd(dev_fd, ext_csd);
> > + if (ret) {
> > + fprintf(stderr, "Could not read EXT_CSD from %s\n",
> > + argv[4]);
> > + goto out;
> > + }
> > + if ((checkup_mask & ext_csd[EXT_CSD_SEC_FEATURE_SUPPORT]) !=
> > + checkup_mask) {
> > + fprintf(stderr, "%s is not supported in %s\n",
> > + print_str, argv[4]);
> > + ret = -ENOTSUP;
> > + goto out;
> > + }
> > +
> > + }
> > + printf("Executing %s from 0x%08x to 0x%08x\n", print_str, start, end);
> > +
> > + ret = erase(dev_fd, arg, start, end);
> > +out:
> > + printf(" %s %s!\n\n", print_str, ret ? "Failed" : "Succeed");
> > + close(dev_fd);
> > + return ret;
> > +}
> > +
> > +
> > int do_ffu(int nargs, char **argv)
> > {
> > #ifndef MMC_IOC_MULTI_CMD
> > diff --git a/mmc_cmds.h b/mmc_cmds.h
> > index 9d3246c..8331ab2 100644
> > --- a/mmc_cmds.h
> > +++ b/mmc_cmds.h
> > @@ -45,3 +45,4 @@ int do_ffu(int nargs, char **argv);
> > int do_read_scr(int argc, char **argv);
> > int do_read_cid(int argc, char **argv);
> > int do_read_csd(int argc, char **argv);
> > +int do_erase(int nargs, char **argv);
> > --
> > 2.24.1 (Apple Git-126)
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [External] : RE: [PATCH RESEND v3] mmc-utils: Re-submit of the erase command addition plus remval of MMC_IOC_MULTI_CMD ifndef for erase. Re-committing the change per request by Avir.
2021-04-23 5:19 ` Avri Altman
2021-04-23 5:31 ` Avri Altman
@ 2021-04-23 14:23 ` kimito.sakata
1 sibling, 0 replies; 8+ messages in thread
From: kimito.sakata @ 2021-04-23 14:23 UTC (permalink / raw)
To: Avri Altman, luserhker, beanhuo, kenny.gibbons, rkamdar, chris,
ulf.hansson
Cc: linux-mmc, linux-kernel, Kimito Sakata
On 4/22/2021 11:19 PM, Avri Altman wrote:
>> From: Kimito Sakata <kimito.sakata@oracle.com>
>>
>> Signed-off-by: Kimito Sakata <ksakata@Kimitos-MBP.hsd1.co.comcast.net>
> Hi Kimito,
> Please use a proper subject and commit log body.
> If you don't understand the difference between those two - please ask.
I don't. I would appreciate an explanation. New to submitting changes
via git.
> Also for consistency, you might want to use your oracle mail for your Signed-off-by tag.
> You need to change your git configs for that.
I tried all kinds of firewall settings and no go via my work network.
Is why I ended up using my personal gmail smtp. Is that a problem?
Thanks
Kimito
>
> Thanks,
> Avri
>
>> ---
>> mmc.c | 8 ++++
>> mmc.h | 13 +++++-
>> mmc_cmds.c | 135
>> +++++++++++++++++++++++++++++++++++++++++++++++++++++
>> mmc_cmds.h | 1 +
>> 4 files changed, 156 insertions(+), 1 deletion(-)
>>
>> diff --git a/mmc.c b/mmc.c
>> index f3d724b..eb2638b 100644
>> --- a/mmc.c
>> +++ b/mmc.c
>> @@ -229,6 +229,14 @@ static struct Command commands[] = {
>> "Run Field Firmware Update with <image name> on <device>.\n",
>> NULL
>> },
>> + { do_erase, -4,
>> + "erase", "<type> " "<start address> " "<end address> " "<device>\n"
>> + "Send Erase CMD38 with specific argument to the <device>\n\n"
>> + "NOTE!: This will delete all user data in the specified region of the
>> device\n"
>> + "<type> must be: legacy | discard | secure-erase | "
>> + "secure-trim1 | secure-trim2 | trim \n",
>> + NULL
>> + },
>> { 0, 0, 0, 0 }
>> };
>>
>> diff --git a/mmc.h b/mmc.h
>> index 5754a9d..e9766d7 100644
>> --- a/mmc.h
>> +++ b/mmc.h
>> @@ -35,7 +35,15 @@
>> #define MMC_SET_WRITE_PROT 28 /* ac [31:0] data addr R1b */
>> #define MMC_CLEAR_WRITE_PROT 29 /* ac [31:0] data addr R1b */
>> #define MMC_SEND_WRITE_PROT_TYPE 31 /* ac [31:0] data addr R1 */
>> -
>> +#define MMC_ERASE_GROUP_START 35 /* ac [31:0] data addr R1 */
>> +#define MMC_ERASE_GROUP_END 36 /* ac [31:0] data addr R1 */
>> +#define MMC_ERASE 38 /* ac [31] Secure request
>> + [30:16] set to 0
>> + [15] Force Garbage Collect request
>> + [14:2] set to 0
>> + [1] Discard Enable
>> + [0] Identify Write Blocks for
>> + Erase (or TRIM Enable) R1b */
>> /*
>> * EXT_CSD fields
>> */
>> @@ -62,6 +70,7 @@
>> #define EXT_CSD_CACHE_SIZE_2 251
>> #define EXT_CSD_CACHE_SIZE_1 250
>> #define EXT_CSD_CACHE_SIZE_0 249
>> +#define EXT_CSD_SEC_FEATURE_SUPPORT 231
>> #define EXT_CSD_BOOT_INFO 228 /* R/W */
>> #define EXT_CSD_HC_ERASE_GRP_SIZE 224
>> #define EXT_CSD_HC_WP_GRP_SIZE 221
>> @@ -190,6 +199,8 @@
>> #define EXT_CSD_REV_V4_2 2
>> #define EXT_CSD_REV_V4_1 1
>> #define EXT_CSD_REV_V4_0 0
>> +#define EXT_CSD_SEC_GB_CL_EN (1<<4)
>> +#define EXT_CSD_SEC_ER_EN (1<<0)
>>
>>
>> /* From kernel linux/mmc/core.h */
>> diff --git a/mmc_cmds.c b/mmc_cmds.c
>> index 6c24cea..9340e3f 100644
>> --- a/mmc_cmds.c
>> +++ b/mmc_cmds.c
>> @@ -2514,6 +2514,141 @@ int do_cache_dis(int nargs, char **argv)
>> return do_cache_ctrl(0, nargs, argv);
>> }
>>
>> +static int erase(int dev_fd, __u32 argin, __u32 start, __u32 end)
>> +{
>> +#ifndef MMC_IOC_MULTI_CMD
>> + fprintf(stderr, "mmc-utils has been compiled without
>> MMC_IOC_MULTI_CMD"
>> + " support, needed by erase.\n");
>> + return -ENOTSUP;
>> +#else
>> + int ret = 0;
>> + struct mmc_ioc_multi_cmd *multi_cmd;
>> +
>> + multi_cmd = calloc(1, sizeof(struct mmc_ioc_multi_cmd) +
>> + 3 * sizeof(struct mmc_ioc_cmd));
>> + if (!multi_cmd) {
>> + perror("Failed to allocate memory");
>> + return -ENOMEM;
>> + }
>> +
>> + multi_cmd->num_of_cmds = 3;
>> + /* Set erase start address */
>> + multi_cmd->cmds[0].opcode = MMC_ERASE_GROUP_START;
>> + multi_cmd->cmds[0].arg = start;
>> + multi_cmd->cmds[0].flags = MMC_RSP_SPI_R1 | MMC_RSP_R1 |
>> MMC_CMD_AC;
>> + multi_cmd->cmds[0].write_flag = 1;
>> +
>> + /* Set erase end address */
>> + multi_cmd->cmds[1].opcode = MMC_ERASE_GROUP_END;
>> + multi_cmd->cmds[1].arg = end;
>> + multi_cmd->cmds[1].flags = MMC_RSP_SPI_R1 | MMC_RSP_R1 |
>> MMC_CMD_AC;
>> + multi_cmd->cmds[1].write_flag = 1;
>> +
>> + /* Send Erase Command */
>> + multi_cmd->cmds[2].opcode = MMC_ERASE;
>> + multi_cmd->cmds[2].arg = argin;
>> + multi_cmd->cmds[2].cmd_timeout_ms = 300*255*255;
>> + multi_cmd->cmds[2].flags = MMC_RSP_SPI_R1B | MMC_RSP_R1B |
>> MMC_CMD_AC;
>> + multi_cmd->cmds[2].write_flag = 1;
>> +
>> + /* send erase cmd with multi-cmd */
>> + ret = ioctl(dev_fd, MMC_IOC_MULTI_CMD, multi_cmd);
>> + if (ret)
>> + perror("Erase multi-cmd ioctl");
>> +
>> + free(multi_cmd);
>> + return ret;
>> +#endif
>> +}
>> +
>> +int do_erase(int nargs, char **argv)
>> +{
>> + int dev_fd, ret;
>> + char *print_str;
>> + char **eptr = NULL;
>> + __u8 ext_csd[512], checkup_mask = 0;
>> + __u32 arg, start, end;
>> +
>> + if (nargs != 5) {
>> + fprintf(stderr, "Usage: erase <type> <start addr> <end addr>
>> </path/to/mmcblkX>\n");
>> + exit(1);
>> + }
>> +
>> + if (strstr(argv[2], "0x") || strstr(argv[2], "0X"))
>> + start = strtol(argv[2], eptr, 16);
>> + else
>> + start = strtol(argv[2], eptr, 10);
>> +
>> + if (strstr(argv[3], "0x") || strstr(argv[3], "0X"))
>> + end = strtol(argv[3], eptr, 16);
>> + else
>> + end = strtol(argv[3], eptr, 10);
>> +
>> + if (end < start) {
>> + fprintf(stderr, "erase start [0x%08x] > erase end [0x%08x]\n",
>> + start, end);
>> + exit(1);
>> + }
>> +
>> + if (strcmp(argv[1], "legacy") == 0) {
>> + arg = 0x00000000;
>> + print_str = "Legacy Erase";
>> + } else if (strcmp(argv[1], "discard") == 0) {
>> + arg = 0x00000003;
>> + print_str = "Discard";
>> + } else if (strcmp(argv[1], "secure-erase") == 0) {
>> + print_str = "Secure Erase";
>> + checkup_mask = EXT_CSD_SEC_ER_EN;
>> + arg = 0x80000000;
>> + } else if (strcmp(argv[1], "secure-trim1") == 0) {
>> + print_str = "Secure Trim Step 1";
>> + checkup_mask = EXT_CSD_SEC_ER_EN | EXT_CSD_SEC_GB_CL_EN;
>> + arg = 0x80000001;
>> + } else if (strcmp(argv[1], "secure-trim2") == 0) {
>> + print_str = "Secure Trim Step 2";
>> + checkup_mask = EXT_CSD_SEC_ER_EN | EXT_CSD_SEC_GB_CL_EN;
>> + arg = 0x80008000;
>> + } else if (strcmp(argv[1], "trim") == 0) {
>> + print_str = "Trim";
>> + checkup_mask = EXT_CSD_SEC_GB_CL_EN;
>> + arg = 0x00000001;
>> + } else {
>> + fprintf(stderr, "Unknown erase type: %s\n", argv[1]);
>> + exit(1);
>> + }
>> +
>> + dev_fd = open(argv[4], O_RDWR);
>> + if (dev_fd < 0) {
>> + perror(argv[4]);
>> + exit(1);
>> + }
>> +
>> + if (checkup_mask) {
>> + ret = read_extcsd(dev_fd, ext_csd);
>> + if (ret) {
>> + fprintf(stderr, "Could not read EXT_CSD from %s\n",
>> + argv[4]);
>> + goto out;
>> + }
>> + if ((checkup_mask & ext_csd[EXT_CSD_SEC_FEATURE_SUPPORT]) !=
>> + checkup_mask) {
>> + fprintf(stderr, "%s is not supported in %s\n",
>> + print_str, argv[4]);
>> + ret = -ENOTSUP;
>> + goto out;
>> + }
>> +
>> + }
>> + printf("Executing %s from 0x%08x to 0x%08x\n", print_str, start, end);
>> +
>> + ret = erase(dev_fd, arg, start, end);
>> +out:
>> + printf(" %s %s!\n\n", print_str, ret ? "Failed" : "Succeed");
>> + close(dev_fd);
>> + return ret;
>> +}
>> +
>> +
>> int do_ffu(int nargs, char **argv)
>> {
>> #ifndef MMC_IOC_MULTI_CMD
>> diff --git a/mmc_cmds.h b/mmc_cmds.h
>> index 9d3246c..8331ab2 100644
>> --- a/mmc_cmds.h
>> +++ b/mmc_cmds.h
>> @@ -45,3 +45,4 @@ int do_ffu(int nargs, char **argv);
>> int do_read_scr(int argc, char **argv);
>> int do_read_cid(int argc, char **argv);
>> int do_read_csd(int argc, char **argv);
>> +int do_erase(int nargs, char **argv);
>> --
>> 2.24.1 (Apple Git-126)
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [External] : RE: [PATCH RESEND v3] mmc-utils: Re-submit of the erase command addition plus remval of MMC_IOC_MULTI_CMD ifndef for erase. Re-committing the change per request by Avir.
2021-04-23 5:31 ` Avri Altman
@ 2021-04-23 14:27 ` kimito.sakata
2021-04-26 9:40 ` Bean Huo
2021-04-26 9:36 ` Bean Huo
1 sibling, 1 reply; 8+ messages in thread
From: kimito.sakata @ 2021-04-23 14:27 UTC (permalink / raw)
To: Avri Altman, luserhker, beanhuo, kenny.gibbons, rkamdar, chris,
ulf.hansson
Cc: linux-mmc, linux-kernel, Kimito Sakata
Its basically the same changes. Bean was coaching me on how to send the
changes.
The only difference between Bean's submit and my last one is Ulf
requested me to make a change in my changes to remove the
MMC_IOC_MULTI_CMD #ifndef segments.
Thanks
Kimito
On 4/22/2021 11:31 PM, Avri Altman wrote:
> +Bean
>
>>> From: Kimito Sakata <kimito.sakata@oracle.com>
>>>
>>> Signed-off-by: Kimito Sakata <ksakata@Kimitos-MBP.hsd1.co.comcast.net>
>> Hi Kimito,
>> Please use a proper subject and commit log body.
>> If you don't understand the difference between those two - please ask.
>> Also for consistency, you might want to use your oracle mail for your Signed-
>> off-by tag.
>> You need to change your git configs for that.
> Also, can you refer to this patch submitted by Bean - https://urldefense.com/v3/__https://www.spinics.net/lists/linux-mmc/msg63582.html__;!!GqivPVa7Brio!PTrg4lGpraGQnZjmxPdNPivVE4IYFevBm1M6vu3_hfHR8REY7LwkcZ3WiZc3MhF21tM$
> Are you co-developing this together?
>
> Thanks,
> Avri
>
>> Thanks,
>> Avri
>>
>>> ---
>>> mmc.c | 8 ++++
>>> mmc.h | 13 +++++-
>>> mmc_cmds.c | 135
>>> +++++++++++++++++++++++++++++++++++++++++++++++++++++
>>> mmc_cmds.h | 1 +
>>> 4 files changed, 156 insertions(+), 1 deletion(-)
>>>
>>> diff --git a/mmc.c b/mmc.c
>>> index f3d724b..eb2638b 100644
>>> --- a/mmc.c
>>> +++ b/mmc.c
>>> @@ -229,6 +229,14 @@ static struct Command commands[] = {
>>> "Run Field Firmware Update with <image name> on <device>.\n",
>>> NULL
>>> },
>>> + { do_erase, -4,
>>> + "erase", "<type> " "<start address> " "<end address> " "<device>\n"
>>> + "Send Erase CMD38 with specific argument to the <device>\n\n"
>>> + "NOTE!: This will delete all user data in the specified region of the
>>> device\n"
>>> + "<type> must be: legacy | discard | secure-erase | "
>>> + "secure-trim1 | secure-trim2 | trim \n",
>>> + NULL
>>> + },
>>> { 0, 0, 0, 0 }
>>> };
>>>
>>> diff --git a/mmc.h b/mmc.h
>>> index 5754a9d..e9766d7 100644
>>> --- a/mmc.h
>>> +++ b/mmc.h
>>> @@ -35,7 +35,15 @@
>>> #define MMC_SET_WRITE_PROT 28 /* ac [31:0] data addr R1b */
>>> #define MMC_CLEAR_WRITE_PROT 29 /* ac [31:0] data addr R1b */
>>> #define MMC_SEND_WRITE_PROT_TYPE 31 /* ac [31:0] data addr R1 */
>>> -
>>> +#define MMC_ERASE_GROUP_START 35 /* ac [31:0] data addr R1 */
>>> +#define MMC_ERASE_GROUP_END 36 /* ac [31:0] data addr R1 */
>>> +#define MMC_ERASE 38 /* ac [31] Secure request
>>> + [30:16] set to 0
>>> + [15] Force Garbage Collect request
>>> + [14:2] set to 0
>>> + [1] Discard Enable
>>> + [0] Identify Write Blocks for
>>> + Erase (or TRIM Enable) R1b */
>>> /*
>>> * EXT_CSD fields
>>> */
>>> @@ -62,6 +70,7 @@
>>> #define EXT_CSD_CACHE_SIZE_2 251
>>> #define EXT_CSD_CACHE_SIZE_1 250
>>> #define EXT_CSD_CACHE_SIZE_0 249
>>> +#define EXT_CSD_SEC_FEATURE_SUPPORT 231
>>> #define EXT_CSD_BOOT_INFO 228 /* R/W */
>>> #define EXT_CSD_HC_ERASE_GRP_SIZE 224
>>> #define EXT_CSD_HC_WP_GRP_SIZE 221
>>> @@ -190,6 +199,8 @@
>>> #define EXT_CSD_REV_V4_2 2
>>> #define EXT_CSD_REV_V4_1 1
>>> #define EXT_CSD_REV_V4_0 0
>>> +#define EXT_CSD_SEC_GB_CL_EN (1<<4)
>>> +#define EXT_CSD_SEC_ER_EN (1<<0)
>>>
>>>
>>> /* From kernel linux/mmc/core.h */
>>> diff --git a/mmc_cmds.c b/mmc_cmds.c
>>> index 6c24cea..9340e3f 100644
>>> --- a/mmc_cmds.c
>>> +++ b/mmc_cmds.c
>>> @@ -2514,6 +2514,141 @@ int do_cache_dis(int nargs, char **argv)
>>> return do_cache_ctrl(0, nargs, argv);
>>> }
>>>
>>> +static int erase(int dev_fd, __u32 argin, __u32 start, __u32 end)
>>> +{
>>> +#ifndef MMC_IOC_MULTI_CMD
>>> + fprintf(stderr, "mmc-utils has been compiled without
>>> MMC_IOC_MULTI_CMD"
>>> + " support, needed by erase.\n");
>>> + return -ENOTSUP;
>>> +#else
>>> + int ret = 0;
>>> + struct mmc_ioc_multi_cmd *multi_cmd;
>>> +
>>> + multi_cmd = calloc(1, sizeof(struct mmc_ioc_multi_cmd) +
>>> + 3 * sizeof(struct mmc_ioc_cmd));
>>> + if (!multi_cmd) {
>>> + perror("Failed to allocate memory");
>>> + return -ENOMEM;
>>> + }
>>> +
>>> + multi_cmd->num_of_cmds = 3;
>>> + /* Set erase start address */
>>> + multi_cmd->cmds[0].opcode = MMC_ERASE_GROUP_START;
>>> + multi_cmd->cmds[0].arg = start;
>>> + multi_cmd->cmds[0].flags = MMC_RSP_SPI_R1 | MMC_RSP_R1 |
>>> MMC_CMD_AC;
>>> + multi_cmd->cmds[0].write_flag = 1;
>>> +
>>> + /* Set erase end address */
>>> + multi_cmd->cmds[1].opcode = MMC_ERASE_GROUP_END;
>>> + multi_cmd->cmds[1].arg = end;
>>> + multi_cmd->cmds[1].flags = MMC_RSP_SPI_R1 | MMC_RSP_R1 |
>>> MMC_CMD_AC;
>>> + multi_cmd->cmds[1].write_flag = 1;
>>> +
>>> + /* Send Erase Command */
>>> + multi_cmd->cmds[2].opcode = MMC_ERASE;
>>> + multi_cmd->cmds[2].arg = argin;
>>> + multi_cmd->cmds[2].cmd_timeout_ms = 300*255*255;
>>> + multi_cmd->cmds[2].flags = MMC_RSP_SPI_R1B | MMC_RSP_R1B |
>>> MMC_CMD_AC;
>>> + multi_cmd->cmds[2].write_flag = 1;
>>> +
>>> + /* send erase cmd with multi-cmd */
>>> + ret = ioctl(dev_fd, MMC_IOC_MULTI_CMD, multi_cmd);
>>> + if (ret)
>>> + perror("Erase multi-cmd ioctl");
>>> +
>>> + free(multi_cmd);
>>> + return ret;
>>> +#endif
>>> +}
>>> +
>>> +int do_erase(int nargs, char **argv)
>>> +{
>>> + int dev_fd, ret;
>>> + char *print_str;
>>> + char **eptr = NULL;
>>> + __u8 ext_csd[512], checkup_mask = 0;
>>> + __u32 arg, start, end;
>>> +
>>> + if (nargs != 5) {
>>> + fprintf(stderr, "Usage: erase <type> <start addr> <end addr>
>>> </path/to/mmcblkX>\n");
>>> + exit(1);
>>> + }
>>> +
>>> + if (strstr(argv[2], "0x") || strstr(argv[2], "0X"))
>>> + start = strtol(argv[2], eptr, 16);
>>> + else
>>> + start = strtol(argv[2], eptr, 10);
>>> +
>>> + if (strstr(argv[3], "0x") || strstr(argv[3], "0X"))
>>> + end = strtol(argv[3], eptr, 16);
>>> + else
>>> + end = strtol(argv[3], eptr, 10);
>>> +
>>> + if (end < start) {
>>> + fprintf(stderr, "erase start [0x%08x] > erase end [0x%08x]\n",
>>> + start, end);
>>> + exit(1);
>>> + }
>>> +
>>> + if (strcmp(argv[1], "legacy") == 0) {
>>> + arg = 0x00000000;
>>> + print_str = "Legacy Erase";
>>> + } else if (strcmp(argv[1], "discard") == 0) {
>>> + arg = 0x00000003;
>>> + print_str = "Discard";
>>> + } else if (strcmp(argv[1], "secure-erase") == 0) {
>>> + print_str = "Secure Erase";
>>> + checkup_mask = EXT_CSD_SEC_ER_EN;
>>> + arg = 0x80000000;
>>> + } else if (strcmp(argv[1], "secure-trim1") == 0) {
>>> + print_str = "Secure Trim Step 1";
>>> + checkup_mask = EXT_CSD_SEC_ER_EN | EXT_CSD_SEC_GB_CL_EN;
>>> + arg = 0x80000001;
>>> + } else if (strcmp(argv[1], "secure-trim2") == 0) {
>>> + print_str = "Secure Trim Step 2";
>>> + checkup_mask = EXT_CSD_SEC_ER_EN | EXT_CSD_SEC_GB_CL_EN;
>>> + arg = 0x80008000;
>>> + } else if (strcmp(argv[1], "trim") == 0) {
>>> + print_str = "Trim";
>>> + checkup_mask = EXT_CSD_SEC_GB_CL_EN;
>>> + arg = 0x00000001;
>>> + } else {
>>> + fprintf(stderr, "Unknown erase type: %s\n", argv[1]);
>>> + exit(1);
>>> + }
>>> +
>>> + dev_fd = open(argv[4], O_RDWR);
>>> + if (dev_fd < 0) {
>>> + perror(argv[4]);
>>> + exit(1);
>>> + }
>>> +
>>> + if (checkup_mask) {
>>> + ret = read_extcsd(dev_fd, ext_csd);
>>> + if (ret) {
>>> + fprintf(stderr, "Could not read EXT_CSD from %s\n",
>>> + argv[4]);
>>> + goto out;
>>> + }
>>> + if ((checkup_mask & ext_csd[EXT_CSD_SEC_FEATURE_SUPPORT]) !=
>>> + checkup_mask) {
>>> + fprintf(stderr, "%s is not supported in %s\n",
>>> + print_str, argv[4]);
>>> + ret = -ENOTSUP;
>>> + goto out;
>>> + }
>>> +
>>> + }
>>> + printf("Executing %s from 0x%08x to 0x%08x\n", print_str, start, end);
>>> +
>>> + ret = erase(dev_fd, arg, start, end);
>>> +out:
>>> + printf(" %s %s!\n\n", print_str, ret ? "Failed" : "Succeed");
>>> + close(dev_fd);
>>> + return ret;
>>> +}
>>> +
>>> +
>>> int do_ffu(int nargs, char **argv)
>>> {
>>> #ifndef MMC_IOC_MULTI_CMD
>>> diff --git a/mmc_cmds.h b/mmc_cmds.h
>>> index 9d3246c..8331ab2 100644
>>> --- a/mmc_cmds.h
>>> +++ b/mmc_cmds.h
>>> @@ -45,3 +45,4 @@ int do_ffu(int nargs, char **argv);
>>> int do_read_scr(int argc, char **argv);
>>> int do_read_cid(int argc, char **argv);
>>> int do_read_csd(int argc, char **argv);
>>> +int do_erase(int nargs, char **argv);
>>> --
>>> 2.24.1 (Apple Git-126)
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH RESEND v3] mmc-utils: Re-submit of the erase command addition plus remval of MMC_IOC_MULTI_CMD ifndef for erase. Re-committing the change per request by Avir.
2021-04-23 5:31 ` Avri Altman
2021-04-23 14:27 ` [External] : " kimito.sakata
@ 2021-04-26 9:36 ` Bean Huo
1 sibling, 0 replies; 8+ messages in thread
From: Bean Huo @ 2021-04-26 9:36 UTC (permalink / raw)
To: Avri Altman, luserhker, beanhuo, kenny.gibbons, kimito.sakata,
rkamdar, chris, ulf.hansson
Cc: linux-mmc, linux-kernel, Kimito Sakata
On Fri, 2021-04-23 at 05:31 +0000, Avri Altman wrote:
> +Bean
>
> > > From: Kimito Sakata <kimito.sakata@oracle.com>
> > >
> > > Signed-off-by: Kimito Sakata <
> > > ksakata@Kimitos-MBP.hsd1.co.comcast.net>
> > Hi Kimito,
> > Please use a proper subject and commit log body.
> > If you don't understand the difference between those two - please
> > ask.
> > Also for consistency, you might want to use your oracle mail for
> > your Signed-
> > off-by tag.
> > You need to change your git configs for that.
> Also, can you refer to this patch submitted by Bean -
> https://www.spinics.net/lists/linux-mmc/msg63582.html
> Are you co-developing this together?
>
> Thanks,
> Avri
>
> > Thanks,
> > Avri
>
Kimito,
If you need my support as before. ping me.
Bean
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [External] : RE: [PATCH RESEND v3] mmc-utils: Re-submit of the erase command addition plus remval of MMC_IOC_MULTI_CMD ifndef for erase. Re-committing the change per request by Avir.
2021-04-23 14:27 ` [External] : " kimito.sakata
@ 2021-04-26 9:40 ` Bean Huo
2021-04-26 16:12 ` [EXT] " Raj Kamdar (rkamdar)
0 siblings, 1 reply; 8+ messages in thread
From: Bean Huo @ 2021-04-26 9:40 UTC (permalink / raw)
To: kimito.sakata, Avri Altman, luserhker, beanhuo, kenny.gibbons,
rkamdar, chris, ulf.hansson
Cc: linux-mmc, linux-kernel, Kimito Sakata
On Fri, 2021-04-23 at 08:27 -0600, kimito.sakata@oracle.com wrote:
> Its basically the same changes. Bean was coaching me on how to send
> the
>
> changes.
>
> The only difference between Bean's submit and my last one is Ulf
>
> requested me to make a change in my changes to remove the
>
> MMC_IOC_MULTI_CMD #ifndef segments.
>
>
>
> Thanks
>
> Kimito
Kimito,
I just saw your last email. we need to re-submit this patch because the
format is not correct. If you need me to resubmit it on behalf of you.
let me know.
Bean
^ permalink raw reply [flat|nested] 8+ messages in thread
* RE: [EXT] Re: [External] : RE: [PATCH RESEND v3] mmc-utils: Re-submit of the erase command addition plus remval of MMC_IOC_MULTI_CMD ifndef for erase. Re-committing the change per request by Avir.
2021-04-26 9:40 ` Bean Huo
@ 2021-04-26 16:12 ` Raj Kamdar (rkamdar)
0 siblings, 0 replies; 8+ messages in thread
From: Raj Kamdar (rkamdar) @ 2021-04-26 16:12 UTC (permalink / raw)
To: Bean Huo, kimito.sakata, Avri Altman, luserhker,
Bean Huo (beanhuo),
kenny.gibbons, chris, ulf.hansson
Cc: linux-mmc, linux-kernel, Kimito Sakata
Micron Confidential
Bean,
If you could help with this it would be appreciated!
Thanks,
Raj Kamdar
Micron Confidential
-----Original Message-----
From: Bean Huo <huobean@gmail.com>
Sent: Monday, April 26, 2021 2:41 AM
To: kimito.sakata@oracle.com; Avri Altman <Avri.Altman@wdc.com>; luserhker@gmail.com; Bean Huo (beanhuo) <beanhuo@micron.com>; kenny.gibbons@oracle.com; Raj Kamdar (rkamdar) <rkamdar@micron.com>; chris@printf.net; ulf.hansson@linaro.org
Cc: linux-mmc@vger.kernel.org; linux-kernel@vger.kernel.org; Kimito Sakata <ksakata@Kimitos-MBP.hsd1.co.comcast.net>
Subject: [EXT] Re: [External] : RE: [PATCH RESEND v3] mmc-utils: Re-submit of the erase command addition plus remval of MMC_IOC_MULTI_CMD ifndef for erase. Re-committing the change per request by Avir.
CAUTION: EXTERNAL EMAIL. Do not click links or open attachments unless you recognize the sender and were expecting this message.
On Fri, 2021-04-23 at 08:27 -0600, kimito.sakata@oracle.com wrote:
> Its basically the same changes. Bean was coaching me on how to send
> the
>
> changes.
>
> The only difference between Bean's submit and my last one is Ulf
>
> requested me to make a change in my changes to remove the
>
> MMC_IOC_MULTI_CMD #ifndef segments.
>
>
>
> Thanks
>
> Kimito
Kimito,
I just saw your last email. we need to re-submit this patch because the format is not correct. If you need me to resubmit it on behalf of you.
let me know.
Bean
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2021-04-26 16:12 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-04-22 16:12 [PATCH RESEND v3] mmc-utils: Re-submit of the erase command addition plus remval of MMC_IOC_MULTI_CMD ifndef for erase. Re-committing the change per request by Avir luserhker
2021-04-23 5:19 ` Avri Altman
2021-04-23 5:31 ` Avri Altman
2021-04-23 14:27 ` [External] : " kimito.sakata
2021-04-26 9:40 ` Bean Huo
2021-04-26 16:12 ` [EXT] " Raj Kamdar (rkamdar)
2021-04-26 9:36 ` Bean Huo
2021-04-23 14:23 ` [External] : " kimito.sakata
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).