From: Lukasz Majewski <lukma@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2 5/6] rename GPT partitions to detect boot failure
Date: Thu, 1 Jun 2017 20:20:37 +0200 [thread overview]
Message-ID: <20170601202037.6eacdf61@jawa> (raw)
In-Reply-To: <080df5158b30ff0e86f63ba00eca6132@she-devel.com>
Hi Alison,
> Lukasz Majewski wrote:
>
> > Maybe it would be better to have
> >
> > gpt flip mmc 0 <optional parameter 'name'>
> >
> > (By default we have "primary" and "backup")
> >
> > In that way we could set other names to GPT partitions without the
> > need to modify the code.
>
> I answered:
>
> >> Another possibility is to support
> >>
> >> gpt flip mmc 0 <name 1> <name 2>
> >>
> >> where two names are required, with defaults 'primary' and
> >> 'backup'. Perhaps I should rework patch 5 that way?
>
> Lukasz responded:
>
> > Please correct my understanding if I'm wrong.
> >
> > You parse GPT partitions to a list and there are only two names
> > possible:
> >
> > "primary" and "backup".
> >
> > When we call gpt filp we rename those names ("primary" -> "backup"
> > and the other way).
> >
> > Ahhh..... so your patch would allow to rename "primary" to <name1>
> > and "backup" to <name2> for all names.
> >
> > My idea was rather to have a gpt call to set name of a GPT
> > partition....
>
> > I can image that somebody would like to change on the fly single GPT
> > partition name (when having/care about 2 partitions for his/her
> > project specific data).
> >
> > But, I don't know how much effort it would take to implement it
> > (and if others would see it beneficial).
>
> I would propose then two options:
>
> gpt rename mmc 0 <name 1> <name 2>
>
> which renames all the 1's to 2's, and
>
> gpt flip mmc 0 <name 1> <name 2>
>
> which swaps the two name strings for all partitions where they're
> found. These two operations together with 'gpt write' then cover all
> the common use cases I can imagine.
I think that this is enough. Let's wait for other's opinions (if
any).
>
> I'm open to any other suggestions, of course. I just posted what I
> already had to get started.
>
> -- Alison
>
> ---
> Alison Chaiken alison at she-devel.com,
> 650-279-5600 http://{ she-devel.com, exerciseforthereader.org }
> "We are giving up our privacy, one convenience at a time." --
> Evangelos Simoudis
Best regards,
Lukasz Majewski
--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
next prev parent reply other threads:[~2017-06-01 18:20 UTC|newest]
Thread overview: 116+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-05-21 2:27 [U-Boot] [PATCH 0/3] add support for GPT partition name manipulation alison at peloton-tech.com
2017-05-21 2:27 ` [U-Boot] [PATCH 1/3] GPT: add accessor function for disk GUID alison at peloton-tech.com
2017-05-26 12:38 ` Tom Rini
2017-05-21 2:27 ` [U-Boot] [PATCH 2/3] GPT: read partition table from device into a data structure alison at peloton-tech.com
2017-05-26 12:39 ` Tom Rini
2017-05-21 2:27 ` [U-Boot] [PATCH 3/3] rename GPT partitions to detect boot failure alison at peloton-tech.com
2017-05-26 12:39 ` Tom Rini
2017-05-29 9:25 ` Lothar Waßmann
2017-05-26 12:38 ` [U-Boot] [PATCH 0/3] add support for GPT partition name manipulation Tom Rini
2017-05-29 16:49 ` [U-Boot] [PATCH v2 0/6] " alison at peloton-tech.com
2017-05-29 16:49 ` [U-Boot] [PATCH v2 1/6] EFI: replace number with UUID_STR_LEN macro alison at peloton-tech.com
2017-05-31 2:07 ` Tom Rini
2017-05-31 7:37 ` Lukasz Majewski
2017-05-29 16:49 ` [U-Boot] [PATCH v2 2/6] disk_partition: introduce macros for description string lengths alison at peloton-tech.com
2017-05-31 7:37 ` Lukasz Majewski
2017-05-31 13:50 ` Tom Rini
2017-05-29 16:49 ` [U-Boot] [PATCH v2 3/6] GPT: add accessor function for disk GUID alison at peloton-tech.com
2017-05-30 6:46 ` Lothar Waßmann
2017-06-03 2:22 ` [U-Boot] [PATCH v3 0/5] add support for GPT partition name manipulation alison at peloton-tech.com
2017-06-03 2:22 ` [U-Boot] [PATCH v3 1/5] GPT: add accessor function for disk GUID alison at peloton-tech.com
2017-06-06 8:20 ` Lothar Waßmann
2017-06-10 5:27 ` [U-Boot] [PATCH v5 1/3] " alison at peloton-tech.com
2017-06-11 13:38 ` Tom Rini
2017-06-03 2:22 ` [U-Boot] [PATCH v3 2/5] partitions: increase MAX_SEARCH_PARTITIONS and move to part.h alison at peloton-tech.com
2017-06-03 11:52 ` Lukasz Majewski
2017-06-03 2:22 ` [U-Boot] [PATCH v3 3/5] GPT: read partition table from device into a data structure alison at peloton-tech.com
2017-06-06 8:28 ` Lothar Waßmann
2017-06-10 5:30 ` [U-Boot] [PATCH v5 2/3] " alison at peloton-tech.com
2017-06-11 13:38 ` Tom Rini
2017-06-06 10:43 ` [U-Boot] [PATCH v3 3/5] " Lothar Waßmann
2017-06-03 2:22 ` [U-Boot] [PATCH v3 4/5] rename GPT partitions to detect boot failure alison at peloton-tech.com
2017-06-06 8:20 ` Lothar Waßmann
2017-06-10 5:35 ` [U-Boot] [PATCH v5 3/3] " alison at peloton-tech.com
2017-06-10 6:51 ` Wolfgang Denk
2017-06-10 23:27 ` Alison Chaiken
2017-06-10 23:33 ` [U-Boot] [PATCH v6 3/3] GPT: provide commands to selectively rename partitions alison at peloton-tech.com
2017-06-11 13:38 ` Tom Rini
2017-06-11 16:03 ` [U-Boot] [PATCH v7] " alison at peloton-tech.com
2017-06-12 7:45 ` [U-Boot] [PATCH v6 3/3] " Wolfgang Denk
2017-06-12 14:24 ` Alison Chaiken
2017-06-12 14:56 ` Tom Rini
2017-06-18 11:08 ` Wolfgang Denk
2017-06-25 23:43 ` [U-Boot] [PATCH v7 0/9] add support for GPT partition name manipulation alison at peloton-tech.com
2017-06-25 23:43 ` [U-Boot] [PATCH v7 1/9] EFI: replace number with UUID_STR_LEN macro alison at peloton-tech.com
2017-08-07 13:54 ` [U-Boot] [U-Boot, v7, " Tom Rini
2017-06-25 23:43 ` [U-Boot] [PATCH v7 2/9] disk_partition: introduce macros for description string lengths alison at peloton-tech.com
2017-08-07 13:54 ` [U-Boot] [U-Boot, v7, " Tom Rini
2017-06-25 23:43 ` [U-Boot] [PATCH v7 3/9] GPT: fix error in partitions string doc alison at peloton-tech.com
2017-08-07 13:54 ` [U-Boot] [U-Boot, v7, " Tom Rini
2017-06-25 23:43 ` [U-Boot] [PATCH v7 4/9] sandbox: README: fix partition command invocation alison at peloton-tech.com
2017-08-07 13:54 ` [U-Boot] [U-Boot, v7, " Tom Rini
2017-06-25 23:43 ` [U-Boot] [PATCH v7 5/9] cmd gpt: test in sandbox alison at peloton-tech.com
2017-08-07 13:54 ` [U-Boot] [U-Boot,v7,5/9] " Tom Rini
2017-06-25 23:43 ` [U-Boot] [PATCH v7 6/9] partitions: increase MAX_SEARCH_PARTITIONS and move to part.h alison at peloton-tech.com
2017-08-07 13:54 ` [U-Boot] [U-Boot, v7, " Tom Rini
2017-06-25 23:43 ` [U-Boot] [PATCH v7 7/9] GPT: add accessor function for disk GUID alison at peloton-tech.com
2017-08-07 13:55 ` [U-Boot] [U-Boot, v7, " Tom Rini
2017-06-25 23:43 ` [U-Boot] [PATCH v7 8/9] GPT: read partition table from device into a data structure alison at peloton-tech.com
2017-06-26 7:34 ` Lothar Waßmann
2017-07-01 22:42 ` [U-Boot] [PATCH v8 8/10] " alison at peloton-tech.com
2017-07-03 6:52 ` Lothar Waßmann
2017-07-04 18:18 ` [U-Boot] [PATCH v8 08/10] " alison at peloton-tech.com
2017-08-07 13:55 ` [U-Boot] [U-Boot, v8, " Tom Rini
2017-06-25 23:43 ` [U-Boot] [PATCH v7 9/9] GPT: provide commands to selectively rename partitions alison at peloton-tech.com
2017-06-26 1:52 ` Bin Meng
2017-06-26 2:11 ` alison at peloton-tech.com
2017-06-26 7:55 ` Lothar Waßmann
2017-07-01 22:44 ` [U-Boot] [PATCH 09/10] " alison at peloton-tech.com
2017-06-18 11:03 ` [U-Boot] [PATCH v6 3/3] " Wolfgang Denk
2017-06-25 21:54 ` Alison Chaiken
2017-06-26 22:47 ` Tom Rini
2017-06-27 7:05 ` Lothar Waßmann
2017-06-27 9:12 ` Lothar Waßmann
2017-07-01 22:44 ` [U-Boot] [PATCH 10/10] gpt: harden set_gpt_info() against non NULL-terminated strings alison at peloton-tech.com
2017-07-03 6:37 ` Lothar Waßmann
2017-07-04 18:19 ` [U-Boot] [PATCH v2 " alison at peloton-tech.com
2017-08-07 13:55 ` [U-Boot] [U-Boot, v2, " Tom Rini
2017-07-01 22:36 ` [U-Boot] [PATCH v6 3/3] GPT: provide commands to selectively rename partitions Alison Chaiken
2017-07-03 6:40 ` Lothar Waßmann
2017-07-04 18:19 ` [U-Boot] [PATCH v8 09/10] " alison at peloton-tech.com
2017-08-07 13:55 ` [U-Boot] [U-Boot, v8, " Tom Rini
2017-06-03 2:22 ` [U-Boot] [PATCH v3 5/5] GPT: fix error in partitions string doc alison at peloton-tech.com
2017-06-03 11:48 ` [U-Boot] [PATCH v3 0/5] add support for GPT partition name manipulation Lukasz Majewski
2017-05-31 7:44 ` [U-Boot] [PATCH v2 3/6] GPT: add accessor function for disk GUID Lukasz Majewski
2017-05-31 8:47 ` Lothar Waßmann
2017-05-29 16:49 ` [U-Boot] [PATCH v2 4/6] GPT: read partition table from device into a data structure alison at peloton-tech.com
2017-05-30 7:37 ` Lothar Waßmann
2017-06-01 6:34 ` Chaiken, Alison
2017-06-01 9:48 ` Lothar Waßmann
2017-05-31 7:48 ` Lukasz Majewski
2017-05-31 8:48 ` Lothar Waßmann
2017-05-31 11:11 ` Lukasz Majewski
2017-05-31 13:42 ` Lothar Waßmann
2017-05-31 14:07 ` Lukasz Majewski
2017-05-29 16:49 ` [U-Boot] [PATCH v2 5/6] rename GPT partitions to detect boot failure alison at peloton-tech.com
2017-05-30 7:38 ` Lothar Waßmann
2017-05-31 8:12 ` Lukasz Majewski
2017-06-01 7:04 ` Chaiken, Alison
2017-06-01 8:21 ` Lukasz Majewski
2017-06-01 15:06 ` Chaiken, Alison
2017-06-01 18:20 ` Lukasz Majewski [this message]
2017-06-04 22:11 ` [U-Boot] [PATCH v4 0/5] add support for GPT partition name manipulation alison at peloton-tech.com
2017-06-04 22:11 ` [U-Boot] [PATCH v4 1/5] GPT: read partition table from device into a data structure alison at peloton-tech.com
2017-06-04 22:11 ` [U-Boot] [PATCH v4 2/5] rename GPT partitions to detect boot failure alison at peloton-tech.com
2017-06-04 22:11 ` [U-Boot] [PATCH v4 3/5] GPT: fix error in partitions string doc alison at peloton-tech.com
2017-06-04 22:11 ` [U-Boot] [PATCH 4/5] sandbox: README: fix partition command invocation alison at peloton-tech.com
2017-06-09 12:28 ` Simon Glass
2017-06-15 19:21 ` sjg at google.com
2017-06-04 22:11 ` [U-Boot] [PATCH 5/5] cmd gpt: test in sandbox alison at peloton-tech.com
2017-06-15 19:21 ` sjg at google.com
2017-08-27 23:02 ` [U-Boot] [PATCH v2 5/6] rename GPT partitions to detect boot failure Chaiken, Alison
2017-08-28 7:54 ` Łukasz Majewski
2017-08-28 11:16 ` Tom Rini
2017-05-29 16:49 ` [U-Boot] [PATCH v2 6/6] GPT: fix error in partitions string doc alison at peloton-tech.com
2017-05-31 8:14 ` Lukasz Majewski
2017-05-31 11:21 ` Lukasz Majewski
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20170601202037.6eacdf61@jawa \
--to=lukma@denx.de \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox