378105 – sys-kernel/genkernel: UUID/Label support for root_keydev and swap_keydev

Bug 378105 - sys-kernel/genkernel: UUID/Label support for root_keydev and swap_keydev

Summary: sys-kernel/genkernel: UUID/Label support for root_keydev and swap_keydev

Status:	RESOLVED FIXED

Alias:	None

Product:	Gentoo Hosted Projects
Classification:	Unclassified
Component:	genkernel (show other bugs)
Hardware:	All Linux

Importance:	Normal enhancement (vote)
Assignee:	Gentoo Genkernel Maintainers

URL:
Whiteboard:
Keywords:

Depends on:
Blocks:

Reported:	2011-08-07 17:26 UTC by Marc Perrudin
Modified:	2012-11-02 17:06 UTC (History)
CC List:	3 users (show)

See Also:
Package list:
Runtime testing required:	---

Attachments
initrd.scripts.diff (initrd.scripts.diff,1.03 KB, patch) 2011-08-07 17:27 UTC, Marc Perrudin	Details \| Diff
initrd.scripts.diff (initrd.scripts.diff,1.87 KB, patch) 2011-08-07 22:03 UTC, Marc Perrudin	Details \| Diff
Another patch to fix this issue. (uuid.patch,4.76 KB, patch) 2012-10-08 13:36 UTC, Marek Sapota	Details \| Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Marc Perrudin 2011-08-07 17:26:15 UTC

Genkernel doen't support UUID and Label for root_keydev and swap_keydev.

Reproducible: Always

Comment 1 Marc Perrudin 2011-08-07 17:27:19 UTC

Created attachment 282457 [details, diff]
initrd.scripts.diff

Here is a patch to add the feature.

Comment 2 Marc Perrudin 2011-08-07 22:03:54 UTC

Created attachment 282473 [details, diff]
initrd.scripts.diff

An other version of the patch.

Comment 3 Alexander Tsoy 2012-04-24 18:34:37 UTC

Similar problem with mounting of /usr partition by initramfs (>=sys-kernel/genkernel-3.4.25.1). Id doesn't understand UUID=/LABEL= in /etc/fstab.


I agree with Marc. A block with the same code

case "${BLABLABLA}" in
  UUID\=*|LABEL\=*)
...
...

is duplicated three or more times in linuxrc and init.scripts. Instead, there should be a reusable function like in a second attached patch.

Comment 4 Marek Sapota 2012-10-08 13:36:47 UTC

Created attachment 325994 [details, diff]
Another patch to fix this issue.

Any progress on this?

I didn’t check the bugzilla before I went on fixing the issue so I produced yet another patch for this.  Anyway that means we have three different patches and it would be cool if one of them made to the official repo.

Comment 5 Alexander Tsoy 2012-10-08 14:28:35 UTC

(In reply to comment #4)
> it would be cool if one of them made to the official repo.

It would be cool to integrate genkernel with dracut. See bug 330127 :)

Comment 6 Marek Sapota 2012-10-08 14:51:29 UTC

(In reply to comment #5)
> It would be cool to integrate genkernel with dracut. See bug 330127 :)

Looks like it is not there yet, so in the meantime can we get this feature in genkernel?=)

Comment 7 Robin Johnson archtester

2012-11-02 17:06:47 UTC

(In reply to comment #4)
> Created attachment 325994 [details, diff] [details, diff]
> Another patch to fix this issue.
> 
> Any progress on this?
> 
> I didn’t check the bugzilla before I went on fixing the issue so I produced
> yet another patch for this.  Anyway that means we have three different
> patches and it would be cool if one of them made to the official repo.

I have merged this version for 3.4.45