All of lore.kernel.org
 help / color / mirror / Atom feed
From: Zhu Yi <yi.zhu@intel.com>
To: The development of GNU GRUB <grub-devel@gnu.org>
Cc: ml@isaac.cedarswampstudios.org, rmh@aybabtu.com
Subject: Re: [PATCH] Backup old boot sectors before installation
Date: Tue, 29 Dec 2009 11:21:03 +0800	[thread overview]
Message-ID: <1262056863.6037.106.camel@debian> (raw)
In-Reply-To: <4B392B6E.2060807@isaac.cedarswampstudios.org>

On Tue, 2009-12-29 at 06:04 +0800, Isaac Dupree wrote:
> > Option 1~3 requires the recover has the knowledge of the backup file
> > format somehow, but a simple dd is enough for option 4. What do you
> > think?
> 
> If the partitioning had been changed in the mean time, 4 would overwrite 
> it with the older values. Similar problem for a weird setup with 
> partitions in between (I dunno, might be possible with GPT or other 
> weirdness).

Good point.

> I think option 2 or 3 is soundest.  Option 2 would have three separate 
> files of information, (right?), and there must be a pretty 
> straightforwards `dd` option to skip to the correct start position. 
> Option 3 would omit the "start position" file... but that file is really 
> cheap, maybe we'd better keep things simple for restorers and stick with 
> three separate files.(Or, we could do something weird like putting 
> start-position somewhere in the filename of the backup.)

This requires the restorer to understand the role of each of the 3 (or
2) files and use `dd' twice with correct options. So an important thing
is, we need to document this somewhere. And I think the best place is to
"document" it in the grub-install script. Because this will enable the
not-so-advanced user to recover her boot sectors easily as well.

If we decide so, we go back to option 1 (my original implementation).
Because the backup file format is not important any more as it is
totally transparent to the restorers. Am I right?

Thanks,
-yi




  reply	other threads:[~2009-12-29  3:21 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-12-11  9:26 [PATCH] Backup old boot sectors before installation Zhu Yi
2009-12-11  9:39 ` Felix Zielcke
2009-12-11 22:23   ` Carles Pina i Estany
2009-12-14  2:48   ` Zhu Yi
2009-12-14  2:58     ` Isaac Dupree
2009-12-14  3:32       ` Zhu Yi
2009-12-14  2:59     ` kashyap garimella
2009-12-14  3:08       ` richardvoigt
2009-12-14  3:24         ` Isaac Dupree
2009-12-14  3:35           ` Zhu Yi
2009-12-14  3:52             ` Isaac Dupree
2009-12-14  3:15       ` Zhu Yi
2009-12-24 21:21 ` Robert Millan
2009-12-28  6:55   ` Zhu Yi
2009-12-28 22:04     ` Isaac Dupree
2009-12-29  3:21       ` Zhu Yi [this message]
2009-12-29  3:59         ` richardvoigt
2009-12-29  7:21           ` Zhu Yi
2010-01-07 19:15             ` rmh

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=1262056863.6037.106.camel@debian \
    --to=yi.zhu@intel.com \
    --cc=grub-devel@gnu.org \
    --cc=ml@isaac.cedarswampstudios.org \
    --cc=rmh@aybabtu.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.