From: "Vladimir 'φ-coder/phcoder' Serbinenko" <phcoder@gmail.com>
To: devkral <devkral@web.de>, The development of GRUB 2 <grub-devel@gnu.org>
Subject: Re: encrypted partitions aren't recognized in a good way
Date: Sun, 24 Jul 2011 14:10:48 +0200 [thread overview]
Message-ID: <4E2C0BC8.4050604@gmail.com> (raw)
In-Reply-To: <1311508112.14473.4.camel@archal>
[-- Attachment #1: Type: text/plain, Size: 1140 bytes --]
On 24.07.2011 13:48, devkral wrote:
> Am Donnerstag, den 21.07.2011, 21:51 +0200 schrieb Vladimir
> 'φ-coder/phcoder' Serbinenko:
>> Please keep the list CC'ed.
Keep the list CC'ed. Idon't accept patches sent privately
>> On 21.07.2011 21:39, devkral wrote:
>>> sorry, I don't understand the uuid-LUKS logic of grub-probe and dmsetup
>>> doesn't support this check.
>> dmsetup info /dev/mapper/mycrypt gives:
>> ...
>> UUID: CRYPT-LUKS1-xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx-mycrypt_unformatted
>> And with grub-probe -t cryptodisk_uuid /mnt/crypt
>> you have:
>> xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
>> Where xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx is the unique UUID which is also
>> used in /dev/disk/by-uuid/xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx, just
>> formatted differently.
>> The patch as is unnecessarily uses a heavily different codepath than
>> non-crypt ones. From maintainance point of view this is unacceptable.
>>
> I have had a logical mistake in my patch (the ':' doesn't vanish like
> dmsetup … -c -o vg_name). Here the corrected version of my patch.
--
Regards
Vladimir 'φ-coder/phcoder' Serbinenko
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 294 bytes --]
next prev parent reply other threads:[~2011-07-24 12:11 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <1311164827.2359.13.camel@archal>
2011-07-20 14:49 ` encrypted partitions aren't recognized in a good way Vladimir 'φ-coder/phcoder' Serbinenko
[not found] ` <1311277164.1790.12.camel@archal>
2011-07-21 19:51 ` Vladimir 'φ-coder/phcoder' Serbinenko
[not found] ` <1311463612.1769.4.camel@archal>
2011-07-24 0:26 ` Vladimir 'φ-coder/phcoder' Serbinenko
[not found] ` <1311508112.14473.4.camel@archal>
2011-07-24 12:10 ` Vladimir 'φ-coder/phcoder' Serbinenko [this message]
2011-11-10 8:58 ` Vladimir 'φ-coder/phcoder' Serbinenko
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=4E2C0BC8.4050604@gmail.com \
--to=phcoder@gmail.com \
--cc=devkral@web.de \
--cc=grub-devel@gnu.org \
/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.