GNU bug report logs - #60057
[PATCH] image: Enhance compatibility of the root ext4 partition.

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 60057 in the body.
You can then email your comments to 60057 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
To: guix-patches <at> gnu.org
Cc: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
Subject: [PATCH] image: Enhance compatibility of the root ext4 partition.
Date: Tue, 13 Dec 2022 21:17:52 -0500

Generating a raw-with-offset image would previously not be bootable with
U-Boot.

* gnu/system/image.scm (root-partition) [file-system-options]: New field.
---
 gnu/system/image.scm | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/gnu/system/image.scm b/gnu/system/image.scm
index f07a4a5217..2bf4559501 100644
--- a/gnu/system/image.scm
+++ b/gnu/system/image.scm
@@ -138,6 +138,9 @@ (define root-partition
    (size 'guess)
    (label root-label)
    (file-system "ext4")
+   ;; Disable the metadata_csum and 64bit features of ext4, for compatibility
+   ;; with U-Boot.
+   (file-system-options (list "-O" "^metadata_csum,^64bit"))
    (flags '(boot))
    (initializer (gexp initialize-root-partition))))
 

base-commit: 0ffa501f2b3e83ae56e9c2bd31418439090e869a
prerequisite-patch-id: 660177fb9eee55d11983ea9360c072730d0d21a5
prerequisite-patch-id: 776778c03bce9b7ad3ab94a120f42b764c00fcae
prerequisite-patch-id: 4910d08bdc27384d76030b6ac491ad2c2ed0957f
prerequisite-patch-id: fd4074fb4cf068ccac3122c19138f098a610542a
prerequisite-patch-id: b9b6f21a2db3f7a5ef82bb11ed23f69749cc6b3d
prerequisite-patch-id: 51cecbabc04c69d4338addaece536c42a075660f
prerequisite-patch-id: 2c67bfd85b343a65440e6c63451e5417129dea6f
prerequisite-patch-id: b24087b240dad97b6417e37176915c24a543e887
prerequisite-patch-id: 2b8e69760af1f7bfb66c1edb44471653b6b229d1
prerequisite-patch-id: 2cb8792eedc8cfc23b4728f9ec44366aa394b954
prerequisite-patch-id: 8684fb6949ced334698dbf32ae71dd76c5eb2cfe
prerequisite-patch-id: 4cc71f61e136fd7288e767ce02bef3b711ea1ad2
prerequisite-patch-id: 69318aca2af97593db0737a035ff646414d61016
-- 
2.38.1

Message #8 received at 60057 <at> debbugs.gnu.org (full text, mbox):

From: Mathieu Othacehe <othacehe <at> gnu.org>
To: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
Cc: 60057 <at> debbugs.gnu.org
Subject: Re: bug#60057: [PATCH] image: Enhance compatibility of the root
 ext4 partition.
Date: Sun, 25 Dec 2022 17:09:58 +0100

Hello Maxim,

> +   ;; Disable the metadata_csum and 64bit features of ext4, for compatibility
> +   ;; with U-Boot.
> +   (file-system-options (list "-O" "^metadata_csum,^64bit"))
>     (flags '(boot))
>     (initializer (gexp initialize-root-partition))))

ext4 man page states that ^64bit will prevent images larger than 2^32
blocks but I highly doubt that we will generate someday images that big.

I think you can go ahead!

Mathieu

Message #13 received at 60057-done <at> debbugs.gnu.org (full text, mbox):

From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
To: Mathieu Othacehe <othacehe <at> gnu.org>
Cc: 60057-done <at> debbugs.gnu.org
Subject: Re: bug#60057: [PATCH] image: Enhance compatibility of the root
 ext4 partition.
Date: Mon, 26 Dec 2022 22:48:23 -0500

Hi Mathieu!

Mathieu Othacehe <othacehe <at> gnu.org> writes:

> Hello Maxim,
>
>> +   ;; Disable the metadata_csum and 64bit features of ext4, for compatibility
>> +   ;; with U-Boot.
>> +   (file-system-options (list "-O" "^metadata_csum,^64bit"))
>>     (flags '(boot))
>>     (initializer (gexp initialize-root-partition))))
>
> ext4 man page states that ^64bit will prevent images larger than 2^32
> blocks but I highly doubt that we will generate someday images that big.
>
> I think you can go ahead!

OK!  Pushed as 60d4652c5d.

Thank you!

-- 
Maxim

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.