unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* [bug#40331] [PATCH] gnu: genimage: Update to 11.
@ 2020-03-30 18:34 Vincent Legoll
  2020-03-30 20:47 ` Danny Milosavljevic
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Vincent Legoll @ 2020-03-30 18:34 UTC (permalink / raw)
  To: 40331

[-- Attachment #1: Type: text/plain, Size: 643 bytes --]

Hello,

here is an update to version 11 for genimage,
this is not the latest, as there is a test suite
failure for version 12 that I cannot understand.

The output for this test failure is in the attached
log file.

It looks like the created image uses 2 filesystem
block more than the expected, and so the bitmap
checksum changes, etc.

Danny, you close the issue:
https://github.com/pengutronix/genimage/issues/36
what was the outcome ?

I want to have your word on this before creating an
issue upstream, if we can solve it on our side.

So, until this test suite failure is solved we can still
get the version 11 in...

-- 
Vincent Legoll

[-- Attachment #2: genimage-v12-test-suite-failure.txt --]
[-- Type: text/plain, Size: 6082 bytes --]

expecting success: 
        run_genimage mke2fs.config mke2fs.ext4 &&
        check_ext images/mke2fs.ext4 mke2fs 33554432
1+0 records in
1+0 records out
1048576 bytes (1.0 MB, 1.0 MiB) copied, 0.00112043 s, 936 MB/s
INFO: cmd: "mkdir -p "/tmp/guix-build-genimage-12.drv-0/source/trash directory.basic-images.test/tmp"" (stderr+stdout):
INFO: cmd: "rm -rf "/tmp/guix-build-genimage-12.drv-0/source/trash directory.basic-images.test/tmp"/*" (stderr+stdout):
INFO: cmd: "mkdir -p "/tmp/guix-build-genimage-12.drv-0/source/trash directory.basic-images.test/tmp"" (stderr+stdout):
INFO: cmd: "cp -a "/tmp/guix-build-genimage-12.drv-0/source/trash directory.basic-images.test/root" "/tmp/guix-build-genimage-12.d
rv-0/source/trash directory.basic-images.test/tmp/root"" (stderr+stdout):
INFO: cmd: "find '/tmp/guix-build-genimage-12.drv-0/source/trash directory.basic-images.test/tmp/root' -depth -type d -printf '%P\0' | xargs -0 -I {} touch -r '/tmp/guix-build-genimage-12.drv-0/source/trash directory.basic-images.test/root/{}' '/tmp/guix-build-genimage-12.drv-0/source/trash directory.basic-images.test/tmp/root/{}'" (stderr+stdout):
INFO: cmd: "mkdir -p "/tmp/guix-build-genimage-12.drv-0/source/trash directory.basic-images.test/images"" (stderr+stdout):
INFO: ext4(mke2fs.ext4): cmd: "MKE2FS_CONFIG="mke2fs.conf" mke2fs -E no_copy_xattrs -t ext4 -E 'root_owner=0:0,lazy_itable_init=0,lazy_journal_init=0' -O '^large_file' -O '^huge_file' -d '/tmp/guix-build-genimage-12.drv-0/source/trash directory.basic-images.test/tmp/root' -U 12345678-1234-1234-1234-1234567890ab -L mke2fs -O '^resize_inode,quota' '/tmp/guix-build-genimage-12.drv-0/source/trash directory.basic-images.test/images/mke2fs.ext4' 32768" (stderr+stdout):
mke2fs 1.45.5 (07-Jan-2020)
Discarding device blocks: done                            
Creating filesystem with 32768 1k blocks and 8192 inodes
Filesystem UUID: 12345678-1234-1234-1234-1234567890ab
Superblock backups stored on blocks: 
        8193, 24577

Allocating group tables: done                            
Writing inode tables: done                            
Creating journal (4096 blocks): done
Copying files into the device: done
Writing superblocks and filesystem accounting information: done

INFO: ext4(mke2fs.ext4): cmd: "/gnu/store/6lqcz57c2r06p3ggjr81c9h88afddgzq-e2fsprogs-1.45.5/sbin/e2fsck -pvfD '/tmp/guix-build-genimage-12.drv-0/source/trash directory.basic-images.test/images/mke2fs.ext4'" (stderr+stdout):
[QUOTA WARNING] Missing quota entry ID 999
mke2fs: Update quota info for quota type 0.
[QUOTA WARNING] Missing quota entry ID 30000
mke2fs: Update quota info for quota type 1.

          51 inodes used (0.62%, out of 8192)
           2 non-contiguous files (3.9%)
           0 non-contiguous directories (0.0%)
             # of inodes with ind/dind/tind blocks: 0/0/0
             Extent depth histogram: 45
        5178 blocks used (15.80%, out of 32768)
           0 bad blocks
           0 large files

          24 regular files
          18 directories
           0 character device files
           0 block device files
           0 fifos
           0 links
           0 symbolic links (0 fast symbolic links)
           0 sockets
------------
          42 files
INFO: ext4(mke2fs.ext4): cmd: "echo 'set_current_time 20000101000000
set_super_value mkfs_time 20000101000000
set_super_value lastcheck 20000101000000
set_super_value mtime 00000000' | /gnu/store/6lqcz57c2r06p3ggjr81c9h88afddgzq-e2fsprogs-1.45.5/sbin/debugfs -w '/tmp/guix-build-genimage-12.drv-0/source/trash directory.basic-images.test/images/mke2fs.ext4'" (stderr+stdout):
debugfs 1.45.5 (07-Jan-2020)
debugfs:  Setting current time to Sat Jan  1 00:00:00 2000

debugfs:  debugfs:  debugfs:  debugfs:  ++ uuid='Filesystem UUID'
++ seed='Directory Hash Seed:'
++ csum1='Checksum: \|Group 0: (Blocks 1-4095) csum'
++ csum2='Group 0: (Blocks 1-4095) \[ITABLE_ZEROED\]\|  Checksum .*, unused inodes 205'
++ dumpe2fs images/mke2fs.ext4
++ grep -v '^\(Filesystem UUID\|Directory Hash Seed:\|Checksum: \|Group 0: (Blocks 1-4095) csum\|Group 0: (Blocks 1-4095) \[ITABLE_ZEROED\]\|  Checksum .*, unused inodes 205\)'
dumpe2fs 1.45.5 (07-Jan-2020)
++ sed -i 's/^\(Journal size:             \)1029k$/\11024k/' dump
++ sed -i 's/^\(  Block bitmap at .*\),\( Inode bitmap.*\)$/\1\n \2/' dump
++ test_cmp /tmp/guix-build-genimage-12.drv-0/source/test/mke2fs.dump dump
++ diff -u /tmp/guix-build-genimage-12.drv-0/source/test/mke2fs.dump dump

--- /tmp/guix-build-genimage-12.drv-0/source/test/mke2fs.dump   2020-03-28 14:27:56.448000000 +0000
+++ dump        2020-03-28 14:28:01.692000000 +0000
@@ -11,7 +11,7 @@
 Inode count:              8192
 Block count:              32768
 Reserved block count:     1638
-Free blocks:              27592
+Free blocks:              27590
 Free inodes:              8141
 First block:              1
 Block size:               1024
@@ -29,7 +29,7 @@
 Maximum mount count:      -1
 Last checked:             Sat Jan  1 00:00:00 2000
 Check interval:           0 (<none>)
-Lifetime writes:          123 kB
+Lifetime writes:          125 kB
 Reserved blocks uid:      0 (user unknown)
 Reserved blocks gid:      0 (group unknown)
 First inode:              11
@@ -47,13 +47,13 @@
 Journal start:            0
 
 
-Group 0: (Blocks 1-8192) csum 0x62cb [ITABLE_ZEROED]
+Group 0: (Blocks 1-8192) csum 0xe083 [ITABLE_ZEROED]
   Primary superblock at 1, Group descriptors at 2-2
-  Block bitmap at 3 (+2), csum 0xb4967804
+  Block bitmap at 3 (+2), csum 0x0fddd371
   Inode bitmap at 7 (+6), csum 0xb1052088
   Inode table at 11-266 (+10)
-  7117 free blocks, 1997 free inodes, 18 directories, 1997 unused inodes
-  Free blocks: 1076-8192
+  7115 free blocks, 1997 free inodes, 18 directories, 1997 unused inodes
+  Free blocks: 1078-8192
   Free inodes: 52-2048
 Group 1: (Blocks 8193-16384) csum 0x7d4b [INODE_UNINIT, BLOCK_UNINIT, ITABLE_ZEROED]
++ func_check
++ local ret=1
++ set +x
Failed to execute 'check_ext'!
   Backup superblock at 8193, Group descriptors at 8194-8194
not ok 7 - mke2fs
FAIL: test/basic-images.test 7 - mke2fs

[-- Attachment #3: 0001-gnu-genimage-Update-to-11.patch --]
[-- Type: text/x-patch, Size: 1539 bytes --]

From e64a6703fa1bbe79364df14848870fdc988745c9 Mon Sep 17 00:00:00 2001
From: Vincent Legoll <vincent.legoll@gmail.com>
Date: Fri, 27 Mar 2020 22:10:06 +0100
Subject: [PATCH] gnu: genimage: Update to 11.

* gnu/packages/genimage.scm (genimage): Update to 11.
[arguments]: add a new test dump file to the patched ones
because of missing /etc/passwd in build environment.
---
 gnu/packages/genimage.scm | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/genimage.scm b/gnu/packages/genimage.scm
index fdf686f9d7..27f55a398a 100644
--- a/gnu/packages/genimage.scm
+++ b/gnu/packages/genimage.scm
@@ -41,7 +41,7 @@
 (define-public genimage
   (package
     (name "genimage")
-    (version "10")
+    (version "11")
     (source (origin
               (method git-fetch)
               (uri (git-reference
@@ -50,7 +50,7 @@
               (file-name (string-append name "-" version "-checkout"))
               (sha256
                (base32
-                "0had00p2why2l1fl14mq7nbhmmfbd3na4qnnpg36akdy05g67jbn"))))
+                "15jmh17lvm3jw9c92bjarly7iwhmnfl322d91mprfv10ppb9ip54"))))
     (build-system gnu-build-system)
     (arguments
      `(#:phases
@@ -149,6 +149,7 @@
              (substitute* '("test/ext2test.dump"
                             "test/ext3test.dump"
                             "test/ext4test.dump"
+                            "test/ext2test-percent.dump"
                             "test/mke2fs.dump")
               (("root") "unknown"))
              #t))
-- 
2.25.2


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [bug#40331] [PATCH] gnu: genimage: Update to 11.
  2020-03-30 18:34 [bug#40331] [PATCH] gnu: genimage: Update to 11 Vincent Legoll
@ 2020-03-30 20:47 ` Danny Milosavljevic
  2020-04-08  9:30 ` Mathieu Othacehe
  2020-04-08  9:32 ` bug#40331: " Mathieu Othacehe
  2 siblings, 0 replies; 4+ messages in thread
From: Danny Milosavljevic @ 2020-03-30 20:47 UTC (permalink / raw)
  To: Vincent Legoll; +Cc: 40331

[-- Attachment #1: Type: text/plain, Size: 520 bytes --]

On Mon, 30 Mar 2020 20:34:42 +0200
Vincent Legoll <vincent.legoll@gmail.com> wrote:

> Danny, you close the issue:
> https://github.com/pengutronix/genimage/issues/36
> what was the outcome ?
> 
> I want to have your word on this before creating an
> issue upstream, if we can solve it on our side.

I can't remember.  I *think* it was because one of our tools had xattr support
and another one of our tools had no xattr support, so the xattrs would get lost
and that would change the image or something.


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [bug#40331] [PATCH] gnu: genimage: Update to 11.
  2020-03-30 18:34 [bug#40331] [PATCH] gnu: genimage: Update to 11 Vincent Legoll
  2020-03-30 20:47 ` Danny Milosavljevic
@ 2020-04-08  9:30 ` Mathieu Othacehe
  2020-04-08  9:32 ` bug#40331: " Mathieu Othacehe
  2 siblings, 0 replies; 4+ messages in thread
From: Mathieu Othacehe @ 2020-04-08  9:30 UTC (permalink / raw)
  To: Vincent Legoll; +Cc: 40331


Hello Vincent,

> here is an update to version 11 for genimage,
> this is not the latest, as there is a test suite
> failure for version 12 that I cannot understand.

Till we find a solution for v12 test failures, I applied your update to
v11.

Thanks for this patch,

Mathieu

^ permalink raw reply	[flat|nested] 4+ messages in thread

* bug#40331: [PATCH] gnu: genimage: Update to 11.
  2020-03-30 18:34 [bug#40331] [PATCH] gnu: genimage: Update to 11 Vincent Legoll
  2020-03-30 20:47 ` Danny Milosavljevic
  2020-04-08  9:30 ` Mathieu Othacehe
@ 2020-04-08  9:32 ` Mathieu Othacehe
  2 siblings, 0 replies; 4+ messages in thread
From: Mathieu Othacehe @ 2020-04-08  9:32 UTC (permalink / raw)
  To: Vincent Legoll; +Cc: 40331-done


And closing!

Mathieu

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2020-04-08  9:33 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-03-30 18:34 [bug#40331] [PATCH] gnu: genimage: Update to 11 Vincent Legoll
2020-03-30 20:47 ` Danny Milosavljevic
2020-04-08  9:30 ` Mathieu Othacehe
2020-04-08  9:32 ` bug#40331: " Mathieu Othacehe

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).