Hello community, here is the log from the commit of package grub2 for openSUSE:Factory checked in at 2018-07-16 10:41:34 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/grub2 (Old) and /work/SRC/openSUSE:Factory/.grub2.new (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "grub2" Mon Jul 16 10:41:34 2018 rev:189 rq:623078 version:2.02 Changes: -------- --- /work/SRC/openSUSE:Factory/grub2/grub2.changes 2018-07-12 09:17:15.722258652 +0200 +++ /work/SRC/openSUSE:Factory/.grub2.new/grub2.changes 2018-07-16 10:41:39.694070729 +0200 @@ -1,0 +2,12 @@ +Mon Jul 16 07:53:19 UTC 2018 - mch...@suse.com + +- Fix install on xfs error (bsc#1101283) + * 0001-xfs-Accept-filesystem-with-sparse-inodes.patch + +------------------------------------------------------------------- +Tue Jul 10 15:57:50 UTC 2018 - jbo...@suse.cz + +- grub2.spec: change %config to %config(noreplace) + Don't overwrite user changes to config files on upgrades. + +------------------------------------------------------------------- New: ---- 0001-xfs-Accept-filesystem-with-sparse-inodes.patch ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ grub2.spec ++++++ --- /var/tmp/diff_new_pack.AMbaAH/_old 2018-07-16 10:41:42.350061868 +0200 +++ /var/tmp/diff_new_pack.AMbaAH/_new 2018-07-16 10:41:42.350061868 +0200 @@ -217,6 +217,7 @@ Patch86: 0001-Fix-packed-not-aligned-error-on-GCC-8.patch Patch87: 0001-Fix-PCIe-LER-when-GRUB2-accesses-non-enabled-MMIO-da.patch Patch88: unix-exec-avoid-atexit-handlers-when-child-exits.patch +Patch89: 0001-xfs-Accept-filesystem-with-sparse-inodes.patch # Btrfs snapshot booting related patches Patch101: grub2-btrfs-01-add-ability-to-boot-from-subvolumes.patch Patch102: grub2-btrfs-02-export-subvolume-envvars.patch @@ -506,6 +507,7 @@ %patch86 -p1 %patch87 -p1 %patch88 -p1 +%patch89 -p1 %patch101 -p1 %patch102 -p1 %patch103 -p1 @@ -1032,13 +1034,13 @@ %config(noreplace) %{_sysconfdir}/default/grub %dir %{_sysconfdir}/grub.d %{_sysconfdir}/grub.d/README -%config %{_sysconfdir}/grub.d/00_header -%config %{_sysconfdir}/grub.d/10_linux -%config %{_sysconfdir}/grub.d/20_linux_xen +%config(noreplace) %{_sysconfdir}/grub.d/00_header +%config(noreplace) %{_sysconfdir}/grub.d/10_linux +%config(noreplace) %{_sysconfdir}/grub.d/20_linux_xen %config(noreplace) %{_sysconfdir}/grub.d/40_custom -%config %{_sysconfdir}/grub.d/41_custom -%config %{_sysconfdir}/grub.d/90_persistent -%config %{_sysconfdir}/grub.d/95_textmode +%config(noreplace) %{_sysconfdir}/grub.d/41_custom +%config(noreplace) %{_sysconfdir}/grub.d/90_persistent +%config(noreplace) %{_sysconfdir}/grub.d/95_textmode %{_sbindir}/%{name}-install %{_sbindir}/%{name}-mkconfig %{_sbindir}/%{name}-once @@ -1100,7 +1102,7 @@ %{_mandir}/man1/%{name}-emu.1.* %endif %ifnarch s390x -%config %{_sysconfdir}/grub.d/30_os-prober +%config(noreplace) %{_sysconfdir}/grub.d/30_os-prober %{_bindir}/%{name}-glue-efi %{_bindir}/%{name}-mount %{_sbindir}/%{name}-bios-setup @@ -1124,11 +1126,11 @@ %files %{grubarch} %defattr(-,root,root,-) %ifarch %{ix86} x86_64 -%config %{_sysconfdir}/grub.d/20_memtest86+ +%config(noreplace) %{_sysconfdir}/grub.d/20_memtest86+ %endif %dir %{_libdir}/%{name}/%{grubarch} %ifarch ppc ppc64 ppc64le -%config %{_sysconfdir}/grub.d/20_ppc_terminfo +%config(noreplace) %{_sysconfdir}/grub.d/20_ppc_terminfo # This is intentionally "grub.chrp" and not "%{name}.chrp" %{_libdir}/%{name}/%{grubarch}/grub.chrp %{_libdir}/%{name}/%{grubarch}/bootinfo.txt @@ -1183,7 +1185,7 @@ %defattr(-,root,root,-) %dir %{_libdir}/snapper %dir %{_libdir}/snapper/plugins -%config %{_sysconfdir}/grub.d/80_suse_btrfs_snapshot +%config(noreplace) %{_sysconfdir}/grub.d/80_suse_btrfs_snapshot %{_libdir}/snapper/plugins/grub %ifarch %{ix86} x86_64 ++++++ 0001-xfs-Accept-filesystem-with-sparse-inodes.patch ++++++ >From cda0a857dd7a27cd5d621747464bfe71e8727fff Mon Sep 17 00:00:00 2001 From: Daniel Kiper <daniel.ki...@oracle.com> Date: Tue, 29 May 2018 16:16:02 +0200 Subject: [PATCH] xfs: Accept filesystem with sparse inodes The sparse inode metadata format became a mkfs.xfs default in xfsprogs-4.16.0, and such filesystems are now rejected by grub as containing an incompatible feature. In essence, this feature allows xfs to allocate inodes into fragmented freespace. (Without this feature, if xfs could not allocate contiguous space for 64 new inodes, inode creation would fail.) In practice, the disk format change is restricted to the inode btree, which as far as I can tell is not used by grub. If all you're doing today is parsing a directory, reading an inode number, and converting that inode number to a disk location, then ignoring this feature should be fine, so I've added it to XFS_SB_FEAT_INCOMPAT_SUPPORTED I did some brief testing of this patch by hacking up the regression tests to completely fragment freespace on the test xfs filesystem, and then write a large-ish number of inodes to consume any existing contiguous 64-inode chunk. This way any files the grub tests add and traverse would be in such a fragmented inode allocation. Tests passed, but I'm not sure how to cleanly integrate that into the test harness. Signed-off-by: Eric Sandeen <sand...@redhat.com> Reviewed-by: Daniel Kiper <daniel.ki...@oracle.com> Tested-by: Chris Murphy <li...@colorremedies.com> --- grub-core/fs/xfs.c | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) Index: grub-2.02/grub-core/fs/xfs.c =================================================================== --- grub-2.02.orig/grub-core/fs/xfs.c +++ grub-2.02/grub-core/fs/xfs.c @@ -79,9 +79,18 @@ GRUB_MOD_LICENSE ("GPLv3+"); #define XFS_SB_FEAT_INCOMPAT_SPINODES (1 << 1) /* sparse inode chunks */ #define XFS_SB_FEAT_INCOMPAT_META_UUID (1 << 2) /* metadata UUID */ -/* We do not currently verify metadata UUID so it is safe to read such filesystem */ +/* + * Directory entries with ftype are explicitly handled by GRUB code. + * + * We do not currently read the inode btrees, so it is safe to read filesystems + * with the XFS_SB_FEAT_INCOMPAT_SPINODES feature. + * + * We do not currently verify metadata UUID, so it is safe to read filesystems + * with the XFS_SB_FEAT_INCOMPAT_META_UUID feature. + */ #define XFS_SB_FEAT_INCOMPAT_SUPPORTED \ (XFS_SB_FEAT_INCOMPAT_FTYPE | \ + XFS_SB_FEAT_INCOMPAT_SPINODES | \ XFS_SB_FEAT_INCOMPAT_META_UUID) struct grub_xfs_sblock