Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 915871

Summary: sys-kernel/gentoo-kernel doesn't enable XFS on PPC32
Product: Gentoo Linux Reporter: immolo <immoloism>
Component: Current packagesAssignee: Distribution Kernel Project <dist-kernel>
Status: RESOLVED FIXED    
Severity: normal CC: csfore, sam
Priority: Normal    
Version: unspecified   
Hardware: All   
OS: Linux   
See Also: https://github.com/projg2/gentoo-kernel-config/pull/9
Whiteboard:
Package list:
Runtime testing required: ---

Description immolo 2023-10-17 01:14:20 UTC
Reported via Xarvatium via Discord

While looking into this issue I've quickly spotted that the config in arch/powerpc/configs/pmac32_defconfig which sys-kernel/gentoo-kernel uses on ppc32 doesn't enable the following:

CONFIG_XFS=FS=y
CONFIG_XFS_SUPPORT_V4=y
CONFIG_XFS_QUOTA=y
CONFIG_XFS_POSIX_ACL=y

This will cause anyone following the handbook recommended default file system to not be able to boot their system.

I have two options to solve this however I'm not sure which is going to be the best method:

1) Add a patch to enable it in pmac32_defconfig
This might cause the maintainer issues if pmac32_defconfig changes upstream

2) Host a default config on projg2 which includes the options set in pmac32_defconfig and add the needed XFS changes.
I think this is the cleanest solution with less chance of unforeseen issues occurring in the future.

If you can let me know which way is better then I can upload a fix to solve this.
Comment 1 Sam James archtester Gentoo Infrastructure gentoo-dev Security 2023-10-23 02:28:00 UTC
As discussed on IRC, it's going to be better to just add these snippets to the base config to:
a) make sure all arches benefit from it in future, not just ppc where it's missing from defconfig;
b) kill 2 birds w/ 1 stone by changing it from M->Y for the arches where it is already enabled

The repo is https://github.com/projg2/gentoo-kernel-config.
Comment 2 immolo 2023-10-26 13:10:59 UTC
This fix has now been merged.